Discussion :

[trecks]

Bonjour à tous

Puiqu'un bout de code vaut mieux qu'une explication^^

Code :

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

1
2
3
4
5
6
7
8
9
10
 
CSpriteManager::~CSpriteManager()
{
    for(m_SpriteIter = m_vSprite.begin(); m_SpriteIter != m_vSprite.end(); ++m_SpriteIter)
    {
        delete (*m_SpriteIter);
        m_vSprite.erase(m_SpriteIter);
    }
	m_vSprite.clear();
}

Quand je quitte le programme voici le message d'erreur que j'obtiens :
Debug Assert Failed

Program : ...
File c:\program files\développement\microsoft visual studio 8\vc\include\vector
Line : 117

Expression "this->_Mycont != NULL", 0)
...

Apparemment il s'agit d'un problème avec mon conteneur vector m_vSprite

Quand je met en commentaire la ligne m_vSprite.erase(m_SpriteIter) le programme ne plante plus mais pourquoi dont?

[Mat007]

Salut,

La FAQ a justement une entrée sur le sujet.
Sinon le 'clear' ne sert à rien, en effet sitôt la fin du destructeur l'objet n'existe plus, ainsi donc que le vecteur qu'il aggrège.

MAT.

[trecks]

Et bien justement, ils disent que s'il s'agit d'un pointeur sur un objet alors il faut supprimer l'objet avec delete, or c'est ce que je fais, puis plus bas dans la FAQ ils disent aussi qu'il faut supprimer l'élément du conteneur vector avec une erase(), mais le problème est la si j'utilise erase() le programme plante

[Laurent Gomila]

La FAQ dit aussi qu'il faut récupérer le retour de erase pour mettre à jour l'itérateur.

Et note que là tu vas vider 3 fois ton vecteur : une fois avec les erase, une autre avec le clear, et une dernière avec le clear que le destructeur de vector fera automatiquement.

Code :

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

1
2
3
4
5
6
7
CSpriteManager::~CSpriteManager()
{
    for(m_SpriteIter = m_vSprite.begin(); m_SpriteIter != m_vSprite.end(); ++m_SpriteIter)
    {
        delete (*m_SpriteIter);
    }
}

Ceci suffit largement.

[trecks]

Ok bas merci ça marche niquel. Dis moi Laurent tu voudrais pas être mon prof?^^^^
A bientôt