Thread et perte de donnée

Version imprimable

Bonjour,

Dans une boucle je declenche un thread qui appele une methode, mais voila la methode n'est pas executé a chaque fois et de facon aleatoire.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
 for (short nI = 0; nI < oMatchCollection.Count; nI++)
            {
                oRegex = new Regex(@"\s");
                film = oRegex.Replace(oMatchCollection[nI].Value, " ");
 
                Thread t = new Thread(() => displayedStatus(data));
                t.Start();
            }
 
 private void displayedStatus(string data) 
        {
            displayed = new displayedStatusDelegate(displayedStatus);
 
            if (textBox1.InvokeRequired)
            {
                textBox1.Invoke(displayed, data);
                return;
            }
            textBox1.AppendText(data);
            textBox1.AppendText(Environment.NewLine);
        }

Avez vous une idee de cette perte de données ?

Merci à vous

En fait ton thread n'as pas le temps de finir son boulot entre chaque itération. Tu peux essayer çà :

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
 
 
for (short nI = 0; nI < oMatchCollection.Count; nI++)
            {
                oRegex = new Regex(@"\s");
                film = oRegex.Replace(oMatchCollection[nI].Value, " ");
 
                Thread t = new Thread(() => displayedStatus(data));
                t.Start();
 
                 while ((t.ThreadState == ThreadState.Running))
                      System.Threading.Thread.Sleep(10);
            }
 
 private void displayedStatus(string data) 
        {
            displayed = new displayedStatusDelegate(displayedStatus);
 
            if (textBox1.InvokeRequired)
            {
                textBox1.Invoke(displayed, data);
                return;
            }
            textBox1.AppendText(data);
            textBox1.AppendText(Environment.NewLine);
        }

La l’intérêt du thread je n'en vois pas ton traitement n'est pas long et tout se fait de manière séquentiel. Pourquoi choisis tu d'utiliser un thread ?

16/03/2011, 13h02
topolino

Ca à l'air de fonctionner.

Mille merci, mais y a pas une facon plus propre que de faire une pause d'un temps mit au pif ? car si y a des latences reseau ou des traitements plus lourd en amont, le 10 mms ne suffira plus.
16/03/2011, 13h12
swif79

Hein ?
le Sleep n'est là que pour éviter d'avoir une boucle trop gourmande en ressource, le temps d'attente est déterminé par le temps d’exécution du thread.

Mais je ne comprends toujours pas pourquoi tu utilise un thread.
16/03/2011, 13h25
topolino

J'utilise un thread car je veux afficher en tps reel sur mon UI les traitements finis
16/03/2011, 13h33
StormimOn

Ton thread ne sert absolument à rien en l'état car le code de ce thread modifie uniquement l'UI (pas de calcul ou de traitement lourd, juste changer le texte d'une TextBox). Autant faire un Invoke direct, voire pas d'Invoke si tu es de base dans le thread UI.

Lorsque l'on gère des opérations longues qui peuvent bloquer l'affichage c'est le traitement en question qui doit être exécuté dans un thread. Et on fait de l'Invoke pour la progression lorsque c'est nécessaire. Là tu fais un thread juste pour faire l'Invoke et ça s'annule mutuellement puisque InvokeRequired sera toujours à true :)
16/03/2011, 13h56
topolino

En faite mon thread appelle une methode qui a pour but de faire de gros traitement de fichier, et cette methode de traitement mets a jour mon UI afin de voir quel fichiers a ete traité.

Est ce coherent ?

Oui mais de ce que je comprends, lorsque tu fais

Code:

Thread t = new Thread(() => displayedStatus(data));

tu es déjà dans ton traitement lourd, et donc dans un thread à part. C'est bien ça ?

A ce moment c'est inutile de faire un autre thread juste pour afficher la progression, puisque si le thread sert juste à faire l'Invoke alors le thread en question ne sert à rien. Il faut faire l'Invoke directement.

Par exemple avec une méthode qui servira à réaliser l'invoke (dans le principe la méthode se trouve sur une Form, this représente donc la Form).
Code:

1 2 3 4 5 6 7 8 9 10 11 public void InvokeMethod(MethodInvoker invoker) { if (this.InvokeRequired) { this.Invoke(invoker); } else { invoker(); } }
L'avantage étant de n'avoir qu'une seule méthode (pas la peine d'en faire une pour chaque contrôle de la Form). Il faut juste remplacer le délégué MethodInvoker (System.Windows.Forms) par Action à partir du framework 3.5.

Et pour l'appel on utilise une méthode anonyme (ou bien une expression lambda si disponible).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public void MonTraitementLourdDansUnAutreThread() { do { // Traitement lourd ... // Progression InvokeMethod( delegate { maTextBox.Text = "..."; }); } while (condition); }

16/03/2011, 15h27
topolino

Merci c'est plus clair pour moi.