Devo evitare i gestori di eventi "async void"?

So che è generalmente considerata una cattiva idea usare il fuoco e dimentica async void metodi per avviare le attività, perché non c'è traccia dell'attività in sospeso ed è difficile gestire le eccezioni che potrebbero essere lanciate all'interno di un tale metodo.

In genere dovrei evitare anche i async voidgestori di eventi? Per esempio,

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

Posso riscriverlo in questo modo:

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
        // ...
} 

Quali sono le rocce sottomarine per i gestori di eventi asincroni, oltre al possibile rientro?

La linea guida è da evitare async void tranne quando viene utilizzato in un gestore di eventi, quindi l'utilizzo async voidin un gestore di eventi è OK.

Detto questo, per ragioni di unit test spesso mi piace escludere la logica di tutti i async voidmetodi. Per esempio,

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);
}

In genere dovrei evitare anche i gestori di eventi void asincroni?

Generalmente i gestori di eventi sono l'unico caso in cui un metodo asincrono vuoto non è un potenziale odore di codice.

Ora, se hai bisogno di monitorare l'attività per qualche motivo, la tecnica che descrivi è perfettamente ragionevole.

Sì, in genere il vuoto asincrono dei gestori di eventi è l'unico caso. Se vuoi saperne di più, puoi guardare un ottimo video qui sul canale 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".

ecco il link

Se utilizzi ReSharper, un'estensione gratuita consigliata potrebbe esserti utile. Analizza i metodi "async void" ed evidenzia se usati in modo inappropriato. L'estensione può distinguere diversi utilizzi di async void e fornire soluzioni rapide appropriate descritte qui: ReCommended-Extension wiki .