C# text boxに表示される値が変化しない

Question

実現したいこと

FormAにあるスタートボタンを押したらデリゲートを作成し、500ms毎に値を1ずつカウントアップし、テキストボックスに表示出来るようにしたいです。
500ms毎に値をカウントアップさせる関数は、FormBから呼び出す仕様にしたいです。

発生している問題・分からないこと

プログラム内部ではFormBから関数が呼び出されていて、500ms毎に値がカウントアップされているところまでは出来ました。(Massagebox.show()を使って確認しました。)。

しかしFormAのテキストボックスには、初期値のみ表示されているだけで数値がカウントアップされていないです。(恐らく、表示を行う関数が1回しか呼ばれていない事が原因です。)。
InvokeをFormBの該当関数内に記載してみたりインスタンスを生成したりしたのですがエラーをはくだけで何も変わりませんでした。色々調べたりもしましたが、解決への糸口が見つけられずに困っています。
テキストボックスの値を変更させるためには、どのようにしたら良いかアドバイス頂けないでしょうか？

該当のソースコード

using System;
using System.CodeDom;
using System.Collections.Generic;
using System.ComponentModel;
using System.ComponentModel.Design;
using System.Data;
using System.Diagnostics.Eventing.Reader;
using System.Drawing;
using System.Globalization;
using System.Linq;
using System.Runtime.ConstrainedExecution;
using System.Runtime.InteropServices;
using System.Security.Cryptography.X509Certificates;
using System.Text;
using System.Threading;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Windows.Forms.DataVisualization.Charting;
using System.Web;
using static System.Windows.Forms.VisualStyles.VisualStyleElement;
using System.Timers;
using TextBox = System.Windows.Forms.TextBox;
using Timer = System.Windows.Forms.Timer;
using System.Runtime.InteropServices.WindowsRuntime;
 
namespace TimeCount
{
    public partial class FormA : Form
    {
        int value1 = 5;
        int value2 = 0;
        int value3 = 100;
        
       public void Value1() {
             value1++;
        }
        public void Value2() {
            value2++;
        }
        public void Value3()
        {
            Value3++;
        }
        
        public System.Timers.Timer timer500ms = new System.Timers.Timer { AutoReset = true, Interval = 1000 };
        public delegate void DelegateProcess();
        public DelegateProcess timer500msDP;

         public void ResultOutput(object sender, EventArgs e)
        {
            textBox1.Text = value1.Tostring();
	    textBox2.Text = value2.Tostring();
	    textBox3.Text = value3.Tostring();
	}

	public void buttonStart(object sender, EventArgs e)
        {
            FormB fmB = new FormB();            
            timer500msDP = new DelegateProcess(ResultOutput);
            this.Invoke(timer500msDP);
            timer500ms.Start();
            timer500ms.Elapsed += fmB.500ms;
        }

        private void buttonStop(object sender, EventArgs e)
        {
            timer1s.Stop();
        }

using System;
using System.CodeDom;
using System.Collections.Generic;
using System.ComponentModel;
using System.ComponentModel.Design;
using System.Data;
using System.Diagnostics.Eventing.Reader;
using System.Drawing;
using System.Globalization;
using System.Linq;
using System.Runtime.ConstrainedExecution;
using System.Runtime.InteropServices;
using System.Security.Cryptography.X509Certificates;
using System.Text;
using System.Threading;
using System.Threading.Tasks;
using System.Windows.Forms;
using System.Windows.Forms.DataVisualization.Charting;
using static System.Windows.Forms.VisualStyles.VisualStyleElement;
using System.Timers;
using static CanTestTool_XlDrvLib.Form2;
using Timer = System.Windows.Forms.Timer;
using vxlapi_NET;
 
namespace TimeCount
{
    public partial class FormB : Form
    {
       FormA fmA = new FormA();

