Discussion :

[betaplus]

Bonjour à tous,

Voici mon problème, je pense que vous pourrez m'aider.

Je retourne une méthode d'une de mes classes me retourne un vecteur. J'essaie ensuite d'y accéder par un itérateur. Dans un cas j'ai la bonne valeur et dans l'autre cas j'ai un 0. A savoir que le premier élément de mon vecteur est initialisé à 10. Cette erreur ne se produite seulement que sur le premier élément de mon vecteur et un accès par indice fonctionne correctement.

Pour exemple, voici mon code :

Code :

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

1
2
3
vector<double>::iterator ite(maClasse.maMethode().begin());
cout<<"cas 1 : "<<*ite<<endl; //me renvoie 0 alors que j'attends 10
cout<<"cas2 : "<<*(maClasse.maMethode().begin())<<endl; // j'obtiens bien 10

code de maMethode :

Code :

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

1
2
3
vector<double> maClasse::maMethode() const{
     return m_monVecteur;
}

Quelqu'un saurait-il m'expliquer ce qu'il se passe ?

Je vous remercie par avance

[ternel]

La, comme ca, je ne vois pas.
Par contre, si tu pouvais nous faire un exemple complet et compilable, on pourrait te dire.

C'est probablement un manque d'initialisation

[betaplus]

En fait je n'initialise pas mon vecteur mais je le remplis avec un push_back() avec des valeurs lues dans un fichier.
J'ai ce problème seulement avec la première valeur. Je compile avec un Makefile, aucune erreur ni warning ne s'affiche (j'ai bien ajouté le flag Wall).

Selon moi, le problème viendrait de mon compilateur (pour le moment GCC 4.1, j’essaierai avec une version plus récente chez moi ce soir, ou cet après midi si le service informatique me fait enfin cette mise à jour).

[Médinoc]

maClasse::maMethode() retourne un temporaire. begin() retourne quelque chose qui référence ce temporaire. Le temporaire est détruit à la fin de l'instruction...

[darkman19320]

Salut,

Je pense tout simplement que tu retourne une copie de ton vecteur et pas une référence constante: du coup tu créé un vecteur temporaire à chaque fois que tu appelles ta méthode mais tu ne stockes pas ce vecteur.

Edit: Medinoc tu as été plus rapide ^^

[betaplus]

Merci pour vos réponses.

Ça pourrait effectivement expliquer cela. Mais alors pourquoi cela fonctionne-t-il avec les éléments suivants de mon vecteur ?

[Médinoc]

Comportement indéfini: Tout peut arriver.

[ternel]

J'aurais réappris quelque chose. Merci d'avoir posé ta question!

[betaplus]

Ok ok, je pensais que les cases mémoires suivantes seraient rapidement réaffectées.

Merci à tous pour vos réponses !

[betaplus]

Petite précision si jamais d'autres se retrouvent dans le même cas que moi :
Dans mon cas, le mieux a été de renvoyer mon vecteur en constant par référence. Ce qui donne ceci :

Code :

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

1
2
3
const vector<double>& maClasse::maMethode() const{
return m_monVecteur;
}