【C＃】非同期の処理順について

前提・実現したいこと

APIの同期・非同期について無知なため教示いただきたいです。
環境
Visual Studio 2018
C#

現状の問題点

ボタンを押した際にAPIにGETリクエストを送り、返ってきた結果をラベルに表示されるというものですが、ブレイクポイントで処理順を確認してみると
表示⇒GETリクエスト
という順で処理がされています。

async/await　の記事をみて実装してみたのですが、awaitで待っているような気配が無く質問させていただきました。

自分の中では
GETリクエストメソッドが実行される⇒awaitで結果が保管されるまで待つ⇒メインメソッドに戻り表示
という認識だったのですが、awaitの使い方が間違っているのでしょうか？

ソースコード

    static public string token;

    private void button1_Click(object sender, EventArgs e)
    {
        string url = "https://hogehoge.com";
        var data = Task.Run(() =>
        {  return HttpGet(url, token); });

        label1.Text = data.Result;

}
//GETリクエスト処理
async public Task<string> HttpGet(string url, string token)
{
var client = new HttpClient();
client.DefaultRequestHeaders.Add("Authorization", "Bearer " + token);

        var response = await client.GetAsync(url);
        return await response.Content.ReadAsStringAsync();
    }

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

回答2件

ベストアンサー

こんにちは。

もしかして、ブレークポイントのステップ順序というのは、「data.Resultの行に行った次にTask.Runの中に行く」という順序ではありませんか？
dataはawaitしていない単なるTask.Runなので、そこで非同期的に実行するだけして自分はそのまま先に進むことになります。
しかし、その直後にdata.Resultの呼び出しを行っており、そこでdataの結果が得られるまでブロックしているため、結果自体はlabel1に正しく反映されるように見えます。実際には、全く非同期になっていません。
正しくは、button1_Click自体を非同期メソッドにして、その中でlabel1.Textへの書き換えを「HttpGetの続き」として記述します。

非同期の部分だけ書き直してみました。テストはしていませんが。

csharp
1static public string token;
2private async void button1_Click(object sender, EventArgs e)
3{
4    string url = "https://hogehoge.com";
5    var data = await HttpGet(url, token);
6    label1.Text = data;
7}
8//GETリクエスト処理
9public async Task<string> HttpGet(string url, string token)
10{
11    var client = new HttpClient();
12    client.DefaultRequestHeaders.Add("Authorization", "Bearer " + token);
13    var response = await client.GetAsync(url).ConfigureAwait(false);
14    return await response.Content.ReadAsStringAsync().ConfigureAwait(false);
15}