¿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:
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();
}
}
Incluso ReSharper genera un método de invocación de la manera mencionada anteriormente.
¿Por qué no hacerlo de esta manera?
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
}
}
¿Alguien puede explicar una razón para no hacer esto? (pro vs. contras)