Stopper le backgroundworker

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour,

J'esssaie de stopper le backgroundworker

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 Private Sub ToolStripButtonNew_MouseDown(ByVal sender As Object, ByVal e As System.Windows.Forms.MouseEventArgs) Handles ToolStripButtonNew.MouseDown
      backgroundWorker1.WorkerSupportsCancellation = True
        backgroundWorker1.CancelAsync()
    End Sub
 
 
 Private Sub Buttonvalid_MouseClick(ByVal sender As Object, ByVal e As System.Windows.Forms.MouseEventArgs) Handles Buttonvalid.MouseClick
 backgroundWorker1.RunWorkerAsync()
end sub
 
Private Sub backgroundWorker1_DoWork(ByVal sender As Object, ByVal e As System.ComponentModel.DoWorkEventArgs) Handles backgroundWorker1.DoWork
 
 If backgroundWorker1.CancellationPending Then
            e.Cancel = True
        End If
'do something 
end sub

le résultat si je clique sur le toolStripButtonNew et ensuite sur le boutonvalid
une exception à backgroundWorker1.RunWorkerAsync()
backgroundWorker1 n'est pas libre

merci

27/01/2014, 09h14
wallace1
Bonjour shayw,

Avant de lancer la méthode "RunWorkerAsync" il faut s'assurer que le BGW ne soit pas déjà en cours d'exécution comme ceci :
Code:

1 2 3 4 If not backgroundWorker1.isBusy backgroundWorker1.RunWorkerAsync() End if
Cela devrait résoudre ton problème.

A+

Bonjour shayw,

Je ne sais pas quel impact

Code:

backgroundWorker1.WorkerSupportsCancellation = True

aura sur le fonctionnnement de ton code dans l'event ToolStripButtonNew_MouseDown, mais je crois qu'il serait plus pratique de mettre ce code dans l'initialisation de ton bgw; ainsi il ne sera appelé qu'une seule fois.
Maintenant, voici comment moi je procède pour arrêter le bgw:
Code:

1 2 3 4 5 6 7 8 9 10 11 private void BtnCancel_Click(object sender, EventArgs e) { // Notify the worker thread that a cancel has been requested. // The cancel will not actually happen until the thread in the // DoWork checks the BgWorker.CancellationPending flag. if (_bgWorker.IsBusy) { _bgWorkerCust.CancelAsync(); } }
Désolé, c'est du c#. Essayes de jouer sur la propriété IsBusy du bgw; cela devrait résoudre ton problème.
[Edit]grillé par wallace :aie:[/Edit]

27/01/2014, 11h20
shayw
Bonjour

Je n'ai pas réussi :(
J'ai bien ajouté la condition
Code:

1 2 3 If not backgroundWorker1.isBusy backgroundWorker1.RunWorkerAsync() End if
et dans le ToolStripButtonNew_MouseDown
Code:

1 2 3 If backgroundWorker1.IsBusy Then backgroundWorker1.CancelAsync() End If
Pour mieux comprendre j'ai un bouton nouveau et un bouton valid
le clique sur nouveau doit stopper le bgw
le bouton valid est enable seulement si des données ont été introduit
et le clique active le bgw

ce que j'essaie possible que j'ai mal expliqué après avoir stopper le bgw
la possibllité de le réutiliser

pour tester

J'ai placé un breakpoint à la ligne If backgroundWorker1.IsBusy
Je lance le prog si je clique sur ToolStripButtonNew_MouseDown
le watch indique backgroundWorker1.IsBusy = false
ensuite je clique sur le bouton valid qui appelle le backgroundWorker1.RunWorkerAsync()
maintenant je clique sur ToolStripButtonNew
le watch indique backgroundWorker1.IsBusy = true
je continue pas à pas
backgroundWorker1.CancelAsync()
je verifie ensuite le backgroundWorker1.IsBusy
bizarre c'est toujours à true :?
27/01/2014, 11h36
tomlev

Le BackgroundWorker ne peut pas s'arrêter tout seul ; c'est à toi, dans le code de l'évènement DoWork, de vérifier régulièrement si CancellationPending vaut true, et d'arrêter ton traitement si c'est le cas.

En général j'ai la structure suivante pour mes bgw et ça fonctionne bien; essayes de voir si tu peux en tirer quelque chose:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
BackgroundWorker _bgWorkerCust;
 
public FrmSearchCustomer()
{
	InitializeComponent();
	_bgWorkerCust = new BackgroundWorker();
	InitBgw();
}
 
private void InitBgw()
{
	_bgWorkerCust.DoWork += BgWorkerCust_DoWork;
	_bgWorkerCust.ProgressChanged += BgWorkerCust_ProgressChanged;
	_bgWorkerCust.RunWorkerCompleted += BgWorkerCust_RunWorkerCompleted; 
	_bgWorkerCust.WorkerReportsProgress = true;
	_bgWorkerCust.WorkerSupportsCancellation = true;
}
 
private void BtnDemarrerBgw_Click(object sender, EventArgs e)
{
	// ...
	LaunchLongProcess();
}
 
private void LaunchLongProcess()
{
	BtnDemarrerBgw.Enabled = false;
	BtnCancel.Enabled = true;
	string keyword = SearchTextBox.Text.Trim();
	_bgWorkerCust.RunWorkerAsync(keyword); 
}
 
private void BgWorkerCust_DoWork(object sender, DoWorkEventArgs e)
{
	string arg = (string)e.Argument;
 
	_bgWorkerCust.ReportProgress(...); 
	//  ....
	_list = BLLClients.GetCustomerByName(arg); // the lengthy operation 
 
	//If cancel button was pressed while the execution is in progress
	//Change the state from cancellation ---> cancel'ed
	if (_bgWorkerCust.CancellationPending)
	{
		e.Cancel = true;
		//...
		return;
	}
}
 
private void BgWorkerCust_ProgressChanged(object sender, ProgressChangedEventArgs e)
{
	LoadingLabel.Visible = true;
	LoadingPictureBox.Visible = true;
}
 
private void BgWorkerCust_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
	LoadingLabel.Visible = false;
	LoadingPictureBox.Visible = false;
 
	BtnDemarrerBgw.Enabled = true;
	BtnCancel.Enabled = false;
 
	if (e.Cancelled)
	{
		MessageBox.Show("Opération de recherche annulée par l'utilisateur!");
	}
 
	else if (e.Error != null)
	{
		MessageBox.Show("Une erreur s'est produite au cours de la recherche :\n\r" + e.Error.Message);
	}
 
	else // opération de effectuée avec succès
	{
		customerListBindingSource.DataSource = new SortableBindingList<CustomerList>(_list);
	    // ...
	}
}
 