       public void 500ms(Object source, System.Timers.ElapsedEventArgs e)
       {
            fmA.Value1();
            fmA.Value2();
            fmA.Value3();
        }
    }
}

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

InvokeやBeginInvokeを使用してみましたが、「ウィンドウハンドルが作成される前コントロールで Invoke または BeginInvoke を呼び出せません」のエラーが出て来ました。
その為、while文を使ってウィンドウハンドルが作成されるまでsleep(100)を繰り返して見ましたが、何も変わらなかったです。

補足

特になし

Comments

[KOZ6.0]

おそらくプログラムの抜粋なのでしょうけど、コンパイルエラーが出ています。
まずは小さいプロジェクトを作ってきちんと動くようにしてみましょう。

[YAmaGNZ]

FormBにて新しいFormAを作っていますよ
FormA fmA = new FormA();
その新しいFormAに対して操作しているので表示しているFormAとは別物なので表示が変化しないのでしょう。

[GORIRAIMO]

YAmaGNZさん

ご回答頂き、ありがとうございます。
悩んでいた事が理解できました！

FormA fmA = new FormA;の箇所をコメントアウトし、FormBのpublic void 500msの関数をpublic static void 500msに直して実行して見ました。
そしたら、初期値が表示されただけでなく値も更新できました。しかし、1秒ごとの更新ではなくボタンを1回押すごとに更新されてしまいました。
私は、FormA fmA = new FormA;を定義してInvokeを使った方がうまく行くと思い、修正&実行も試みましたが、エラーが出てきてしまいました。

行き詰まりを感じているのですが、アドバイスやヒントを頂けないでしょうか？

[GORIRAIMO]

KOZ6.0さん

ご回答頂き、ありがとうございます。
仰る通り、プログラムの抜粋となります。(抜粋とは言え、現在私が修正を行なっているプログラムはFormAとFormBの2つになります。)

他のプログラムはサンプルとしてもらったもので、全てある状態で実行すると、エラーは出ないのですが、期待値(1秒ごとにテキストボックスの値を1ずつ増やす)には程遠い結果となっています。

お手数おかけしてしまい、申し訳ございません。

[YAmaGNZ]

FormBにてFormAをnewするのではなく
FormAを受け取るようなプロパティなり、コンストラクタの引数なりにしてFormB側へFormAのインスタンスを渡せばいいだけだと思います。

[len_souko]

FormA fmA = new FormA;
これは呪文ではありません
ちゃんと意味があります
その意味を理解していないようですので、
> 私は、FormA fmA = new FormA;を定義してInvokeを使った方がうまく行くと思い
等という返答をしています
まずは最低限、クラスとインスタンスについて勉強してください
その理解がないと上記のコードがダメだという指摘を一生理解できません

[GORIRAIMO]

YAmaGNZさん

コメント頂き、ありがとうございます。
また、返信遅くなってしまい申し訳ございません。

参照渡しを使ってやって見ました。1秒ずつ値をカウントアップするところまでは出来ましたが、テキストボックスの値は変化せずでした。

まだ私の理解が甘かった所があると思うので、もう少し自分なりに考えてみます。

[GORIRAIMO]

len_soukoさん

コメント頂き、ありがとうございます。 また、返信遅くなってしまい申し訳ございません。

ご指摘頂いた通り、私自身完璧に理解できていなかったと思います。
もっと、クラスやインスタンスについて基本的なところを勉強し、今回出来なかった所の原因を考えてみます！

[len_souko]

変更したソースの提示がないので断言はできませんが、ほかの方が書かれているコードを見た感じ、
> 参照渡しを使ってやって見ました。
とありますが多分参照渡しは使っていないかと思われます。
直感ですがよくjavaの人が参照渡しと誤って説明していると例に出る参照型の値渡しのことかと思われます
この誤解の原因は個人的にはほとんどの言語で入門者向けに行われている変数の説明に「値を入れる箱」とあるのが原因だと思っています
その言葉を流用するならば、実際には「値を入れている箱への参照のエイリアス（別名）」の方が近いかと思います
値型の場合は変数の示すポインタからその型の必要とするサイズ分連続したメモリ領域をその型として読み込んで使用しますし、参照型の場合は今だと大抵64bitの動作環境になるので、64bitつまり8byte分の連続した領域をポインタとして読み込み、そのポインタの示すメモリ領域を起点として型の示す連続した領域を使用する感じですね
参照渡しの場合は変数（の示すエイリアス）そのものを渡しているので、newした場合は呼び出し先と呼び出し元の両方のインスタンスが常に同じですが、参照型の値渡しの場合は変数自体は別で変数の（示すそれぞれのポインタに格納されている）値が同じなだけなので、呼び出し先でnewした場合は呼び出し元の変数の値と呼び出し先の変数の値は別物となってしまい、呼び出し先でそのインスタンスに対して変更した値は呼び出し下では参照されていないので見失ってしまいます
c#では引数にrefなどのキーワードで参照渡しを指定できます

[GORIRAIMO]

len_soukoさん

ご回答頂き、ありがとうございます！
参照渡しの解説、分かりやすいです！
参考になりました^ ^

また今回の質問なのですが、無事に解決致しました。
色々アドバイスをくださり、ありがとうございます！

勉強不足の私ですが、少しずつ精進していきます！

Answer 1

FormAにあるスタートボタンを押したらデリゲートを作成し、500ms毎に値を1ずつカウントアップし、テキストボックスに表示出来るようにしたいです。

System.Timers.Timerを使っているので「デリゲート」うんぬんが出てくるのでしょうが、System.Windows.Forms.Timerに変更はできないのでしょうか？
そうすれば難しいことを考えずに済むのですが...
Timer クラス (System.Timers) | Microsoft Learn
Timer クラス (System.Windows.Forms) | Microsoft Learn

using System;
using System.Windows.Forms;


namespace Qcqonfgr3inq2r5
{
    public partial class FormA : Form
    {
        private int value1 = 5;
        private int value2 = 0;
        private int value3 = 100;
        private System.Timers.Timer timer500ms = new System.Timers.Timer
        {
            AutoReset = true,
            Interval = 500,
        };
        //private System.Windows.Forms.Timer timer500ms = new System.Windows.Forms.Timer
        //{
        //    Interval = 500,
        //};

