Devrais-je éviter les gestionnaires d'événements 'async void'?

Question

Je sais qu'il est généralement considéré comme une mauvaise idée d'utiliser des méthodes async void de feu et d'oublier pour démarrer des tâches, car il n'y a pas de suivi de la tâche en attente et il est délicat de gérer les exceptions qui pourraient être levées à l'intérieur d'une telle méthode.

Devrais-je généralement éviter les gestionnaires d'événements async void, également? Par exemple,

private async void Form_Load(object sender, System.EventArgs e)
{
        await Task.Delay(2000); // faire un travail asynchrone
        // ...
} 

Je peux le réécrire comme ceci:

Tâche onFormLoadTask = null; // suivre la tâche, peut implémenter l'annulation

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); // faire un travail asynchrone
        // ...
} 

Quels sont les risques cachés pour les gestionnaires d'événements asynchrones, à part la possible réentrance?

Answer 1

La directive est d'éviter async void sauf lorsqu'il est utilisé dans un gestionnaire d'événements, donc l'utilisation de async void dans un gestionnaire d'événements est OK.

Cela dit, pour des raisons de tests unitaires, j'aime souvent factoriser la logique de tous les méthodes async void. Par exemple,

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

Answer 2

Devrais-je généralement éviter les gestionnaires d'événements async void, aussi?

Généralement, les gestionnaires d'événements sont le seul cas où une méthode async void n'est pas un potentiel code suspect.

Maintenant, si vous devez suivre la tâche pour une raison quelconque, alors la technique que vous décrivez est parfaitement raisonnable.

Answer 3

Si vous utilisez ReSharper, une extension ReCommended gratuite pourrait vous être utile. Elle analyse les méthodes "async void" et met en évidence leur utilisation inappropriée. L'extension peut distinguer les différents usages de async void et fournir les correctifs rapides appropriés décrits ici : wiki de l'extension ReCommended.

Answer 4

Oui, généralement async void des gestionnaires d'événements est le seul cas. Si vous voulez en savoir plus à ce sujet, vous pouvez consulter une excellente vidéo ici sur la chaîne 9

Le seul cas où ce type de déclenchement et oubli est approprié est dans les gestionnaires d'événements de niveau supérieur. Chaque autre méthode asynchrone dans votre code devrait retourner "async Task".

voici le lien