views:

135

answers:

4

I have a base class A and a derived class B:

class A
{
public:
    virtual f();
};

class B : public A
{
public:
     B()
     {
         p = new char [100];
     }
     ~B()
     {
         delete [] p;
     }
     f();
private:
    char *p;
};

For any reason the destructor is never called - why? I dont understand this.

+6  A: 

Class A should have a virtual destructor. Without that, derive class destructors won't be called.

Andrew
Won't be called iff the destructor is called via a pointer to A.
Martin York
+15  A: 

Your base class needs a virtual destructor. Otherwise the destructor of the derived class will not be called, if only a pointer of type A* is usd.

Add

virtual ~A() {};

to class A.

Manfred Meyer
+2  A: 

try this:

class A
{
public:
    virtual ~A() {}
    virtual f();
};

class B : public A
{
public:
     B()
     {
         p = new char [100];
     }
     virtual ~B() // virtual keywork optional but occasionally helpful for self documentation.
     {
         delete [] p;
     }
     f();
private:
    char *p;
};
shoosh
+2  A: 

If your variable is of type A it doesn't have a virtual destructor and so it won't look at the actual runtime type of the object to determine it needs to call the desstructor

Add an empty destructor to A

virtual ~A() {}

and that should fix it.

In general you need to do this on any class that can possibly be used as a base class.

John Burton