危険ですか？イベントについて

Question

多分愚かな質問ですが、

私のコードでは、私はいくつかの場所で以下の構造を使用しています。

void MyFunction() 
{ 
    DoSomething(myClass.myProperty) 

    myClass.PropertyChanged += (s,e) => { 
     if (e.PropertyName == "myProperty") { 
     DoSomething(myClass.myProperty); 
     } 
    } 
} 

私は最初に何かしたいと思いますし、将来的にプロパティが変更されても同じことをしたいと思います。

今、問題はMyFunction（）がプログラムの実行中に数回呼び出されることです。私がPropertyChangedに割り当てるデリゲートは、このメソッドを通過する際に追加されますか？ （より多くのメモリを消費する毎回の反復とプログラムの減速）またはコンパイラ/ランタイムは私が最初に追加する必要があることを理解するのに十分なスマートなのですか？もしそうなら、これはどのように機能しますか？

Answer 1

多くは、あなたがPropertyChangedイベントは、いくつかのリスナーを追加することを防止するnullあるかどうかを確認すべきであることを示唆しています。その解決策の問題点は、PropertyChangedは、nullである可能性があります。コードの他の部分が同じイベントをリッスンする場合は、このような別のプロパティーをリッスンします。

void AnotherFunction() 
{ 
    myClass.PropertyChanged += (s,e) => { 
    if (e.PropertyName == "anotherProperty") { 
     DoSomethingElse(myClass.anotherProperty); 
    } 
    } 
} 

よりよい解決策は、私の意見では、最初はfalse、およびMyFunctionが呼び出されたときにのみtrueに設定されているブールフラグを維持するだろう。次に、このフラグをチェックして、イベントハンドラを追加する必要があるかどうかを確認します。

Answer 2

コンパイラはあなたの意図を知ることができません...それは忠実にMyFunction（）の各呼び出しでイベントハンドラを添付します。

このように見てください。コンパイラは、関数内の参照変数（例：myclass）がハンドラを以前にアタッチしたオブジェクトの同じインスタンスを参照していることを知ることができません。また、コール間でハンドラを別の場所に切り離さなかったことも知りません。このような仮定はできません。

ただし、ハンドラが1回だけ接続されるようにコードを再構成できます。他の消費者はPropertyChangedイベントを購読することができるので、あなたが購読しているかどうかを知るために、独立した州を維持する必要があります。例：他の回答の

if(!haveSubscribedToPropertyChanged) { 
    myClass.PropertyChanged += (s,e) => { 
      if (e.PropertyName == "myProperty") { 
       DoSomething(myClass.myProperty); 
      } 
     } 
     haveSubscribedToPropertyChanged = true; 
     }