meilleure façon de Refactor Hiérarchie des classes

Question

J'ai une classe CGrandMother ayant une méthode publique virtual bool Compute() cela ne

virtual bool Compute() 
{ 
    return false; 
} 

De CGrandMother publiquement CMother qui dérive ne met pas en œuvre Compute. Maintenant de CMother dérivez publiquement C1 et C2 qui implémentent virtual bool Compute(). Maintenant, virtual bool C1::Compute() et virtual bool C2::Compute() respectivement faire beaucoup de choses propres à respectivement C1 et C2, mais aussi beaucoup de choses identiques à CMother. Maintenant, il y a une classe CFamily ayant en tant que membre d'un pointeur vers CMother et presque partout dans le code Compute est appelé par les lignes de la forme

ptrCMother->Compute(); 

Comment pourrais-je factoriser les choses communes liées à CMother fait dans C1 et C2 donc que je n'aurais pas à changer toutes ces lignes ptrCMother->Compute();?

Answer 1

La réponse devrait être assez simple. Vous avez dit "beaucoup de choses identiques à CMother". Donc, vous devriez les factoriser dans une fonction membre de CMother. Comme il semble que la fonctionnalité ne soit nécessaire que pour les classes dérivées de CMother, vous devez marquer la nouvelle fonction membre "protected". Ce que @ 0x5453 a dit est une manière; mais je suggère une nouvelle fonction, afin de laisser la méthode publique CMother :: Compute seule. Il pourrait y avoir une autre classe enfant de CMother qui n'implémente pas Compute et s'appuie sur CMother :: Compute pour faire certaines choses.