Est-ce une bonne pratique de mettre en œuvre la méthode modèle par modèle des événements C #?

Question

Je suis maintenant en train de comprendre un peu de code et je l'ai trouvé un modèle, qui semble un peu étrange pour moi. Il y a une classe de contrôle de l'utilisateur avec l'événement « EditorOpen ». Au début, je pensais que ce nom est incorrect, car il ne se termine pas avec « -ment » ou « -ed », comme le suggère MSDN. Cependant, plus tard, je l'ai découvert, que cet événement n'INFORM quelque chose qui se passe, mais il est plutôt une sorte de en faire la demande l'opération proprement dite. Ceci est le code client, qui devrait effectuer l ' « ouverture de l'éditeur »!

Je suis un peu surpris de découvrir que c'est en fait une forme de modèle de modèle Méthode de conception, dans laquelle il peut y avoir plusieurs actions liées à un seul espace réservé d'action.

Je pense qu'il est très intéressant, mais je crains aussi que l'utilisation des événements dans de tels cas peut être très trompeur. Quoi qu'il en soit, nous ne parlons pas d'ici, mais sur les demandes. Hm ... peut-être qu'il serait correct, si seul le nom de l'événement était « EditorOpeningRequest » ou « EditorOpeningRequested ». Qu'est-ce que tu penses? Comment voulez-vous commenter cela lors d'une revue de code?

Answer 1

À moins d'une très bonne raison pour vous d'avoir plusieurs objets de traitement d'une demande d'ouverture d'un éditeur, je dirais que cela devrait être un délégué au lieu d'un événement. Cela vous limite à avoir un gestionnaire de la demande, dans ce cas semble plus logique.

De plus je changerais le nom OpenEditor car cela est plus descriptif de ce que vous allez attendre. Si vous restez avec le modèle d'événement, alors OpenEditorRequested peut être un meilleur nom.

HTH.