C#で画面を再ロードする方法

Question

実現したいこと

ここに実現したいことを箇条書きで書いてください。

• Form2の画面が閉じられたとき、Form1の画面を再度ロードする

前提

C#で

このようにForm2で入力したデータを元にForm1に結果を表示するシステムを作っています。
Form2のデータはテキストファイルで保存され、Form1のロード時や更新ボタンを押したときに読み込んで使用します。
このシステムは画面を閉じずに常駐して動かしたいので、Form2で新しい入力があったときに保存ボタンで新しくデータを保存して、Form2が閉じられたときにその新しく保存されたファイルを再読み込みしたいのですが、何か良い方法はありませんでしょうか？

該当のソースコード

//ボタン１をクリックしてForm2を表示する
private void button1_Click(object sender, EventArgs e)
        {
            // Form2のインスタンスを生成
            Form2 form2 = new Form2();
            // form2を表示
            form2.Show();
}

private void Form1_Load(object sender, EventArgs e)
{
//ロード時のファイル読み込み
if (File.Exists(mastFile))
            {
                //アラームの名前でグループ化してリストを作る
　　　　　//名前をキー,1行全体をvalueとする
                var alarm = File.ReadLines(mastFile, Encoding.GetEncoding("Shift_JIS"))
                    .Select(x => new { Key = x.Split(',')[0], Value = x, })     
                    .GroupBy(x => x.Key) //名前でグループ化する            
                    .Select(x => string.Join(",", x.Select(y => y.Value)))
                    .ToList();

                string[] alarmList = alarm.ToArray();     //リストを配列化する

                dataGridView1.Rows.Add(alarmList.Length);

                //リストの名前をDataGridViewに入れる
                for (int i = 0; i < alarmList.Length; i++)
                {
                    string fileData = alarmList[i];
                    string[] alarmData = fileData.Split(',');
                    name = alarmData[0].ToString().Replace("\"", "");
                    dataGridView1.Rows[i].Cells[0].Value = name;

                }

                //マスタファイルを読み込む
                StreamReader sr = new StreamReader(mastFile, Encoding.GetEncoding("SHIFT_JIS"));
                int valNum = 0;     //数値変換用の変数

                while (!sr.EndOfStream)
                {
                    //読んだ一行をカンマで区切って配列にする
                    string file = sr.ReadLine();
                    string[] fileData = file.Split(',');
　　　　　 }


            
      

### 試したこと

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

.NET Framework4.8

Comments

[hqf00342]

実現したいことの「Form2の画面が閉じられたとき、Form1の画面を再度ロードする」は

(1) 新しいForm1を生成する
(2) 既に表示してあるForm1の画面を更新する

のどちらでしょうか（今のコードでは(1))

また、「新しく保存されたファイルを再読み込み」し画面に反映する機能（メソッド）はForm1に既に存在しているのでしょうか？

[KOZ6.0]

Form2 を表示するときのコードが重要です。それによって処理が変わりますので提示してください。

[fana]

> Form2が閉じたときにForm1をアクティブにして、Form1がアクティブになったときにファイルの読み込み処理をしよう

なんというか，回りくどくないですか？
やりたいことが「ファイルの読込処理」なのであれば，単純に/直接的にその「ファイルの読込処理」を実施する手段を用意すれば良いのでは？　（「アクティブ」とかいう余計な要素をわざわざ間に挟む理由とは何か？）

[fana]

「アクティブになったとき」という実装だと，「Form2が閉じたとき」以外のタイミングでもその処理が走ることになりかねないと思うのですが，そこは問題ないという考えなのでしょうか？

[fana]

あなたの提示コードでは

> Form1 form1 = new Form1();

として，新しいインスタンスを生成していますが，これが所望の処理なのだとすれば，
「Form1のロード時」には件の処理が走る，という話なのですから，特別な対処は何も要らないのではありませんか？
（：単にその新しく生成したフォームを表示すればそこで「ロード時」の処理が走るのでは？）

[fana]

おそらく，

(1)最初，Form1 が存在し，表示されている
(2)あるとき，Form2 を表示する
(3) (2)のForm2が閉じられる
(4) あなたがやりたいこと（Form1がファイルを読込んで何かする）

…という順序で物事が進むのだろうと想像しますが，
上記(2)の時点において，(1)で表示されていた Form1 のインスタンスというのはどうなるのですか？
(1)と(4)に登場する Form1 というのは同一のインスタンスなのですか？　それとも別人ですか？

…といったあたりの話を明確にするべきではないでしょうか？
あなたの提示コードだと「(1)と(4)のForm1は別人」という話と見えますが，そうであれば前記したように特別な対処は不要と思えます．
そうではなくて「同一人物」たるべきなのであれば，別人をnewで生成するのではなく，元から存在した Form1インスタンスに処理をさせるべきでしょう．

[退会済みユーザー]

おっしゃるように、
(1)最初Form1 が存在し，表示されている
(2)Form2画面へのボタンがクリックされたときにForm2 を表示する
(3) (2)のForm2が閉じられる
(4) Form1がファイルを読込む(読み込んだあと、Form1の画面は更新する)
の順序なので、別のインスタンスを生成する必要はなさそうです。勉強不足で申し訳ありません。

ファイルの読み込み処理自体はロード時に実装済みです。Form2の表示とともに該当ソースを質問に追加しました。
Form1は最初に開いた時以外は常に開いている状態なので、そこから再びロードの処理を走らせる方法が質問の趣旨でした。

[KOZ6.0]

コードが崩れています。
編集がうまくいっていないようです。

[退会済みユーザー]

修正しました。

Answer 1

用途にもよりますがForm2は、ShowDialogのほうがいいんじゃないでしょうか。
Form.ShowDialog メソッド (System.Windows.Forms) | Microsoft Learn

ShowDialogは、モーダル（いわゆるダイアログ。閉じないと親を操作できない）でウィンドウを表示します。
Showは、モードレス（ペイントソフトの色選択窓のように常に表示され、どちらも操作可能）でウィンドウを表示します。

Showだとボタンを押すたびにウィンドウが増えていきますし、閉じるのを待つこともできません。
ShowDialogであれば複数開く心配はない（開いている間Form1はいじれないのでそもそもボタンが押せない）し、閉じるのを待つことができます。

もちろんモードレスのまま複数開かないように作ることも可能ですが、いろいろ考慮してそのためのコードを書かなくてはなりません。

using System;
using System.IO;
using System.Windows.Forms;


namespace Qp111xlug8rrwo4
{
    public partial class Form1 : Form
    {
        public static string FilePath => @"test.txt";

        public Form1()
        {
            InitializeComponent();

            // デザイナで変更した各プロパティ（デザイナでやっていればここに書く必要はない）
            textBox1.Multiline = true;
            textBox1.ReadOnly = true;

            // ファイルは必ず存在しているとする
            File.WriteAllText(FilePath, "ようこそ\r\n");

            LoadFile();
        }

        private void LoadFile()
        {
            // ファイル読み込み
            textBox1.Text = File.ReadAllText(FilePath);
        }

        private void EditButton_Click(object sender, EventArgs e)
        {
            var form2 = new Form2();

            // Form2をモーダルで開き、閉じるのを待つ
            if (form2.ShowDialog() == DialogResult.OK)
            {
                // OKを押して閉じたらならファイル読み直し
                LoadFile();
            }
            // キャンセルや×で閉じたなら何もしない

            form2.Dispose();
        }
    }
}

using System;
using System.IO;
using System.Windows.Forms;

namespace Qp111xlug8rrwo4
{
    public partial class Form2 : Form
    {
        public Form2()
        {
            InitializeComponent();

            // デザイナで変更した各プロパティ（デザイナでやっていればここに書く必要はない）
            okButton.DialogResult = DialogResult.OK; // 押すとShowDialogの戻り値にOKをセットし自動で閉じる
            cancelButton.DialogResult = DialogResult.Cancel; // 押すとShowDialogの戻り値にCancelをセットし自動で閉じる
            textBox1.AcceptsReturn = true;
            textBox1.AcceptsTab = true;
            textBox1.Multiline = true;

            // ファイル読み込み
            textBox1.Text = File.ReadAllText(Form1.FilePath);
        }

        private void OkButton_Click(object sender, EventArgs e)
        {
            // OKの時はファイル上書き
            File.WriteAllText(Form1.FilePath, textBox1.Text);

            // DialogResultを設定してるので勝手に閉じる
        }

        // Cancelの時は何もする必要はないのでハンドラ自体不要
    }
}

Comments

[TN8001]

個人的にはなんでもかんでもモーダルダイアログにするUIは大っ嫌いですが^^;

[TN8001]

> とりあえずForm1のロードとデータの読み込み処理を分ける方法で解決しそうです！
> 処理を分けるという発想がなかなかできなかったので、大変勉強になりました。

４件も回答貰って感想これだけかぁ。全然話が通じてないんすね。。。
１年後にもう一度見直してもらうとどういうことを言っていたのかわかるかもしれませんね。

Answer 2

Form1 の修正サンプルです。

private void Form1_Load(object sender, EventArgs e) {
    // ロード時のファイル読み込み処理は別メソッドとして分離
    LoadData();
}

private void LoadData() {
    // データ読み込み処理を記述
}

private void button1_Click(object sender, EventArgs e) {
    // Form2のインスタンスを生成
    Form2 form2 = new Form2();
    // Form2 が閉じられたイベントをキャッチする
    form2.FormClosed += Form2_FormClosed;       // 追加
    // form2を表示
    form2.Show();
}

private void Form2_FormClosed(object sender, FormClosedEventArgs e) {
    // Form2 が閉じられたら読み込み処理を行う
    LoadData();
}

1. Form_Load の処理中に、データを読み込む処理が書いてあるんですかね？読み込みの処理は分離しましょう。サンプルでは LoadData というメソッドに分離してます。

2. Form2 のインスタンスを作成したら、FormClosed イベントに Form2_FormClosed メソッドを関連付けます。これにより、Form2 が閉じられたことがわかります。

3. Form2 が閉じられたら読み込み処理を行います。

注釈なんですが、「Form2 のインスタンスを作成する」なんて必要でしょうか？
それは見ればわかるので、どんな画面を表示するのか書いたほうがいいような気がします。

Comments

[退会済みユーザー]

コメントが遅くなりました。とりあえずForm1のロードとデータの読み込み処理を分ける方法で解決しそうです！
ありがとうございました！処理を分けるという発想がなかなかできなかったので、大変勉強になりました。

Answer 3

些細なことかもしれませんが，

Form2が閉じられたとき

という話は，今々の Form2 側の動作から来ているのでしょうけど，
本来的(?)には Form1 側がデータを読込んで表示更新するという処理を実施する必要性があるのは「データが保存（更新）されたとき」だと思う（：「Form2が閉じたとき」ではなくて）．

登場人物が２つのフォームしかいない現状では，最も簡単な(直接的な?)形で済ませれば良いでしょうが，
今後あれこれとプログラムが発展していくような話があり得る場合には，個々のフォーム間で直接的にやりとりするのではなく，例えば…

• 【データを保存（更新）し → データが更新されたことを知る必要があるやつらに「更新されましたぜ」と通知する，　という仕事を行うやつ】というのを用意して…
• Form1 はあらかじめそいつに「じゃあ，そんときには俺に通知してくださいね」と依頼しておき…
• Form2 はデータの保存処理をそいつにやらせる（→結果としてそいつから Form1 に通知がいく）

…みたいな形を考えてみておいても良いかも…？　とか．

Answer 4

お二人からモーダルウィンドウとイベントハンドラを使った回答が出ているので、別の方法としてForm2からForm1内の関数を呼び出す例を書きます。
（Form1が先に閉じられる問題はこの例では考慮しません。）

// == Form1 ====================
public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
    }

    private void フォーム2起動ボタン_Click(object sender, EventArgs e)
    {
        var form2 = new Form2(this);
        form2.Show();
    }

    public void UpdateScreen(string message)
    {
        //画面を更新。好きに書き換えてください。
        textBox1.Text = messge;
    }
}

// == Form2 ====================
public partial class Form2 : Form
{
    private Form1 _form1;

    public Form2(Form1 form1)
    {
        InitializeComponent();
        _form1 = form1;
    }

    private void 保存_Click(object sender, EventArgs e)
    {
        //Form1の関数を呼び出してデータを反映
        _form1?.UpdateScreen(textBox1.Text);
        //必要ならFrom2を閉じる
        //this.Close();
    }
}