在引发事件之前是否有任何理由将事件分配给局部变量？

我经常看到如下代码，想知道是否有任何理由为事件使用局部变量，而不是仅仅使用事件本身。有没有？

var handler = OnQueryComplete;
if (handler != null)
    handler(this, new RepositoryEventArgs<T>(results));

是的，绝对 - 它使无效检查变得安全。

如果你只有：

// Bad code, do not use
if (OnQueryComplete != null)
{
    OnQueryComplete(this, ...);
}

那么最后一个订阅者可能会在检查和调用之间取消订阅，从而导致NullReferenceException.

有lots这里的选项：

• 决定您不关心线程安全，即使它抛出异常。 （在许多情况下，这可能是合理的，特别是如果这都是您自己的代码库。）
• 使用扩展方法可以使此操作变得简单，因此无效检查无论如何都会消失。

• 在 C# 6 中，使用条件 null 运算符：

  OnQueryComplete?.Invoke(this, ...);
  

• 使用一个永远不会被删除的空处理程序设置事件：

  public event FooEventHander OnQueryComplete = delegate {};
  

您可能还想使用Interlocked确保您获得变量的最新值，以避免内存模型问题。看我的博客文章（包括评论）对此进行更多讨论。