iteration sur un conteneur membre

Version imprimable

Bonjour à tous, j'ai un problème: j'ai une class A qui contient un vecteur d'éléments de type B:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 #include "B.h" #include <vector> class A { public typedef std::vector<B> listB; private: listB m_lstB; public: listB::iterator beginB() throw() { return m_lstB.begin(); } listB::iterator endB() throw() { return m_lstB.end(); } };
mon problème est lorsque j'itère sur les éléments de B, en mode debug sous visual 2005, j'ai une erreur qui me dit en gros que l'iterateur n'a pas de conteneur:
Code:

1 2 3 4 5 6 7 8 9 10 11 #include "A.h" int main() { A a; A::listB::iterator it = a.beginB(); for (; it!=a.endB(); ++it) std::cout << *it << std::endl; return 0; }
Avez vous déjà eu ce type d'erreur, j'ai l'impression que c'est lié au fait qu'il y a une copie lors du renvoi de l'iterateur de début et de fin dans mes méthodes beginB et endB. Je n'utilise peut être pas la bonne méthodologie pour permettre d'itérer sur mon conteneur?

29/07/2008, 09h30
agamitsudo
Bonjour,

Tu écris :
Code:

1 2 3 4 listB::iterator beginB() throw() { return listB.begin(); }
Alors que tu pensais certainement à :
Code:

1 2 3 4 listB::iterator beginB() throw() { return m_lstB.begin(); }
---

2e point à vérifier, je ne suis pas sur que :

Code:

listB::iterator

Peut être faudra-t'il écrire :

Code:

vector<B>::iterator

Cordialement,
Agamitsudo.

Probablement :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
class A
{
public:
  typedef std::vector<B> listB;
private:
  listB m_lstB;
public:
  listB::iterator beginB() throw()
  {
     return m_lstB.begin();
  }
 
  listB::iterator endB() throw()
  {
     return m_lstB.end();
  }
};

int main()
{
  A a;
  A::listB::iterator it = a.beginB();
  for (; it!=a.endB(); ++it)
    std::cout << *it << std::endl;
  return 0;
}

29/07/2008, 09h39
Spout

Citation:

Envoyé par agamitsudo

2e point à vérifier, je ne suis pas sur que :

Code:

listB::iterator

Peut être faudra-t'il écrire :

Code:

vector<B>::iterator

listB::iterator fonctionne très bien.
29/07/2008, 09h42
CedricMocquillon

Effectivement vous avez raison pour le m_lstB, j'édite tout de suite mon premier post, mais c'est juste une erreur d'inattention. L'erreur est ailleurs...

Effectivement.
Le code suivant plante:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
  A a;
  B b;
 
  A::listB::iterator it = a.beginB();
 
  a.push_back(b);
  a.push_back(b);
  a.push_back(b);
  a.push_back(b);
 
  for (; it!=a.endB(); ++it)
    std::cout << *it << std::endl;
  return 0;

Par contre le code suivant fonctionne:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
  A a;
  B b;
 
  a.push_back(b);
  a.push_back(b);
  a.push_back(b);
  a.push_back(b);
 
  A::listB::iterator it = a.beginB();
 
  for (; it!=a.endB(); ++it)
    std::cout << *it << std::endl;
 
  return 0;

Aurais-u dans ton main ajouter les éléments entre l'obtention de l'itérateur et son utilisation?

29/07/2008, 10h08
CedricMocquillon

En fait, il semblerait que mon erreur survienne sur un code plus complet que celui que j'ai posté, je n'ai pas visual 2005 à mon travail, je vérifierai ce soir en essayant de poster un code minimal d'erreur (l'exemple que j'ai posté fonctionne très bien sur visual 2003). Sinon pour ce qui est de ton exemple qui plante 3DArchi, c'est normal puisque lors d'un push_back sur un vecteur, il peut y avoir réallocation et donc l'iterateur de début est invalidé, mais ce n'est pas de là que provient mon erreur. Je vous tiens au courant ce soir en tout cas merci