UWPでWebAPIを使ったサンプルでエラー

Question

C#でUWPのWebAPIを調べるため、ネットにあったサンプルを利用しようとしているのですが、「'await' 演算子は、非同期メソッド内でのみ使用できます。このメソッドに 'async' 修飾子を指定し、戻り値の型を 'Task' に変更することを検討してください。」とエラーが出ます。
まだ勉強中なため、根本がわかっていないのだと思いますが、このエラーは何が悪いためのエラーなのでしょうか？
初心者にわかりやすく教えて頂けるとありがたいです。
よろしくお願いいたします。

using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
using System.Runtime.InteropServices.WindowsRuntime;
using Windows.Foundation;
using Windows.Foundation.Collections;
using Windows.UI.Xaml;
using Windows.UI.Xaml.Controls;
using Windows.UI.Xaml.Controls.Primitives;
using Windows.UI.Xaml.Data;
using Windows.UI.Xaml.Input;
using Windows.UI.Xaml.Media;
using Windows.UI.Xaml.Navigation;
using Windows.Web.Http;
using Newtonsoft.Json.Linq;
// 空白ページのアイテム テンプレートについては、http://go.microsoft.com/fwlink/?LinkId=402352&clcid=0x409 を参照してください

namespace MaterialMatching_WebAPI
{
    /// <summary>
    /// それ自体で使用できる空白ページまたはフレーム内に移動できる空白ページ。
    /// </summary>
    public sealed partial class MainPage : Page
    {
        public MainPage()
        {
            this.InitializeComponent();
        }

        private void button_Click(object sender, RoutedEventArgs e)
        {
            //ここにボタンがクリックされた時の処理を書く
            string apiKey = "key";
            string zipCode = "123-4567,jp";
            string endpoint = string.Format(
                "http://api.openweathermap.org/data/2.5/weather?zip={0}&appid={1}",
                zipCode,
                apiKey
                );
            //HttpClientを使って天気情報を取得する
            HttpClient client = new HttpClient();
            HttpResponseMessage result = await client.GetAsync(new Uri(endpoint));
            string json = await result.Content.ReadAsStringAsync();
            //jsonを解析して結果を取得する
            JObject root = JObject.Parse(json);
            string weather = root["weather"].First()["main"].ToString();
            textWeather.Text = weather;

        }
    }
}

Answer 1

private void button_Click(object sender, RoutedEventArgs e)

は

private async void button_Click(object sender, RoutedEventArgs e)

と書くべきでは？

Comments

[picotoyou8]

回答、ありがとうございます。
無事にエラーは回避できました。
ただ、実行してみるとボタンを押しても何の反応もしないのですが・・・
現在、原因を探しているところです。

[hidori]

ブレークポイント設定してデバッグ実行、デバッガでステップ実行などしてみるといいんじゃないでしょうか。

[hidori]

```C#
string endpoint = string.Format(
"http://api.openweathermap.org/data/2.5/weather?zip={0}&appid={1}",
zipCode,
apiKey
);
```
query パラメータ名は `appid` なのに、appKey という名前の変数に設定された値を当てはめていますが、そこは大丈夫ですか？

[picotoyou8]

回答、ありがとうございます。
private void button_Click(object sender, RoutedEventArgs e)が作成したボタンとリンクしていないだけでした。
おかげさまで、何とか動くようにはなったのですが、ループをさせて値を取得した時、取得元の値を変更しても同じ値しか取得できません。
そもそも、private void button_Click(object sender, RoutedEventArgs e)内でループさせて値を取得しようとすることが間違いなのでしょうか？

[hidori]

> ループをさせて値を取得した時、取得元の値を変更しても同じ値しか取得できません。

キャッシュされてるんじゃないですか？
レスポンスヘッダにキャッシュ制御のヘッダが含まれてたりはしませんか？

> そもそも、private void button_Click(object sender, RoutedEventArgs e)内でループさせて値を取得しようとすることが間違いなのでしょうか？

ループが長時間に及ぶようなら、キャンセル可能とする等考えるべき点はありますが、
ループさせること自体は間違いではないです。