L'utilisation de cette "fonction de raccourci" est-elle une bonne pratique?

Question

I ont un système de communication basé sur des événements , ce qui est la version simplifed de celui-ci:

classe A: contient des éléments issus de la classe classe B

B: a un pointeur sur son propriétaire un

Alors, quand B veut communiquer avec d'autres hôtes, il appelle une émission () méthode de:

• m_owner-> Broadcast();

Mais je me demandais: méthode B, qui est juste ce

• Dois-je faire une protégée Broadcast():
  • m_owner-> Broadcast();

Plus:

• Au lieu de m_owner-> Broadcast() Je peux écrire simplement de diffusion()

  • ce qui rend le code plus propre.

Moins:

• Il y aura une fonction d'appel dans la procédure.

  • Mais cela peut être évité en faisant la méthode en ligne

Est-ce une bonne pratique? Pourquoi oui et pourquoi pas?

Answer 1

Dans une situation que vous avez décrite, cela ne fait probablement pas beaucoup de différence. Mais à l'avenir, il se peut que vous deviez ajouter du code supplémentaire à chaque endroit où est appelé Broadcast, par exemple la journalisation ou le verrouillage de mutex. Dans ce cas, une fonction séparée sera vraiment utile.

De même, vous mentionnez que vous avez des classes dérivées de B. S'il s'agit d'une classe dérivée qui appelle m_owner->Broadcast() et m_owner est un champ de classe de base, ce n'est pas un bon exemple.Les classes dérivées devraient mieux accéder aux fonctions protégées du parent, pas directement aux membres de données.