C# と VB.NET の質問掲示板

こんにちは。
重たい処理をするのに↓のようなことをしているのですが、
(1)を書くとなぜか(2)に来なくなってしまいます。
何が良くないのでしょうか。
よろしくお願いします。

using System.Threading;
using System.Threading.Tasks;
using System.Windows;

namespace Sketch {

  public partial class MainWindow : Window {

    public MainWindow() {
      InitializeComponent();
    }

    class AsyncTest {
      private bool busy = false;

      public bool IsBusy() {
        return busy;
      }

      public async void AsyncMethod() {
        busy = true;
        await Task.Run(() => {
          HeavyWeightMethod();
        });
        busy = false; //(2)ここへ来なくなる
      }

      public void HeavyWeightMethod() {
        Thread.Sleep(1000);
      }
    }

    private void Button_Click(object sender, RoutedEventArgs e) {
      var asyncTest = new AsyncTest();

      asyncTest.AsyncMethod();

      while(asyncTest.IsBusy()) { //(1)これを書くと...
        Thread.Sleep(100);
      }

      MessageBox.Show("Done");
    }
  }
}

肝心のUI側でSleepしてたら台無しです。
まずUI側をasync/awaitで書く必要があります。

private async void Button_Click(object sender, RoutedEventArgs e)
{
    var asyncTest = new AsyncTest();
    await asyncTest.MethodAsync(); // awaitableなメソッドはAsync接尾辞がお約束
    MessageBox.Show("Done");
}

await MethodAsyncするにはMethodAsyncがTaskを返す必要があるので、
MethodAsyncはこんな感じになるでしょう。

public async Task MethodAsync()
{
    await Task.Run(() => {
        HeavyWeightMethod();
    });
    // または単純に、asyncを外したうえで
    // return Task.Run(() => HeavyWeightMethod());
}

2019/07/18(Thu) 09:26:47 編集(投稿者)
2019/07/18(Thu) 09:26:38 編集(投稿者)

<pre><pre>みなさんありがとうございます。
魔界の仮面弁士さんの2番目ので思った動きになりました。

これに関して教えてほしいことが3つあります。
1) 私のコードで(1)をすると(2)へ来なくなる理由(デッドロック?)
2) 上記に関係すると思いますが、仮面弁士さんの2番目の様にするとうまくいく理由
3) ContinueWith()に渡すAction<Task>の引数tとは何か

お手数ですが、よろしくお願いします。

public async Task AsyncMethod() {
  busy = true;
  var task = Task.Run(() => {
    HeavyWeightMethod();
  });
  await task.ContinueWith(t => {
    busy = false;
  });
}
</pre></pre>

■No91660 (魔界の仮面弁士 さん) に返信
> 以下、自分の認識です。
> おかしな所があったら突っ込み歓迎。

魔界の仮面弁士さんはいつも完璧なので
突っ込みなどないのですが、
同じことの箇条書き説明してみました。

01  class AsyncTest {
02    private bool busy = false;
03    public bool IsBusy() {
04      return busy;
05    }
06    public async void AsyncMethod() {
07      busy = true;
08      await Task.Run(() => {
09        HeavyWeightMethod();
10      });
11      busy = false; //(2)ここへ来なくなる
12    }
13    public void HeavyWeightMethod() {
14      Thread.Sleep(1000);
15    }
16  }
17  private void Button_Click(object sender, RoutedEventArgs e) {
18    var asyncTest = new AsyncTest();
19    asyncTest.AsyncMethod();
20    while(asyncTest.IsBusy()) { //(1)これを書くと...
21      Thread.Sleep(100);
22    }
23    MessageBox.Show("Done");
24  }

UIスレッド
　18,19,07,08,20,04,21,22（ループ）

スレッドA（08から）
　09,14

08のawaitの機能でスレッドAの処理が終了したら、
08はUIスレッドで実行されたので
11もUIスレッドで実行しようとするが、
UIスレッドは20から22のループ中であるため、
UIスレッドに切り替えができない（デットロック）。

kikuさんの行番号を借りて、ソースコードの実行順序を表すと以下の通りです。

ボタンクリックによって、18行目に移行します。
以降、
  18 → 19  → 06 → 07 → 08 → 09 → 20 → 03 → 04 → 05 → 20 (以降、無限ループ)
                                 ↓(別スレッド)
                                 13 → 14 → 15

HeavyWeightMethod()が終わったら、11行目へ移行したいわけですが、
そのためには同一スレッドで動作するButton_Click()の終了を待つ必要があります。
しかし、20行目のループが終了しない(無限ループ)ので、移行できません。

つまり、スレッド待ちによるデッドロックではなく、無限ループによるフリーズが起きて
います。
デバッガで20行目からステップインするとその様子が分かります。

01  class AsyncTest {
02    private bool busy = false;
03    public bool IsBusy() {
04      return busy;
05    }
06    public async void AsyncMethod() {
07      busy = true;
08      await Task.Run(() => {
09        HeavyWeightMethod();
10      });
11      busy = false; //(2)ここへ来なくなる
12    }
13    public void HeavyWeightMethod() {
14      Thread.Sleep(1000);
15    }
16  }
17  private void Button_Click(object sender, RoutedEventArgs e) {
18    var asyncTest = new AsyncTest();
19    asyncTest.AsyncMethod();
20    while(asyncTest.IsBusy()) { //(1)これを書くと...
21      Thread.Sleep(100);
22    }
23    MessageBox.Show("Done");
24  }