C# と VB.NET の質問掲示板

普通に Designer で各ボタンを事前に生成しておき、各ボタンに別ハンドラを生成すればそれで済むわけですが、
そうしたくない理由がある？

private void button2_Click(object sender, EventArgs e) { Form2 を表示 }
private void button3_Click(object sender, EventArgs e) { Form3 を表示 }
private void button4_Click(object sender, EventArgs e) { Form4 を表示 }


Tag プロパティに格納しておいた値でほげほげするとか。

■No87891 (なっとう さん) に返信

> 仰る通りFormXは自力で作ります。試行錯誤しながら追加・コピペ・削除を繰り返すと思います。
> 
> ＞であれば、Form1 のコードにその時に手を加えれば済むことだと思うのですが
> 前に述べたように、条件に応じてボタンのON/OFFが必要になります。
> ON/OFFは設定ファイル等によって制御することになるかもしれません。

上の質問者さんのレスを読むと、私の提案を理解いただけてないような気がしますので補足します。

一番最初の私のレスで、

> Form2 〜 FormX クラスは定義済・ビルド済みで、Button.Name などに応じて当該 Form を 
> new して Show すればいいのであれば、switch 文を使って分ければよさそうな気がします。

と言った switch 文とは以下のようにすることです。

private void Button_Click(object sender, EventArgs e)
{
    Form frm = null;
    switch (((Button)sender).Name)
    {
        case "button2":
            frm = new Form2();
            break;
        case "button3":
            frm = new Form3();
            break;

        // ・・・中略・・・

        case "buttonX":
            frm = new FormX();
            break;
    }
    frm.Show();
}

FormX+1 のコードを自力で書いて追加する際に、上記のコードに case "buttonX+1" ...　を 3 行
追加すれば良いはずです。

上記には、

> 条件に応じてボタンのON/OFFが必要になります。

は影響ないはず。また、既存のコード（すでに完成しているのでは？）はほとんど変更せずに済む
はずです。

他の回答者の方の案と比較して、簡単、現実的、保守がしやすいかなどを考えてどうするかを検討す
ることをお勧めします。

Dictionaryでボタンとフォームのセットを保持する

private Dictionary<Button, Type> Btns = new Dictionary<Button, Type>();

Btns.Add(btn1, typeof(Form1));
Btns.Add(btn2, typeof(Form2));


private void button_Click(object sender, EventArgs e)
{
	Type t = Btns[(Button)sender];
	Form frm = (Form)Activator.CreateInstance(t);
	frm.Show();
}

リフレクションを使わない方法

public interface IFormMaker {
	Form Make();
}

public class FormMaker<T> : IFormMaker where T : Form, new() 	
{
	public Form Make()
	{
		return new T();
	}
}

Dictionary<Button, IFormMaker> Btns = new Dictionary<Button, IFormMaker>();
Btns.Add(btn1, new FormMaker<Form1>());
Btns.Add(btn2, new FormMaker<Form2>());


private void button_Click(object sender, EventArgs e)
{
	IFormMaker maker = Btns[(Button)sender];
	Form frm = maker.Make();
	frm.Show();
}

---
追記
あるいは、辞書化せずに

private void AddButton<T>(string name) where T : Form, new()
{
	var button1 = new System.Windows.Forms.Button();
	button1.Click += delegate {
		Form form = new T();
		form.Show();
	};
}

提示コードを出来るだけ活かしたサンプル：
TabIndexはなんとかした方がよいと思います。

    public partial class Form1 : Form
    {
        private List<System.Windows.Forms.Button> Btns = new List<Button>();
        public Form1()
        {
            InitializeComponent();
            CreateBtns();
        }

        private void CreateBtns()
        {
            AddButton<Form2>("Open Form2");
            AddButton<Form3>("Open Form3");
        }

        private void AddButton<T>(string name) where T : Form, new()
        {
            var button1 = new System.Windows.Forms.Button()
            {
                //Location = new System.Drawing.Point(3, 3),
                Name = name,
                Size = new System.Drawing.Size(75, 23),
                TabIndex = 0,　
                Text = name,
                UseVisualStyleBackColor = true
            };
            button1.Click += (s, e) => OpenForm<T>();
            Btns.Add(button1);
            this.flowLayoutPanel1.Controls.Add(button1);
        }
        
        private void OpenForm<T>() where T : Form, new()
        {
            T frm = new T();
            frm.Show();
            frm.FormClosed += Frm_closed;
        }

        private void Frm_closed(object sender,EventArgs e)
        {
            Form frm = (Form)sender;
            frm.Dispose();
        }

    }

話がかなり進んでいるので、他の方の発言は読んでいません。
ただ、なんとなく話がバラバラに見えます。

なっとうさんがやりたいのって、こういうこと？
手元にFormのサンプルがないので、WPFで試してしまいましたが、
普通に動作しましたよ。

// nameの文字列が表示されたボタンを押すと、Wndが表示される
private void AddButton(string name, Window Wnd)
{
    var button1 = new Button();
    button1.Name = name;
    button1.Content = name;
    button1.Click += (obj, e) =>
    {
        Wnd.Show();
    };
    MainPanel.Children.Add(button1);
}

Form流に書き直すとこんな感じかな？
こっちは動作確認してないです。
private void AddButton(string name, Form frm)
{
    var button1 = new Button();
    button1.Name = name;
    button1.Text = name;
    button1.Click += (sender, e) =>
    {
        frm.Show();
    }
    this.flowLayoutPanel1.Controls.Add(button1);
}