[cast] dans le cas d'une Std::list

Version imprimable

salut à tous,

l'erreur :
error C2440: 'cast de type'*: impossible de convertir de 'std::list<_Ty>::iterator' en 'CInstance' with [_Ty=CInstance]

mes definitions

Code:

1
2
3
4
5
 
typedef std::list<CInstance> type_listOfInstances;//mon type de list
	type_listOfInstances m_Instances;						//ma liste d instances
 
	type_listOfInstances::iterator m_InstanceIterator;		//un iterateur sur ma liste

mon code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
m_InstanceIterator=m_Instances.begin();
for(m_InstanceIterator;m_InstanceIterator!=m_Instances.end();m_InstanceIterator++)
{
	if (m_InstanceIterator->GetID()==_iNoInstance)
	{
	//suppresion de l'instance dans la liste
         m_Instances.erase(m_InstanceIterator);
	delete (CInstance) m_InstanceIterator;//ERREUR C2440
 
	return;
	}			
}

comment faire pour bien caster afin de supprimer l instance de la liste ??

23/10/2006, 11h16
ZaaN

résolu ! il me manquait un &

Code:

delete &m_InstanceIterator;
23/10/2006, 11h26
Médinoc

Euh... J'avoue que j'ai des doutes, là...
C'est on iterator sur la pile, que tu essayes d'effacer là...

Ta liste est une liste d'objets, non de pointeurs: Je ne pense pas que delete marche avec ce type de liste.
23/10/2006, 13h27
bolhrak

Citation:

Envoyé par Médinoc

Euh... J'avoue que j'ai des doutes, là...
C'est on iterator sur la pile, que tu essayes d'effacer là...

Ta liste est une liste d'objets, non de pointeurs: Je ne pense pas que delete marche avec ce type de liste.

Non en fait il n'y a pas besoin de détruire l'objet, vu qu'il s'agit d'un objet et non d'un pointeur, il est détruit au cours de l'opération erase, de manière automatique puisqu'il sort du scope.
23/10/2006, 13h37
loufoque

Et indexer ta liste par des IDs ça te semble pas plus pertinent ?
23/10/2006, 14h09
Matthieu Brucher

Pour supprimer l'élément de la liste, c'est plutôt un list.erase(pos) ?
23/10/2006, 15h10
bolhrak

Citation:

Envoyé par Miles

Pour supprimer l'élément de la liste, c'est plutôt un list.erase(pos) ?

Oui et c'est ce qu'il fait. Après lui veut également détruire l'objet, alors que cela n'est pas nécessaire s'il s'agit d'une liste d'objets et non de pointeurs.

Par contre je ne crois pas que l'itérateur soit mis à jour lorsqu'un élément est retiré de la liste, du coup le code risque de bugger (l'itérateur se retrouve après la fin de la liste au bout d'un moment :s).
23/10/2006, 15h11
Matthieu Brucher

Citation:

Envoyé par bolhrak

Oui et c'est ce qu'il fait. Après lui veut également détruire l'objet, alors que cela n'est pas nécessaire s'il s'agit d'une liste d'objets et non de pointeurs.

Ah oui, je n'avais même pas vu... Donc il essaie d'effacer un objet non alloué sur le tas... Gros crash en perspective.
26/10/2006, 09h27
ZaaN

en effet le delete est inutile dans cet ECM!

Mais j avais simplifier mon code pour produire cet ECM.
Dans ma version definitive ma list contient des ptr d'objet.

merci a tous
a+
26/10/2006, 10h07
Luc Hermitte

Ce n'est pas une question d'inutilité, il y est dangereux.