3

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

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)或信任编译器/运行时？

4

1 回答 1

8

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

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

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 
}
于 2012-07-18T06:47:01.647 回答