The correct code is: stopPoints.erase(std::remove_if(stopPoints.begin(), stopPoints.end(), [&](const stopPointPair stopPoint)-> bool { return stopPoint.first == 4; }), stopPoints.end()); You need to remove the range starting from the iterator returned from std::remove_if to the end of the vector, not only a single element. “Why?” std::remove_if swaps elements inside the vector in order to put all elements that … Read more
Well, you could erase() the first character too (note that erase() modifies the string): m_VirtualHostName.erase(0, 1); m_VirtualHostName.erase(m_VirtualHostName.size() – 1); But in this case, a simpler way is to take a substring: m_VirtualHostName = m_VirtualHostName.substr(1, m_VirtualHostName.size() – 2); Be careful to validate that the string actually has at least two characters in it first…
You need to call erase on the iterator returned from remove_if, It should look something like this: auto new_end = std::remove_if(MyDisplayDevices.begin(), MyDisplayDevices.end(), [](const DisplayDevice3d& device) { return device.Modes.size() == 0; }); MyDisplayDevices.erase(new_end, MyDisplayDevices.end());
How about std::remove() instead: #include <algorithm> … vec.erase(std::remove(vec.begin(), vec.end(), 8), vec.end()); This combination is also known as the erase-remove idiom.