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

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

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

3回答

16836閲覧

C# 複数のTimerを処理する際に、実行されない場合がある

ryo_se

総合スコア68

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

1クリップ

投稿2016/06/16 09:21

ボタンを押したときにTimerでボタンの色を変更する処理を作成しています。

下記コードで基本的な動きはできるようになったのですが、
ボタンAとBを交互に素早く押すという動作を行ったとき、
まれにTimerの処理が行われずボタンの色が変化しない現象が発生します。

ログで追ってみたところ、changeメソッドには確かに処理が通っているのですが、
changeBtnColorメソッドの中には処理が入っていないことがわかりました。

原因が分かる方がいらっしゃいましたらご教示お願いいたします。

private void change(int ptn){ //省略 //ボタンAを押したとき if(ptn == 0){ //0.5秒後にボタンAの色を変更 System.Threading.Timer timer1 = new System.Threading.Timer(changeBtnColor, buttonA, 500, 0); //ボタンBを押したとき }else if(ptn == 1){ //0.5秒後にボタンBの色を変更 System.Threading.Timer timer2 = new System.Threading.Timer(changeBtnColor, buttonB, 500, 0); } } private void changeBtnColor(Object state) { //ボタンの色を変える処理 }

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

System.Threading.Timer はインスタンスをどこかに保存しておかないとガベージコレクトされます。

投稿2016/06/16 09:50

Zuishin

総合スコア28656

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ryo_se

2016/06/16 22:58

イメージがわかないのですが、つまりはこういうことでしょうか? 具体例を教えていただければ幸いです。 int i=0; System.Threading.Timer timerArray[] = new System.Threading.Timer[100] System.Threading.Timer timer1 = new System.Threading.Timer(changeBtnColor, buttonA, 500, 0); timerArray[i] = timer1; i++;
Tak1wa

2016/06/16 23:40 編集

timerArrayの宣言をメソッド内ではなくクラスメンバーにて行ってみてはどうでしょう。
guest

0

ベストアンサー

.NET Framework 4 をターゲットにビルドし、動きましたので、今度は大丈夫かと思います。
フォームに複数のボタンを配置し、そのすべての Click イベントに下記のソース中の button_Click を割り当てています。

C#

1using System; 2using System.Collections.Generic; 3using System.ComponentModel; 4using System.Data; 5using System.Drawing; 6using System.Linq; 7using System.Text; 8using System.Threading; 9using System.Threading.Tasks; 10using System.Windows.Forms; 11 12namespace WindowsFormsApplication1 13{ 14 public partial class Form1 : Form 15 { 16 public Form1() 17 { 18 InitializeComponent(); 19 } 20 21 private void button_Click(object sender, EventArgs e) 22 { 23 var callback = new WaitCallback(state => 24 { 25 Thread.Sleep(1000); 26 var button = (Button)sender; 27 button.Invoke((Action)(() => 28 { 29 button.BackColor = button.BackColor == Color.Blue ? Color.Red : Color.Blue; 30 numericUpDown1.Value++; 31 })); 32 }); 33 ThreadPool.QueueUserWorkItem(callback, sender); 34 } 35 } 36}

投稿2016/06/17 06:20

編集2016/06/17 06:21
Zuishin

総合スコア28656

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ryo_se

2016/06/17 11:33

ありがとうございます、処理の方が無事正常に動きました! ちなみに下の配列の方法でも、処理がスルーされることなく動作することを確認いたしました。
Zuishin

2016/06/17 11:38

配列を使ってももちろん良いのですが、使用済みのタイマーをきちんと回収しないとメモリリークしますよ。
ryo_se

2016/06/18 04:28

回収というのはTimerインスタンスを入れた配列の要素にnullを入れる処理でよいのでしょうか?
Zuishin

2016/06/18 05:00

手順を言えば、 1 その Timer に関連付けられたスレッドが終了していることを確認 2 Timer.Dispose() 3 配列からの参照を外す となります。
guest

0

そもそも System.Threading.Timer の使用自体が Windows Forms で推奨されていませんので、例えばタスクを作ってその中で一定時間待つというのはどうでしょうか?
タスクならば終了までインスタンスは保持されます。

ソースを書くので、コメントではなく別回答とさせてください。

ボタンを二つ置いて、その Click イベントはどちらも同じ button_Click を参照しています。
Task.Delay(1000) は 1 秒待つことを表します。
フォームの UI を触る時には Control.Invoke() を使います。
ここでは Button.Invoke() を使っていますが、「このボタンを作ったスレッドで実行しますよ」という意味なので、コントロール毎に変えなくて結構です。

C#

1using System; 2using System.Collections.Generic; 3using System.ComponentModel; 4using System.Data; 5using System.Drawing; 6using System.Linq; 7using System.Text; 8using System.Threading.Tasks; 9using System.Windows.Forms; 10 11namespace WindowsFormsApplication1 12{ 13 public partial class Form1 : Form 14 { 15 public Form1() 16 { 17 InitializeComponent(); 18 } 19 20 private void button_Click(object sender, EventArgs e) 21 { 22 Task.Factory.StartNew(() => 23 { 24 changeBtnColor((Button)sender); 25 }); 26 } 27 28 private async void changeBtnColor(Button button) 29 { 30 await Task.Delay(1000); 31 button.Invoke((Action)(() => 32 { 33 button.BackColor = button.BackColor == Color.Blue ? Color.Red : Color.Blue; 34 })); 35 } 36 } 37} 38

投稿2016/06/17 00:20

Zuishin

総合スコア28656

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ryo_se

2016/06/17 02:40

ありがとうございます。 開発環境がvisualstudio 2010を使用しているため.NETのバージョンが4であり、 ご提示していただいたコードのasync等が使用できないため実装することが難しいです・・。
Zuishin

2016/06/17 03:06

では async を取って、await Task.Delay(1000); を Task.Delay(1000).Wait() に変更してみてください。
Zuishin

2016/06/17 03:08

色の変化だけ見ると、タイミングによっては取りこぼしがあるように見えることがあるかもしれないので、NumericUpDown を一つ追加し、ボタンの色を変えているところでそれのカウントも上げてみてください。
Zuishin

2016/06/17 03:11

具体的には、button.BackColor = ... の次の行に numericUpDown1.Value++; を入れるだけです。
ryo_se

2016/06/17 05:01

言葉足らずで申し訳ありません。 awaitも.NETのバージョン関係で使用ができないです。
Zuishin

2016/06/17 05:11

だから修正してくださいと書いています。
ryo_se

2016/06/17 05:45

すみません、間違えていました。 Delayの部分で定義がないと怒られてしまうのです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問