private void BtnCancel_Click(object sender, EventArgs e)
{
	// Notify the worker thread that a cancel has been requested.
	// The cancel will not actually happen until the thread in the
	// DoWork checks the BgWorkerCust.CancellationPending flag. 
	if (_bgWorkerCust.IsBusy)
	{
		_bgWorkerCust.CancelAsync();
        }
}

27/01/2014, 14h22
wallace1

J'avais posté sur le forum un projet exemple qui visait à lire un gros fichier tout en affichant une barre de progression à l'aide du BackgroundWorker.
Ce projet permettait aussi d'annuler l'opération.
Le bouton sert à lancer et à annuler la tache.

Voilà le projet :

http://wallace87000.upd.fr/ReadBigFile.zip

PS : N'oublies pas de remplacer la valeur de la variable nommée "_BigFilePath" par le chemin d'un fichier d'une assez grande capacité !
http://imabox.fr/a2/1427011421pshbtj42.png

EDIT : voilà le sujet : http://www.developpez.net/forums/d13...r/#post7569916

A+
27/01/2014, 16h52
Pol63
Code:

1 2 3 4 If backgroundWorker1.CancellationPending Then e.Cancel = True End If 'do something
pendant le do something il faut aussi vérifier si CancellationPending est à true
autrement dit il faut que le do something soit une boucle pour pouvoir vérifier à chaque tour
si ton do something est "une" ligne de code longue et qu'il n'y a pas de boucle, il faut trouver le moyen de faire la même chose avec une boucle ou utiliser un thread, qui lui peut etre interrompu pas proprement :aie:

si c'est une récupération de fichier par exemple transformer la ligne de code qui récupère tout en une boucle qui récupère par bloc de x kilo octets

En fait le problème est plus complexe la sub appellé dans l'évenement
backgroundWorker1_DoWork le "do something" est executé par une autre class

Il y a form1 et classx
le code du form1
Code:

