Windows Form　サブフォームからメインフォームに値を渡す方法

Question

### **WindowsForm(C#)について質問です。**
**サブフォームからメインフォームに値を渡す**やり方を調べていたところ、サブフォームからメインフォームへ情報を受け渡す場合、サブフォームはメインフォームのインスタンスを保持していない為、メインフォームのインスタンスをサブフォームの変数に渡さなければならないという記事を見つけたのですが、**staticでメインフォームの変数を静的**にしてもサブフォームから値を渡すことができたので、後者のやりかたでやっても問題ないのでしょうか？
```C#
public partial class Form1 : Form
{   
	private static Boolean a;
	
	private void button1_Click(object sender, EventArgs e)
	{
		Form2 form2 = new Form2();
		
		if (!a)
		{
			form2.Show();
		}
		a = true;
	}
	
	public Boolean A
	{
		get { return a; }
		set { a = value; }
	}
         
         
         
public partial class Form2 : Form
{
	private void button1_Click(object sender, EventArgs e)
	{
		Form1 form1 = new Form1();
		form1.A = false;
	}
	
```

Accepted Answer

不必要なインスタンスを作るのがよくないかと思います。
例えばForm2を生成して表示する部分は下記のように必要な時にインスタンスを作成すべきかと
```C#
    private void button1_Click(object sender, EventArgs e)
    {
        if (!a)
        {
            Form2 form2 = new Form2();
            form2.Show();
        }
        a = true;
    }

```

また、staticにするのも静的なもの考え方からするとよくないかと思います。

方法論としては
0. Form2にForm1のインスタンスを渡してForm1のプロパティにアクセスする
0. Form2にデータを受け渡したいタイミングで発生するイベントを作り、Form1側でそのイベント時にForm2のプロパティ等からデータを取り出す。

私は2の方法のほうがForm2がForm1に依存しなくなるので好みです。



質問の趣旨とは少し外れますが、提示されたソースはForm2の表示制御を行いたいだけに見えます。
Form1とForm2の関係性は分からないので的外れかもしれませんが
```C#
public partial class Form1 : Form
{   
    private void button1_Click(object sender, EventArgs e)
    {
        Form2 form2 = new Form2();
        form2.ShowDialog();
    }
}


public partial class Form2 : Form
{
    private void button1_Click(object sender, EventArgs e)
    {
        this.Close();
    }
}
```
これで終わる話だったりしませんか？

Answer

> サブフォームはメインフォームのインスタンスを保持していない為、メインフォームのインスタンスをサブフォームの変数に渡さなければならないという記事...

メインをサブから直接いじくるというのは１つのやり方だけども，
別に，その形じゃなくてもいい．

「サブからメインに値を渡すための何らかの手段」があればよい．
例えば，「サブ→何か→メイン」のように，間に何かを経由しようが，目的が果たせれば何でもいい．

---

提示コードに関しては（staticでやることの良し悪しは別として），

プロパティを介してstaticメンバをいじくるためだけに `new Form1();` とかするんじゃなくて，
そのプロパティ自体をstaticにすればよいのでは．

```
//こいつをstaticに
public static Boolean A
{
   get { return a; }
    set { a = value; }
}
```

WindowsForm(C#)について質問です。

関連した質問