Discussion :

[so.generously]

Bonjour,

J'ai un souci avec le backgroundworker, et je souhaite savoir si quelqu'un peut m'aider. mon application est composé d'un TabControl auquel est aloué dynamiquement un controle utilisateur.

Quand je fonctionne en monothread, il n'y a pas de souci. Mais ceci demandant beaucoup de ressources et affectant un freeze de mon interface graphique, j'ai décidé de le mettre une partie du traitement dans le backgroundWorker.

mon backgroudworker est créé dans la form principale. elle fait appel à une méthode qui va se charger d'alouer dynamiquement des controles utilisateur dans les TabPages correspondant.

Tout ceci à l'air de fonctionner jusqu'au moment où je souhaite afficher le tour (dans la méthode du backgroundworker : RunWorkerCompleted). Je demande l'affichage via des méthodes. Et voici le message d'érreur qui j'ai à l'execution :

Opération inter-threads non valide : le contrôle 'StatsTab' a fait l'objet d'un accès à partir d'un thread autre que celui sur lequel il a été créé.

Voici la partie du code du backgroundworker :

Code :

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

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
 
 public void AddClient(AveragesData[][] Data)
        {
            //this.DelAllClientTabs();
 
            CurrentData = Data;
            ClientStat = new SimotionTab[Data.Length];            //create as many Simotion tab's group as number of client
            LoadForm = new Loading();
            backgroundWorker1.RunWorkerAsync();
 
        }
 
private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
            for (int Client = 0; Client < CurrentData.Length; Client++)
            {
                ClientStat[Client] = new SimotionTab();
                ClientStat[Client].Dock = DockStyle.Fill;
                ClientStat[Client].BackColor = Color.LightGray;
                ClientStat[Client].AddSimotionTab(CurrentData[Client]);
            }
        }
 
        private void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
        {
            ClientTabControl = new TabControl();
 
            for (int Client = 0; Client < CurrentData.Length; Client++)
            {
                this.ClientTabControl.TabPages.Add(CurrentData[Client][0].ClientName);
                this.ClientTabControl.TabPages[Client].Controls.Add(ClientStat[Client]);
            }
            this.ClientTabControl.Dock = DockStyle.Fill;
            this.Controls.Add(ClientTabControl);
            this.QuickMenu.Visible = true;
 
            Sender.showClienttab();
 
        }

La méthode qui affiche le tout dans l'interface principale :

Code :

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

1
2
3
4
public void showClienttab()
        {
            this.splitContainer1.Panel2.Controls.Add(clientTab1);
        }

Pouvez-vous m'aider car je ne retrouve pas ce genre de problème sur google?

[clenoir]

Il me semble qu'en .Net (au moins en winform) il n'y ait qu'un unique Thread qui maintienne l'UI en .Net (c'est une limitation).
Utiliser le background worker pour paralléliser des opérations uniquement sur l'UI n'a donc pas de sens : en effet pour éviter les opérations inter-threads non valides, il faut utiliser la méthode Invoke(), qui va répercuter le travail sur le thread principal qui maintient l'UI (donc au final on n'a rien gagné).
Bref la méthode DoWork() du background worker devrait surtout travailler sur des données , et les répercutions sur l'UI devraient plutôt être effectuées dans les méthodes ReportProgress() ou WorkerCompleted() qui s'exécutent dans le thread principal.
Après il est toujours possible de placer des Invoke() dans le DoWork(), mais le background worker n'a d'intérêt que si le DoWork() fait autre chose que gérer l'affichage graphique.
Plus de détails là car je ne suis pas forcément très clair : http://msdn.microsoft.com/en-us/library/ms171728.aspx

[so.generously]

Le but de mon backgroundworker est de pouvoir mettre en forme une présentation de résultats pour être affiché ensuite. Pendant cette oppération qui est lourde, je voudrais éviter le freeze de ma form. Voilà pourquoi, je souhaite utiliser un thread.

[ostenhard]

Bonjour,

Comme dit par clenoir, tout ce qui est UI doit être fait dans le thread principal.
Donc retourne sur ton premier code et essaye d'ajouter un SuspendLayout avant l'ajout des tabls et usercontrols puis un ResumeLayout juste après.

Sinon je vois que tu as mis le new des SimotionTab (usercontrol) dans le DoWork est-ce ceci qui prend du temps ? Si oui, que fais-tu dans leur constructeur qui peut prendre du temps ?

[clenoir]

Pendant cette oppération qui est lourde, je voudrais éviter le freeze de ma form.

Eventuellement tu peux tenter d'utiliser des Thread.Sleep() dans ta boucle du DoWork(), et lancer donc ponctuellement tes mises à jour de l'UI (via un Invoke() ou backgroundWorker1.ReportProgress()). Si ça marche, ton composant risque de se charger de manière un peu aléatoire mais l'UI ne sera peut-être pas freezée.
(sinon +1 pour l'utilisation des SuspendLayout())

[so.generously]

Après analyse de ce qui se passe j'ai soupsonne que le problème vienne du fait que je créer des user control (SimotionTab et Statstabs) dans mon thread. Ces usercontrols, je souhaite les affichés.

Mais le problème interviens quand mon thread à terminé sont boulot. le thread supprime les usercontrols qu'il a créé et donc appel la methode dispose qui donne ce genre de message :

Opération inter-threads non valide : le contrôle 'StatsTab' a fait l'objet d'un accès à partir d'un thread autre que celui sur lequel il a été créé.

Y a il un moyen pour récupérer les usercontrols que le thread a créé dans mon programme pour pouvoir les afficher tranquillement?

[Pol63]

pas lu, mais seul le thread qui a créé un controle peut le modifier, sachant qu'il convient de créer tous les controles sur le thread principal

on peut utiliser des délégués pour revenir temporairement sur le thread principal puis retourner sur l'autre thread (this.invoke(le délégué))

donc si dans ton thread tu ne veux faire que des manipulations de controles, il ne sert à rien
par contre si tu fais du rapatriement de données ca peut être utile

[so.generously]

ok donc en fait ce que je veux faire est impossible.
Merci à tous pour vos éclairages.