2

В приложении ASP.NET Core есть участок кода такого вида:

public class ContactsController : Controller {
        /// Для упрощения параметры и несущественные детали удалены.
        [HttpGet]
        public async void SomeGetMethod() {
        /// Операция тяжеловестная, поэтому максимально быстро отдаём ответ пользователю 
       /// и намеренно не ждём завершение операции!
            Task.Run(async () => {
                /// Какой-то код
                ...
                /// Тут exception
               throw new System.Exception("error");
            });
        }
}    

Естественно этот Eexception не перехватывается ExceptionFilter'ом.

Встал вопрос как перехватывать такие исключения , так как было бы неплохо логировать эти случаи наиболее элегантным способом без написания try catch в каждом подобном асинхронном методе?

6
  • 1
    А зачем вы делаете Task.Run в уже асинхронном методе? Это же запускает код внутри Run в отдельном потоке, а SomeGetMethod дорабатывает до конца, и возвращает клиенту 'ничего'. К моменту, когда бросается исключение, возвращать ошибку уже некому. Метод вашего контроллера и так запускается в своем собственном потоке (поток под запрос), зачем эти манипуляции с Task.Run?
    – user177221
    30 июл 2019 в 8:35
  • Это сделано специально для того что бы клиент не ждал ответа, он запускает некую тяжеловестную операцию окончание которой ему не надо ждать.
    – Vas Mil
    30 июл 2019 в 8:43
  • 1
    Чисто на заметку, как возможная альтернатива. В asp.net core начиная с 2.1 есть вполне удобный механизм фоновых задач (тынц), клиент может просто сохранить задание в базу (как правило входные данные небольшие по объёму) и спокойно отваливаться, а задача сама стартанёт и обработается.
    – A K
    30 июл 2019 в 9:25
  • Можно через ContinueWith. Правда придется логер везде таскать.
    – Vasek
    30 июл 2019 в 10:26
  • @AK спасибо за совет, почитал оф доку про фоновые задачи, не не понял преимуществ против подхода с Task.Run. И не совсем понятны юзкейсы для чего онфоновые задачи подходят.
    – Vas Mil
    30 июл 2019 в 10:42

2 ответа 2

0

В моём случае мне может помочь такой хелпер. Ваше мнение?

public static class TaskHelper {
    public static void SafeRun(Func<Task> action) {
        Task.Run(async () => {
            try {
                await action.Invoke();
            } catch(Exception e) {
                /// логируем ошибку
            }
        });
    }
}
8
  • 1
    Только не Action action, а Func<Task> action. И дальше делать await action.Invoke() 30 июл 2019 в 10:43
  • Инстанс логера как протащите?
    – Vasek
    30 июл 2019 в 10:47
  • @Vasek я использую rollbar там один инстанс на все приложение (singleton)
    – Vas Mil
    30 июл 2019 в 11:24
  • @PavelMayorov спасибо! А почему именно так?
    – Vas Mil
    30 июл 2019 в 11:25
  • 1
    @VasyaMilovidov а что если вам понадобится запускать асинхронную задачу? 30 июл 2019 в 11:30
0

Для того что бы метод вернул Exception необходимо:

  1. Перед Task.Run использовать await.
  2. Вместо void использовать Task. Таким образом, вы скажете C# что ваш метод будет работать в другом потоке, и по окончанию его работы вернётся его результат.

О асинхронных методах рекомендую почитать: https://metanit.com/sharp/tutorial/13.3.php

Добавлю: Создайте сервис и только через него запускайте все "сложные" таски пример ниже: (Используйте входные параметры на ваше усмотрение)

public class TaskManagerService
{
    /***/

    public TaskManagerService(Logger logger)
    {
        /***/
    }

    public void Run(Action action)
    {
        Task.Run(() =>
        {
            try
            {
                action.Invoke();
            }
            catch (Exception e)
            {
                //логируем
            }

        });
    }
}

public class FooController : ApiController
{
    private TaskManagerService taskManager;
    public FooController(TaskManagerService taskManager)
    {
        this.taskManager = taskManager;
    }

    [HttpGet]
    public void GetFoo()
    {
        taskManager.Run(() =>
        {
            //Сложная задача
        });
    }
}

И не забудьте его добавить в

public void ConfigureServices(IServiceCollection services)
{
     //Если только для логирования используйте AddTransient
     service.AddTransient<TaskManagerService>()
}
4
  • 2
    по поводу использования await автор же ответил Это сделано специально для того что бы клиент не ждал ответа, он запускает некую тяжеловестную операцию окончание которой ему не надо ждать.
    – tym32167
    30 июл 2019 в 10:18
  • Какой смысл делать ConfigureAwait без, собственно, await? 30 июл 2019 в 11:32
  • Спасибо хороший вариант. Кстати я использую NET Core и поэтому .ConfigureAwait(false); не обязательно. blog.stephencleary.com/2017/03/… И да соглашусь с @PavelMayorov надо передавать Func<Task> и вызывать так await action.Invoke()
    – Vas Mil
    30 июл 2019 в 11:33
  • @Pavel Mayorov,@Vasya Milovidov Согласен. Убрал. 30 июл 2019 в 12:02

Ваш ответ

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

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.