iterator de vector

**je®ome** · 27/10/2007, 15h54

Bonjour à tous,

Code :

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

1
2
3
4
5
6
7
vector<Ball*>::iterator it;
for(it=b.begin();it!=b.end();it++){
   Ball *sf=(*it);
   cout << "ball : " << sf->getName() <<endl; // fonctionne pas
}
Ball *sf1=b[0];
cout << "ball : " << sf1->getName() <<endl; // fonctionne

Je pense ne pas avoir bien compris le fonctionnement des iterator(s) de vector(s).
Quelqu'un peut-il me dire où est la différence, dumoins pourquoi le 1ier exemple ne marche pas.

Merci et bon weekend à tous

**Mat007** · 27/10/2007, 20h25

Salut,

Qu'entends-tu par ça ne fonctionne pas ?
Parce que ça l'air correct là comme ça en 3 lignes, même si en général on écrirait sans doute plutôt :

Code :

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

1
2
for(vector<Ball*>::const_iterator it=b.begin();it!=b.end();++it)
  ...

Mais c'est un détail à la limite...

MAT.

**je®ome** · 28/10/2007, 00h16

J'ai une erreur de segmentation sur l'un, et sur l'autre le nom est affiché comme souhaité.
je posterai l'erreur demain, car je n'ai pas envie de redémarrer mon ordi(Win/linux) et ai déjà modifié le code.
Demain, je mets l'erreur.
Bonne soirée à tous

**yan** · 28/10/2007, 12h05

Salut.
Tes bout de code sont correctes.
ton problème, ça sent l'erreur mémoire à plein nez..
Faudra le reste de ton code pour t'aider.
Aprés pourquoi

Code :

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

1
2
Ball *sf1=b[0];
cout << "ball : " << sf1->getName() <<endl;

fonctionne et non

Code :

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

1
2
3
4
for(it=b.begin();it!=b.end();it++){
   Ball *sf=(*it);
   cout << "ball : " << sf->getName() <<endl; // fonctionne pas
}

Parce que t'as de la chance (ou pas, a toi de voir).Mais l'autre plantera un jour ou l'autre.
Les erreurs mémoire génère des bug aléatoire. L'apparition du bug(qui est bien là) va dépendre si tu est en debug ou en release, de l'état de ta machine,...

**je®ome** · 28/10/2007, 15h15

Envoyé par Mongaulois

Parce que t'as de la chance (ou pas, a toi de voir).Mais l'autre plantera un jour ou l'autre.
Les erreurs mémoire génère des bug aléatoire. L'apparition du bug(qui est bien là) va dépendre si tu est en debug ou en release, de l'état de ta machine,...

J'avais pensé à un problème mémoire.
Il vient bien du contenu du vector, pas du vector lui-même ?

Voici ce que me dit gdb

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Program received signal SIGSEGV, Segmentation fault.
0x04ba40fa in std::basic_string<char, std::char_traits<char>, std::allocator<char> >::basic_string ()
   from /usr/lib/libstdc++.so.6
(gdb) up
#1  0x0804e727 in File::getName (this=0x0) at File.C:30
30              return this->name;
Current language:  auto; currently c++
(gdb) cont
Continuing.
 
Program terminated with signal SIGSEGV, Segmentation fault.
The program no longer exists.
(gdb) quit

**Mat007** · 28/10/2007, 15h21

Envoyé par je®ome

#1 0x0804e727 in File::getName (this=0x0) at File.C:30
30 return this->name;

Tu accèdes manifestement à un pointeur nul (this=0x0)...

MAT.

**yan** · 28/10/2007, 15h31

Envoyé par Mat007

Tu accèdes manifestement à un pointeur nul (this=0x0)...

MAT.

Je rajouterai que tu accède (surement) a une adresse memoire d'un element detruit.
Ca marche dans un cas car la memoire n'as pas encore été réutilisé(coup de chance)
Ilfaudrait plus de code pour t'aider

**je®ome** · 28/10/2007, 16h20

Re,

je suis en train de chercher avec valgrind, mais ce n'est pas du gateau.
Mon code est trop long et je ne peux (ne veux) pas le mettre.

dans Hes.h

Code :

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

1
2
3
4
5
6
7
class Hes
{
   private:
   ...
   vector<Ball*> b;
   ...
}

