Discussion :

[ZaaN]

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 :

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

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 :

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

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 ??

[ZaaN]

résolu ! il me manquait un &

Code :

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

delete  &m_InstanceIterator;

[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.

[bolhrak]

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.

[loufoque]

Et indexer ta liste par des IDs ça te semble pas plus pertinent ?

[Matthieu Brucher]

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

[bolhrak]

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).

[Matthieu Brucher]

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.

[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+

[Luc Hermitte]

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