c# - .NET 事件引发和空对象模式

Question

我们可以通过两种方式引发事件:

public event EventHandler MyEvent;

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

和

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

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

我更喜欢最后一个。它更短。
我的同事坚持第一种变体。

第一个比第二个有什么优势吗？

Answer 1

C# 6 更新

在 C# 6 中，您只需像这样使用 null 条件运算符:

PropertyChanged?.Invoke(this, args);

这是 recommended by the Roslyn wiki

原始答案

Eric Lippert 有 a great blog post on Events and Races ，如果您还没有阅读，您应该阅读。

第一个选项可以被认为比第二个选项更安全，因为事件可能会被设置为 null。有人可能会不小心修改类。此外，如果您反序列化实例，则第二种方法将不起作用(取决于您使用的序列化机制)。

我有时会使用辅助方法来引发事件

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);
    }
}