【C#WPF】ボタンが押されたら処理をやり直したい

Question

###前提・実現したいこと
ボタンA,Bがあり、ボタンAを押した後に関数Aが実行され関数Aの処理が終わらない内にボタンBを押すと関数Bが実行され、関数Aと関数Bが並列に実行されるため処理が変になります。
ボタンを押したら今の処理を終わって新しく処理を開始したいです。
グローバル変数で管理しようと思ったのですが関数内にfor文が幾つかあり、いちいちフラグ管理を入れると長くなるのでもっとスマートに出来たら嬉しいのですが、何かいい方法がありませんか。

###該当のソースコード

 //ボタンAが押された時
 private async void funcA() {
   for(int i =0;i<50;i++) {
   //処理A
　　Slider.Value = i;
　　await Task.Delay(50);
 }
}

//ボタンBが押された時
private async void funcB() {
   for(int i =50;i>0;i--) {
   //処理B
　　Slider.Value = i;
　　await Task.Delay(50);
 }
}

//何かしらのボタンが押された時
 private void Buttons_Clicked(object sender, RoutedEventArgs e)
        {
            String btnName = ((Button)sender).Name.Replace("Button", "");
            switch (btnName)
            {
                case "A": //ボタンAが押された
                    funcA();
                    break;
                case "B": //ボタンBが押された
                    funcB();
                    break;
            }
        }

関数はイメージです。徐々にスライドバーを動かすような処理をしています。実際は複数のスライドバーを順番に動かしているためfor文が幾つかあります。

Answer 1

キャンセルトークンの出番では？

Comments

[nura_nura]

回答ありがとうございます。
調べてみたのですが、クリックイベント(Buttons_Clicked)関数内でキャンセルするということでしょうか。
CancellationTokenSourceとCancellationTokenをグローバルにして処理自体は関数のまま置いときたいのでTask.Run(()=>funcA(),token);などしてクリックイベントの最初でトークンソースをキャンセルしてみたのですが上手くいきませんでした。
初心者なもので具体的な処理方法教えていただけると助かります。

[kiichi54321]

まず、「一度に処理をやり直したい」というのを考えるのをやめて、「適切に終了する」「再開する」ということに問題を分けます。
適切に終了するために、CancellationToken　の　ThrowIfCancellationRequested　で例外を引き起こすことで、Taskを終了させます。
これは、ややこしいフラグ管理をしないために使います。
そして、すべてのTaskの終了を検知するために、Task.WaitAll　をつかます。
これを使うためにも、すべてのTaskを、コレクションで、プライベートフィールドに持つようにします。
Task.WaitAllを待ってから、また、TaskとCancellationTokenを再作成をすれば、再開ができると思います。

ただ、これは、典型的な難易度の高いマルチスレッドプログラミングになるので、初心者であるという自覚があるなら、問題をもっと簡単にしたほうがいいと思います。
（ちなみに、C#には、グローバルという概念は、一応ありません。staticかな。とはいえ、staticを使う話でもありません。）

[nura_nura]

返信ありがとうございます。
色々調べてみましたが難しいようですので今回は諦めます。

Answer 2

お二方の回答ありがとうございました。
私の力不足のため解決に至らず申し訳ありません。
同じボタンの二度押しの対応は諦めました。
根本的な解決にはなってませんがスライダーの値を直接変える処理がfor文で沢山あっただけなので一つ関数をかませて無理やりしました。

int currentMotion = -1;
//ボタンAが押された時
 private async void funcA() {
   int motion = 1;
   for(int i =0;i<50;i++) {
   //処理A
　　setSliderValue(i, motion);
　　await Task.Delay(50);
 }
}

//ボタンBが押された時
private async void funcB() {
   int motion = 2;
   for(int i =50;i>0;i--) {
   //処理B
　　setSliderValue(i, motion);
　　await Task.Delay(50);
 }
}

// スライダーに値を送る
void setSliderValue(int value, int motion) {
  if(motion != currentMotion)
    return;
  Slider.Value = value;
}

//何かしらのボタンが押された時
 private void Buttons_Clicked(object sender, RoutedEventArgs e)
{
 String btnName = ((Button)sender).Name.Replace("Button", "");
 switch (btnName)
 {
   case "A": //ボタンAが押された
       currentMotion = 1;
   　　funcA();
       break;
   case "B": //ボタンBが押された
       currentMotion = 2;
       funcB();
       break;
 }
}

Answer 3

細かいタイミングまで気にしないで良いなら、画面ロックかボタンを使用不可とすれば良いんじゃないでしょうか。

Comments

[nura_nura]

回答ありがとうございます。
すみません、ボタン二重クリックで再実行されるようにしたいです。

[sazi]

＞二重クリックで再実行
とは、実行中の処理をキャンセルして再度実行するということですか？
そもそも、二重クリックとはダブルクリックのことでは無いですよね？
C#は詳しくないですが、実行中をキャンセルするとかであればスレッドでの管理が必要じゃないのでしょうか

それよりも、「実行中に別の処理をされたらおかしくなる」というのを抑制する方が先ではないですか

[nura_nura]

返信ありがとうございます。
はい、二重クリックは２度押しの事です。
ボタン２度押しで二つのスレッドが同時に１つのスライドバーを動かすことになるので処理がおかしくなるのは仕方がないです。
難しそうなので諦めます。