NET podnoszenie zdarzenie i NullObject wzór

Question

możemy podnieść zdarzenie na dwa sposoby:

public event EventHandler MyEvent; 

private void DoSomething() 
{ 
    ... 
    var handler = MyEvent; 
    if (handler != null) 
     handler(this,EventArgs.Empty); 
} 

i

public event EventHandler MyEvent = (o,e) => {} ; 

private void DoSomething() 
{ 
    ... 
    MyEvent(this, EventArgs.Empty); 
} 

Wolę ostatni. Jest krótszy.
Moi koledzy nalegają na pierwszy wariant.

Czy istnieje przewaga pierwszego nad drugą?

Answer 1

Aktualizacja dla C# 6

W języku C# 6 wystarczy użyć operatora null warunkowy tak:

PropertyChanged?.Invoke(this, args); 

To recommended by the Roslyn wiki

Original Odpowiedź

Eric Lippert ma a great blog post on Events and Races, którą powinieneś przeczytać, jeśli nie masz.

Pierwsza opcja może być uważana za bezpieczniejszą niż druga, ponieważ zdarzenie może zostać ustawione na wartość null. Ktoś mógłby bezmyślnie modyfikować klasę. Ponadto, jeśli deserializuje się instancje, druga metoda nie będzie działać (w zależności od używanego mechanizmu szeregowania).

czasami użyć metody pomocnika podnieść zdarzenia

static class Raiser 
{ 
    public static void Raise<T>(this EventHandler<T> evnt, object sender, T args) 
     where T : EventArgs 
    { 
     if (evnt != null) 
     { 
      evnt(sender, args); 
     } 
    } 
} 

class SomeClass 
{ 
    public event EventHandler<EventArgs> MyEvent; 

    private void DoSomething() 
    { 
     MyEvent.Raise(this, EventArgs.Empty); 
    } 
}