Discussion :

[je®ome]

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]

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]

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]

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]

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]

#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]

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]

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