Discussion :

[Nasky]

Bonjour

J'ai un problème avec un overloading.

Dans un fichier matrix.cpp j'ai la ligne suivante :

Code :

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

d=d+(isEven(i)*mat.values[i][0]*det(subMat));

d est de type Complex.
isEven(i) retourne 1 ou -1 mais caster en double donc ça retourne un double en fait.
values[i][0] est de type Complex
det(subMat) est une méthode (récursive) qui retourne un Complex

Le problème, c'est qu'à la compilation, j'ai l'erreur suivante :

binary '*' : no operator found which takes a right-hand operand of type 'Complex' (or there is no acceptable conversion)
1> could be 'Matrix operator *(Complex,Matrix &)'
1> while trying to match the argument list '(int, Complex)'

Je ne vois pas pourquoi il me propose un overloading avec un type Matrix sachant que ça n'intervient pas dans cette ligne. Mais peut-être est-ce parce que cette ligne se trouve dans un Matrix.cpp ...

Bref, j'ai donc voulu overloader l'opérateur de multiplication pour qu'il comprenne l'opération : double * Complex (Complex * Complex étant déjà overloadé et marche).

J'ai fait donc dans le fichier Complex.cpp ceci :

Code :

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

1
2
3
4
Complex Complex::operator*(const double nb)
{
	return Complex(getRePart()*nb, getImPart()*nb);
}

Mais j'ai toujours la même erreur!

Sauriez-vous d'où ça provient ?

Merci

Nas'

[Nasky]

Bon, j'ai trouvé tout seul en fait

Il fallait aussi faire dans l'autre sens pour l'overloading c'est-à-dire ajouter :

Code :

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

1
2
3
4
5
6
friend Complex operator*(double, Complex &);
 
Complex operator*(double nb, Complex & cnum)
{
   return cnum * nb ;
}

Bon, là ça compile maintenant...

Problème résolu, vous pouvez supprimer si vous jugez la solution inutile sur le forum

Nas'

[Jean-Marc.Bourguet]

Il vaut en général mieux ne pas implémenter les opérateurs binaires comme fonctions membres (sauf les assignations). Sinon il y a une assymétrie qui se manifeste en particulier quand il y a des conversions implicites.

[Nasky]

Comment ça ? Il faut les créer en dehors de toute classe? Ca aurait changé quelque chose dans mon code (à part virer le Complex:: ) ?

Nas'

[Jean-Marc.Bourguet]

Comment ça ? Il faut les créer en dehors de toute classe?

Comme l'autre opérateur que tu as montré (en passant, pourquoi est-ce que le deuxième opérante est passé par référence non constante?).

Ca aurait changé quelque chose dans mon code (à part virer le Complex:: ) ?

Je n'en sais rien. Tu a des conversions implicites vers Complex ou pas?

[Laurent Gomila]

Attention aussi à ne pas tomber dans le piège du "friend" systématique pour les opérateurs non membres. Là ta fonction n'accède à aucune donnée privée de la classe, inutile de la rendre amie de celle-ci.