C＃のイベントやLambda？

Question

私はあなただけを提供する1つの方法を望む/期待していると仮定すると...ただ好奇心の問題としてそこ

これを投げていますが、これは悪い習慣時または眉をひそめたことでしょうか？

public class Something { 
    public Action OnRemove =() => { }; 
    public Action<object, EventArgs> OnFinishedLoading = (sender, e) => { }; 
} 

// then used like... 
something.OnRemove =() => { /*do something...*/ }; 
something.OnFinishedLoading = (sender, e) => { /*do something else...*/ }; 

私は、このイベントで不正行為のようなものです実現が、私はこのアプローチについて悪いものはありますか？長期的にはアプリケーションの潜在的な問題を引き起こすでしょうか？

複数のメソッドを実行したい場合は、イベントがうまくいくことがわかります。これは、主に1つのメソッドを欲しい/期待している場合の主な質問です。

Answer 1

イベントでは、あなたがこれを行うことはできません。

something.OnRemove = null; 

をイベントは、あなたのラッパー、プロパティと属性を持つラッパーの種類を提供しますので、あなたは、デリゲートメソッドを超えるカプセル化を確保することができます。

Answer 2

よく分かりませんが、「間違っている」というのは、公開されているフィールドが公開されているということです。私は少なくともプロパティを使用していました - その時点で、イベントはおそらくもっと簡単になります。これは、プロパティよりも少ないコードで柔軟性を提供します（特定のデフォルト値で自動実装されたプロパティを使用できなかったためです）。

もう1つの方法は、コンストラクタでそれを取得し、プライベートの読み取り専用フィールドとして保持することです。それはおそらく私が本当にやりたいことです...本当にプロパティ/フィールドとしてアクションを公開する必要はありますか？

Answer 3

この方法では何も問題はなく、私は非常に頻繁に使用します。シンプルなハンドラはシンプルなコードを持っています（これは良いことです）。

この方法の唯一の欠点は、この方法で作成されたハンドラを簡単に削除できないことです。通常、あなたは言うことができます

something.OnRemove -= SomeEvent; 

しかし、これはすべてのラムダ式では当てはまりません。テキストで等価なラムダ式はほぼ確実に別々の実装を持つため、イベントハンドラを追加したり削除したりする目的で一致しません。

たとえば、次のコードではハンドラを削除できません。

something.OnRemove +=() { MessageBox.Show("Removed!"); } 
something.OnRemove -=() { MessageBox.Show("Removed!"); } 

ただし、追加したい限り、これは問題ではありません。

Answer 4

潜在的にあなたが望む以上に露出します。通常、イベントはクライアントがイベントを返すMulticastDelegateの内容を検査することを許可しません。

Answer 5

私はこのアプローチがうまくいくと思います。それはデリゲートを宣言するためのちょっとした方法です（Actionファミリの定義を見てください）。