Discussion :

[Staithes]

Bonjour,
j'ai une classe classeA et une classeB.
Dans classeA, j'ai :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
class ClasseA{
  list<ClasseB*> *ptrListe;
 
  ~ClasseA(){
    for( list<ClasseB*>::iterator i = ptrListe->begin(); i != ptrListe->end(); ++i){
	delete(*i);
  }
}
};

Je supprime bien le contenu des pointeurs contenu dans la liste, mais est ce que je dois rajouter ? :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

delete(ptrListe)

Est ce qu'il manque un truc?

Merci.

[3DArchi]

Salut,

Je supprime bien le contenu des pointeurs contenu dans la liste, mais est ce que je dois rajouter ? :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

delete(ptrListe)

Oui il faut aussi détruire ton pointeur sur la liste si tu l'as allouée dynamiquement (chaque new doit avoir son delete).

Est ce qu'il manque un truc?

Une politique de copie ? L'exception safety lors de leur allocation ? Bref, des pointeurs intelligents.
Pourquoi gérer la liste par pointeur ? Les conteneurs n'ont en général aucun intérêt à être gérer par pointeur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::list<ClasseB*> Liste

voir

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::list<std::shared_ptr<ClasseB>> Liste

[Staithes]

Merci.
Je retiens :

chaque new doit avoir son delete

Je suis partie sur cette idée et ca ne surcharge pas trop le code.