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();
}