Discussion :

[spottt]

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 :

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

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

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

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 :

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

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.

[spottt]

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 :

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

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

CountJobs

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

[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

[spottt]

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.

[Alro_gr]

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 :

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

public ManualResetEvent m_SleepEvent = new ManualResetEvent(false);

dans la boucle :

Code :

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

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 :

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

1
2
 
m_SleepEvent.Set();

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

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