Discussion :

[a_lincoln54]

Bonjour à tous,

Je voudrais savoir si quelqu'un sait comment utiliser les <vector> au sein de fonctions. Par exemple, soit la fonction :

Code :

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

1
2
3
4
5
6
7
 
void add(std::vector<int> vec, int z)
{
 
vec.push_back(z);
 
}

qui à un vecteur vec et un entier z associe vec(z). Cette fonction est compilable sans problème mais son execution et la récupération du résultat :

Code :

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

1
2
3
4
5
 
 
add(vec_1, 4);
 
cout << vec_1[0] << endl;

plante et génère une violation d'accès. Si quelqu'un a une idée pour pouvoir faire fonctionner cette procédure de manière normale, je le remercie par avance.

[koala01]

Salut,

C'est l'éternel problème posé par le passage d'arguments par valeur VS le passage d'arguments par référence

Telle que tu écris ta fonction, lorsqu'elle est appelée, il y a une copie de ton vecteur qui est créée, et c'est à cette copie de vecteur que tu applique la méthode push_back.

Manque de bol, cette copie de vecteur est détruite au sortir de la fonction, et n'a strictement aucun rapport avec l'objet d'origine

Pour résoudre le problème, rien de plus simple: passe ton vecteur sous la forme d'une référence (non constante, étant donné que tu veux, justement, le modifier).

Cela se fait très simplement en rajoutant une esperluette "&" entre le > de vector<int> et le v de vec

Au final, la fonction devient donc

Code :

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

1
2
3
4
5
6
void add(std::vector<int> & vec, int z)
{
 
vec.push_back(z);
 
}

[a_lincoln54]

OK, merci beaucoup pour votre réponse.

[befalimpertinent]

Au passage petite piqûre de rappel : Comment passer correctement des paramètres à ma fonction ?

[Sylvain Togni]

Attention, la dernière partie de cette FAQ est doublement incorrecte

Cette remarque vaut également pour les pointeurs :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void FonctionIncorrecte(char* s)
{
    // ...
}
 
void FonctionCorrecte(const char* s)
{
    // ...
}
 
FonctionIncorrecte("bonjour"); // Erreur : "bonjour" est un temporaire
FonctionCorrecte("bonjour"); // Ok

- "bonjour" n'est pas un temporaire mais un objet statique
- passer un temporaire par pointeur contant n'est pas valide