C # Бросить OperationCanceledException изнутри CancellationToken.Register

Question

C # Бросить OperationCanceledException изнутри CancellationToken.Register

2

У меня длительная операция, которую я хочу отменить, скажем, через 5 секунд. К сожалению, опрос для IsCancellationRequested невозможен (длинная история).

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

Похоже, что это не работает должным образом, так как это приводит к необработанному исключению, и приложение не завершается корректно.

Любая помощь приветствуется. Спасибо!

    void TestTimeOut()
    {
        var cts = new CancellationTokenSource();
        cts.CancelAfter(5000);

        try
        {
            var task = Task.Run(() => LongRunningOperation(cts.Token));
            task.ContinueWith(t => Console.WriteLine("Operation cancelled"), TaskContinuationOptions.OnlyOnFaulted);
            task.Wait();
        }
        catch (AggregateException e)
        {
            //Handle
        }
    }


    void LongRunningOperation(CancellationToken token)
    {
        CancellationTokenRegistration registration = token.Register(
            () =>
            {
                throw new OperationCanceledException(token); 
            });

        using (registration)
        {
            // long running operation here
        }
    }

tiswingpower 20 нояб. 2018, в 18:51

Источник

1

Какой смысл запускать что-то в задаче, если вы .Wait() это? Использование CancellationToken является кооперативным. Может быть, вы можете просто создать поток, если вы на самом деле не используете асинхронные функции (ключевое слово await).
Dennis Kuypers 20 нояб. 2018, в 17:33
0

@DennisKuypers После Task.Wait () нужно выполнить другие действия, но только если долгосрочное задание завершено. В противном случае я хотел бы поймать отмененное исключение, сделать несколько уборок и выйти из приложения.
tiswingpower 20 нояб. 2018, в 17:43
0

Я Task.Wait() что если вы блокируете с помощью Task.Wait() зачем ставить его в другой поток? Нет никакой разницы, если запускать долго работающий объект в фоновом потоке + блокировать ваш вызывающий поток VS просто запустить работу в вызывающем потоке
Dennis Kuypers 20 нояб. 2018, в 17:44
0

@DennisKuypers, потому что мне нужно было тайм-аут задачи, поэтому я использовал CancelAfter. Я попробовал async / await ... но главная проблема здесь в том, как правильно вызвать исключение OperationCanceledException из задачи.
tiswingpower 20 нояб. 2018, в 17:48
0

Можете ли вы предоставить больше информации о длительной работе? Возможно, к нему можно добавить отмену (или он предлагает механизм для фактического прерывания операции). Похоже, вопрос должен быть об операции и о том, как правильно ее остановить, а не убивать.
Dennis Kuypers 20 нояб. 2018, в 18:03

Показать ещё 3 комментария

Теги:

c#

multithreading

.net

cancellation

2 ответа

Ещё вопросы

Какой смысл запускать что-то в задаче, если вы .Wait() это? Использование CancellationToken является кооперативным. Может быть, вы можете просто создать поток, если вы на самом деле не используете асинхронные функции (ключевое слово await).
@DennisKuypers После Task.Wait () нужно выполнить другие действия, но только если долгосрочное задание завершено. В противном случае я хотел бы поймать отмененное исключение, сделать несколько уборок и выйти из приложения.
Я Task.Wait() что если вы блокируете с помощью Task.Wait() зачем ставить его в другой поток? Нет никакой разницы, если запускать долго работающий объект в фоновом потоке + блокировать ваш вызывающий поток VS просто запустить работу в вызывающем потоке
@DennisKuypers, потому что мне нужно было тайм-аут задачи, поэтому я использовал CancelAfter. Я попробовал async / await ... но главная проблема здесь в том, как правильно вызвать исключение OperationCanceledException из задачи.
Можете ли вы предоставить больше информации о длительной работе? Возможно, к нему можно добавить отмену (или он предлагает механизм для фактического прерывания операции). Похоже, вопрос должен быть об операции и о том, как правильно ее остановить, а не убивать.

LeY · Answer 1 · 2018-11-20T15-47-00.000Z

В вашем коде много No-No-es, но я думаю, вы просто используете его в качестве демонстрации для своей проблемы. TaskCompletionSource является TaskCompletionSource, моя демка тоже ужасна, слишком много слоев Task.Run(), если вы используете Async, вы должны выполнить асинхронизацию до конца. Так что не используйте его в PRD, изучите TaskCompletionSource самостоятельно и TaskCompletionSource лучшее решение.

    static void LongRunningOperation(CancellationToken token)
    {
        TaskCompletionSource<int> tcs1 = new TaskCompletionSource<int>();
        token.Register(() => { tcs1.TrySetCanceled(token); });
        Task.Run(() =>
        {
            // long running operation here
            Thread.Sleep(10000);
            tcs1.TrySetResult(0);
        }, token);
        tcs1.Task.Wait();
    }

Marco Diniz · Answer 2 · 2018-11-20T14-54-00.000Z

Вы перехватываете AggregateException но фактически создаете исключение OperationCanceledException которое не будет перехвачено.

Изменить, чтобы поймать все типы исключений, таких как

catch (Exception ex) { ... }

Разрешить.