0

Имеется метод, который внутри себя запускает дважды другой метод, но второй раз асинхронно. И вот внутри Task не срабатывает SaveToDB().. Подскажите, пожалуйста в чём моя ошибка

public IEnumerable<UserFeedPrerender> GetFeedPrerenders(int userId, FilterPaging paging)
    {
        var feedPrerenders = PreRenderFeed(userId, paging);

        Task.Run(() =>
        {
            var fullPrerenders = PreRenderFeed(userId, null);
            SaveToDB(fullPrerenders, userId);
        });

        return feedPrerenders;
    }


 public async void SaveToDB(IEnumerable<UserFeedPrerender> userFeedPrerenderElements, int userId)
    {
        await Task.Run(() => SaveSelectedElementsToDB(userFeedPrerenderElements, userId));
    }
6
  • 1
    Ну ошибка в том, что вы не дожидаетесь выполнения таска и теряете его.
    – iluxa1810
    18 фев 2019 в 10:07
  • @iluxa1810, я так понимаю нужно заменить возвращаемый тип с void на Task?
    – user310525
    18 фев 2019 в 10:09
  • для начала да. Тем не менее, проблема останется.
    – iluxa1810
    18 фев 2019 в 10:11
  • @iluxa1810, будьте добры - подскажите как исправить её полностью
    – user310525
    18 фев 2019 в 10:12
  • Попробуй, вместо IEnumerable время Task<T>, поставь модификатор async и через await дождись выполнения таска.
    – iluxa1810
    18 фев 2019 в 10:17

1 ответ 1

1

Предположу, что вам надо что то вроде этого

public async Task<IEnumerable<UserFeedPrerender>> GetFeedPrerenders(int userId, FilterPaging paging)
{
    var feedPrerenders = PreRenderFeed(userId, paging);
    await Task.Run(() =>
    {
        var fullPrerenders = PreRenderFeed(userId, null);
        SaveSelectedElementsToDB(fullPrerenders, userId);
    });
    return feedPrerenders;
}

Вариант без асинхронного ожидания (в случае, если вас не волнует ни статус выполнения задачи, ни потенциально возникающие исключения в ней)

public IEnumerable<UserFeedPrerender> GetFeedPrerenders(int userId, FilterPaging paging)
{
    var feedPrerenders = PreRenderFeed(userId, paging);
    Task.Run(() =>
    {
        var fullPrerenders = PreRenderFeed(userId, null);
        SaveSelectedElementsToDB(fullPrerenders, userId);
    });
    return feedPrerenders;
}
10
  • Тогда ведь не надо асинхронно SaveToDB() вызывать 18 фев 2019 в 10:53
  • @RomanIeromenko у меня вообще нет метода SaveToDB()
    – tym32167
    18 фев 2019 в 10:54
  • Да, согласен, был у него 18 фев 2019 в 10:58
  • Работает, но тогда у меня return срабатывает только после того, как выполняется Task. А я хотел бы чтоб этот Task в фоне работал
    – user310525
    18 фев 2019 в 11:10
  • Асинхронность от многопоточности отличается почитай ru.stackoverflow.com/questions/445768/… 18 фев 2019 в 11:18

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.