Re: Memory deallocation
On Nov 1, 3:57 am, Shan <m.shanmugara...@gmail.com> wrote:
FieldValues = new vector<char*>();
I have the above FieldValues in a class as private member. How should
I deallocate the memory allocated by this vector.
Preferably, by allowing it to happen automatically in your class's
destructor by keeping the vector itself as a class member, rather than
a pointer to it. It sounds like your class semantics are that the
vector and its contents are all owned and managed by your class, so
there is no reason to involve pointers and new/delete at all.
Consider this instead:
#include <vector>
#include <string>
class Foo {
private:
std::vector<std::string> FieldValues; // not a pointer
// ...
};
Now it will be automatically initialized to an empty vector<string> in
the constructor (regardless of whether user-defined or default), and
destroyed along with all its contained string values in the
destructor. And you no longer have to worry about remembering all the
fiddling with new and delete, to say nothing of exception safety, in
the assignment operator.
Also I am populating
the vector with <char*> dynamically allocated. I want make sure that
char* memory also deallocated when I destroy the instance of the
class.
If for some reason not clear from your post, you absolutely have to do
it that way, then you need to run delete[] (I'm guessing that it's
operator delete[] you'll want, since it would seem rather silly to
allocate a single character for each char pointer) on each char* in a
loop over the vector elements before deleting the vector itself in
your class's destructor (and also in its assignment op prior to
copying from the RHS):
Foo::~Foo() {
// e.g.:
for (size_t i = 0, n = FieldValues->size(); i < n; ++i)
delete [] (*FieldValues)[i];
delete FieldValues;
}
- Kevin
--
[ See http://www.gotw.ca/resources/clcm.htm for info about ]
[ comp.lang.c++.moderated. First time posters: Do this! ]