C#, EventHandlerの多重登録回避について教えてください。

Question

前提・実現したいこと

EventHandlerの登録について教えてください。
Windows Formアプリケーションを作っています。
EventHandlerをコードで登録する場合、繰り返し登録すると、イベントハンドラは多重登録できます。

試したこと

たとえば下記のコードでButton1をクリックすると、Button2のイベントハンドラを多重登録します。
Button1を1回クリック、Button2を1回クリック。-MessageBoxを１回表示。内容は1。
Button1を2回クリック、Button2を1回クリック。-MessageBoxを2回表示。内容は1,2。
Button1を3回クリック、Button2を1回クリック。-MessageBoxを3回表示。内容は1,２,3。

該当のソースコード

using System;
using System.Windows.Forms;

namespace WindowsFormsApplication1
{
	public partial class Form1 : Form
	{
		public Form1()
		{
			InitializeComponent();
		}

		private void button1_Click(object sender, EventArgs e)
		{
			counter = 0;
			//if (this.button2.Click==null)
				this.button2.Click += new System.EventHandler(this.button2_Click);
		}

		int counter = 0;
		private void button2_Click(object sender, EventArgs e)
		{
			counter++;
			MessageBox.Show(counter.ToString());
		}
	}
}

namespace WindowsFormsApplication1
{
	partial class Form1
	{
		/// <summary>
		/// 必要なデザイナー変数です。
		/// </summary>
		private System.ComponentModel.IContainer components = null;

		/// <summary>
		/// 使用中のリソースをすべてクリーンアップします。
		/// </summary>
		/// <param name="disposing">マネージ リソースを破棄する場合は true を指定し、その他の場合は false を指定します。</param>
		protected override void Dispose(bool disposing)
		{
			if (disposing && (components != null))
			{
				components.Dispose();
			}
			base.Dispose(disposing);
		}

		#region Windows フォーム デザイナーで生成されたコード

		/// <summary>
		/// デザイナー サポートに必要なメソッドです。このメソッドの内容を
		/// コード エディターで変更しないでください。
		/// </summary>
		private void InitializeComponent()
		{
			this.button1 = new System.Windows.Forms.Button();
			this.button2 = new System.Windows.Forms.Button();
			this.SuspendLayout();
			// 
			// button1
			// 
			this.button1.Location = new System.Drawing.Point(207, 168);
			this.button1.Name = "button1";
			this.button1.Size = new System.Drawing.Size(75, 23);
			this.button1.TabIndex = 0;
			this.button1.Text = "button1";
			this.button1.UseVisualStyleBackColor = true;
			this.button1.Click += new System.EventHandler(this.button1_Click);
			// 
			// button2
			// 
			this.button2.Location = new System.Drawing.Point(146, 99);
			this.button2.Name = "button2";
			this.button2.Size = new System.Drawing.Size(75, 23);
			this.button2.TabIndex = 1;
			this.button2.Text = "button2";
			this.button2.UseVisualStyleBackColor = true;
			// 
			// Form1
			// 
			this.AutoScaleDimensions = new System.Drawing.SizeF(6F, 12F);
			this.AutoScaleMode = System.Windows.Forms.AutoScaleMode.Font;
			this.ClientSize = new System.Drawing.Size(284, 261);
			this.Controls.Add(this.button2);
			this.Controls.Add(this.button1);
			this.Name = "Form1";
			this.Text = "Form1";
			this.ResumeLayout(false);

		}

		#endregion

		private System.Windows.Forms.Button button1;
		private System.Windows.Forms.Button button2;
	}
}

発生している問題・エラーメッセージ

これを回避するために、

	if (this.button2.Click==null)
		this.button2.Click += new System.EventHandler(this.button2_Click);

としたところ、
イベントControl.Clickは+=または-=の左側にのみ使用できます。
とエラーになります。

EventHandlerをコードで登録する場合、多重登録を回避する方法を知りたいです。
よろしくお願いします。

補足情報（FW/ツールのバージョンなど）

Visual Studio 2015 Pro
C#

Comments

[mattn]

多重登録された場合、先の物を生かすのでしょうか？それとも先の物を取り消し後の物を有効にするのでしょうか？

[meshkit]

先のものを生かします。

Answer 1

mattin さんが書かれているようにイベントの add, remove を明示的に実装すれば制御できます。

もし、同じメソッドが重複して登録されないようにしたいのであれば、add に渡された value からそのメソッドのクラス名やメソッド名を取得できます。

既に登録されているメソッドは GetInvocationList で取得できます。
mattin さんの例示コードでは fireEvent.GetInvocationList() です。Delegate の配列が取得できますので、EventHandler にキャストしてください。

とはいえ、イベントハンドラは初期処理やフォームデザイナで定義しておいて、

・その処理を実行するかどうかを制御するフラグを設ける。既定値は無効。
・フラグが有効でない場合は何もしない。
・button2 がクリックされたときにそのフラグを有効にする。

としたほうがよいような気がします。

Comments

[meshkit]

ありがとうございます。mattinさんの回答、なるほどと思ったものの、実際にはbuttonではなくハードウェアからやってくるイベントで、ちょっと現在のわたしには参考にはなるものの実装は無理とあきらめてました。
フラグ方式ならわたしにも使えそうです。こちらでやってみます。ありがとうございます。感謝。

Answer 2

イベントハンドラを自前で実装すれば出来なくないです。

using System;
using System.Linq;

class Publisher 
{
    private event EventHandler fireEvent;

	public event EventHandler FireEvent
	{
		add
		{
			if (fireEvent == null)
			{
				fireEvent += value;
			}
			else
			{
				Console.WriteLine("重複登録ですよ");
			}
		}
		remove
		{
			fireEvent -= value;
		}
	}

	public void Fire()
	{
		OnFire(new EventArgs());
	}

    protected virtual void OnFire(EventArgs e)
    {
        if (fireEvent != null)
            fireEvent(this, e);
    }
}

public class Program
{
    static void Main(string[] args)
    {
		var publisher = new Publisher();
		publisher.FireEvent += (o, e) =>
		{
        	Console.WriteLine("fire event!");
		};
		publisher.FireEvent += (o, e) =>
		{
        	Console.WriteLine("more fire event!");
		};
		publisher.Fire();
    }
}

ただしコントロールの Click 等はこれが実装されてはいませんので継承するなりしてカスタムコントロール化しないといけないかもしれません。