Пожар и забудь подход

В связанном вопросе я изначально хотел использовать static void Forget(this Task task) в следующем контексте:

var task = DoWorkAsync();
QueueAsync(task).Forget();

// ...

async Task QueueAsync(Task task)
{
    // keep failed/cancelled tasks in the list
    // they will be observed outside
    _pendingTasks.Add(task);
    await task;
    _pendingTasks.Remove(tasks)
}

Это выглядело великолепно, но потом я понял, что фатально исключения, которые могут быть сгенерированы _pendingTasks.Add / _pendingTasks.Remove, пропадут незамеченными и потеряны, что нехорошо.

Поэтому я просто сделал QueueTask методом async void, чем он и является:

var task = DoWorkAsync();
QueueAsync(task);

// ...

async void QueueAsync(Task task)
{
    // keep failed/cancelled tasks in the list
    // they will be observed outside
    _pendingTasks.Add(task);
    try
    {
        await task;
    }
    catch
    {
        return;
    }
    _pendingTasks.Remove(tasks)
}

Хотя мне не нравится пустой catch {}, я думаю, что здесь есть смысл.

В этом сценарии сохранение async Task QueueAsync() и использование async void Forget(this Task task), как вы предлагаете, было бы излишним, ИМО.

Да, если вас интересует, вызывает ли задача исключение, тогда вам нужно await получить результат, но, с другой стороны, это в значительной степени противоречит цели "запустить и забыть ". ".

В сценарии, когда вы хотите узнать, произошло ли что-то плохое , рекомендуется использовать продолжение, например

public static void ForgetOrThrow(this Task task)
{
    task.ContinueWith((t) => {
        Console.WriteLine(t.Exception);
    }, TaskContinuationOptions.OnlyOnFaulted);
}