タイマーの中で非同期処理はやっちゃだめですか？

Question

助けてほしいこと

こんにちは。
今、C#の基本をやっています。
Form1クラスでタイマーをセットしているのですが、timer.Stop()でエラーになります。
これはどのようにすればよいか教えてほしいのです。

詳細

タイマーをセットしているところ

//タイマー
System.Timers.Timer timer;

private void Setimer(int time)
{
  // タイマーの生成
  timer = new System.Timers.Timer(time);
  timer.Elapsed +=  (sender, e) =>
  {
    test_async();
    timer.Stop();
    label_Timer.Text = "タイマーとめたよ";
　};

  timer.Start();
  label_Timer.Text = "タイマーはじめました";
}

タイマー時間経過で呼び出している関数

private async Task test_async()
{
	txtdata = textBox.Text;
	if (txtdata == "その１")
	{
		while (Test.Proc1(txtdata) == false)
		{
			//停止になってたらとめる
			if (Stopflg == true) break;
			//再読み込みまで100ms空ける
			await Task.Delay(100);
		}
		return;
	}

	//	非同期で動かす(固まっちゃうので)	
	await Task.Run(async () =>
	{
		while((Test.Proc2(txtdata) == false)
		{
			//停止になってたらとめる
			if (Stopflg == true) break;
			//100ms待ってから
			await Task.Delay(100);
		}
	});
	return;
}

わからないこと

test_async();

がなければ、エラーになりませんので、ここが悪いとは思います。

await test_async();

にしてもだめでしたので、悩んでいます。
タイマーの呼び出しの中で非同期処理をしている関数を呼んではいけない、ということですか？

参考

参考にさせていただいたのはこちらの中ほどのコードでした。
https://webbibouroku.com/Blog/Article/cs-timer

エラー

例外がスローされました: 'System.InvalidOperationException' (System.Windows.Forms.dll の中)
型 'System.InvalidOperationException' の例外が System.Windows.Forms.dll で発生しましたが、ユーザー コード内ではハンドルされませんでした
有効ではないスレッド間の操作: コントロールが作成されたスレッド以外のスレッドからコントロール 'label_Timer' がアクセスされました。

改めて

右も左もわからず、思うがまま組んでみてます。
そもそも、それじゃダメ、という意見もあるかと思います。
よろしければ、教えていただければうれしいです。

よろしくお願いいたします(- -)(_ _)ペコリ

Comments

[y_waiwai]

エラーメッセージを提示しましょう
いらぬ加工をせず、出たそのママを提示してください

[退会済みユーザー]

ありがとうございます
記入しました

[BluOxy]

コントロールが作成されたスレッド以外のスレッドからlabel_Timerにアクセスしたことが原因と書かれています。添付されたコードにそのような処理は書かれていないので、上記の例外と添付されたコードは関係がないように見えます。

[退会済みユーザー]

すみません、変数名label_Timerが上のコードでいうlabelなんです。
短くしようとして変えてました。

[BluOxy]

誤解が生じかねないので、質問には実際のコードを記載してください。

Answer 1

test_async();

timer.Stop();

タイマーを止める前に、test_async()を呼び出しますが、問題ありません?
多重になって問題を起こす気がします。

Comments

[BluOxy]

InvalidOperationException とは別の問題が起きそうですね。

[退会済みユーザー]

ありがとうございます。さらに問題あるのですか。
タイマーは一回きりのつもりで書いていたのですが、多重になるんですね・・・(´・ω・`)
順番ですか。
タイマー止めてからテスト関数して、デリゲート？でラベル書き込みしてみます。
（ラベルは外から見えるようにしたかっただけなので、別になくてもいいのですが・・・）

[BluOxy]

timer.Stop();
test_async();
と順番を入れ替えたら良いです。
現在のコードの場合はtimer.Stop()が呼ばれる前に次のイベントが実行され、多重にtest_asyncが呼ばれる場合があります。

[退会済みユーザー]

ありがとうございます。
その順番にしてみます。
ただ、「次のイベント」というものがしっくり来てないので、間違ったコードでも何が起こるか確認してみます！
ちょっとしたことでエラーが出るの難しいですね。
がんばります！

Answer 2

System.Timers.Timer クラスを見ると、下記のように記載されています。

SynchronizingObject プロパティが null場合、ThreadPool スレッドで Elapsed イベントが発生します。

label_TimerはWindows Formsのコントロールですから、そのコントロールに対して変更ができるスレッドはエラーメッセージに出ている通り「コントロールが作成されたスレッド」だけです。このスレッドは一般的にメイン スレッドと呼ばれます。

それ以外のスレッド(すなわちThreadPoolスレッド もといtimer.Elapsedイベント)でlabel_Timer.Text = "タイマーとめたよ";のようにWindows Formsのコントロールに対してアクセスを行うとInvalidOperationExceptionが発生します。これが今回の原因です。

解決策として一般的な方法はControl.Invoke メソッドを使ってメイン スレッドに処理を委譲することです。

---

今回の問題に関連する記事が公式にありますので、参考にしてください。
方法 : Windows フォーム コントロールをスレッド セーフに呼び出す

Comments

[退会済みユーザー]

ありがとうございます。
test_async();を足したタイミングでエラーになったと思い込んでましたので、これのせいかと思っていましたが、label_Timerを触ったのがダメだったんですね。
(勘違いの可能性もありますが、test_async前は動いていた気がしてます…
リンク先を読ませていただいた限りでは、安全「でない」ということなので、たまたま動いてただけっぽいですが…)

もう少しじっくりいただいたリンク読んでみます！
ありがとうございました。

Answer 3

そのエラーメッセージを読んでください。
別のスレッドからコントロールを操作したがための例外です