Discussion :

[BasicZX81]

Bonjour à tous,
J'ai un bug dans ma ligne cbCol.datasource = ListCars lorsque je tente de fermer le formulaire avant la fin du chargement des données effectuées par des Backgroundworker.
En effet, mon code fonctionne très bien sauf dans le cas ou je tente de fermer le formulaire, à ce moment là cbCol devient égale à Nothing puisque les contrôles ont été détruit.

J'ai dans l'idée qu'il faudrait soit pouvoir arrêter le backgroundworker dans le Form.closing ou alors tester une variable dans l’évent du bgw pour savoir quand l'UI est inaccessible et ainsi pouvoir faire un exit sub mais je ne sais pas si une telle variable existe ni comment faire exactement.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 Private Sub bgwCars_RunWorkerCompleted(sender As System.Object, e As System.ComponentModel.RunWorkerCompletedEventArgs) Handles bgwCars.RunWorkerCompleted
    Dim cbCol As DataGridViewComboBoxColumn = CType(DataGridViewCars.Columns("Dépendance"), DataGridViewComboBoxColumn)
    ListCars.Sort()
    cbCol.DataSource = ListCars         
 
 End Sub

Merci beaucoup d'avance pour votre aide.

[Pol63]

IsDisposed permet de savoir si un control a été détruit
https://msdn.microsoft.com/fr-fr/lib...vs.110%29.aspx

si tu fermes tes fenetres avec .close au lieu de .dispose pas sur que ca aille
après tu peux setter un booléen dans le formclosing


pour arreter le bgw, ca dépend de ce qu'il fait, s'i c'est un boucle tu peux tester un booléen dans la boucle pour arreter au plus tot
si c'est une seule instruction non annulable tu ne peux que faire interrupt (enfin ca se fait sur un thread, sur un bgw je ne sais pas si on peut)
le bgw encapsule un peu le traitement d'une annulation (WorkerSupportsCancellation, CancellationPending, CancelAsync()) mais c'est à toi de la coder, la méthode ne fait que setter le booléen à true ...

[BasicZX81]

Merci beaucoup Pol63.

Je ferme ma fenêtre en faisant un clique sur la croix en haut à droite, j'imagine que ça revient à faire un .close.
J'ai setté un boolean dans Form.closing et je m'en sert pour sortir de l'event RunWorkerCompleted. Après une trentaine de tentatives je n'ai plus l'erreur bien que je reste prudent car ça dépend du moment ou je clique....

Je passe en résolu néanmoins, je pense que ça fonctionne....

Merci pour tout....

[Pol63]

pour tester les trucs qu'un jedi devrait tester, tu peux rajouter un peu de rien pendant un certains temps, genre au début du bgw.dowork :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

system.threading.thread.sleep(2000)

ce qui te laisse 2 seconde pour cliquer avant qu'il commence à exécuter le code qui suit

[sinople]

Déjà fondamentalement ton exemple m'a l'air boiteux.

Normalement, dans les règles de l'art, le background worker doit fournir ces valeurs de retour (et d'entrée) à l'aide de la propriété Result (respectivement Argument) de l'évenement DoWork. Ensuite dans l'événement RunWorkerCompleted, on récupère ce résultat puis on l'utilise (pour modifier un contrôle ou peupler un objet de donnée liée aux contrôle).

La je vois pas exactement ce que tu fais mais tu m'a l'air de passer par la variable ListCars pour gérer le passage du résultats et que cette variable est par la suite utilisée comme source de données. C'est le meilleur moyen de se prendre des bonnes veilles Exceptions de thread dans la tronche.

Après concernant ton problème précis, le plus juste (ça n'engage que moi) et de prendre le problème à l'envers. C'est à dire de vérifier dans l'événement Form Closing que le background worker n'est pas en train de bosser (Propriété IsBusy) et d'annuler la fermeture du formulaire si c'est le cas. Ainsi tu t'assure que le worker a non seulement terminé son travail, mais aussi que le formulaire a traité le retour (ce qui peut avoir son importance).

Ca n'empêche pas d'aller plus loin en ajoutant une gestion d'annulation dans le backgroundworker et d'essayer d'annuler le travail avant d'annuler la fermeture du formulaire.

[wallace1]

@sinople : Je plussois.