¿Es una buena práctica definir un cuerpo de delegado vacío para un evento?

Question

Duplicar posible:
Is there a downside to adding an anonymous empty delegate on event declaration?

¿Es una buena práctica para definir un cuerpo de delegados vacío para un evento para que usted no tiene que preocuparse plantear un evento que no tienen ningún controlador de eventos ? (no es necesario verificar si el evento es nulo).

código, como a continuación:

public event EventHandler<LoadEventArgs> LoadedData = delegate { }; 

Answer 1

ciertamente he encontró útil, sí. Habrá un pequeño y diminuto costo de rendimiento, pero el beneficio en legibilidad por no tener que realizar la prueba de nulidad hace que valga la pena IMO.

Vale la pena señalar que este es uno de los pocos momentos en que es bueno utilizar un método anónimo en lugar de una expresión lambda - de lo contrario hay que nombrar los parámetros que vas a ignorar, como esto:

public event EventHandler<LoadEventArgs> LoadedData = (sender, args) => {}; 

no me gusta tener que nombrar las cosas que no estoy con la intención de usar :)

Answer 2

yo no haría esto.

Existen dos motivos para esto. En primer lugar, es estándar simplemente verificar el controlador para null antes de llamar a esto. Ve esto por todos lados, en ejemplos, en la fuente de referencia de Microsoft y in the manuals.

En segundo lugar, la inicialización de sus eventos de esta manera asigna objetos innecesarios al crear su clase. Esto pondrá presión de memoria adicional en su aplicación sin una necesidad real. Esto se debe a:

public event EventHandler<LoadEventArgs> LoadedData = delegate { }; 

se traduce en:

public event EventHandler<LoadEventArgs> LoadedData = new EventHandler<LoadEventArgs>delegate { }); 

Si no lo hace ya, envolviendo sus eventos en los métodos específicos ayuda mucho:

public event EventHandler MyEvent; 

protected virtual void OnMyEvent(EventArgs e) 
{ 
    if (MyEvent != null) 
     MyEvent(this, e); 
} 

Usted puede hacer este hilo es seguro (r) usando lo siguiente:

public event EventHandler MyEvent; 

protected virtual void OnMyEvent(EventArgs e) 
{ 
    var handler = MyEvent; 

    if (handler != null) 
     handler(this, e); 
} 

Answer 3

Normalmente no uso C#, pero me parece una buena práctica. Esta es una aplicación de libro de texto del patrón 'Null Object'. Como se señaló, esto costará en términos de rendimiento cuando el delegado vacío realmente se ejecuta, pero gana en términos de rendimiento en cualquier otro momento, cuando no tiene que verificar un nulo explícito, por lo que también debería ser una ganancia neta. sabio siempre que la frecuencia de delegados vacíos sea baja.