【C#.NET】非同期処理(async)でハマった話

asyncを使った非同期処理でそこそこハマってしまったので、結果から言えば大した話ではないのだけど自戒を込めてメモ。

以下のようなコードがありました。

private async void message()
{
    MessageBox.Show("メッセージ１");

    messageAsync();

    MessageBox.Show("メッセージ３");
}

private async void messageAsync()
    {
    await Task.Run(() =>
    {
        System.Threading.Thread.Sleep(2000);

        MessageBox.Show("メッセージ２");
    });
}

本来は
メッセージ１
↓
メッセージ２
↓
メッセージ３

と表示して欲しいのですが、実際には以下のような動きとなります。

メッセージ１
↓
メッセージ３
↓
メッセージ２

この場合の正しいコードは以下。

private async void message()
{
    MessageBox.Show("メッセージ１");

    await messageAsync();

    MessageBox.Show("メッセージ３");
}

private async Task messageAsync()
{
    await Task.Run(() =>
    {
        System.Threading.Thread.Sleep(2000);

        MessageBox.Show("メッセージ２");
    });
}

非同期処理を行うメソッド側でTaskを戻り値とし、呼び出し元はきちんと待ち合わせしてあげましょうと。
これをやらないと単なる投げっぱなしのスレッドになってしまいますよ・・・というお話でした。。