Duda al crear un desctructor en c++

[varuk]

Buenas.

Tengo la siguiente estructura:

Código:

struct Usuario {

	string nombre;
	int numeroID;
	list<miTipo *> listaMensajes;

};

class AlmacenaUsuarios {

      private:
		list<Usuario> * arrayNombres[1000];
		list<Usuario>::iterator it;
		
	
		
	
      public:
		 AlmacenaUsuarios();
		~ AlmacenaUsuarios() {

			for (int i = 0; i<1000; i++){
				arrayNombres[i]->clear();
			}

		 }

El constructor es este:

Código:

AlmacenaUsuarios::AlmacenaUsuarios(){

	for (int i = 0; i<1000; i++){
		arrayNombres[i] = new list<Usuario>;	
	}

	list<Usuario>::iterator it;	//Iterador de la lista general de cada posición
	

}

¿Ese destructor borraria cada posición del arrayNombres?


Por si hace falta, en la clase "miTipo" no he puesto ningún destructor, pero no sé si hace falta especificar alguno. La clase "miTipo" se crea así:

Código:

miTipo::miTipo(){
	
	this->tipoMensaje = "NULL";
	this->nombre = "NULL";
	this->texto = "NULL";
	this->fechaHora = new FechaHora();	
}

FechaHora, a su vez, se crea así:

Código:

FechaHora::FechaHora(){

	this->fecha = "NULL";
	this->hora = "NULL";
}

[alexg88]

Si usas new para crear, tienes que usar delete para destruir, es decir:

Código C++:

class AlmacenaUsuarios {
 
      private:
        list<Usuario> * arrayNombres[1000];
        list<Usuario>::iterator it;
        
    
        
    
      public:
         AlmacenaUsuarios();
        ~ AlmacenaUsuarios() {
 
            for (int i = 0; i<1000; i++){
                delete arrayNombres[i];
            }
 
         }
 

[varuk]

Ya lo solucioné.

Es que antes tenía puesto eso que tu dices y no me iba bien pero lo puse así:

Código:

for (int i = 0; i<1000; i++){
				arrayNombres[i]->clear();
			}

Ya que arrayNombres son punteros a listas.

[alexg88]

Pero es que tal y como lo haces estás dejando unas lagunas de memoria, aunque no las veas.

Ten en cuenta que cada vez que haces un new, tienes que hacer un delete para liberar la memoria.

EDITO:

Tienes que hacer ambas cosas:

Código C++:

for (int i = 0; i<1000; i++){
                arrayNombres[i]->clear();
delete arrayNombres[i];
            }