Discussion :

[rouliane]

Bonsoir,

J'ai crée une classe vecteur suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
class Vecteur
{
public:
	Vecteur(double a=0., double b=0., double c=0.){x=a;y=b;z=c;} // constructeur par défaut
	Vecteur operator +( const Vecteur & );
	Vecteur& operator *( double ) ;
	void affiche() { cout<<"("<<x<<","<<y<<","<<z<<")"<<endl;}
private:
double x,y,z; // coordonnees
};

Seulement j'ai un problème pour surcharger l'opérateur *, qui sera ici la multiplication d'un vecteur par un double.

J'ai comme fonction :

Code :

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

1
2
3
4
5
6
7
Vecteur& Vecteur::operator * ( double a)
{
	x=a*x;
	y=a*y;
	z=z*a;
	return *this;
}

Seulement, avec ce code, je peux seulement multiplier à droite par un double, et pas à gauche :

Comment faire pour pouvoir multiplier par un double à la fois à droite et à gauche ?

Merci beaucoup.

[Davidbrcz]

Tu ne peux pas comme ca.
C++ ne fait aucune prédiction sur la commutativité d'un opérateur.
=> 2 Solutions :
crée un opérateur* avec ses paramètre en position inversé comparé au 1er
Jouer sur les casts.

Personelement, je préfère le 1 car le 2 peux avoir des concéquences néfastes ailleurs

[Médinoc]

/!\ ATTENTION ton code actuel confond les opérateurs * et *= /!\

Typiquement, pour la commutativité il faut mettre l'opérateur hors de la classe:

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
Vecteur& Vecteur::operator *= ( double a)
{
	x=a*x;
	y=a*y;
	z=z*a;
	return *this;
}
Vecteur operator * (Vecteur const &v, double a)
{
	Vecteur ret(v);
	ret *= a;
	return ret;
}
Vecteur operator * (double a , Vecteur const &v)
{
	return v * a;
}

[rouliane]

Merci pour vos réponses.

Medinoc >> Je ne comprends pas pourquoi mon code confond les opérateurs * et *= ??
Si je tape :

Code :

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

1
2
3
4
Vecteur v1(1.,1.,1.);
Vecteur c;
c=v1*2;
c.affiche();

J'obtiens bien le vecteur (2,2,2).

Davidbrcz >> J'avais effectivement pensé à inverser mais types mais je ne vois pas comment, parce que quand on écrit :

Code :

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

Vecteur Vecteur::operator * ( type )

J'aurais de toute façon un type Vecteur à gauche du * ( par défaut ? ) donc je vois pas comment changer ça

[Médinoc]

Ton code pour l'opérateur * modifie le vecteur, c'est donc l'équivalent d'un opérateur *=...

[rouliane]

ah d'accord, je viens de comprendre !

J'avais pas compris que quand on écrit Vecteur::operator * , on modifie le vecteur.

Merci en tout cas !

[rouliane]

J'ai un petit problème par contre, j'essaye de faire l'addition de 2 vecteurs, j'ai donc rajouté dans ma classe la ligne :

Code :

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

friend Vecteur operator + (  const Vecteur & ,  const Vecteur & );

et hors de la classe la fonction :

Code :

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

1
2
3
4
5
6
7
8
Vecteur operator + (  Vecteur &v1,  Vecteur &v2)
{
	Vecteur res;
	res.x=v1.x+v2.x;
	res.y=v1.y+v2.y;
	res.z=v1.z+v2.z;
	return res;
}

Mais le compilateur me dit qu'il ne peut pas accéder à x, y et z car ils sont privés. Mais je ne comprends pas le problème vu que j'ai définie la fonction comme amie ?