Discussion :

[Anonymouse]

Bonjour

J'ai créée une classe Wall qui contient un vecteur de vecteur d'objet "Brick" et une methode getConstWall() qui me retourne la référence du vecteur:

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
Wall::Wall(int numberW, int numberH, string path, int brickWidth, int brickHeight)
{
  for(int i=0; i < numberH; i++)
  {
    wall.push_back(vector<Brick>());
 
    for(int j=0; j < 1; j++)
    {
      wall.front().push_back
      (
        Brick(path, (brickWidth*j) + (brickWidth/2) , (brickHeight*i) + brickHeight/2, brickWidth, brickHeight)
      );
    }
  }
}
 
const vector<vector<Brick> >& Wall::getConstWall() const
{
  return wall;
}

Dans une autre classe je fais un appel à cette méthode pour récupérer un itérateur dessus:

Code :

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

1
2
3
4
 
  cout << "==== Debut ===" << endl;
  vector< vector<Brick> >::iterator itV = ((vector< vector<Brick> >)game.getConstBoard().getConstWall().getConstWall()).begin();
  cout << "=== Fin ===" << endl;

Or j'observe qu'a chaque appel de l'itérateur j'ai de multiples appels aux destructeurs des bricks (J'ai fait un print dans chaque destructeurs):

Je ne comprend pas pourquoi l'appel à la methode bgin provoque une recopie de mes objets? Je pourrais bien sur initialiser dynamiquement mes Briques mais je me demande si il n'y à pas une autre solution?

Merci d'avance de vos réponses.

[loufoque]

Quand tu fais ça

Code :

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

((vector< vector<Brick> >)game.getConstBoard().getConstWall().getConstWall())

Tu fais une copie.

La solution c'est d'écrire

Code :

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

game.getConstBoard().getConstWall().getConstWall()

[Nogane]

Bonjour,
Cet horrible cast pose problème: (vector< vector<Brick> >)

Je suppose que tu le fait car getConstWall renvoie un vector const, alors que ton iterator est non const.

Je voie deux bonnes solutions, et une mauvaise:
- Crée une fonction getWall() qui renvoie un objet non const.
- Declare vector< vector<Brick> >::const_iterator itV

Et la mauvaise solution: fait ton cast comme ça:

Code :

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

const_cast<vector< vector<Brick> >&>(game.getConstBoard().getConstWall().getConstWall())

Tu ne devrait jamais avoir a faire une chose pareil. C'est comme si le vector te disait:
- "Ne me touche pas!"
Et que toi tu lui répondais:
- "Si!! Hahaha!! Je vais te faire tout ce que j'ai envie!!"

... c'est un viol en fait...

[Anonymouse]

Ok merci de vos réponses le const_iterator fait exactement ce que je veux. Je retourne une constante car je veux éviter, par erreur, de modifier mon double vector en passant par cette méthode.

Je suis désolé que mon code aie choqué quelqu'un mais je début en C++ et j'ai vraiment du mal à manipuler la STL.

Merci de votre aide.