        public FormA()
        {
            InitializeComponent();
        }

        public void Value1() { value1++; }
        public void Value2() { value2++; }
        public void Value3() { value3++; }
        public void ResultOutput()
        {
            textBox1.Text = value1.ToString();
            textBox2.Text = value2.ToString();
            textBox3.Text = value3.ToString();
        }

        private void buttonStart(object sender, EventArgs e)
        {
            FormB fmB = new FormB(this);
            fmB.Show();

            timer500ms.Start();
            timer500ms.Elapsed += fmB.Timer500ms_Elapsed;
            //timer500ms.Tick += fmB.Timer500ms_Tick;
        }

        private void buttonStop(object sender, EventArgs e)
        {
            timer500ms.Stop();
        }
    }
}

using System;
using System.Windows.Forms;


namespace Qcqonfgr3inq2r5
{
    public partial class FormB : Form
    {
        private FormA fmA;

        public FormB(FormA formA)
        {
            InitializeComponent();
            fmA = formA;
        }

        public void Timer500ms_Elapsed(object sender, System.Timers.ElapsedEventArgs e)
        {
            fmA.Value1();
            fmA.Value2();
            fmA.Value3();

            // Invokeで検索すると（歴史的経緯から）色々出てくるが、記述が短くて済む
            // ラムダ式を組み込みのActionデリゲートにキャストして呼び出し
            fmA.Invoke((Action)(() => fmA.ResultOutput()));

        }
        public void Timer500ms_Tick(object sender, EventArgs e)
        {
            fmA.Value1();
            fmA.Value2();
            fmA.Value3();

            // System.Windows.Forms.Timerなら（UIスレッドで呼ばれるので）何も考えないでよい
            fmA.ResultOutput();
        }
    }
}

FormAとFormBの関係がわからないので、回答コードではタイマーのイベント解除をしていません。
そのため「スタートボタン」を押すたび、１ずつ・２ずつ・３ずつ...とカウントアップ数が増えていきます。

こういった方法で都度newしなければ、そのあたりを気にしないで済むかもしれません。
フォームの×ボタンで閉じないで、Hide にする。 - SN工房

追記

using System;
using System.Windows.Forms;


namespace Qcqonfgr3inq2r5
{
    public partial class FormA : Form
    {
        public FormA()
        {
            InitializeComponent();

            // FormCは閉じずにHide作戦なので、newは１回だけ
            fmC = new FormC(this);
            timer2.Tick += fmC.Timer_Tick;
        }

        #region FormB
        private readonly Timer timer1 = new Timer { Interval = 500, };
        private FormB fmB; // 最初はnull
        private int value1;

        public void Value1() => value1++;
        public void ResultOutput1() => textBox1.Text = value1.ToString();

