Discussion :

[donkeyquote]

Bonjour à tous !
Est-ce quelqu'un peut me dire pourquoi je trouve le message d'erreur :
"error: passing ‘const CompositeProduct’ as ‘this’ argument of ‘virtual double CompositeProduct::getPrice()’ discards qualifiers".
L'erreur se produit sur la sentence suivante:

Code :

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

1
2
 
std::cout<<"###"<<iComposite.getPrice()<<"###"<<std::endl;

La declaration de la fonction getPrice() est :

Code :

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

1
2
3
 
Public:
virtual double CompositeProduct::getPrice();

Comment je peut le corriger ?

Merci d'avance de votre aide !

[bolhrak]

Salut,

C'est simplement un problème de const. Ton paramètre iComposite est déclaré comme tel, mais tu appelles une fonction non const dessus, donc susceptible de le modifier.

Si ta fonction getPrice ne modifie pas l'instance sur laquelle elle est appelée, tu devrais la déclarer de la manière suivante :

Code :

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

1
2
 
virtual double CompositeProduct::getPrice() const;

Sinon il faut que tu enlèves le const devant la déclaration de iComposite.

[donkeyquote]

Le problème c'est que je ne peux pas le mettre come const parce que ma fonction getPrice() modifie un atribut de sa classe...

[bolhrak]

Lis la dernière phrase de mon post

Et un getter qui modifie un attribut de classe ne porte pas vraiment bien son nom :p

[nicroman]

Un getter qui modifie un attribut.... ben voyons....
A moins que ce ne soit un cache ? (genre on cache un accès couteux à une DB ou un site web pour calculer le prix), mais dans ce cas, l'attribut modifié est transient. Et le mot clef ad-hoc en C++ est mutable, et la fonction peut rester constante.

Dans tous les cas, Toto.getXXXX() ne devrait jamais modifier Toto, jamais !

Si ce n'est pas le cas... alors oui... c'est normal, tu ne PEUX pas appeler getPrice() sur un objet constant

Je suis un fourbe... je corrige inline

[Luc Hermitte]

s/volatile/mutable/