Асинхронный, два ожидаемых метода. Только один возвращает

Question

У меня есть проблема с асинхронным программированием.

Если вы посмотрите на этот пример:

public async void SaveUsersThings(List<User> Users) 
{      
    await SaveUsersCars(Users); 
    await SaveUsersHouses(Users);   
} 

Так я ожидаю SaveUsersThings выполнить следующие два метода:

public async Task SaveUsersCars(List<User> Users) 
{ 
    var listofCars = new List<Car>(); 
    foreach (var item in Users) 
    { 
     var cars = await _sqlrepo.GetAllCars(item.UserId); 
     foreach (var car in cars) 
     { 
      listofCars.Add(car); 
     } 
    } 

    _sqlrepo.SaveListOfCars(listofCars); 
} 

И тогда это (идентично, как указано выше для OBJ исключением)

public async Task SaveUsersHouses(List<User> Users) 
{ 
    var listofHouses = new List<House>(); 
    foreach (var item in Users) 
    { 
     var houses = await _sqlrepo.GetAllHouses(item.UserId); 
     foreach (var house in houses) 
     { 
      listofHouses.Add(house); 
     } 
    } 
    _sqlrepo.SaveListOfHouses(listofHouses); 
} 

Проблема в том, что только первый метод завершает то, что он (чтобы сохранить некоторые вещи в db). Думаю, это связано с тем, что я не совсем понимаю этот вид программирования. Может кто-нибудь объяснить, как я могу заставить оба метода «работать»?

Спасибо

Answer 1

public async void SaveUsersThings(List<User> Users) 

Как вы можете знать, если есть ошибка? async void эффективно игнорирует любые ошибки. (Я подозреваю, что команда C# действительно не хотела поддерживать async void вообще, но для обратной совместимости с некоторыми существующими парадигмами. Что-то о обработчиках событий, я не знаю.) В основном void предотвращает использование любого кода потребления ожидая результата или рассматривая этот результат. Таким образом, вполне может быть AggregateException, которому просто некуда идти.

async должно быть "async all the way down". Сделать метод верхнего уровня awaitable:

public async Task SaveUsersThings(List<User> Users) 

Этот путь любой вызывающую SaveUsersThings() может ждать его и изучить результат. Это может не гарантировать, что «оба метода будут работать», но он хотя бы разоблачает причину , что что-то не работает. Вместо угадывания.