You need to be careful because erase() will invalidate existing iterators. However, it will return a new valid iterator you can use: for ( it = Entities.begin(); it != Entities.end(); ) { if( (*it)->getXPos() > 1.5f ) { delete * it; it = Entities.erase(it); } else { ++it; } }
The standard doesn’t quite spell it out, but v.erase(q) is defined, “Erases the element pointed to by q” in [sequence.reqmts]. This means that q must actually point to an element, which the end iterator doesn’t. Passing in the end iterator is undefined behavior. Unfortunately, you need to write: auto it = std::find(…); if (it != … Read more
You should do itVid = videoObjects.erase(itVid); Quote from cplusplus.com: [vector::erase] invalidates all iterator and references to elements after position or first. Return value: A random access iterator pointing to the new location of the element that followed the last element erased by the function call, which is the vector end if the operation erased the … Read more
Using algorithm remove_copy_if :- string text,result; std::remove_copy_if(text.begin(), text.end(), std::back_inserter(result), //Store output std::ptr_fun<int, int>(&std::ispunct) );
You can use VT100 escape codes. Most terminals, including xterm, are VT100 aware. For erasing a line, this is ^[[2K. In C this gives: printf(“\33[2K\r”);
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.
You should not increment it in the for loop: for (vector<Player>::iterator it=allPlayers.begin(); it!=allPlayers.end(); /*it++*/) <———– I commented it. { if(it->getpMoney()<=0) it = allPlayers.erase(it); else ++it; } Notice the commented part;it++ is not needed there, as it is getting incremented in the for-body itself. As for the error “‘operator=” function is unavailable in “Player’“, it comes … Read more
To delete a single element, you could do: std::vector<int> vec; vec.push_back(6); vec.push_back(-17); vec.push_back(12); // Deletes the second element (vec[1]) vec.erase(std::next(vec.begin())); Or, to delete more than one element at once: // Deletes the second through third elements (vec[1], vec[2]) vec.erase(std::next(vec.begin(), 1), std::next(vec.begin(), 3));
Use the remove/erase idiom: std::vector<int>& vec = myNumbers; // use shorter name vec.erase(std::remove(vec.begin(), vec.end(), number_in), vec.end()); What happens is that remove compacts the elements that differ from the value to be removed (number_in) in the beginning of the vector and returns the iterator to the first element after that range. Then erase removes these elements … Read more