安全地引发事件线程-最佳实践

Question

为了引发事件，我们使用如下方法OnEventName：

protected virtual void OnSomethingHappened(EventArgs e) 
{
    EventHandler handler = SomethingHappened;
    if (handler != null) 
    {
        handler(this, e);
    }
}

但是，这有什么区别？

protected virtual void OnSomethingHappened(EventArgs e) 
{
    if (SomethingHappened!= null) 
    {
        SomethingHappened(this, e);
    }
}

显然第一个是线程安全的，但是为什么以及如何执行？

不必启动新线程吗？

Answer 1

有一个微小的机会SomethingHappened成为null空校验之后，但在调用之前。但是，MulticastDelagates是不可变的，因此，如果您首先分配一个变量，对该变量进行null检查并通过它进行调用，那么在这种情况下是安全的（自我插入：我之前写过一篇有关此问题的博客文章）。

但是硬币的背面。如果您使用temp变量方法，那么您的代码将受到NullReferenceExceptions的保护，但可能是该事件将在事件监听器
与事件分离后 调用事件监听器。这只是以最优雅的方式处理的事情。

为了解决这个问题，我有一个扩展方法，我有时会使用：

public static class EventHandlerExtensions
{
    public static void SafeInvoke<T>(this EventHandler<T> evt, object sender, T e) where T : EventArgs
    {
        if (evt != null)
        {
            evt(sender, e);
        }
    }
}

使用该方法，您可以像这样调用事件：

protected void OnSomeEvent(EventArgs e)
{
    SomeEvent.SafeInvoke(this, e);
}