There is no need for any of the code in the Delete them all loop. This is one of the benefits of using local variables instead of dynamically allocated ones. Everything is cleaned up automatically.
You are using a const_iterator, which is a good choice, but if you use that you must make your cPizza class const-correct. That means that any function that doesn't change the visible state of the class data should be declared as const. Since value() only uses the member values, but doesn't change them, it should be a const function. That will allow you to call it from a const_iterator.
