Discussion :

[vdumont]

Alors j'ai une classe MyFrame dans laquelle j'aimerais faire.

Code :

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

1
2
3
4
5
6
7
8
     std::vector<MG_NOEUD*> coord;
     for (unsigned int j = 0 ; j < mai->get_nb_mg_noeud() ; j++) {
          MG_NOEUD* no = new MG_NOEUD(*mai->get_mg_noeud(j));
          coord.push_back(no);
     }
        m_glCanvas->PushNoeuds(coord); // m_glCanvas == Objet GL_MAILLAGE
       coord.clear();

Code :

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

1
2
3
4
5
void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> &noeuds) 
{ 
  coord_noeuds.push_back(noeuds); 
}

Cependant le programme plante à l'exécution donc je crois que je fais quelque chose de pas correcte avec les références

Pourtant: je le passe par référence et le push_back fait une copie donc ca devrait être correct non!?

Ca marchait avant car ma classe était friend et je faisais

Code :

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

1
2
 
m_glCanvas->coord_noeuds.push_back(coord);

Mais j'aimerais me débarasser de ce friend.

[bigboomshakala]

hello,

on ne peut pas faire de tableaux de références.
tu peux utiliser les pointeurs ici.

Code :

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

1
2
3
4
 void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> * noeuds) 
{ 
  coord_noeuds.push_back(*noeuds); 
}

ou

Code :

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

1
2
3
4
 void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> noeuds) 
{ 
  coord_noeuds.push_back(noeuds); 
}

[vdumont]

Donc si je comprend bien:


Ici PushNoeuds reçoit un pointeur (qui pointe sur le vecteur coord passé en paramètre)


Mais quand on fait &noeuds on se trouve à insérer la référence non? Pourquoi ne fait-on pas push_back(*noeuds) (Ce sur quoi le pointeur pointe) ???

Code :

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

1
2
3
4
5
6
7
8
 
void PushNoeuds(const std::vector<MG_NOEUD*> *);
 
 
void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> * noeuds) 
{ 
coord_noeuds.push_back(&noeuds); 
}

Code :

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

1
2
 
m_glCanvas->PushNoeuds(coord);

Il me dit no matching function for call to `GL_MAILLAGE:ushNoeuds(std::vector<MG_NOEUD*, std::allocator<MG_NOEUD*> >&)'


Y'a que la fonction PushNoeuds s'attend a avoir un pointeur alors que moi j'lui donne a bouffer un vecteur normal...

[Jan Rendek]

on ne peut pas faire de tableaux de références.

Je ne vois pas de tableau de référence dans le code donné.

Quel est le type de coord_noeuds ?

[vdumont]

std::vector<std::vector<MG_NOEUD*> > coord_noeuds

[Matthieu Brucher]

C'est un tableau de pointeurs, pas de références, ça.

[vdumont]

Ok la ca marche après avoir essayer quelques trucs.
Mais j'ai de la légère difficulté à comprendre le mécanisme.

Code :

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

1
2
3
4
5
6
 
m_glCanvas->PushNoeuds(&coord);
 
void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> * noeuds) { 
   coord_noeuds.push_back(*noeuds); 
}

En envoyant &coord en paramètre, on se trouve à passer sa référence.
Donc la fonction PushNoeuds, recoit un pointeur. Donc le pointeur noeud pointe à l'adresse mémoire de coord (&coord).

Lorsque je push_back ensuite je fais *noeuds pour aller chercher la valeur pointée? Donc je push_back une copie de l'élément pointé?

J'ai bien comprit ou j'y suis pas du tout?

[bigboomshakala]

Je ne vois pas de tableau de référence dans le code donné.

Quel est le type de coord_noeuds ?

et ça ?

Code :

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

1
2
3
4
void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> &noeuds) 
{ 
  coord_noeuds.push_back(noeuds); // noeud est une référence
}

[Matthieu Brucher]

Code :

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

1
2
3
4
5
6
 
m_glCanvas->PushNoeuds(&coord);
 
void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> * noeuds) { 
   coord_noeuds.push_back(*noeuds); 
}

En envoyant &coord en paramètre, on se trouve à passer sa référence.

En fait, ce que tu mets dans le conteneur coord_noeuds, c'est une copie du vecteur pointé par noeud. Autan passer à la fonction une référence constante plutôt qu'un pointeur !

[vdumont]

C'est ce que je voulais faire initialement en faisant

Code :

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

1
2
3
4
void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> &noeuds) 
{ 
  coord_noeuds.push_back(noeuds); 
}

Mais ca plante à l'exécution

[Laurent Gomila]

bigboomshakala -> ce n'est pas parce qu'on passe le paramètre par référence, que c'est une référence qui sera stockée dans le tableau.

Mais ca plante à l'exécution

Vive les debuggers...

Ceci-dit le problème vient certainement d'ailleurs, car il est clair qu'entre le passage par adresse et le passage par référence, il y n'y a aucune différence.

[Jan Rendek]

Pour moi la fonction PushNoeuds est parfaitement correcte, si ça plante c'est que l'erreur est ailleurs.

@bigboomshakala

Code :

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

1
2
3
4
5
 
void GL_MAILLAGE::PushNoeuds(const std::vector<MG_NOEUD*> &noeuds) 
{ 
  coord_noeuds.push_back(noeuds);
}

Tous les passage de paramètres dans la STL se font par valeur. La conséquence de ce code est qu'une copie du vecteur passé par référence sera inséré dans coord_noeuds.