Элегантно обрабатывать отмену задачи

Ответ 1

Итак, в чем проблема? Просто отбросьте блок catch (OperationCanceledException) и установите правильные продолжения:

var cts = new CancellationTokenSource();
var task = Task.Factory.StartNew(() =>
    {
        var i = 0;
        try
        {
            while (true)
            {
                Thread.Sleep(1000);

                cts.Token.ThrowIfCancellationRequested();

                i++;

                if (i > 5)
                    throw new InvalidOperationException();
            }
        }
        catch
        {
            Console.WriteLine("i = {0}", i);
            throw;
        }
    }, cts.Token);

task.ContinueWith(t => 
        Console.WriteLine("{0} with {1}: {2}", 
            t.Status, 
            t.Exception.InnerExceptions[0].GetType(), 
            t.Exception.InnerExceptions[0].Message
        ), 
        TaskContinuationOptions.OnlyOnFaulted);

task.ContinueWith(t => 
        Console.WriteLine(t.Status), 
        TaskContinuationOptions.OnlyOnCanceled);

Console.ReadLine();

cts.Cancel();

Console.ReadLine();

TPL отличает отмену и ошибку. Следовательно, аннулирование (т.е. Бросание OperationCancelledException внутри тела задачи) не является ошибкой.

Главное: не обрабатывать исключения внутри тела задачи, не перебрасывая их.

Ответ 2

Вот как вы элегантно обрабатываете отмену задачи:

Работа с задачами "огонь и забвение"

var cts = new CancellationTokenSource( 5000 );  // auto-cancel in 5 sec.
Task.Run( () => {
    cts.Token.ThrowIfCancellationRequested();

    // do background work

    cts.Token.ThrowIfCancellationRequested();

    // more work

}, cts.Token ).ContinueWith( task => {
    if ( !task.IsCanceled && task.IsFaulted )   // suppress cancel exception
        Logger.Log( task.Exception );           // log others
} );

Обработка ожидания Ожидание завершения задачи/аннулирование

var cts = new CancellationTokenSource( 5000 ); // auto-cancel in 5 sec.
var taskToCancel = Task.Delay( 10000, cts.Token );  

// do work

try { await taskToCancel; }           // await cancellation
catch ( OperationCanceledException ) {}    // suppress cancel exception, re-throw others

Ответ 3

С# 6.0 имеет решение для этого.. Фильтрация исключений

int denom;

try
{
     denom = 0;
    int x = 5 / denom;
}

// Catch /0 on all days but Saturday

catch (DivideByZeroException xx) when (DateTime.Now.DayOfWeek != DayOfWeek.Saturday)
{
     Console.WriteLine(xx);
}

Ответ 4

Согласно этому сообщению в блоге MSDN, вы должны поймать OperationCanceledException, например

async Task UserSubmitClickAsync(CancellationToken cancellationToken)
{
   try
   {
      await SendResultAsync(cancellationToken);
   }
   catch (OperationCanceledException) // includes TaskCanceledException
   {
      MessageBox.Show('Your submission was canceled.');
   }
}

Если ваш отменяемый метод находится между другими отменяемыми операциями, вам может потребоваться выполнить очистку при отмене. При этом вы можете использовать вышеуказанный блок catch, но не забудьте правильно перебрасывать его:

async Task SendResultAsync(CancellationToken cancellationToken)
{
   try
   {
      await httpClient.SendAsync(form, cancellationToken);
   }
   catch (OperationCanceledException)
   {
      // perform your cleanup
      form.Dispose();

      // rethrow exception so caller knows youve canceled.
      // DONT 'throw ex;' because that stomps on 
      // the Exception.StackTrace property.
      throw; 
   }
}

Ответ 5

Вы могли бы сделать что-то вроде этого:

public void DoWork(CancellationToken cancelToken)
{
    try
    {
        //do work
        cancelToken.ThrowIfCancellationRequested();
        //more work
    }
    catch (OperationCanceledException) when (cancelToken.IsCancellationRequested)
    {
        throw;
    }
    catch (Exception ex)
    {
        Log.Exception(ex);
        throw;
    }
}

Ответ 6

Я не совсем уверен, чего вы пытаетесь достичь здесь, но я думаю, что следующий шаблон может помочь

public void DoWork(CancellationToken cancelToken)
{
    try
    {
        //do work
        cancelToken.ThrowIfCancellationRequested();
        //more work
    }
    catch (OperationCanceledException) {}
    catch (Exception ex)
    {
        Log.Exception(ex);
    }
}

Возможно, вы заметили, что я удалил инструкцию throw отсюда. Это не вызовет исключения, а просто проигнорирует его.

Сообщите мне, если вы намерены сделать что-то еще.

Существует еще один способ, который очень близок к тому, что вы показали в своем коде

    catch (Exception ex)
    {
        if (!ex.GetType().Equals(<Type of Exception you don't want to raise>)
        {
            Log.Exception(ex);

        }
    }