I always have memory errors in destructors

Crikey. Whenever I allocate memory off the heap and delete it in the destructor, my program crashes. A la:

Code:

---

num_steps = 2;
ai = new aistep [num_steps];

/* .... */

enemy::~enemy ()
{
        if (ai) {
                if (num_steps > 1)
                        delete[] ai;
                else
                        delete ai;
               
                ai = NULL;
        }
}

---

Now if I comment all that code in the destructor out, the program starts and ends fine. No matter how I do it (no num_steps check , no if (ai) check, no ai = NULL, etc) it always crashes. And it does thise for other things too, not just classes but for basic types too. Anything allocated dynamically works, but deleting it and the whole shebang crashes.

What am I doing wrong? Am I not supposed to delete stuff in the destructor? Is dynamically allocated memory deleted automatically in destructors? (I didn't think it was...)

Thanks!

given your code you should always use delete []. Calling ordinary delete even on somthing allocated with new foo[1] is undefined, in this case a compiler vendor would have to go fairly far out of thier way to define it as anything other than works correctly, but we must always obey the holy standard. Secondly, if you need to write a destructor you almost always need to write a copy constructor and assignment operator. This is almost certainly your bug. Somewhere you have a copy of your enemy object that when it gets destroyed rips the guts out of your original object. You need to ether make your copy ctor and assignment operator private, and have them throw something just to make absolutely sure you aren't creating any temporaries in any members or friends. Or write a copy ctor that performs the deep copy. Naturally the prefered and obvious solution is to use std::vector.

I should've just use vector in the first place. What was I thinkin.

Thanks, problem solved