Discussion :

[CedricMocquillon]

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

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

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

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?

[agamitsudo]

Bonjour,

Tu écris :

Code :

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

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

Alors que tu pensais certainement à :

Code :

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

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

---

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

Code :

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

listB::iterator

Peut être faudra-t'il écrire :

Code :

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

vector<B>::iterator

Cordialement,
Agamitsudo.

[3DArchi]

Probablement :

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
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;
}

[Spout]

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

Code :

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

listB::iterator

Peut être faudra-t'il écrire :

Code :

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

vector<B>::iterator

listB::iterator fonctionne très bien.

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

[3DArchi]

Effectivement.
Le code suivant plante:

Code :

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

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 :

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

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?

[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