Unikaj zwracania typu Zadanie <Object> z funkcji asynchronicznej podczas korzystania z HttpClient

Question

Wykonuję połączenia HTTP za pomocą System.Net.Http.HttpClient. Wygląda na to, że wszystkie połączenia muszą być asynchroniczne.

Powiedzmy mam strukturę projektu z następujących czynności: MVC Web App -> warstwy biznesowej -> Warstwa danych

w warstwie danych Robię HTTP wywołania API Web, aby powrócić danych i kończę przy użyciu funkcji takich jak to:

public async Task<IList<Product>> GetProducts() 
{ 
    HttpResponseMessage response = await client.GetAsync("api/products"); 
    string data = await response.Content.ReadAsStringAsync(); 
    IList<Product> products = JsonConvert.DeserializeObject<IList<Product>>(data); 

    return products; 
} 

to następnie przechodzi do BusinessLayer:

public Task<IList<Product>> GetProducts(string name = null) 
{ 
    return _repository.GetProducts(name); 
} 

wreszcie w kontroler MVC:

public IActionResult Index() 
{ 
    Task<IList<Product>> ProductsTask = _manager.GetProducts(); 

    IList<Product> ProductsNonTask = products.Result.ToList(); 

    return View(); 
} 

Czy naprawdę muszę zrobić każda funkcja zwraca listę typu Task<IList<Product>> prowadzącej do mojego kontrolera MVC? Jak widać w kontrolerze MVC, muszę najpierw pobrać produkty z Zawijanym Zadaniem. Debugger wygląda trochę dziwnie, kiedy przejdę do przeglądania listy produktów za jego pośrednictwem. Więc jak widzisz, zamieniam je na regularną listę produktów.

Zastanawiam się, czy to jest właściwe, aby wszystkie funkcje zwracały typ Task<IList<Product>> do mojego kontrolera MVC lub jeśli istnieje obejście, aby moje funkcje mogły nadal zwracać standardową listę ale nadal używasz asynchronicznych możliwości urządzenia HttpClient?

UPDATE: Czy jest coś złego w następujący sposób:

public IList<Product> GetProducts() 
{ 
    Task<HttpResponseMessage> response = client.GetAsync("api/products"); 
    if (response.Result.IsSuccessStatusCode) 
    { 
      string data = response.Result.Content.ReadAsStringAsync().Result; 
      IList<Product> products = JsonConvert.DeserializeObject<IList<Product>>(data); 

      retVal = products; 
    } 
} 

Answer 1

Jeśli chcesz operacja faktycznie być asynchroniczny musi być asynchroniczny aż do góry.

Moment zablokowania na Task z Wait lub nie jest już asynchroniczny.

Tak, nie. Albo użyj asynchronicznie do góry, albo nie używaj go w ogóle.

... Zastanawiam się, czy istnieje obejście tego problemu tak, że moje funkcje mogą jeszcze powrócić do standardowej listy produktów, ale nadal korzystać z możliwości async Spośród HttpClient?

Nie, tak naprawdę nie ma.

Powinieneś po prostu sprawić, by twój kod był asynchroniczny, ponieważ zapewnia wiele korzyści w zakresie skalowalności i wydajności. Ale jeśli jesteś przeciwko temu, możesz użyć WebClient, który ma synchroniczne operacje sieciowe.

Answer 2

Czy naprawdę muszę sprawić, aby każda funkcja zwróciła listę typu Zadanie> prowadzącą do mojego kontrolera MVC?

Tak, musisz i nawet nie złapałeś wszystkich miejsc, ponieważ samo Index musi być również asynchroniczne.

Async jest naprawdę zakaźny.Użyj, gdy to konieczne, a nie domyślnie.