Discussion :

[Hans von Dall]

Bonjour, je dispose d'une classe dont un champ est un vecteur d'objets Voiture.

Code :

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

1
2
3
4
5
6
7
8
class Classa{
vector<Voiture> voitures;
 
 Classa(vector<Voiture>& v )
    {
        voitures = v;
    }
}

Lorsque j'exécute mon code, le vecteur voitures devient une copie du vecteur fourni en paramètre au constructeur. Hors ce n'est pas le comportement que j'attends. Je souhaites que voitures soit une référence à v et que lorsque j'opère des modifications sur un objet Voiture du vecteur voitures, les modifications s'opère sur les éléments du vecteur v passé en paramètre du constructeur de Classa.

Dois-je utiliser des pointeur (je désirerais conservé le champ vector<Voiture> voitures; dans ma classe classa plutôt que d'utiliser un pointeur de vector<Voiture> .


Débutant en c++, la notion de référence m'est quelque peu obscure.

Cordialement,

[jabbounet]

As tu jeté un oeil dans les FAQ C/C++ il y'a quelques explications sur le sujet.

[Hans von Dall]

As tu jeté un oeil dans les FAQ C/C++ il y'a quelques explications sur le sujet.

Oui mais j'ai quelques problèmes de compréhension. Avez-vous une suggestion pour résoudre mon soucis (qui au demeurant, me semble simple pour un habitué
du langage)?

[Dalini71]

Bonjour,

J'suis pas expert et j'ai pas tout compris, mais il semble que tu veuilles que les éléments de voitures pointent sur les éléments du vector v passé en paramètre.

Essayes ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class Classa
{
vector<Voiture*> voitures;
 
Classa(vector<Voiture>& v )
{
for (unsigned int i = 0; i < v.size(); ++i)
{
voitures.push_back(&(v[i]));
}
}
}

J'suis débutant aussi donc pataper si sa marche pas ^^.

[ymoreau]

Si j'ai bien compris tu veux stocker dans ta classe un vecteur de voitures mais qui correspondrait à un vecteur existant déjà hors de la classe. Et que lors de modifications sur l'un ou l'autre, les deux soient modifiés ?

Je trouve ça un peu étrange. Ton vecteur est censé être un attribut protégé par ta classe, il n'est pas logique de vouloir le rendre modifiable au même titre qu'un vecteur externe à la classe.

Si tu veux vraiment que tes vecteurs dans la classe et hors classe représentent la même chose, alors fais une référence sur le vecteur directement plutôt que sur les éléments qu'il contient.

Code :

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

1
2
3
4
5
6
7
8
class Classa
{
vector<Voiture>& voitures;
 
public:
Classa(vector<Voiture>& v ) : voitures(v)
{}
};

[Hans von Dall]

merci pour vos réponse.

Lorsque j'utilise votre code:

Code :

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

1
2
3
4
5
6
7
8
9
 
class Classa
{
vector<Voiture>& voitures;
 
public:
Classa(vector<Voiture>& v ) : voitures(v)
{}
};

La compilation (avec mingw32 ) ne passe pas, m'indiquant que la référence vector<Voiture>& voitures n'a pas été utilisée.

[ymoreau]

Ce n'était qu'un exemple basique pour le principe (à compléter donc), mon code tout seul ne sert strictement à rien. Tu peux voir que j'ai directement mis le code du constructeur dans la déclaration de la classe, mais dans ton vrai programme il faut séparer ça en un .h et un .cpp.

Est-ce une erreur ou un warning que tu obtiens ?
Car la donnée membre "voitures" n'est effectivement pas utilisée, c'est à toi d'ajouter les fonctions dans ta classe qui feront ce qu'elles ont à faire sur le vecteur de voitures.