après dans une méthode de la même classe hes.C

Code :

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

1
2
3
 
   Ball *c=new Ball(name);
   b.push_pack(c);

après dans une autre méthode, je fais ce que je vous ai montré en début de post.

Il n'y a pas de constructeur à appeler pour un vector, ou si ?

Merci de votre aide

**yan** · 28/10/2007, 16h32

tu fait quoi entre la fonction ou il y as :
Ball *c=new Ball(name);
b.push_pack(c);

et le moment ou tu affiche?

**je®ome** · 28/10/2007, 16h36

Je lis d'un pipe.
Je crée un objet à partir de la chaine reçue du pipe.
Ce dernier recrée deux autres objets(en appelant leur constructeur) arrivés en sérialisé par le même pipe.

**je®ome** · 28/10/2007, 16h46

A présent je ne fais plus rien d'autre entre.
Par contre il y a d'autres processus crée avec des fork qui tourne en même temps.
Mais leur mémoire n'est pas partagée ......
Hmmm, ce n'est pas gagné

**yan** · 28/10/2007, 17h08

Envoyé par je®ome

Je lis d'un pipe.
Je crée un objet à partir de la chaine reçue du pipe.
Ce dernier recrée deux autres objets(en appelant leur constructeur) arrivés en sérialisé par le même pipe.

Es qut u ne ferait un truc comme

Code :

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

1
2
3
4
5
6
7
std::vector<AA*> vect1;
std::vector<AA*> vect2;
InitVect(vect1);
 
 
vect2 =vect1
deleteElement(vect1);

Ici vect2 a recopie les pointeur de vect1. Qui sont invalide, car on detruit les élément pointér par vect1 et donc par vect2

**je®ome** · 28/10/2007, 17h16

Non du tout, j'utilise un seul vecteur.
Un contenu lui est inséré dans une méthode appartenant à sa classe.
le traitement après est fait en appelant une méthode renvoyant ce vecteur, mais cela ne devrait pas jouer.

Il n'y a pas de constructeur à appeler pour un vector, ou si ?

**je®ome** · 28/10/2007, 17h35

Après d'autres essais, j'ai constaté qu'en ramenant tout dans la classe ou l'insertion se fait, ça fonctionne correctement. Ai-je quand même un problème mémoire ou pas ?
Comment savoir ?
valgrind ne me dit rien et gdb non plus.

**yan** · 28/10/2007, 18h01

Envoyé par je®ome

Non du tout, j'utilise un seul vecteur.
Un contenu lui est inséré dans une méthode appartenant à sa classe.
le traitement après est fait en appelant une méthode renvoyant ce vecteur, mais cela ne devrait pas jouer.

Il n'y a pas de constructeur à appeler pour un vector, ou si ?

normalement non

Envoyé par je®ome

Après d'autres essais, j'ai constaté qu'en ramenant tout dans la classe ou l'insertion se fait, ça fonctionne correctement. Ai-je quand même un problème mémoire ou pas ?
Comment savoir ?
valgrind ne me dit rien et gdb non plus.

je pense que t'as déplacer le problème (plus le même code généré)
Ca n'as peut etre rien avoir avec cette class ton erreur mémoire. Regarde ce que tu fait entre...
Par exemple, le nom est une string ou un tableau de char?

**je®ome** · 28/10/2007, 18h48

le nom en question est assigné à un string dans un constructeur.

Code :

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

1
2
3
4
5
6
7
8
class Req
{
   private:
     string name;
   public:
     Req(string n);
     string getName();
}

Code :

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

1
2
3
4
5
6
7
8
9
Req::Req(string n)
{
   name=n;
}
 
string Req::getName()
{
   return name;
}

L'appel est fait comme suit:

Code :

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

Req* req=new Req("nom");

ensuite c'est passé en paramètre à une méthode d'une autre classe.
c'est là que Ball est construit et inséré dans le vector.

Code :

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

Ball* b=new Ball(req->getName());

Un parcours du vector dans la même méthode affiche bien son contenu, le nom de req.
J'ai essayé avant de ne rien faire du tout entre, mais le résultat reste le même.
Je vais continuer à essayer de trouver avec valgrind.

Merci

iterator de vector

SL & STL C++

Discussions similaires

Partager

Partager