Skal jeg utelate parameteren “ e ” når jeg bruker EventHandler < EventArgs >?

Når jeg definerer mine egne hendelser, bruker jeg et mønster som følgende (som jeg tror er slik MSDN anbefaler gjør det):

public class MyEventClass { private bool _value; // Backing variable public bool Value { get { return _value; } set { if (value != _value) // Only raise event if value is changed/different { _value = value; OnValueChanged(EventArgs.Empty); } } } public event EventHandler ValueChanged; // Anything can subscribe to the event protected void OnValueChanged(EventArgs e) // Only this and children can invoke event { if (ValueChanged != null) ValueChanged(this, e); } } 

Dermed trenger klientkoden bare å abonnere på hendelsen, og den fungerer automatisk.

Imidlertid, den føles tungvint å overføre parameteren EventArgs e, spesielt med en klasse som bruker EventHandler til å heve hendelsene sine, siden basen EventArgs ikke har data til den, bare EventArgs.Empty.

Ville det betraktes som en faux pas å endre det til dette?

protected void OnValueChanged() { if (ValueChanged != null) ValueChanged(this, EventArgs.Empty); } 

Kommentarer

• Hva skader det å være der? Må du legge den tilbake senere når du finner et behov for å sende hendelsesargumenter? Sannsynligvis …
• Jeg tenkte bare på det som en bekvemmelighet å spare meg litt arbeid mens jeg skrev timen. Men du ' har helt rett i at den kunne endres senere (den vant sannsynligvis ' t, men fortsatt …), og det ville være en bruddendring, så jeg kan like godt gjøre det riktig første gang. Takk!
• I dette tilfellet er det ikke ' t det er mer normalt å bruke en PropertyChangedEventArgs og -Handler, og sende egenskapens navn som argument? Eller er dette bare et dårlig eksempel?
• Koden min kan være et dårlig eksempel, men jeg hentet design- og navngivningskonvensjonene fra noen .NET-klasser, som bruker standard EventArgs og -Changed nomenklatur.