Surcharge opérateurs

Bonjour, j'ai un problème avec les surcharges d'opérateurs.

J'ai une classe vecteur:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

class vecteur { private: int taille ; float *tab ;   public: vecteur() ; vecteur(int); vecteur(vecteur&); ~vecteur(); ... }

---

puis :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

vecteur& vecteur::operator=(vecteur& v) { if (this!=&v){ if(tab!=NULL) delete []tab; if (taille!=0){ taille=v.taille; tab=new float[taille]; for(int i=0;i<taille;i++) tab[i]=v.tab[i]; }else tab=NULL; } return *this; }   vecteur vecteur::operator+(vecteur& v) { vecteur z(*this); for(int i=0;(i<taille)&&(i<v.taille);i++) z.tab[i]+=v.tab[i]; return z; }

---

lorsque je teste :

Code:

---

v3=v1+v2

---

le compilateur me dit no match for 'operator=' in 'v3 = vecteur::operator+(vecteur&)(((vecteur&)(&v2)))' 8O 8O 8O

c'est pareil avec v3=v1*v2, par contre si je fais v2=v1 ça marche très bien

Merci de votre aide !

v1 + v2 produit une variable temporaire, qui ne peut donc être acceptée comme paramètre pris par référence. Il faut prendre des références constantes.

Pareil pour ton constructeur par copie, d'ailleurs.

Ensuite tu devrais regarder la FAQ C++ pour l'opérateur d'affectation, il y a plein de bons conseils.

ça ne change strictement rien :roll:

bien le bonsoir,

comme l'a dit loulou24, ton constructeur par copie doit prendre en argument un const vecteur&. de même que l'opérateur = et l'opérateur+ . et si ça ne change rien, c'est peut-être que tu n'as pas reporté les modifications dans les deux fichiers .h et .cpp

bon j'ai réussi à m'en sortir je ne sais pas trop comment

merci

Ce serait bien si tu développais un peu les "ça ne change rien" et "je m'en suis sorti" avec du code, des explications ou des messages du compilo.