        private void ButtonStart1(object sender, EventArgs e)
        {
            // もしFormBがすでにnewされていたら、イベント解除＆閉じる
            if (fmB != null)
            {
                timer1.Tick -= fmB.Timer_Tick;
                fmB.Close();
            }

            // FormBを作ってイベント登録＆表示
            fmB = new FormB(this);
            timer1.Tick += fmB.Timer_Tick;
            fmB.Show();

            timer1.Start();
        }
        private void ButtonStop1(object sender, EventArgs e) => timer1.Stop();
        #endregion


        #region FormC
        private readonly Timer timer2 = new Timer { Interval = 500, };
        private readonly FormC fmC;
        private int value2;

        public void Value2() => value2++;
        public void ResultOutput2() => textBox2.Text = value2.ToString();

        private void ButtonStart2(object sender, EventArgs e)
        {
            fmC.Show();
            fmC.Activate(); // ほかのウィンドウに隠れていたら気が付かないのでアクティブ化
            fmC.WindowState = FormWindowState.Normal; // アイコン化してたら復帰

            timer2.Start();
        }
        private void ButtonStop2(object sender, EventArgs e) => timer2.Stop();
        #endregion
    }
}

using System;
using System.Windows.Forms;


namespace Qcqonfgr3inq2r5
{
    public partial class FormB : Form
    {
        private readonly FormA fmA;

        public FormB(FormA formA)
        {
            fmA = formA;
            InitializeComponent();
        }

        public void Timer_Tick(object sender, EventArgs e)
        {
            fmA.Value1();
            fmA.ResultOutput1();
        }
    }
}

using System;
using System.Windows.Forms;


namespace Qcqonfgr3inq2r5
{
    public partial class FormC : Form
    {
        private readonly FormA fmA;

        public FormC(FormA formA)
        {
            fmA = formA;
            InitializeComponent();
            FormClosing += FormC_FormClosing;
        }

        public void Timer_Tick(object sender, EventArgs e)
        {
            fmA.Value2();
            fmA.ResultOutput2();
        }

        private void FormC_FormClosing(object sender, FormClosingEventArgs e)
        {
            if (e.CloseReason == CloseReason.UserClosing) // ユーザーが閉じようとしたら...
            {
                e.Cancel = true; // 閉じるのを中止して
                Hide(); // 非表示にする
            }
        }
    }
}

Comments

[GORIRAIMO]

TN8001さん

コメント頂き、ありがとうございます。
また、返信遅くなってしまい申し訳ございません。

私の知識不足だったのでSystem.Windows.Forms.Timerを使った方が良かった事知らなかったです。申し訳ございません。

仰る通り、私がやりたかった事が出来ました！
別のフォームから関数を呼び出し、1秒ずつカウントアップするというプログラムを作成し、C#の理解を深める事を目的として行なっています。

なので、FormAとFormBに分けて作成していました。
ただ、カウントアップ数は1ずつアップするで固定したいので、タイマーのイベント解除方法について、1度考えてみたいと思います!

[TN8001]

> 私の知識不足だったのでSystem.Windows.Forms.Timerを使った方が良かった事知らなかったです。申し訳ございません。

いえいえ謝る必要はありません（最初はみな初心者だったのですから）
使用していないusingが大量にあったので、「実際のコードではTimers.Timerを使う必要があるのかな？」と思っていました。
Forms.Timerならデザイナ上で設定もできます。

> ただ、カウントアップ数は1ずつアップするで固定したいので、タイマーのイベント解除方法について、1度考えてみたいと思います!

ということは、「FormBは同時に複数開くことはない」という想定でしょうかね？
「複数開こうとしたら単に無視をする」でいいなら一番楽ですが、FormBを閉じてしまうと再度開く手段がなくなります。

２つ目を開こうとした場合は、１つ目をイベント解除して閉じれば常に１つになりますね。
そのかわり開くたびFormBの状態（位置やサイズ・TextBoxの内容等）がリセットされます。

回答内のリンクのような作りにすればイベント解除は不要ですが、FormClosingの処理が必要です（FormC）
個人的にはFormCのほうが、状態が保存されるので好みです。

回答に追記しました^^

[GORIRAIMO]

TN8001さん

ご回答頂き、ありがとうございます。
「FormBは同時に複数開くことはない」と言う想定です。

追記もありがとうございます^ ^
ご回答頂いた内容を参考にして、プログラムを修正し、私の希望通りの動作を得る事が出来ました！

本当にありがとうございます！