UI & Threading

Version imprimable

UI & Threading

Bonjour,

J'aimerais afficher sur mon UI le nombre de fichiers pdf se trouvant dans certains dossiers spécifiques du pc host. Sachant que ce nombre peut varier d'un moment à l'autre, j'aimerais le calculer toutes les 15 secondes.
Pour ce faire, je démarrer un thread qui se charge de compter le nombre de fichiers, et de mettre à jour l'UI.

Cependant, j'ai l'impression que ce thread ne rend pas la main au thread UI, ce qui fait que mon UI n'est pas rafraichie (le calcul s'effectue bien, lui).

Mon code :

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
 
while (true)
            {              
                if (this.listviewHotfolders.InvokeRequired)
                {
                    this.Invoke(new countJobsCallBack(countJobs));
                }
                else
                {
                    for (int i = 0; i < this.listviewHotfolders.Items.Count; i++)
                    {
                        String[] files = Directory.GetFiles(this.listviewHotfolders.Items[i].SubItems[4].Text);
                        int count = 0;
                        foreach (String file in files)
                        {
                            if (Path.GetExtension(file) == ".pdf")
                                count++;
                        }
                        this.listviewHotfolders.Items[i].SubItems[6].Text = count.ToString();
                    }
                }
            }
 
// Les 15 secondes ne sont pas prises en compte ici, mais le problème reste le même

23/04/2012, 17h45
ketan

Bonjour,

Pourrait-on avoir plus de code ?

La le code que tu nous montre vérifie si il se trouve sur le thread UI (via le InvokeRequired) et si il s'y trouve, il invoke la méthode CountJobs sur le thread ui et sinon il compte le nombre de fichier pdf (si j'ai bien tout compris).

Je vois plusieurs points d'erreur :
- ta boucle pour le compte du nombre de fichier pdf devrait être exécuté en dehors du thread UI (ce que tu souhaite) via un thread de la pool (http://msdn.microsoft.com/fr-fr/libr...(v=vs.90).aspx).
- tu ne devrait pas (ce n'est que mon avis) lire tes données depuis l'UI dans ta boucle de travail (ce que tu fais en faisant

Code:

this.listviewHotfolders.Items[i].SubItems[4].Text

)
- une fois avoir fait tout tes calculs, tu devrait utiliser this.Invoke pour mettre à jour ton UI (ou bien mettre chacun de tes

Code:

this.listviewHotfolders.Items[i].SubItems[6].Text = count.ToString();

dans this.Invoke).

Enfin, pour demander à un thread d'attendre, tu as http://msdn.microsoft.com/fr-fr/libr...(v=vs.90).aspx.
23/04/2012, 17h47
GuruuMeditation

Pas de rapport avec l'UI, mais tu peux utiliser la classe FileSystemWatcher, qui envoie un événement si un fichier est ajouté, effacé, modifié : http://msdn.microsoft.com/en-us/libr...emwatcher.aspx

Merci pour ta réponse ketan. La méthode que je montre ci-haut est la méthode "CountJobs" et est dans un autre thread que le thread UI (thread lancé dans le constructeur de ma form, je ne sais d'ailleurs pas si c'est le bon endroit pour le faire, bien que je n'en vois pas de meilleur).

Ce que je veux faire, donc, c'est compter le nombre de fichiers dans chaque dossier repris dans ma listview (et j'ai donc besoin de lire des données dans celle-ci depuis un thread autre que l'UI, puis de la màj).

Pour y voir plus clair :

Constructeur

Code:

1
2
3
4
5
6
7
8
 
        public MainWindow()
        {
            InitializeComponent();
            // ...
            Thread inputJobs = new Thread(new ThreadStart(countJobs));            
            inputJobs.Start();
        }

CountJobs

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
 
        public void countJobs()
        {
            while (true)
            {              
                if (this.listviewHotfolders.InvokeRequired)
                {
                    this.Invoke(new countJobsCallBack(countJobs));
                }
                else
                {
                    for (int i = 0; i < this.listviewHotfolders.Items.Count; i++)
                    {
                        String[] files = Directory.GetFiles(this.listviewHotfolders.Items[i].SubItems[4].Text);
                        int count = 0;
                        foreach (String file in files)
                        {
                            if (Path.GetExtension(file) == ".pdf")
                                count++;
                        }
                        this.listviewHotfolders.Items[i].SubItems[6].Text = count.ToString();
                    }
                }
                Thread.Sleep(5);
            }
        }

24/04/2012, 08h43
spottt

Citation:

Envoyé par GuruuMeditation

Pas de rapport avec l'UI, mais tu peux utiliser la classe FileSystemWatcher, qui envoie un événement si un fichier est ajouté, effacé, modifié : http://msdn.microsoft.com/en-us/libr...emwatcher.aspx

Salut Guruu, et merci pour ta réponse.
Le système est basé en grande partie sur les FileSystemWatchers, mais ceux-ci dépendent d'un service qui n'est pas fortement lié à l'UI, ce qui rend la communication assez difficile. De plus, le nombre de fichiers traités étant assez importants, je ne veux pas un refresh trop régulier de l'UI, et gérer cela serait, je pense, plus compliqué que ce que je propose ici.

Je pense que ton thread prends trop de ressources.
Tu dois l'endormir lorsque ton traitement est terminé.
Sleep(5) n'est probablement pas efficace car le système est incapable de faire une attente de 5ms.
J'utilise plutôt :

Code:

public ManualResetEvent m_SleepEvent = new ManualResetEvent(false);

dans la boucle :
Code:

1 2 3 4 5 6 7 8 m_SleepEvent.Reset(); // Initialisation while (true) { // Ton traitement m_SleepEvent.WaitOne(); // On reste bloqué m_SleepEvent.Reset(); // On réarme }
Dans la routine qui gère les 15 s, tu fait un Set pour le réveiller.
Tu peux aussi utiliser un système équivalent pour quitter le thread "proprement".
Je ne suis pas rentré dans les détails, mais le principe est là.
Code:

1 2 m_SleepEvent.Set();

24/04/2012, 10h04
spottt

Bonjour alro,

Merci pour ta réponse. J'avais essayé via des AutoResetEvent, l'affichage ne se faisait qu'une fois, mais le soucis restait le même par la suite. Pareil ici, avec les ManualResetEvent...

J'ai réussi à faire ce que je voulais via un timer (et non un thread travail), mais je ne suis pas sur que ce soit la meilleure solution dans ce cas-ci...
24/04/2012, 10h50
Alro_gr

Le problème du timer, c'est que ça bloque tout le reste pendant son exécution.
Par contre, le timer peut très bien lancer le thread (ton thread sans la boucle while(true)).
24/04/2012, 11h39
spottt

Je vais essayer comme ça, ça m'a l'air d'une solution plus facile.

Merci ! Je vous tiendrai au courant.

Le freeze de ton UI vient du fait que tout ton code relatif à ton calcul est exécuté dans ton thread UI et aussi que tu ne fait jamais de pause entre deux calcul.

La ligne en cause est ici :

Code:

1
2
3
4
5
 
if (this.listviewHotfolders.InvokeRequired)
{
    this.Invoke(new countJobsCallBack(countJobs));
}

Une solution qui devrait fonctionner en modifiant un peu ton code :

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
 
       public void countJobs()
        {
            while (true)
            {              
 
                    for (int i = 0; i < this.listviewHotfolders.Items.Count; i++)
                    {
                        String[] files = Directory.GetFiles(this.listviewHotfolders.Items[i].SubItems[4].Text);
                        int count = 0;
                        foreach (String file in files)
                        {
                            if (Path.GetExtension(file) == ".pdf")
                                count++;
                        }
                        if (this.listviewHotfolders.InvokeRequired)
                        {
                            this.Invoke(UpdateCount, i, count.ToString());
                        }
                        else
                        {
                            UpdateCount(i, count.ToString());
                        }
                    }
 
                Thread.Sleep(15000);
            }
        }
 
public UpdateCount(int index, string  val)
{
    this.listviewHotfolders.Items[index].SubItems[6].Text = val;
}

Ce code fait quoi comparé au tien ?
- il n'exécute que la mise à jour de l'UI dans le thread UI si besoin
- il attend 15 secondes entre chaque lancement de ton calcul (Thread.Sleep attend des millisecondes, donc il faut multiplier ton nombre de seconde par 1000).

24/04/2012, 13h43
spottt

Merci pour ton temps ketan.

Cependant, ce bout de code ne fonctionne pas étant donné que j'essaie d'accéder aux données de l'UI en dehors du thread UI. C'est là la base de mon soucis :(

Je vais essayer la solution de Alro_gr : un timer qui lance le thread toutes les 15 secondes, si faisable.

Merci à tous en tout cas !
24/04/2012, 14h30
Alro_gr

Le problème de faire un sleep(15000), c'est que le thread peut mettre 15 secondes pour quitter, si par exemple tu fermes l'application juste après le traitement.
Pour accéder à tes info. depuis ton thread, tu dois passer par une fonction déléguée, je crois..... je débute en C# et je dois bien avouer que je ne comprends pas encore tout. Dans ma logique de C++ (que je maitrise carrément mieux), le thread devrait envoyer un message à la form pour qu'elle se mette à jour.
24/04/2012, 14h52
spottt

Tu as raison de bout en bout Alro, le soucis est que je n'arrive pas à l'implémenter. Je l'ai théoriquement, reste à le mettre en pratique.

En passant par les délégués, le travail s'effectue uniquement dans le thread UI et bloque donc l'affichage...

Je suis perdu ! (Et je vais donc rester sur un timer...)
24/04/2012, 15h23
ketan

Pour le problème de l'attente à la fermeture, tu peux passer ton thread en fond après sa création comme le runtime détruit le thread lors de la demande de fermeture, sans attendre sa fin (le problème d'un while(true) dans un thread peut complètement bloquer une application et l'empêcher de se fermer).
http://msdn.microsoft.com/fr-fr/libr...(v=vs.90).aspx

Personnellement je suis plus découplage de l'interface et des données, ce qui implique que pour le calcul je n'ai pas besoin de demander à la vue mes données vu que je les ais déjà.

Dans ton cas, le mieux que je vois c'est quelque chose comme ça dans ton thread :
1 - passage dans le thread UI pour récupérer la liste des dossiers à vérifier (via àun appel à Invoke)
2 - une fois de retour dans le thread de travail, faire mes calculs
3 - une fois les calculs finis, passer dans le thread UI pour mettre à jour l'interface avec les nouveaux résultats (via Invoke ou BeginInvoke)
4 - mettre en attente le thread de travail (sleep, Event, autre ?)
5 - recommencer à l'étape 1.

Après, je suis d'accord que la boucle while(true) n'est pas terrible et qu'il faut mieux utiliser des systèmes de contrôles avec les thread plutôt que des sleep, mais si on veut rester simple, c'est quand même le mieux.
24/04/2012, 16h49
spottt

Un immense merci !

Ca fonctionne parfaitement, sans bloquer mon UI. C'est exactement ce que je voulais.

Une nouvelle fois merci pour le temps que tu as passé pour m'aider ketan (et ça vaut pour les autres :zoubi:)!