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

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