c# - 为什么我们不直接调用事件？

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> 进行另一个重载，也是。