¿Es esta una mejor manera de desencadenar / invocar eventos sin una comprobación nula en C #?
La mayoría del código que he visto utiliza la siguiente forma de declarar e invocar la activación de eventos:
<code>public class MyExample { public event Action MyEvent; // could be an event EventHandler<EventArgs>, too private void OnMyEvent() { var handler = this.MyEvent; // copy before access (to aviod race cond.) if (handler != null) { handler(); } } public void DoSomeThingsAndFireEvent() { // ... doing some things here OnMyEvent(); } } </code>
Incluso ReSharper genera un método de invocación de la manera mencionada anteriormente.
¿Por qué no hacerlo de esta manera?
<code>public class MyExample { public event Action MyEvent = delegate {}; // init here, so it's never null public void DoSomeThingsAndFireEvent() { // ... doing some things here OnMyEvent(); // save to call directly because this can't be null } } </code>
¿Alguien puede explicar una razón para no hacer esto? (pro vs. contras)