push_back vecteurs privés avec 2 méthodes, on ne travaille pas

Question

J'ai une classe avec un vecteur privé de doubles. Pour accéder ou modifier ces valeurs, tout d'abord je des méthodes telles que

void classA::pushVector(double i) 
{ 
this->vector.push_back(i); 
} 
double classA::getVector(int i) 
{ 
return vector[i]; 
} 

Cela a fonctionné pendant un certain temps jusqu'à ce que je trouve que je dois surcharger beaucoup d'opérateurs pour ce que je avais besoin, j'ai donc essayé de le changer pour obtenir et définir le vecteur directement au lieu des valeurs, à savoir

void classA::setVector(vector<double> vector) 
{ 
this->vector = vector; 
} 
vector<double> classA::getVector() 
{ 
return vector; 
} 

maintenant, dire qu'il ya un ClassB, qui a un élément de CLASSA privé, qui a également méthodes get et set pour lire et écrire. Le problème était quand j'ai essayé de repousser une valeur au vecteur final dans la classe A.

void classB::setFirstValue(double first) 
{ 
this->getClassA().getVector().push_back(first); 
} 

Cela ne fait absolument rien au vecteur. Il reste inchangé et je n'arrive pas à comprendre pourquoi ... Des idées?

Answer 1

Vous renvoyez le vecteur par valeur dans votre getVector(). Cela signifie que dans votre appel "this-> getClassA(). GetVector(). Push_back (first);", vous copiez le vecteur, puis vous appuyez sur l'élément sur la copie. La copie est ensuite immédiatement rejetée.

Afin de le faire fonctionner comme vous le souhaitez, vous devez renvoyer le vecteur par référence. Votre méthode ressemblera:

"vecteur & CLASSA :: getVector()"

Answer 2

Dans cette méthode:

void classA::setVector(vector<double> vector) 
{ 
this->vector = vector; 
} 

vous passez une copie du vecteur. Pour passer le vecteur réel, vous avez besoin d'une référence:

void classA::setVector(vector<double> & vector) 
{ 
this->vector = &vector; 
} 

et la variable vecteur dans votre classe devra être un pointeur.

Tout cela me semble plutôt indésirable - je repense à ce que fait votre classe. Il est beaucoup plus logique pour la classe de gérer le vecteur que de maintenir un pointeur vers l'extérieur de la classe. et si vous avez besoin de ré-implémenter toutes les méthodes de classe, peut-être n'avez-vous pas du tout besoin de classe, mais juste un vecteur.