Discussion :

[Ange_blond]

Bonjour,

J'ai un petit soucis de C++...

Ma classe hérite d'une classe abstraite. Certaines méthodes sont donc obligatoirement const. Dans ces méthodes je souhaite apeller une méthode private à moi... seulement cette derniere ne passe pas car elle n'est pas const, et donc le compilo refuse.

Je ne peux pas passer ma méthode en const elle fait au moins 200 lignes ça sera trop pénible ...

Y'a t il une feinte ? une idée ?

Merci.

[nikko34]

Ma classe hérite d'une classe abstraite. Certaines méthodes sont donc obligatoirement const.

hein? pourquoi?

[Médinoc]

@nikko34 : Sans doute possède-t-elle des méthodes virtuelles const que Ange_blond veut redéfinir...

@Ange_blond : Si ta méthode peut être const, alors elle doit l'être, je ne vois pas trop d'autre solution. De plus, quand une méthode fait 200 lignes, c'est souvent un signe qu'elle devrait être fragmentée un peu...

[Ange_blond]

Heuu faut partir du principe que c'est comme ça.
C'est du code déjà existant, je cherche juste à faire un test et donc je ne vais pas réécrire tout le code juste pour tester ça...

Si je ne les met pas en const :

Error : impossible d'instancier une classe abstraite
osgGA::MatrixManipulator::getInverseMatrix(void) const' : est abstrait

(sachant que ma classe herite de osgGA::MatrixManipulator)

donc pas tellement le choix... retour au probleme original...

[Médinoc]

Au pire, la solution la plus sale est de faire de la méthode const un wrapper qui fait un const_cast sur this pour appeler la méthode non-const, mais c'est vraiment dégueu...

[Ange_blond]

Au pire, la solution la plus sale est de faire de la méthode const un wrapper qui fait un const_cast sur this pour appeler la méthode non-const, mais c'est vraiment dégueu...

ce n'est que pour tester un effet... et voir si oui ou non une modif pourrait changer qqch... donc dans tous les cas ça sera retravaillé proprement par la suite... donc je susi preneur des trucs meme pas nets...

Comment je peux faire ça Médinoc ?

[Médinoc]

Code C++ sale :

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

1
2
3
4
5
6
7
8
9
TypeDeRetour TaClasse::getInverseMatrix(void) const
{
	return const_cast<TaClasse*>(this)->getInverseMatrix();
}
 
//Ta fonction de 200 lignes
TypeDeRetour TaClasse::getInverseMatrix(void)
{
}

Mais je rappelle, c'est vraiment sale. La seule chose plus sale que ça aurait été d'utiliser un cast C-style au lieu du const_cast.

[Ange_blond]

ca compile.

Merci bien, et désolé pour t'avoir fait sortir un code comme celui ci ^^

[nikko34]

ta méthode, elle ne peut pas être const non plus?

et si tu as des attributs private à toi, tu peux aussi les mettre "mutable" pour qu'ils puissent être modifiés par des méthodes consts.

[Médinoc]

et si tu as des attributs private à toi, tu peux aussi les mettre "mutable" pour qu'ils puissent être modifiés par des méthodes consts.

Mais à part pour des caches, des compteurs de références ou des compteurs d'appels, c'est généralement une mauvaise idée.