Question

在类中考虑以下与事件相关的基本代码：

public event EventHandler Updated;

public void OnUpdated() {
    if (Updated != null) Updated(sender: this, e: null)
}

如果没有人订阅了Updated事件，我不希望第四行引起任何重大的性能拖累（想法是让订阅者选择要订阅的最细粒度的事件只有最少数量的事件触发并阻止消息队列超载）。

我是否应该关心并跟踪订阅者的存在（例如使用if (Updated != null && OnUpdateSubscribed) Updated(sender: this, e: null)或信任编译器/运行时？

Answer 1

支票OnUpdate != null 定义正在订阅/取消订阅的活动。实际上，布尔检查和空检查之间没有真正的区别，因为它们最终都只是一个“加载字段”，“如果为false则分支” - 因为就逻辑检查而言，空引用计为“假”关心。

我唯一的建议是：将它存储在局部变量中，以防止（非常不可能，但可能）竞争条件：

var snapshot = Updated;
if(snapshot != null) snapshot(this, EventArgs.Empty);

所以：不，基本上：这没有开销。

示例：

public event EventHandler SomeEvent;
protected virtual void OnSomeEvent()
{
    var snapshot = SomeEvent;
    if (snapshot != null) snapshot(this, EventArgs.Empty);
}

编译到（评论是我的）：

.method family hidebysig newslot virtual instance void OnSomeEvent() cil managed
{
    .maxstack 3
    .locals init (
        [0] class [mscorlib]System.EventHandler snapshot)

    // var snapshot = SomeEvent
    L_0000: ldarg.0 
    L_0001: ldfld class [mscorlib]System.EventHandler Foo::SomeEvent
    L_0006: stloc.0

    // if(snapshot == null) goto L_0016;
    L_0007: ldloc.0 
    L_0008: brfalse.s L_0016

    // snapshot(this, EventArgs.Empty);
    L_000a: ldloc.0 
    L_000b: ldarg.0 
    L_000c: ldsfld class [mscorlib]System.EventArgs [mscorlib]System.EventArgs::Empty
    L_0011: callvirt instance void [mscorlib]System.EventHandler::Invoke(object, class [mscorlib]System.EventArgs)

    // L_0016: return;
    L_0016: ret 
}

没有听众的事件会对性能产生相当大的影响吗？

1 个答案: