Discussion :

[topolino]

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 :

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
 
 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

[swif79]

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

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
 
 
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 ?

[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.

[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.

[topolino]

J'utilise un thread car je veux afficher en tps reel sur mon UI les traitements finis

[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

[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 ?

[StormimOn]

Oui mais de ce que je comprends, lorsque tu fais

Code :

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

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 :

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

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 :

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

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);
}

[topolino]

Merci c'est plus clair pour moi.