質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.78%

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

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 3,619

score 13

助けてほしいこと

こんにちは。
今、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 == "その1")
    {
        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' がアクセスされました。

改めて

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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • BluOxy

    2020/06/17 21:00 編集

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

    キャンセル

  • ann

    2020/06/17 21:01

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

    キャンセル

  • BluOxy

    2020/06/17 21:05

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

    キャンセル

回答 3

checkベストアンサー

+1

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

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

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

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

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


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/06/17 21:23 編集

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

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

    キャンセル

+1

    test_async();
   timer.Stop();

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/06/17 21:30

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

    キャンセル

  • 2020/06/17 21:52

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

    キャンセル

  • 2020/06/17 22:34

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

    キャンセル

  • 2020/06/17 22:43

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

    キャンセル

0

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.78%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る