我经常看到类似下面的代码,想知道是否有任何理由为事件使用局部变量,而不是仅仅使用事件本身。有吗?
var handler = OnQueryComplete;
if (handler != null)
handler(this, new RepositoryEventArgs<T>(results));
最佳答案
是的,绝对 - 它使无效检查变得安全。
如果你有:
// Bad code, do not use
if (OnQueryComplete != null)
{
OnQueryComplete(this, ...);
}
那么最后一个订阅者可能会在检查和调用之间取消订阅,从而导致 NullReferenceException
。
这里有很多选项:
- 决定您不关心线程安全,甚至不关心它抛出异常的程度。 (在许多情况下,这可能是合理的,尤其是当这都是您自己的代码库时。)
- 使用扩展方法使实现变得简单,因此无效检查无论如何都会消失。
在 C# 6 中,使用条件 null 运算符:
OnQueryComplete?.Invoke(this, ...);
使用一个永远不会删除的空处理程序设置事件:
public event FooEventHander OnQueryComplete = delegate {};
您可能还想使用 Interlocked
来确保您获得变量的最新值,以避免内存模型问题。参见 my blog post (including comments)对此进行更多讨论。
关于c# - 是否有任何理由在引发事件之前将事件分配给局部变量?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/28655778/