Следует ли мне избегать обработчиков событий async void?

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

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

private async void Form_Load(object sender, System.EventArgs e)
{
        await Task.Delay(2000); // do async work
        // ...
} 

Я могу переписать это так:

Task onFormLoadTask = null; // track the task, can implement cancellation

private void Form_Load(object sender, System.EventArgs e)
{
        this.onFormLoadTask = OnFormLoadTaskAsync(sender, e);
} 

private async Task OnFormLoadTaskAsync(object sender, System.EventArgs e)
{
        await Task.Delay(2000); // do async work
        // ...
} 

Каковы подводные камни для обработчиков асинхронных событий, помимо возможного повторного входа?

Рекомендуется избегать, async void за исключением случаев использования в обработчике событий, поэтому использование async voidв обработчике событий - это нормально.

Тем не менее, по причинам модульного тестирования мне часто нравится исключать логику всех async voidметодов. Например,

public async Task OnFormLoadAsync(object sender, EventArgs e)
{
  await Task.Delay(2000);
  ...
}

private async void Form_Load(object sender, EventArgs e)
{
  await OnFormLoadAsync(sender, e);
}

Следует ли мне вообще избегать обработчиков событий async void?

Как правило, обработчики событий - это единственный случай, когда метод void async не является потенциальным запахом кода.

Теперь, если вам по какой-то причине нужно отслеживать задачу, то описанная вами методика вполне разумна.

Да, обычно async без обработчиков событий - единственный случай. Если вы хотите узнать об этом больше, вы можете посмотреть отличное видео здесь, на канале 9.

The only case where this kind of fire-and-forget is appropriate is in top-level event-handlers. Every other async method in your code should return "async Task".

вот ссылка

Если вы используете ReSharper, вам может быть полезно бесплатное расширение ReCommended . Он анализирует методы "async void" и выделяет при неправильном использовании. Расширение может различать различные варианты использования async void и предоставлять соответствующие быстрые исправления, описанные здесь: ReCommended-Extension wiki .