1 2 3 4 5 6 7 8 9 10 11 12 classmath as new classx private sub initialize() Private Sub Form1_Load(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles MyBase.Load initialize end sub Private WithEvents backgroundWorker1 As New System.ComponentModel.BackgroundWorker Public Sub backgroundWorker1_DoWork(ByVal sender As Object, ByVal e As System.ComponentModel.DoWorkEventArgs) Handles backgroundWorker1.DoWork If backgroundWorker1.CancellationPending Then e.Cancel = True End If classx.dosomething end sub
dans le code de la classx.dosomething
dans la boucle principal
Code:

1 2 3 if form1.backgroundWorker1.CancellationPending then exit sub end if
la le problème form1 est chargé de nouveau donc le backgroundWorker1
n'est pas le meme objet
peut etre il faut déclarer le bgw pas dans la form1 mais dans un module ?

27/01/2014, 21h17
tomlev
Citation:

Envoyé par shayw

la le problème form1 est chargé de nouveau donc le backgroundWorker1

Voilà exactement le genre de trucs pour lesquels je déteste VB.NET... Ca fait plein de trucs "magiques" derrière ton dos (comme pouvoir appeler les membres d'une Form sans indiquer une instance spécifique), mais tu sais pas vraiment ce qui se passe derrière, et dès que tu n'es plus exactement dans le scénario prévu (en l'occurrence tu es sur un autre thread), ça ne marche plus... Si on ajoute à ça le fait que par défaut les composants d'une Form sont déclarés comme internal (Friend), on a tous les ingrédients pour faire des catastrophes :aie:

Bon, fin de mon coup de gueule, je vais essayer d'être plus constructif :mrgreen:

Le noeud du problème, c'est que ta classe qui fait le boulot ne devrait pas manipuler Form1 ; ça crée un couplage entre les deux qui non seulement n'est pas utile, mais risque de te poser des problèmes par la suite (par exemple si tu veux utiliser cette même classe depuis une autre Form). Le plus simple, ce serait que tu passes en paramètre de la méthode DoSomething une fonction qui indique s'il faut s'arrêter ou pas :
Code:

1 2 3 4 5 6 7 8 9 Public Sub DoSomething(ByVal isCancellationPending As Func(Of Boolean)) ... if isCancellationPending() then exit sub end if ... End Sub
Et dans to DoWork, tu appelles DoSomething comme ça :

Code:

classx.DoSomething(Function() backgroundWorker1.CancellationPending)

De cette façon, tu peux tester si l'annulation a été demandée, sans pour autant avoir une dépendance vers la Form dans la classe métier

Il y a bien sûr d'autres façons de faire, mais c'est probablement celle-là qui nécessite le moins de code...
27/01/2014, 22h37
wallace1

Merci tomlev. Bon coup de gueule mais très bonne leçon. J'apprécie :ccool:
28/01/2014, 00h03
Pol63

Citation:

Envoyé par tomlev

Voilà exactement le genre de trucs pour lesquels je déteste VB.NET... Ca fait plein de trucs "magiques" derrière ton dos (comme pouvoir appeler les membres d'une Form sans indiquer une instance spécifique), mais tu sais pas vraiment ce qui se passe derrière, et dès que tu n'es plus exactement dans le scénario prévu (en l'occurrence tu es sur un autre thread), ça ne marche plus ...

un fois qu'on connait les problèmes on trouve les solutions
de ce que j'en avais vu au début dans 80% des cas la variable magique du nom de l'instance te ressort la dernière instance créée
pour les 20% restants ca serait peut etre la 1ère instance créée qui sait ... :aie:
à moins que ca prenne en compte le passage du garbage collector ou la météo locale :lol:

c'est vrai qu'ils ont bien abusé microsoft là dessus

et le pire c'est les paramètres de compilation par défaut qui laisse écrire totalement n'importe quoi et du coup on voit des débutants arriver avec des fonctions non typées et des assignations qui le sont encore moins
28/01/2014, 00h27
tomlev

Citation:

Envoyé par Pol63

de ce que j'en avais vu au début dans 80% des cas la variable magique du nom de l'instance te ressort la dernière instance créée
pour les 20% restants ca serait peut etre la 1ère instance créée qui sait ... :aie:
à moins que ca prenne en compte le passage du garbage collector ou la météo locale :lol:

C'est pas tout à fait ça en fait... J'avais un peu creusé la question il y a quelque temps, et apparemment ça crée une instance pour chaque thread (l'instance par défaut d'un thread est initialisée lors de la première utilisation par ce thread). Dans le cas de shayw, quand il fait référence à Form1 depuis le DoWork du BackgroundWorker, ce n'est pas le même thread, donc pas la même instance par défaut de Form1...

Citation:

Envoyé par Pol63

c'est vrai qu'ils ont bien abusé microsoft là dessus

C'est clair... Ce concept d'instance par défaut est une énorme aberration du point de vue de la POO. Tout ça pour rendre le langage "plus facile" pour les débutants... bah on voit le résultat.

Pour moi, VB.NET ne mérite pas sa réputation de langage "facile". Ce n'est pas plus facile que C#, par exemple... La plupart des principes sont exactement les mêmes (liés à la plateforme .NET plutôt qu'au langage lui-même) ; c'est juste moins strict, et donc c'est plus facile de faire des erreurs. Les fonctionnalités "magiques" ajoutées pour rendre le langage plus abordable peuvent facilement devenir des pièges...
28/01/2014, 19h53
shayw

Merci Tomlev :D
ça marche

Question les "surprises vb.net" c'est seulement en vb.net ou aussi en
c# ou c++ ?

Je lis msdn pour comprendre
28/01/2014, 20h15
tomlev

Citation:

Envoyé par shayw

Question les "surprises vb.net" c'est seulement en vb.net ou aussi en
c# ou c++ ?

Bah ça dépend de quelles surprises tu parles, mais les trucs vraiment aberrants comme l'instance par défaut d'un Form, c'est spécifique à VB.NET ;)

La solution que je t'ai proposée, en revanche, est utilisable dans n'importe quel langage .NET
28/01/2014, 20h32
ebastien

Bonjour,

Citation:

Envoyé par tomlev

Bah ça dépend de quelles surprises tu parles, mais les trucs vraiment aberrants comme l'instance par défaut d'un Form, c'est spécifique à VB.NET ;)

Non, Delphi le fait aussi... Je suis d'accord c'est une aberration. Heureusement, dans Delphi, on peut supprimer le code correspondant aux créations automatiques des instances des Forms. Mais c'est vrai que quand on débute, c'est la porte ouverte à toutes les fenêtres !...

Citation:

Le BackgroundWorker ne peut pas s'arrêter tout seul ; c'est à toi, dans le code de l'évènement DoWork, de vérifier régulièrement si CancellationPending vaut true, et d'arrêter ton traitement si c'est le cas.

Bonsoir a tous,
je rebondi sur ce post car je suis justement en train d’implémenter l'annulation dans mon programme. Hors d'après la lecture de ce post on a la possibilité d'annuler dans son propre code (j'ai mis un petit résumé du code ci-dessous) mais qu'en est t'il si on lance un processus externe (ce qui est mon cas) ?
J'imagine qu'il faut "tuer" le processus mais je ne sais pas du tout comment. Savez vous comment on procède dans ce cas ?
Code:

1 2 3 4 5 6 7 8 9 10 11 ' Demande d'annulation If backgroundWorker1.IsBusy Then backgroundWorker1.CancelAsync() End If ' Test de la demande d'annulation et sortie de la tache en cours Public Sub DoSomething if backgroundWorker1.CancellationPending then exit sub end if End Sud

28/01/2014, 20h53
Pol63

pas besoin de bgw pour lancer un processus, et pour le tuer c'est .kill
28/01/2014, 21h01
tomlev

Citation:

Envoyé par BasicZX81

J'imagine qu'il faut "tuer" le processus mais je ne sais pas du tout comment. Savez vous comment on procède dans ce cas ?

Tu pourrais utiliser Process.Kill, mais c'est pas très propre, parce que le programme en question ne supporte pas forcément bien d'être arrêté brutalement. Une autre solution est d'utiliser Process.CloseMainWindow, si le programme a une fenêtre.

Si le process ne peut pas être interrompu proprement, il vaut sans doute mieux ne pas donner la possibilité d'annuler...
28/01/2014, 21h31
BasicZX81

Citation:

pas besoin de bgw pour lancer un processus

En effet j'ai réagi un peu vite. Je me suis replongé dans mon code pour savoir pourquoi j'avais fait cela. J'ai retrouvé une ligne commentée qui indique que "MonProcess.WaitForExit" bloque le thread principale tant que le processus n'est pas terminé, ce qui me figé l'affichage de la progression.
En utilisant un backgroundworker je libère le thread qui exécute l'affichage. C'est ce que j'avais noté à l’époque et cela fonctionne, bien que je ne soit pas certain que la méthode soit très orthodoxe :aie:. (Pourtant a l'époque j'avais déjà eu connaissance de la méthode invoque mais il faut croire que je n'avais pas su faire autrement).

EDIT: j'ai fait une confusion, la méthode invoque qui n'a rien a voir avec le gel de l'affichage.

Voir 40 message(s) de cette discussion en une page