Чи слід уникати обробників подій "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);
}

Чи варто взагалі уникати асинхронних обробників недійсних подій?

Як правило, обробники подій - це один випадок, коли метод асинхронізації, який недійсний, не є потенційним запахом коду.

Тепер, якщо вам потрібно чомусь відстежувати завдання, то описана вами техніка цілком розумна.

Так, як правило, асинхронізація недійсних обробників подій є єдиним випадком. Якщо ви хочете дізнатися більше про це, ви можете подивитися чудове відео тут на каналі 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, вам може бути корисно безкоштовне рекомендоване розширення . Він аналізує методи "асинхронізовані порожнечі" та виділяє при неправильному використанні. Розширення може розрізняти різні способи використання асинхроністих пустот і надавати відповідні швидкі виправлення, описані тут: Вікі для перегляду рекомендацій .