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

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

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

Answer 1

是的，绝对 – 它使无效检查安全。

如果你有：

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

然后最后一个订阅者可以在检查和调用之间取消订阅，从而导致NullReferenceException 。

这里有很多选择：

• 决定你不关心线程安全性，甚至是它抛出exception的程度。 （在许多情况下，这可能是合理的，特别是如果这是您自己的代码库。）
• 使用扩展方法使这很简单，无论如何无效检查都会消失。

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

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

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

   public event FooEventHander OnQueryComplete = delegate {}; 

您可能还希望使用Interlocked来确保获得变量的最新值，以避免内存模型问题。 有关此问题的更多讨论，请参阅我的博客文章（包括评论） 。