Problema con destructores y delete

[vangodp]

Muy bien señalado por lareto.
El destructor es una función que se ejecuta automáticamente al cerrar el programa. No tienes por que llamarlo.
Pero si quieres liberar memoria te aconsejo que hagas una función freeMem() a la que puedas llamar cuando quieras y así puedes prescindir de usar el destructor. Pero no te olvides de liberar la memoria XD
Mire eso:

Código C++:

#include <iostream>
using namespace std;
 
class lol {
    public:
        lol(){ cout << "constructor llamado, se ha creado el objeto!" << endl; }
        ~lol(){ 
            cout << "Destructor llamado! Aqui se destruye automatico" << endl; 
            cin.ignore(); //Se va pausar para que puedas ver el mensaje antes de salir de main
        }
        
    private:
    //blabla bla bla!
};
 
int main (){
    
    lol miLol;    //Crea el objeto
    
    cin.ignore(); // Pausa
    
    cout << "aqui debajo se va llamar automaticamente el destructor" << endl;
    return 0;
}
 

como buena practica de programación es asignar NULL al puntero después de liberar memoria, para que no haya problemas por una doble limpieza de memoria pues haz algo como:

Código C++:

~C() {
    if ( puntero != NULL ) {
        delete [] puntero;
        puntero = NULL; // ahora sabemos que apunta a NADA y podemos averiguar posteriormente.
    }
}
 

Con eso si ya habías llamado el destructor pues asigna NULL al puntero y cuando el destructor es llamado automáticamente al final va a ver que ya no apunta a nada y no va haber problemas XD

Te dejo la posible solución:

Código C++:

#include <iostream>
using namespace std;
 
 
 
class C {
        int * elem;
    public:
        C ( long n ) {
            elem = new int [n];
        }
        ~C() {
                cout << "Destructor llamado"<<endl;
            if ( elem != NULL ) {
                cout << "se ha liberado la memoria y elem apunta a NULL" <<endl;
                delete [] elem;
                elem = NULL;
            }else{
                cout << "elem ya habia sido liberado previamente" << endl;
            }
        }
};
 
int main() {
    C a ( 5 );
    a.~C();
    
    return 0;
}