C# Веб-запрос вызывает сбой

Question

Я спросил об этом раньше, но ни один из ответов не закончился. Итак, у меня есть класс, который делает веб-запрос для получения строки данных и метод в этом классе, который вызывает метод для этого, а затем возвращает строковое значение.

У меня есть класс MainPage, класс City (WebRequest возвращает строку JSON, и этот класс анализирует ее и сохраняет данные), и у меня есть класс WebRetrieval.

В MainPage у меня есть это:

private void City_GoButton_Click(object sender, RoutedEventArgs e) 
{ 
    City cityObject = new City(); 
    WebRetrieval wr = new WebRetrieval(); 

    string jsonString; 
    string formattedString; 

    if (String.IsNullOrWhiteSpace(CityID_Textbox.Text)) 
    { 
     MessageDialog md = new MessageDialog("Please enter in a number"); 
     md.ShowAsync(); 
    } 
    else 
    { 
     long cityNum = Convert.ToInt64(CityID_Textbox.Text); 
     string url = "http://honey.computing.dcu.ie/city/city.php?id=" + cityNum; 

     jsonString = wr.GetResult(url); 
     formattedString = cityObject.ReturnFormattedString(jsonString); 
    } 
} 

А в классе Web Retrieval я это:

async public void GetInformation(string url) 
{ 
    var client = new HttpClient(); 
    var response = await client.GetAsync(new Uri(url)); 
    result = await response.Content.ReadAsStringAsync(); 
} 

public string GetResult(string url) 
{ 
    GetInformation(url); 
    return result; 
} 

И что странно, что, в классе WebRetrieval, внутри GetInformation метод, когда он пытается выполнить ответ, программа переходит в мой класс City и терпит неудачу, потому что полученная строка равна null. Он никогда не попадает в результирующую строку внутри метода GetInformation.

У меня такое чувство, что это связано с тем, что GetInformation является асинхронным, и ответ ожидал в нем, но я не знаю, как это исправить, поскольку предложения в другом вопросе не закончились за работой.

Надеюсь, кто-то может мне помочь. Приветствия

Answer 1

Вы должны избегать async void. Я подробно объясняю «почему» в моей статье MSDN Best Practices in Asynchronous Programming.

void - очень неестественный тип возврата для async методов. Его следует использовать только при записи обработчика событий async (который должен вернуть void).

Таким образом, преобразование всех async void методы async Task (и following the naming convention):

async public Task GetInformationAsync(string url) 
{ 
    var client = new HttpClient(); 
    var response = await client.GetAsync(new Uri(url)); 
    result = await response.Content.ReadAsStringAsync(); 
} 

Эти задачи должны быть await изд:

public string GetResult(string url) 
{ 
    await GetInformationAsync(url); 
    return result; 
} 

Но тогда компилятор будет жаловаться, что если вы используете await, вы должны сделать свой метод async, поэтому мы делаем:

public async Task<string> GetResultAsync(string url) 
{ 
    await GetInformationAsync(url); 
    return result; 
} 

И await, что тоже:

private void City_GoButton_Click(object sender, RoutedEventArgs e) 
{ 
    ... 
    jsonString = await wr.GetResultAsync(url); 
    formattedString = cityObject.ReturnFormattedString(jsonString); 
} 

Еще раз, компилятор жалуется, так что мы это исправить, сделав метод вызова async (обычно, мы бы сделать его async Task, но это должно быть, потому что async void это обработчик событий):

private async void City_GoButton_Click(object sender, RoutedEventArgs e) 
{ 
    ... 
    jsonString = await wr.GetResultAsync(url); 
    formattedString = cityObject.ReturnFormattedString(jsonString); 
} 

И все готово.

Вы можете найти мой async intro полезный.

Answer 2

Если я правильно понять это, GetResult звонит GetInformation, который является async метод. В принципе, внутренности GetResult не нужно ждать завершения звонка GetInformation, чтобы закончить. У вас есть проблема с синхронизацией.

некоторые возможные решения, это может быть либо отметки GetResult в async, и он ждет GetInformation, или иметь GetInformation возвращают значение, которое затем можно заставить выполнить путем вызова .Result на задаче.