awaitの後の挙動

問題

C#初心者です。
ボタン押した後、Main→TaskSampleが呼ばれ、
awaitを実行したあと、SubTask1が動きますが、すぐに戻ってきて"test"が出力されてしまいます。
こういう仕様なのでしょうか。そもそも書きかたがおかしいでしょうか。

実現したいこと

SubTask1が終わり、"Taskおわり"が出力後、"test”が出力されるような動きにしたいです。

該当のソースコード

C#
1        private void Main()
2        {
3            TaskSample();
4            Console.WriteLine("test");
5          
6        }
7
8        private async void TaskSample()
9        {
10            await Task.Run(()=>SubTask1());
11            Console.WriteLine("Taskおわり");
12        }
13
14        private void SubTask1()
15        {
16            Thread.Sleep(10000);
17           
18        }
19
20        private void button1_Click(object sender, EventArgs e)
21        {
22            Main();
23        }
24
25

退会済みユーザー

2022/09/13 01:19

質問者さん、無言ですが、回答したのでそれに対するフィードバックを返してください。役に立った/立たなかったぐらいはすぐに返せるのでは？役に立たなかったならどこがダメかを書くとより期待に近い回答が出てくるかも。とにかく無言は NG です。

退会済みユーザー

2022/09/13 23:29

スレッドを放置しておかないでクローズしてください。問題は解決したはず。解決したら、解決に役に立った回答にベストアンサーをつけてスレッドをクローズするのがここでのマナーです。

行動規範の内容に同意します

回答1件

難しいかもしれませんが、読んでみましょう。

非同期プログラミングのベストプラクティス
https://docs.microsoft.com/ja-jp/archive/msdn-magazine/2013/march/async-await-best-practices-in-asynchronous-programming

「async void を避ける」と「すべて非同期にする」・・・できてないですよね。特に後者が問題。

【追記】

具体的にどう直せばいいかを追記しておきます。

(1) すべて非同期にする

"上から下に (または下から上に) 非同期コードが他の非同期コードを呼び出す"

⇒ button1_Click, Main も async/await を付与して非同期メソッドにします。

(2) async void を避ける

"async メソッドで自然な戻り値の型は Task と Task<T> だけです"

⇒ 戻り値がない場合は async Task とします。ただしイベントハンドラは例外で、async void とするほかありませんので注意。

質問者さんのコードを上の (1), (2) に従って書き換えると以下のようになります（注: button1 ⇒ button3、Main ⇒ Test、Console ⇒ System.Diagnostics.Debug と変更しています）

private async void button3_Click(object sender, EventArgs e)
{
    await Test();
}

private async Task Test()
{
    await TaskSample();
    System.Diagnostics.Debug.WriteLine("test");
}

private async Task TaskSample()
{
    await Task.Run(() => SubTask1());
    System.Diagnostics.Debug.WriteLine("Taskおわり");
}

private void SubTask1()
{
    Thread.Sleep(10000);
}

結果は: