为什么我们不直接调用事件

Question

我见过许多开发人员在想要调用Event时，他们将其分配给名为handler的局部变量并调用handler，而不是直接调用Event。为什么我们不直接调用事件？

private void OnSomethingChanged(EventArgs e)
{
    if (SomethingEvent != null)
    {
        SomethingEvent(this, e);
    }
}

Answer 1

您发布的代码基本上不是线程安全的。如果最终订阅者在if检查之后但在调用之前在不同的线程中取消订阅，则会得到NullReferenceException。

一种选择是编写一个扩展方法：

public static void NullSafeInvoke(this EventHandler handler,
                                  object sender, EventArgs e)
{
    if (handler != null)
    {
        handler(this, e);
    }
}

然后你可以写：

private void OnSomethingChanged(EventArgs e)
{
    SomethingEvent.NullSafeInvoke(this, e);
}

您可能也想要EventHandler<T>的另一个过载。

Answer 2

如果不复制事件（仅与多线程应用程序相关），则可能出现竞争条件。

如果一个线程在null检查后立即取消订阅该事件，而没有任何订阅，那么您将获得一个NullReferenceException。

Answer 3

我也不明白为什么。有一种简单而安全的方法：

// when constructing of instance, to create empty subscription
public event EventHandler SomethingEvent = delegate { }; 
private void OnSomethingChanged(EventArgs e)
{
    // and call it directly
    SomethingEvent(this, e);
}