affichage barre de progression

Version imprimable

bonjour,

mon programme compare les valeurs contenues dans deux tableaux.

je voudrais afficher le niveau de correspondance à l'ai d'une progress bar. j'ai récupéré le code de codeguru pour avoir une barre personnalisée avec le pourcentage.

malheureusement la barre ne s'affiche pas (ou trop vite et donc invisible). voici mon code:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void CEssaiDlg::OnComparaison() //je clique su le bouton comparaison { int count =0; for (int i=0; i<100; i++) { if (moy[i]==moy2[i]) //deux tableau définis dans ma fonction d'initialisation { count = count + 1; } //en plaçant ici m_progress.SetPos(count) j'aperçois la barre puis elle disparait très vite } m_progress.SetPos(count); m_progress.SetStep(1); }
je n'arrive pas à laisser la barre de progression affichée

merci d'avance

Il faut séparer l'affichage de ta barre de progression de ton calcul. Tant que tu reste dans ta boucle for tu ne redonnes pas la main à la boucle de message pour rafraîchir l'affichage. C'est pour ça que tu as l'impression que ta barre de progression ne fonctionne pas.
Cette boucle :
Code:

1 2 3 4 5 6 7 8 for (int i=0; i<100; i++) { if (moy[i]==moy2[i]) //deux tableau définis dans ma fonction d'initialisation { count = count + 1; } //en plaçant ici m_progress.SetPos(count) j'aperçois la barre puis elle disparait très vite }
se déroulera certainement plus vite qu'un mécanisme de suivi mis en place.

Maintenant, si c'était juste pour l'exemple, et qu'en réalité ta boucle dure beaucoup plus longtemps, tu peux procéder de plusieurs façons :
1/ 2 threads : un pour ta boucle et un pour l'affichage
2/ timer qui vient faire un step de ta boucle toutes les n ms
3/ le step de ta boucle dans OnIdle

17/03/2009, 13h26
marie6631

merci

mais je n'ai pas tout compris...

j'ai déplacé la boucle dans la fonction qui remplit mes tableaux et j'ai toujours l'affichage de la barre associée au bouton. ça ne fonctionne pas.

il se peut aussi que le compteur ne s'implémente pas car la condition de comparaison n'est pas bonne.......?

quelle est la meilleure solution pour ce genre d'application?

merci d'avance

Ton application est mono-thread. Donc pendant que tu exécutes la boucle :
Code:

1 2 3 4 5 6 7 8 9 10 m_progress.SetStep(1); for (int i=0; i<100; i++) { if (moy[i]==moy2[i]) //deux tableau définis dans ma fonction d'initialisation { count = count + 1; } m_progress.StepIt() // ici c'est bien incrémentée }
L'affichage n'aura lieu que quand tu auras terminé le traitement sur ton bouton et rendu la main à l'application... Donc m_progress sera arrivé à la fin. D'où, ton impression visuelle que ça ne fonctionne pas.

La première question est : vas-tu vraiment faire cette boucle de 100 itération ? Car, si elle est aussi courte, je te conseil de ne pas te compliquer la vie et d'oublier la barre de progression.

En revanche, si ta boucle est vraiment longue, à ce moment tu as plusieurs possibilité :
1/ Faire du multi-thread. Sur OnComparaison, tu créés un nouveau thread qui aura la charge de faire le traitement et à chaque boucle demandera à incrémenter la barre de progression. Si tu es débutant, oublies.
2/ Sur OnComparaison, initier un contexte (indice_courant=0), démarrer un timer. Et sur chaque tick de ton timer, tu fais un step de ton traitement. Tu arrêtes ton timer dès que tu as terminé (en pseudo-code) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 Début OnComparaison Si traitement_en_cours Rien Sinon Initialiser traitement : indice_courant = 0 Demarrer timer Fin OnComparaison Début On Timer if (moy[indice_courant]==moy2[indice_courant]) { count = count + 1; } indice_courant++ m_progress.StepIt() Si indice_courant>= max Arreter timer Dérouler résultat Fin Si Fin On Timer
3/ Même principe que précédemment, sauf qu'au lieu de déclencher ton traitement dans On Timer, tu le fais s'exécuter sur OnIdle.

18/03/2009, 08h30
marie6631

merci beaucoup.

en effet je suis débutante.

ma boucle doit sucruter 600 points. c'est beaucoup. je vais essayer ta méthode 2.
18/03/2009, 09h05
marie6631

en essayant de coder selon ton pseudo code je ne comprends pas ce qu'est le traitement.

Quand je clique sur comparaison j'ai la fonction OnComparaison qui est associée. C'est à ce niveau que j'ai
"si traitement en cours : rien". Je ne comprends pas ce qu'est cette fonction.
"sinon:init traitement et démarrer timer". Je place donc un SetTimer(x,y,z).

est ce bon?
18/03/2009, 09h37
farscape

salut,
une méthode simple: rajouter une pompe a messages dans la boucle
voir faq:
http://cpp.developpez.com/faq/vc/ind...rk#PumpMessage
18/03/2009, 10h19
3DArchi

Citation:

Envoyé par farscape

salut,
une méthode simple: rajouter une pompe a messages dans la boucle
voir faq:
http://cpp.developpez.com/faq/vc/ind...rk#PumpMessage

C'est effectivement une solution. Mais, j'avoue que ça me plait jamais trop d'insérer des pompes à messages. Question de goût probablement.

Citation:

Envoyé par marie6631

en essayant de coder selon ton pseudo code je ne comprends pas ce qu'est le traitement.

Quand je clique sur comparaison j'ai la fonction OnComparaison qui est associée. C'est à ce niveau que j'ai
"si traitement en cours : rien". Je ne comprends pas ce qu'est cette fonction.

C'est pour que tu ne redémarres pas un nouveau traitement si ton traitement précédent est encore en cours.

Citation:

Envoyé par marie6631

"sinon:init traitement et démarrer timer". Je place donc un SetTimer(x,y,z).

est ce bon?

A priori, oui. Tu devrais aussi initialiser un contexte (indice de ta boucle, ID du timer pour pouvoir le stopper...).
18/03/2009, 13h42
marie6631

la solution de la pompe à messages semble fonctionnait. mais j'ai un problème avec mes pointeurs

merci à tous
18/03/2009, 15h50
marie6631

excusez moi mais en fait la pompe à message ne fonctionne pas. j'ai une erreur d'application.

de quoi cela peut il venir ?

merci d'avance
18/03/2009, 15h54
3DArchi

Bonjour,
Peux-tu préciser ton erreur et poster le code correspondant ?

Bien sur

voila le code complet

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
 
void CEssaiDlg::OnFiltre()
{
  moy=pFiltre(m_pdData);
  m_cvcDraw.AddDataBuffer(paramètre pour afficher le graph);
 
  moy2=pFiltre(m_pdData2);
  m_cvcDraw2.AddDataBuffer(paramètre pour afficher le graph);
}
 
void CEssai22Dlg::OnComparaison()
{
  int count = 0;
  for(int i=0; i<582; i++)
  {
    if (moy[i]==moy2[i])
      {
      PumpMessage();
      count = count + 1;
      }
  }
  m_progress.StepIt();
  m_progress.SetPos(count);
}

pour MessagePump()

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
void MessagePump()
{
  MSG msg;
  while(PeekMessage(&msg,NULL,0,0,PM_REMOVE))
  {
    if(AfxGetApp()->PreTranslateMessage(&msg))
    {
      ::TranslateMessage(&msg);
      ::DispatchMessage(&msg);
    }
    AfxGetApp()->OnIdle(0);
    AfxGetApp()->OnIdle(1);
  }
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
void CEssai22Dlg::OnComparaison()
{
  int count = 0;
  m_progress.SetPos(0);
  m_progress.SetRange (0,582); // ?
  for(int i=0; i<582; i++)
  {
    
    if (moy[i]==moy2[i])
      {
      m_progress.StepIt();
      MessagePump(); // ?
      count = count + 1;
      }
  }
  
}

18/03/2009, 16h21
farscape

il est clair que le positionnement de Messagepump n'est pas correcte ....
et si ça plante ce n'est certainement pas la pompe a message la responsable.
18/03/2009, 17h37
marie6631

avec ces modifications, la barre de progression s'affiche puis disparait très vite avec une erreur d'application qui suit me disant qu'une instruction ne peut pas être lue.

je pense que ca vient d'un rafraichissement de mes variables....

merci