Discussion :

[poussinphp]

Bonjour,

J'ai crée une class qui fais un liste doublement chainé. Cette classe est en faite un monstre pour un jeu. J'ai une boucle principal de jeu qui affiche les monstres. Mais j'ai une fuite mémoire (quand je fais ctrl+alt+suppr je remarque que l'espace mémoire grandi au fur et à mesure.

J'ai essayé pas mal de méthode mais sans résultat.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void Cmonstre::afficher(SDL_Surface *surface)
{
	Cmonstre *monstre = new Cmonstre(this->fichier,this->tileH,this->tileL,this->positionX,this->positionY);
	*monstre = *this->suivant;
	this->image->afficher(surface,this->numeroImage,this->positionX, this->positionY);
	while(monstre->suivant != NULL)
	{
		monstre->image->afficher(surface,monstre->numeroImage,monstre->positionX, monstre->positionY);
		*monstre = *monstre->suivant;
	}
	delete monstre;
}

Je comprend pas d'ou peut venir le problème, je détruit bien l'objet à la fin et je fais pas de copie...

[JolyLoic]

Code :

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

1
2
 
	*monstre = *this->suivant;

je fais pas de copie...

Là tu ne fais pas une affectation de pointeurs, mais une affectation d'objets. Si tu veux être certain de ne pas faire de copie, déclare privés, sans les définir, les constructeurs de copie et opérateurs d'affectation de ta classe.

[Laurent Gomila]

A chaque fois que tu dois afficher un monstre (donc des dizaines de fois par seconde) tu en alloue un nouveau dynamiquement, tu y recopies ton monstre original, et pour finir ton monstre affiche les autres monstres qui lui sont reliés

Parfois on se demande comment les gens en arrivent à des trucs aussi tordus

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
class Cmonstre
{
public :
 
    void Afficher()
    {
        image->afficher(surface, numeroImage, positionX, positionY);
    }
};
 
std::list<Cmonstre> Monstres;
for (std::list<Cmonstre>::iterator It = Monstres.begin(); It != Monstres.end(); ++It)
{
    It->Afficher();
}