Gel de formulaire à l'aide de BackgroundWorker

Question

Je travaille avec un formulaire WinForm à partir duquel tous les processus dont j'ai besoin sont pilotés. Maintenant, j'essaie d'intégrer un BackgroundWorker avec un ProgressBar et un bouton d'annulation dans mon code. Je veux que ce soit localement autour de mon code et non dans une méthode séparée. Pour tester cela, un nouveau formulaire est créé avec une barre de progression (pas encore active) et un bouton pour arrêter une boucle for. Cependant, le code ne fonctionne pas (et la barre de progression n'est même pas encore incluse). Le formulaire se fige immédiatement (voir image), donc je ne peux pas tester le bouton d'annulation. La boucle for est cependant exécutée et "Done: " + l.ToString() est affichée. Comment puis-je resoudre ceci?

void stopMeasurement(object sender, EventArgs e) 
{ 
    stopMeas = true;  
} 

public void testcancel() // Test method which is triggered manually 
{ 
    int l = 0; 

    MetingProgress metingProgress = new MetingProgress(); 
    metingProgress.btnCancelmeting.Click += new EventHandler(stopMeasurement); 

    BackgroundWorker worker = new BackgroundWorker(); 
    worker.WorkerSupportsCancellation = true; 
    worker.DoWork += (sender, args) => 
    {      
     for (int k = 0; k < 10; k++) 
     { 
      Thread.Sleep(1000); 
      l++; 

      if (worker.CancellationPending) 
       break; 
     } 

     MessageBox.Show("Done: " + l.ToString()); 

    }; 
    worker.RunWorkerAsync(); 

    while (worker.IsBusy) 
    { 
     if (stopMeas) 
      worker.CancelAsync(); 
    } 

    metingProgress.Dispose(); 
    MessageBox.Show("All done"); 

} 

Answer 1

La forme gèle immédiatement

c'est parce que vous avez un while encore en cours d'exécution sur le thread principal! Donc le formulaire ne sera pas réactif. C'est ce qu'on appelle l'attente de Buisy. Vous ne serez pas en mesure d'appeler la méthode CancelAsync.

Une solution pourrait être d'enlever la boucle while et placez l'annuler appel dans le code d'événement bouton:

void stopMeasurement(object sender, EventArgs e) 
{ 
    stopMeas = true; 
    worker.CancelAsync(); 

} 

Ce que vous avez fait est essentiellement: vous avez créé un second jeton d'annulation. Donc, une autre possibilité pourrait être d'utiliser uniquement stopMeas pour annuler l'opération de fond:

worker.DoWork += (sender, args) => 
{      
    for (int k = 0; k < 10; k++) 
    { 
     Thread.Sleep(1000); 
     l++; 

     if (stopMeas) 
      break; 
    } 

    string mes = stopMeas ? "Done: " + l.ToString() : "Task aborted!"; 
    MessageBox.Show(mes); 

}; 

EDIT: aussi cette ligne:

metingProgress.Dispose(); 

pourrait conduire à une exception ObjectDisposed. Si le processus d'arrière-plan est toujours en cours d'exécution et tentez de mettre à jour votre barre de progression et que vous disposez déjà du formulaire. Vous devriez enlever cette ligne et la laisser au garbage collector.

Answer 2

Ce code est votre problème:

while (worker.IsBusy) 
{ 
    if (stopMeas) 
     worker.CancelAsync(); 
} 

Votre GUI-fil est dans cette boucle jusqu'à ce que votre travailleur se fait. Vous devez rendre votre instance de travail accessible depuis EventHandler et appeler worker.CancelAsync() à partir de là.

---

En dehors, je voudrais personnellement améliorer le code en 2 étapes:

1. Déplacer l'ensemble BackgroundWorker dans la classe MetingProgress et faire son constructeur prendre un délégué pour la mise en œuvre réelle de travail.

2. Utilisez TAP (modèle asynchrone de tâches), c'est-à-dire async/await Tâche avec Progress et CancellationToken.