Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Should EventHandler always be used for events?

I've been merrily writing events using custom delegate types and the generic Action delegate type, without really thinking about what I was doing.

I have some nice extension helpers for Action and EventHandler which makes me tend to use those predefined delegate types rather than my own, but aside from that...

Is there a good reason other than convention to favour EventHandler and EventHandler<T> over custom delegate types or generic Action delegate types?

like image 237
Andy Avatar asked Dec 28 '22 02:12

Andy


2 Answers

The main advantage of the signature EventHandler<T> over using one parameter for each member of your EventArgs is that you can add additional properties to your EventArgs without breaking compatibility.
IMO this is the most important argument. Being able to extent your EventArgs without breaking subscribing code is very nice. But of course you can achieve the same with any signature that uses some kind of property-bag parameter instead of a parameter per property.

Then there is variance, EventHandler<Base> is convertible to EventHander<Derived>, so you can write an EventHandler with parameter EventArgs and it can subscribe to events which have more specific EventArgs.

Extension methods are another plus, but you already mentioned that.

like image 149
CodesInChaos Avatar answered Jan 06 '23 07:01

CodesInChaos


No, no good reason.

If your events do not require EventArgs or a sender object, then you don't need to use EventHandler or EventHandler<T>.

like image 22
Oded Avatar answered Jan 06 '23 08:01

Oded