イベントデリゲートを呼び出す方がよいでしょうか？

Question

private void NotifyFreeChannelsChanged() //1. 
{ 
    if (FreeChannelsChanged != null) 
    { 
     FreeChannelsChanged(this, null); 
    } 
} 

private void NotifyFreeChannelsChanged() //2. 
{ 
    NotifyCollectionChangedEventHandler h = FreeChannelsChanged ; 
    if (h != null) 
     h(this, e); 
} 

これより優れている理由は何ですか。それはちょっとしたチェックです。大きな違いはありません。

Answer 1

"Better"？まあ、（特定の）競合状態を含んでいない、そうです。 MultiCastDelegate型は不変であり、意味のあるすべての方法で値型セマンティクスを使用します（は参照型ですが、this、さらに重要なのはthisを参照）。そのため、最初に割り当ててチェックします。問題は次のとおりです。

// this evaluates to true... 
if(SomeEvent != null) 
{ 
    // ...but before this line executes, the last 
    // subscriber detached, and now SomeEvent is null. Oops. 
    SomeEvent(this, e); 
} 

"なぜ例2を使用しますか？"

---

さておき、これは暗黙に型付けされた変数（var）を使用するのに最適な場所です。これらの代理人タイプの名前は長くなります...

また、競争状態がまだ存在するということは、それはちょっと微妙です。割り当て後に加入者が削除されるとどうなりますか？さて、それはまだ呼び出されますが、実際にあなたがそれについて行うことができるものは（私が知っている）何もありません。

Answer 2

第2の問題は、ヌルチェックとスニペット1でイベントが発生する間にコンテキストスイッチが発生するスレッドバグを防ぐことです。他のスレッドは、その間にヌルに設定できます。

あなたのメソッドがマルチスレッド化されやすいものでなければ、より単純なスニペットを使うことができます。

Answer 3

2番目のメソッドは、私たちが使用する拡張メソッドのセットに簡単に対応します。ここでは、EventHandlerがメソッドのターゲットです。

public static void Raise(this EventHandler handler, object sender) 
{ 
    if (handler != null) 
    handler(sender, EventArgs.Empty); 
} 

// And then... 
TheEvent.Raise(this); 

私が知る限り、ハンドラをメソッドに渡すという事実は、競合状態を避けるためにそのローカルコピーを提供します。