Form2でForm1の値を変更

Question

初歩的な質問かもしれませんが、題の通り、Form2の画面のボタンを押したらForm1のlabelのTextを変更したいです。

public partial class Form1 : Form
    {
        Form2 form2 = new Form2();
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            form2.Show();
        }
    }

public partial class Form2 : Form
    {
        Form1 form1;
        public Form2()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            if(form1 == null)
            {
                form1 = new Form1();
            }
            form1.label1.Text = "aaa";
        }
    }

これではできませんでした。
なぜダメなのかも教えていただけると幸いです。
よろしくお願いいたします。

Answer 1

Form2 に From1 を引数に取るコンストラクタを追加して、

public partial class Form2 : Form
{
    Form1 form1;

    // Form1 を引数に取るコンストラクタ
    public Form2(Form1 form1)
    {
        InitializeComponent();
        this.form1 = form1;
    }

    // ・・・略・・・
}

それを Form1 で以下のように使うというのはいかがですか？

public partial class Form1 : Form
{
    Form2 form2;

    public Form1()
    {
        InitializeComponent();
        this.form2 = new Form2(this);
    }

    // ・・・略・・・
}

Comments

[退会済みユーザー]

このような方法があるとは・・・勉強になりました。ありがとうございます。

[KSwordOfHaste]

自分のコード例は「説明のためのもの」という気持ちで書いてしまい実際に使うコードとしてはとても良くないコードです。SuferOnWwwさんのコードの方がはるかによいです。
変な回答をしてしまって申し訳ないです。実際のコードはぜひSuferOnWwwさんの回答どうりにしてください。BAもぜひSuferOnWwwさんの方へ変えていただければと思います。

[退会済みユーザー]

ご丁寧にありがとうございます。BAを変更させていただきましたが、KSwordOfHasteさんのご説明わかりやすかったです。ありがとうございます。

Answer 2

追記：SuferOnWwwさんの回答へコメントしましたが、自分の対処例は「考え方」を書いたつもりで、コードとしては非常によくありませんでした。特にform1をpublicにしてForm2クラス以外から設定する点がまったくイケてません。普通はSuferOnWwwさんのコードを書くと思います。

ヘンテコなコード例を挙げてしまい申し訳ありませんでした。

---

form1がnullだからです。

form1.label1.Text = "aaa"を実行する際、まずform1.label1を参照しようとしますが、form1がnull(どのインスタンスも指していない)ならlabel1の参照のしようがありません。

Form2.form1の初期化をしましょう。とりあえず動けばよいのであれば、

• Form2のform1フィールドをpublicにする
• Form2のform1フィールドへForm1のインスタンスを初期設定する

Form1のコンストラクターを次のようにすると初期設定できます。

public Form1() {
  InitializeComponent();
  form2.form1 = this;    //<==この行を追加
}

Comments

[退会済みユーザー]

やりたいことができました。
ありがとうございました。