Discussion :

[ke2007]

Bonjour,

Je suis sous VS2008 et dans mon programme, j'obtiens l'erreur:

Erreur 1 error C3892: 'this'*: vous ne pouvez pas assigner une variable const c:\....\caserailstation.cpp 37 monop2

En gros, il me dit que je peux pas modifier la valeur d'une constante (ici this) ce que je comprend...

Mais ce que je comprend pas, c'est d'o� vient le probl�me.
J'ai 2 classes l'une h�ritant de l'autre.

Classe 1:

Code :

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
18
19
20
21
22
23
24
25
26
27
 
class CaseProperty :
	public Case
{
protected:
	int pricePurchasing; // prix d'achat
	int priceMortGage; // prix d'hypothèque
	std::string name;
	Player *owner;
 
public:
	CaseProperty(std::string = "", int = 0, int = 0);
	virtual ~CaseProperty(void);
 
	virtual int getRent() const = 0;
 
	int getPricePurchasing() const;
	int getPriceMortGage() const;
	std::string getName() const;
	Player* getOwner();
 
	void setOwner(Player* p);
 
	bool isOwned() const;
 
	bool operator ==(const CaseProperty&);
};

Classe 2:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
 
class CaseRailStation :
	public CaseProperty
{
public:
	CaseRailStation(std::string = "", int = 0, int = 0);
	~CaseRailStation(void);
 
	std::string getType() const;
	void onCase(Player& p) const;
	int getRent() const;
};

Le probl�me survient dans la m�thode onCase qui est une m�thode virtual pure de la classe m�re Case.
Sa d�finition est:

Code :

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
18
19
20
21
22
23
24
 
void CaseRailStation::onCase(Player &p) const
{
	if(this->isOwned())
		cout << "Vous devez payer à " << this->owner->getName() << endl;
	else 
	{
		cout << "Voulez vous acheter la " << this->name << " au prix de " << this->pricePurchasing << " F? (y/n)" << endl;
		string res;
		do
		{
			cout << "votre reponse: ";
			cin >> res;
			cout << endl;
		}
		while(res != "y" && res != "n");
 
		if(res == "y" && p.getCash() >= this->pricePurchasing)
		{
			p.retryCash(this->pricePurchasing);
			this->owner = &p; // L'erreur est sur cette ligne !!!!!
		}
	}
}

Je ne vois pas pourquoi je n'aurais pas le droit de modifier la valeur de la propri�t� "owner". Elle est prot�g� et non priv� dans la classe CaseProperty.

Si quelqu'un pouvait m'aider et m'expliquer mon probl�me ce serait super
Merci d'avance !!

[JulienDuSud]

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
 
	void onCase(Player& p) const;

En pla�ant le const ce dette mani�re, tu dis que cette m�thode ne modifiera pas l'�tat de l'instance courrante de ta classe.

Or, c'est ce que tu fais dans cette m�me m�thode en tentant de modifier l'�tat de ton instance en modifiant l'un de ses attributs.

[ke2007]

Ok merci.

Mais comment je peux modifier mon attribut?

J'ai tent� de mettre une m�thode setOwner(Player& p) dans ma classe CaseProperty.
Cette m�thode doit �tre commune � toute les classes filles et n'a pas besoin d'�tre red�finie.

Mais quand je l'appel dans ma m�thode de CaseRailStation, il me met l'erreur:

Erreur 1 error C2662: 'CaseProperty::setOwner'*: impossible de convertir un pointeur 'this' de 'const CaseRailStation' en 'CaseProperty &' c:\...\caserailstation.cpp 37 monop2

[JulienDuSud]

Tu refais exactement la m�me chose que ton probl�me original, tu ne fais que le d�placer

Dans une m�thode const, ton pointeur this est en r�alit� un constant. Tu ne peux donc pas le modifier directement. Pour pouvoir le modifier, tu dois passer par une m�thode non constante.

Pour r�soudre ton probl�me, il suffit de rendre ta m�thode onCase non constante, c'est � dire enlever le const qui se situe � la fin de la d�claration de ta classe. Si cette m�thode modifie l'�tat de ta classe, elle n'est pas constante.

Par contre, tu as un effet de bord: cette m�thode ne pourra plus �tre utilis�e sur les objets constants ! (ce qui est logique)

[ke2007]

Ok, mais ma m�thode est m�thode virtual pure, donc elle est d�clar� comme ceci dans la classe m�re Case:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
 
virtual void onCase(Player& p) const = 0;

Je suis oblig� d'avoir un const non?

[JulienDuSud]

Si ta classe d�rive d'une certaine interface ou classe qui d�finis cette m�thode comme const (ce qui n'�tait pas le cas dans le code que tu avais montr�), alors oui, tu as tout int�r�t � ce que cette m�thode soit const. Tu as alors un probl�me dans le r�le de ta m�thode qui est ambig�e. Modifie-t-elle ou ne modifie-t-elle pas l'�tat courant ? Il faut le savoir. Est-ce que ta classe est elle r�ellement du type de son parent ?
Tu peux �galement surcharger ta m�thode afin de ne pas avoir de probl�me d'abstraction, et avoir d'un c�t� ta m�thode const qui pourra �tre appell�e par les classes manipulant des objets du type parent, et avoir une m�thode non const pour les classes manipulant directement l'objet et ayant besoin de la m�thode non const. Cependant, tes 2 m�thodes ne feront strictement pas la m�me chose, et c'est un probl�me de design r�ellement important.

[ke2007]

Ouai je pense que je vais revoir la conception � ce niveau...
Mais �a m'embete que je puisse pas modifi� l'attribut, m�me en appelant une autre m�thode qui le fasse...

En tt cas merci pour la r�ponse �a ma aid� � comprendre le bug

[white_tentacle]

Mais �a m'embete que je puisse pas modifi� l'attribut, m�me en appelant une autre m�thode qui le fasse...

Et pourtant, c'est tout � fait logique. En d�clarant ta m�thode "const", tu t'engages � ne pas modifier l'objet lors de l'appel. Et le client (celui qui appelle ta m�thode) se repose sur cet engagement. Si tu pouvais rompre ton engagement comme si de rien �tait, autant ne pas prendre d'engagement du tout, il ne servirait plus � rien.