Discussion :

[franc82]

le code suivant est-il bon pour la memoire? En fait je voudrais si a la fin du programme, la destruction du pointeur *ps me garanti qu'il n'y a pas de memory leak?

Code :

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

1
2
3
4
5
6
7
 
void Initialize()
{
    ps = new CSmoke(CVector(0,0,0), LoadTextureWithAlphaMask("particle.bmp", "particle.bmp"));
    ps->Initialize();
    //delete ps;
}

[Mat.M]

Si tu fais à la fin du programme delete ps cela suffit car CVector(0,0,0) est comme une variable locale déclarée sur la pile et non un pointeur déclaré sur le "tas" ( ou heap en anglais ) .
Cela revient à déclarer :

CVector vect(0,0,0);
ps = new CSmoke(vect, LoadTextureWithAlphaMask("particle.bmp", "particle.bmp"));
ps->Initialize();
//....
delete ps;
ps=NULL;

Il faut voir si la texture chargée avec LoadTextureWithAlphaMask est bien effacée après

[franc82]

mais il y a appel recursif dans la definition de ps, puisque ps fait d'abord un new puis un appel a la fonction qui l'a permis d'avoir de la memoire sur le heap

[loufoque]

Pourquoi tu déclares pas ton CSmoke sur la pile ?

[franc82]

parce qu'il va utiliser de la memoire dynamique.

[Luc Hermitte]

Et ?


Au fait, si ta fonction initialise ne réalise pas de la post-construction polymorphique, elle n'a pas à exister. Oui, j'ai une dent contre ce genre de code

Code :

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

1
2
3
4
T * p = new T();
p -> init("toto", 42); // lève une exception
code quelconque...
delete p;

alors que le rôle des constructeurs est ... de construire les objets pour les amener dans un état valide et exploitable.

PS: il n'y a pas que la mémoire qui peut fuir. Les OS savent récupérer la mémoire non désallouée, mais pas les autres ressources. Du coup, certains constructeurs ne doivent en aucun cas être oubliés.