Discussion :

[Isythiel]

Bonjour

J'ai dans mon appli une tâche assez lourde lancée lorsque l'on appuie sur un bouton.

Code C# :

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

1
2
3
4
5
6
7
 
private void commencer_Click_1(object sender, EventArgs e)
{
   ...
   speechRecognitionEngine.LoadGrammar(magrammaire);
   ...
}

Cette tâche est "brute de décoffrage" : pas moyen de suivre son évolution, pas moyen d'en extraire un quelconque évenement, ni de la scinder en plusieurs partie. Elle dure environ 5 secondes (mais peu importe).

Pour spécifier à l'utilisateur que la tâche est en cours, j'ai pensé à mettre en place une progressbar (en suivant ce tutoriel) :

Code C# :

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
private void LanceThreadProgress()
{
   if (progressbarCharg.InvokeRequired)
   {
      progressbarCharg.Invoke(new progressbarChargInvokeHandler(Progress));
   }
            else Progress();                
}
 
private delegate void progressbarChargInvokeHandler();
 
private void Progress()
{
  while (progressbarCharg.Value < progressbarCharg.Maximum)
     progressbarCharg.Value++;
}
 
private void commencer_Click_1(object sender, EventArgs e)
{
   thrProgress = new Thread(new ThreadStart(LanceThreadProgress));
   thrProgress.Start();
 
   speechRecognitionEngine.LoadGrammar(magrammaire);
   ... 
}

Pour l'instant, je ne cherche pas à caler le step de la progessbar sur le temps réel de chargement. C'est un autre problème.

Le problème, c'est que dans tous les cas, l'incrémentation de la progressbar commence après le chargement de la grammaire, alors que mon thread est appelé avant.

Est-ce que j'utilise bien le thread ?
Existe-t-il un autre moyen de procéder ?

[rakakabe]

Et si tu mets LoadGrammar() dans un autre thread au lieu du progressBar ?

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
 
private event EventHandler<EventArgs> TaskStart;
 
 
void commencer_Click_1(...)
{
   this.TaskStart += new EventHandler(DemarrerProgression);
  Thread th = new Thread(new ThreadStart(LancerLoadGrammar));
  th.Run();
}
 
void LancerLoadGrammar()
{
   if(TaskStart != null)
      TaskStart(this, new EventArgs());
   speechRecognitionEngine.LoadGrammar(magrammaire);
}
 
// Cette methode est a ameliorer
void DemarrerProgression()
{
   progressBar.Step = 1;
   progressBar.Maximum = 10000000;
   progressBar.Start();
   Progress(); // ta methode progress
   progressBar.End();
  ....
}

.

[kineton]

J'ai déjà été confronté à ce problème et le moyen que j'ai trouvé à l'époque fut de créer une form dans laquelle j'exécutais un backgroundworker et via ce backgroundworker je parvenais à récupérer une configuration lourde en base de données (environ 15 secondes je crois me rappeler) et à mettre à jour simultanément une barre de progression.

Je n'ai pas l'exemple de code sous la main, mais je pense que si tu cherches de ce côté là, tu devrais trouver ton bonheur.

C'est pas forcément la meilleure des solutions bien sur,mais elle fonctionnais et fut rapidement implémentée.

Je suis preneur d'une meilleur solution.

Bon courage.

[theMonz31]

salut

le problème est simple

Fait une fonction qui fait un while() ce que tu veux sur une IHM. En théorie, l'IHM n'est mise à jour que si tu sors de la fonction ou que si tu appelles Application.DoEvents() sinon, l'IHM ne se raffraichit pas

T'as beau lancé un thread pour faire ta mise à jour, celà n'a pas de sens du fait de ce while... si tu insères un Application.DoEvents() tu auras "surement" un comportement plus proche de ce que tu veux !!!

[Isythiel]

Merci pour vos réponses rapides, je vais étudier un peu tout ça, et je reviendrai mettre la progression

[Isythiel]

Bien ! J'ai potassé tout ça, et je dois malheureusement dire que je n'obtiens toujours pas ce que j'attendais.

Et si tu mets LoadGrammar() dans un autre thread au lieu du progressBar ?

Code :

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

1
2
 
   this.TaskStart += new EventHandler(DemarrerProgression);

Qu'est-ce que this.TaskStart ? Je n'ai pas compris comment l'implémenter...

J'ai déjà été confronté à ce problème et le moyen que j'ai trouvé à l'époque fut de créer une form dans laquelle j'exécutais un backgroundworker et via ce backgroundworker je parvenais à récupérer une configuration lourde en base de données (environ 15 secondes je crois me rappeler) et à mettre à jour simultanément une barre de progression.

