Question

I feel a little stupid for making a question about the deletion of pointers but I need to make sure I’m deleting in the correct way as I’m currently going through the debugging process of my program.

Basically I have a few arrays of pointers which are defined in my header file as follows:

AsteroidView *_asteroidView[16];

In a for loop I then initialise them:

for(int i = 0; i < 16; i++)  
{  
      _asteroidView[i] = new AsteroidView();  
} 

Ok, so far so good, everything works fine.
When I eventually need to delete these in the destructor I use this code:

for(int i = 0; i < 16; i++)  
{  
        delete _asteroidView[i];  
}

Is this all I need to do? I feel like it is, but I’m worried about getting memory leaks.

Out of interest…
Is there much of a difference between an Array of Points to Objects compared with an Array of Objects?

Answer 1

This is correct. However, you may want to consider using Boost.PointerContainer, and save you hassle the hassle of manual resource management:

boost::ptr_vector<AsteroidView> _asteroidView;
for(int i = 0; i < 16; i++)
{
    _asteroidView.push_back(new AsteroidView());
}

You do not have to manage the deletion, the container does that for you. This technique is called RAII, and you should learn about it if you want to have fun using C++ 🙂

About your edit: There are several difference, but I guess the most important are these:

1. An array of pointers can contain objects of different types, if these are subclasses of the array type.
2. An array of objects does not need any deletion, all objects are destroyed when the array is destroyed.