Q&A
実行し続けたいということは中でループが実行されているんだと思います。ループを外からのイベントで終了するのは、CancellationToken を使うのが定石です。またそのループはフォームの動きを止めてしまうので、非同期で行うと良いでしょう。
または System.Reactive を使います。こちらの方が簡単かもしれません。
ボタンを右クリックし続けている間に指定した関数を実行し続けたいと考えています。
MouseDown.Up等で実装できるかもしれないと思い試してみたのですが長押しにうまく対応することができません。
具体的に「テキストに100と入力されている→ボタンを押す→テキストが99になりラベル0が1になる」処理を長押しでリアルタイムで確認できるように実装したいと考えています。
どうすれば実装することができるでしょうか？ご教授お願い致します。
C#のFormApplicationを使用しています。
ご自身が回答に対して試されたことを具体的なコードも出さずに「出来ませんでした。どうやればいいですか？」とだけ聞かれても回答者の方は困るのではないでしょうか？
「テキストに100と入力されている→ボタンを押す→テキストが99になりラベル0が1になる」とのことですが
ボタンを押している間、テキストは１づつ減っていき、ラベルは１づつ増えていくということですか？
それはどれくらいのスピードですか？
ただループで減らす（増やす）のであれば、100くらいであれば一瞬で終わってしまい視認することは無理だと思います。
UI の話なら一瞬で数字がいくつも変わったのでは使いにくいのでタイマー処理は必須でしょうね。それも押し始めと中頃では速度を変える方が使いやすくなると思います。System.Reactive の出番ですね。
＞YAmaGNZ
>テキストは１づつ減っていき、ラベルは１づつ増えていくということですか？
それを想定しています。スピードに関しては視認できるものを想定していたのですが厳しいですかね...
>Zuishin
timerの処理調べて試してみます！ありがとうございます。
NumericUpDown+ValueChangedで満足できるならそれが一番簡単です。
タイマーでやるならこちらはどうですか。
[Auto-repeat Button in 10 Minutes - CodeProject](https://www.codeproject.com/Articles/629644/Auto-repeat-Button-in-10-Minutes
回答4件
0
- MouseDown で目的の処理を開始
- MouseUp で目的の処理を終了
すればよいです。
ただし、マウスイベントのハンドラに目的の処理をまんま書いてしまうと UI の処理がブロックしてしまうので、目的の処理は
- BackgraoundWorker
- Task
- Thread
などから、用途に適したものを使用して、UI 処理とは異なるスレッドで実行してください。
投稿2020/07/14 12:54
総合スコア403
回答ありがとうございます。
非同期処理について調べ、中でもTaskが適していると思い、試したのですが1つ1つ実行されずに規定数一気に実行されてしまいます。
1つ1つ実行される方法はないでしょうか？
どういう動作を望んでいて、どういうコードを書いたのでしょうか？
何の脈絡もなく、「１つ１つ」や「規定数」と言われても、よく分からないです。
仮に、要件が
* 実行するべき処理がいくつかあって、ボタンを押している間にそれらを１つずつ実行したい
* ボタンを離したら、処理を中断したい
だとして
* ボタンを離した時に実行中の処理はどうしますか？（中断？継続？）
* 再度ボタンを押された時には、どの処理から実行しますか？
などなど。
動作に関しては追記させていただきました。
コードに関しては
>for(i;i<規定数;i++)
>await Task.Run(() => Lottery(i));
のように書きました。説明不足で申し訳ありません。
そのコードをどこに書きましたか？ MouseDown イベントのハンドラの中ですか？
どこに書いてるかにもよりますが、await が機能してるなら、Task.Run() は並列ではなく、順次実行されるはずですよ。
MouseButtondown内に書きました。
Windows Forms では、イベントハンドラの中に await が書けなかったような気がするのですが、ビルド出来てますか？
awaitでエラーが起こりビルドできなかったので調べたところasyncをMouseButtonに追記することで使用することができるとサイトい書いてありビルドエラーは回避することができました。
なるほど。WPF ではそれで良かった気がしますが、Windows Forms でもそれでいいんですね。
で、話は戻りますが
```
for(i;i<規定数;i++)
>await Task.Run(() => Lottery(i));
```
には「MouseUp で処理を中断する」実装が無いですよね。
処理を中断するコードが書かれていなければ、例えば規定数＝１００であれば、中断されることなく１００回ループが回るのは当然な気がしますが。。
> Windows Forms では、イベントハンドラの中に await が書けなかったような気がするのですが、
最近できるようになりました。
★+1
0
長押し
「押した」と「離した」とをハンドリングすればよいのではないでしょうか．
（「押されてから→離されるまで」が処理実行期間）
投稿2020/07/14 07:43
総合スコア11996
ベストアンサー
回答ありがとうございます。
試してたところ、1度長押しした際には1づつ増減するのですが、1度離し再度長押ししたところ1づつ増減するのではなく、2づつ増減するようになりました。その後同じように離して長押しすると3づつになっていしまいました。コードは下記のように行いました。
timer.Elapsed += (sender, e) =>
{
int a+=1;
int b-=1;
text=a;
label=b;
}
常に1づつ増減する場合は+＝や-＝ではうまくいかないのでしょうか？
ボタンを押した時にイベントハンドラを登録しているなら、離した時にイベントハンドラを削除しなけばなりません。
そうしないのであれば、フォームのロード等で1回だけイベントハンドラを登録し、ボタンの押した/離したのところはタイマーの開始/停止のみにしてください。
timerのイベントハンドラの削除していませんでしたが、削除方法が調べてもよくわかりませんでした...
timer.Elapsed -= new ElapsedEventHandler();
の()にどの引数を指定すればハンドラ削除できますでしょうか？またはまずハンドラ削除の方法が違いますでしょうか？併せて教えてほしいです。
ラムダ式を使ってのイベントハンドラを登録すると削除できません。
EventHandler handler;
handler = (sender, e) =>
といった感じで一旦変数に受けてから
timer.Elapsed += handler;
と追加
timer.Elapsed -= handler;
と削除してください。
handlerは適切なスコープで宣言してください。
ElapsedEventHandler handler;
// タイマーを開始する
timer.Start();
// タイマーの処理
handler = (c, a) =>
{
timer.Elapsed += handler;
.
.
.
}
のように書いた場合「割り当てされていないローカル変数が使用されました」とビルドエラーが起こってしまいます...
何故、ハンドラーの定義の中でハンドラーの追加を行うのですか？
ボタンが押されたときにイベントハンドラを追加するのではなく、フォームのロードなど、起動されてすぐに1回だけ追加してください。
もしくは、フォームデザイナーでタイマーを追加してイベントハンドラを設定してください。
そのほうが簡単です。
＞フォームデザイナーでタイマーを追加してイベントハンドラを設定してください。
そのほうが簡単です。
試してみたらできました！ありがとうございます！
0
ボタンを右クリックし続けている間
マウスの右クリックという事でしたら、ボタンのマウスイベントを使えば、良いのでは?
例えば、bool progRun とかいうメンバ変数を用意して、
MouseDown(右ボタンを検出し)で、progRun = true;
MouseUpで、progRun = false;
別スレッドで、動いているメソッドで、この progRunが、trueの間だけ、処理を行うようにすれば、出来ます。
投稿2020/07/14 12:18
総合スコア6385
回答ありがとうございます。
trueの間do..whileで実行するように試してみたのですが、一気に実行されてしまい1つ1つ実行されずに規定数一気に実行されてしまいました...
1つ1つ実行する方法はないでしょうか？
他の回答へのコメントまで追い切れてませんので、解決済みかもしれませんが。
> 1つ1つ実行する方法はないでしょうか？
1つ終わったら、true->false にするだけでは?
あなたの回答
tips
プレビュー
