Hello guys here's the problem. I want to delete the objects within a vector, although I'm not sure whether I should clear the vector afterwards. Is it really necessary?
Code:for (i = 0; i < allSales.size(); i++) delete allSales[i]; allSales.clear(); //Is this step necessary?
Is allSales storing pointers that you allocated with new? If not, then delete itself is incorrect.
The purpose of clear is to remove the objects from the vector, so if that's what you want done, then clear is the way to go.
Yes, it is a vector of pointers.
In that case, the delete calls will release the objects being pointed to, and clear will then remove the pointers from the vector (I would assume you don't want them hanging around still, cluttering up your vector now that they don't point anywhere).Originally Posted by Khabz
If the vector is going to be destroyed (e.g., by going out of scope) soon afterwards, then you don't need the call to clear.
Look up a C++ Reference and learn How To Ask Questions The Smart Way
Call clear() if there is the possibility that something else could or might see the vector full of pointers and mistakenly think they are valid.
Even then, it might be less confusing to someone debugging the code later if a tidy call to clear() ensured that the pointers were gone.
My homepage
Advice: Take only as directed - If symptoms persist, please see your debugger
Linus Torvalds: "But it clearly is the only right way. The fact that everybody else does it some other way only means that they are wrong"