La solution du backgroundworker donne un résultat pas trop mal.

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
40
class Donnees
{
   public SpeechRecognitionEngine sre;
   public Grammar gram;
   public ProgressBar pbar;
   public int taille;
   //constructeur
   public Donnees(SpeechRecognitionEngine sre, Grammar C, ProgressBar bar, int sz)
      {
                this.sre = sre;
                gram = C;
                pbar = bar;
                taille = sz;
      }
}
 
private void commencer_Click_1(object sender, EventArgs e)
{
   Donnees maDonnee = new Donnees(sre, gramCmd, gram_refPdt, gram_qte_condit, progressbarCharg, poids);
   backgroundWorker1.RunWorkerAsync(maDonnee);
   progressbarCharg.Visible = true;
   backgroundWorker1.RunWorkerCompleted += new    RunWorkerCompletedEventHandler(backgroundWorker1_RunWorkerCompleted);
}
 
private void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
{
   BackgroundWorker worker = sender as BackgroundWorker;
   Donnees d = e.Argument as Donnees;
   MessageBox.Show("dowork ! ");
   d.sre.LoadGrammar(gram);  //traitement lourd de 5 sec
   MessageBox.Show("done !");
   e.Result = d;
}
 
private void backgroundWorker1_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
   Donnees d = e.Result as Donnees;
   d.pbar.Visible = false;
   MessageBox.Show("progress finie");
}

Comme ça, tout se passe bien : ma progressbar étant de style 'Masquee', elle apparait bien avant le traitement lourd, et elle s'anime correctement.

Le problème, c'est que lorsque j'enlève la MsgBox

Code :

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

MessageBox.Show("dowork ! ");

Le problème réapparait...
A savoir, la progressbar apparait bien, mais ne s'anime pas tant que le traitement lourd n'est pas fini...

C'est dommage parce que j'y étais presque ^^
D'où cela peut-il venir ? Je soupçonne l'ordonnanceur CPU de choisir les tâches selon son bon vouloir ^^

Quelqu'un a-t-il une idée précise de ce comportement ?

Fait une fonction qui fait un while() ce que tu veux sur une IHM. En théorie, l'IHM n'est mise à jour que si tu sors de la fonction ou que si tu appelles Application.DoEvents() sinon, l'IHM ne se raffraichit pas

T'as beau lancé un thread pour faire ta mise à jour, celà n'a pas de sens du fait de ce while... si tu insères un Application.DoEvents() tu auras "surement" un comportement plus proche de ce que tu veux !!!

Je n'ai pas tellement compris ta solution. J'ai cherché comment implémenter Application.DeEvents() proprement, et une explication précise, mais je n'ai rien trouvé de très détaillé. De plus, dans certains posts, ta solution est dite "barbare", pas naturelle.

Pourquoi dis-tu que l'IHM ne se rafraichit pas ? Dans mes premiers posts, lorsque j'utilisais un while pour incrémenter la progressbar, je voyais tout à fait la progression. Mais après le traitement lourd.

Pourrais-tu me détailler le fond de ta pensée ?

[Isythiel]

Moké, en suivant les instructions de sovitec, un :

Code :

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

Application.DoEvents();

à la place de :

Code :

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

MessageBox.Show("dowork ! ");

Débloque tout.

Un grand merci à tous

[rakakabe]

Bien que t'as resolu ton probleme, je defends ma proposition () :

Qu'est-ce que this.TaskStart ? Je n'ai pas compris comment l'implémenter...

voir :

Code :

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

private event EventHandler<EventArgs> TaskStart; (a mettre dans la meme classe, c'est un evenement)

Et essaie de lire et de comprendre un peu le code en entier. Cet evenement sert a notifier le thread responsable de la mise a jour du progressBar que la tache lourde a commencee.
Comme l'evenement est geree prioritairement, il est fort probable que progressBar se met a jour selon ton souhait en meme temps que le demarrage de la tache lourde. D'ailleurs, tu peux tres bien utiliser BackgroundWorker au lieu du Thread.

Par ailleurs, l'interet de la tache lourde dans un nouveau thread et la progression pour la tache principale est que, dans le cas contraite, la tache lourde pourrait se terminer au moment ou le thread mettant a jour l'affichage vient d'etre cree (prendre en compte le temps de creation du thread, et la lourdeur de InvokeRequired).

[Isythiel]

Je te remercie, je mettrai ta proposition en œuvre lorsque j'aurai le temps d'optimiser mon application (et ainsi comparer les propositions).