Discussion :

[camboui]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
struct S0
{
 ...
 virtual void Close();
 virtual ~S0() { Close(); }
};
 
struct S0D: S0
{
 ...
 virtual void Close() { ...; S0::Close(); }
};

J'ai une méthode virtual Close() dans une classe de base qui est appelée par son destructeur. La classe dérivée surcharge cette méthode Close() en faisant un traitement spécifique puis appelle la méthode Close() parente. Il me semble que c'est une manière assez classique de coder.
La problème survient quand on "oublie" de surcharger le destructeur de la classe dérivée. En fait je trouve assez pompant de devoir répéter le même code dans chaque destructor des classes dérivées, à savoir

Code :

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

 virtual ~S0_derived() { Close(); }

Quelles sont vos habitudes de conception dans ce genre de cas ?
Merci.

[3DArchi]

Quelles sont vos habitudes de conception dans ce genre de cas ?
Merci.

Ne pas appeler de méthodes virtuelles dans le constructeur et le destructeur.

[Médinoc]

Pour ça, je conseillerais de rendre le destructeur abstrait, pour forcer à le redéfinir.

[Davidbrcz]

Ne pas appeler de méthodes virtuelles dans le constructeur et le destructeur.

Tu peux appeler des fonctions virtuelles mais il n'y aura pas de polymorphisme, la résolution des liens sera statique (cf là).
D'où cette règle générale de ne pas le faire, mais du point de vue syntaxique, on peut le faire.

Par contre, appeler une fonction virtuelle pure donne un comportement indéfini.

Cf la norme en 10.4.6

[3DArchi]

Tu peux appeler des fonctions virtuelles mais il n'y aura pas de polymorphisme, la résolution des liens sera statique (cf là).
D'où cette règle générale de ne pas le faire, mais du point de vue syntaxique, on peut le faire.

Oui. Mais j'ai préféré être catégorique car
1/je n'ai jamais croisé de cas où c'était pertinent
2/ca va tellement à l'encontre de ce qu'on peut penser à priori, que je serais tenté de considérer cette utilisation même en connaissance de cause comme une faute.

[camboui]

OK, merci à tous.

[Noxen]

Code :

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

 virtual void Close() { ...; S0::Close(); }

Est-ce-que la méthode Close() peut-être appelée autre-part que dans le destructeur ? Si ce n'est pas le cas, ça n'a aucun intérêt de faire ça : le destructeur de la classe dérivée appellera celui de la classe mère, qui appellera sa méthode Close(). Et dans le cas présent, SO::Close() sera appelée 2 fois : 1 fois dans SOD::Close(), puis une deuxième fois dans le destructeur de la classe mère.