Discussion :

[gonzo59]

Bonjour,

j'ai un problème de platage lors d'un parcour de vecteur. ça ne plante que pour le suivant alors que j'utilise la même logique que pour le précédent. Le message d'erreur est "iterator not dereferencable" lors de l'affichage, je suppose que c'est parce que je sors du vecteur.... Logiquement ça devrait pas vu que je test si l'iterateur est egal à la fin du vecteur.

.cpp

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
16
17
18
 
vector<Perso*>::iterator conteneurp::suivant()
{
	if (ipb == dpperso.end())
		cout<<"Dernier perso!"<<endl;
	else
	ipb++;
	return ipb;
}
 
vector<Perso*>::iterator conteneurp::precedent()
{
	if (ipb == dpperso.begin())
		cout<<"Premier perso!"<<endl;
	else
	ipb--;
	return ipb;
}

main:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
vector<Perso*>::iterator ipb; //iterator de travail
	case '4':
		system("cls");
		ipb=cont.suivant();
		(*ipb)->AfficherPerso();
		break;
 
	case '5':
		system("cls");
		ipb=cont.precedent();
		(*ipb)->AfficherPerso();
		break;

Merci d'avance.

[koala01]

Salut,

Le problème, c'est que tu obtiens bel et bien un itérateur valide, mais pointant sur ... rien lorsque tu essaye d'accéder à tab.end() (ou tab est n'importe quelle collection STL)

Ce que tu dois donc faire, si tu utilise une des collection de la STL comme membre de classe, et que tu souhaite disposer d'un itérateur sur la collection disponible depuis l'extérieur (ce qui n'est, à bien y penser, pas forcément l'idéal ), c'est définir des fonctions membres qui renvoient le résultat de tab.begin() et tab.end() (en version const et / non const selon le cas), et vérifier que l'itérateur que tu as obtenu différent du résultat renvoyé par cette fonction.

Par exemple:

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
16
class MyClass
{
 
    typedef std::vector<Perso*> vector;
    public:
        typedef vector::iterator iterator;
        typedef vector::const_itertator const_iterator;
        /*je passe sur les constructeurs ;) */
        iterator begin(){return tab.begin();}
        iterator end(){return tab.end();}
        const_iterator begin() const{return tab.begin();}
        const_iterator end() const{return tab.end();}
        /* les différentes fonctions qui vont bien */
    private:
        vector tab;
};

(tu n'as même pas à t'inquiéter de permettre le passage à l'élément suivant ou à l'élément précédent, vu que ce sont les itérateurs qui s'en chargent)

Tu pourra utiliser ces fonctions sous une forme proche de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
/* soit mc est un objet de type MaClass */ 
for(MaClass::iterator it=mc.begin();it!=mc.end();++it)
{
    /* tout ce qui doit être fait */
}
/* ou, la version "constante" */
for(MaClass::const_iterator it=mc.begin();it!=mc.end();++it)
{
    /* tout ce qui doit être fait */
}
/* et tu pourra même envisager d'utiliser les fonctions qui manipulent des
 * intervalles
 */
MaClass::const_iterator result=std::find_if(mc.begin(),mc.end(),predicat);

[gonzo59]

Excellent! Merci beaucoup ça fonctionne

[koala01]

Au passage, et bien que cela n'ait rien à voir, méfie toi des collections qui manipulent des pointeurs "classiques" sur des objets...

C'est un gros tas de pièges dans lequel tu risque de te retrouver avec des fuites mémoire, des double libérations de la mémoire ou des pointeurs invalides en pagaille