Discussion :

[betaplus]

Bonjour à tous,

Je reviens solliciter vos lumières car j'ai une petite question (j'apprends sur le tas et parfois, les livres et les forums ne suffisent pas...).

Pour décrire mon problème :
J'ai une classe (appelons là c1) qui contient une méthode qui prend des arguments par référence.
J'ai une seconde classe (appelons là c2) qui contient une surcharge de l'opérateur * (pour me permettre de faire une produit matrice*vecteur)

Je passe donc une instance de c2 à la méthode de c1 par référence. Dans cette méthode de c1 j'appelle l'opérateur * (surcharge dans c2) mais je me fait jeter par le compilateur qui me sort "no match operator * in..."
Je sais que c'est un peut tordu et que j'oublie certainement quelque chose d'important mais je n'arrive pas à savoir ce que c'est.

j'ai donc quelque chose qui ressemble à celà pour ma méthode de c1 :

Code :

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

1
2
3
4
5
6
7
8
void c1::setMatrix(c2& aC2)
{
<div style="margin-left:40px">vector<double> testVector;
//... je remplie ce vecteur
 
vector<double> bufferVector = aC2*testVector; //ceci me sort une erreur lors de la compilation
vector<double> bufferVector2 = c2(aC2)*testVector; //ceci fonctionne mais ce n'est pas ce que je veux faire</div>
}

à savoir que la surcharge fonctionne très bien lorsque je l'utilise directement sur l'instance de ma classe (comme mis en commentaire au dessus)

voici ce à quoi ressemble mon overload de * dans c2 :

Code :

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

1
2
3
4
5
6
vector<double> operator*(const vector<double>& aVector)
{
<div style="margin-left:40px">vector<double> toBeReturned;
//... je fais mon mic-mac
return toBeReturned;</div>
}

Je ne sais pas si tout ceci est bien clair, mais je suis preneur d'explications et d'une solution pour pouvoir utiliser mon passage par référence et ma surcharge.

Merci d'avance,

betaplus

[Bousk]

Salut,

essaye de passer ta méthode const, peut-être que c'est juste ça qui bloque la première version.

[therwald]

compilateur qui me sort "no match operator * in..."

ça aiderait de donner le message complet, vu qu'il donne des informations sur la recherche de la fonction à appeler, donc sur la raison pour laquelle il ne trouve pas...

[betaplus]

J'ai essayé sur une autre machine et voici l'erreur du compilateur :

Code :

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

error: invalid operands to binary expression ('const C2', and 'C1')

J'ai ensuite essayé de passé la méthode en const et là ça a marché (il me semblait avoir pourtant déjà essayé). Bref pour le moment ça semble fonctionner, j'en saurai plus lundi pour voir si ça fonctionne sur la première machine.

[Bousk]

Ton erreur est pourtant claire..
Tu as un const& C2 et ta méthode n'est pas const.

[betaplus]

Oui Bousk, c'est ce que j'ai compris hier... Tout marche maintenant.

Merci à tous !

[therwald]

Accessoirement, de toutes façons c'est mieux de déclarer const les méthodes qui ne modifient pas l'état de l'instance:
1) ça permet de détecter des bugs (une méthode qui n'est pas censée modifier l'instance mais qui la modifie quand même peut donner des effets de bord tordus...)
2) ça permet des optimisations