Discussion :

[Cyr69]

Bonsoir,

Je vous explique mon problème : J'ai un programme qui comporte 2 threads de comptage (incremente une progressBar). Jusque là tout va bien. Maintenant je dois gerer les threads en fonction des messages.

Un compteur a son propre thread. a la fin du comptage le thread est détruit et recreer pour recommencé le comptage...

diagramme de classe :
ihm->Sablier (par composition)

dans Sablier il y a une methode Execute() pour faire le thread dans ihm.
dans la méthode Execute j'ai :

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
 
UINT Sablier::Execute()
{
	value->SetRange(mini,maxi);
 
        do
	{
		value->SetPos(i);
		Sleep(100);
		i++;
 
	}while(i!=maxi);
 
	ihm->PostMessage(valmess,0,0);
 
	return 0;
}

dans l'ihm pour gerer les messages :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
BEGIN_MESSAGE_MAP(CQuizzDlg, CDialog)
	ON_WM_SYSCOMMAND()
	ON_WM_PAINT()
	ON_WM_QUERYDRAGICON()
	//}}AFX_MSG_MAP
	ON_BN_CLICKED(B_REPONSE, &CQuizzDlg::OnBnClickedReponse)
	ON_MESSAGE(WM_USER,&CQuizzDlg::FinTempsQuestion)
	ON_MESSAGE(WM_USER+1,&CQuizzDlg::FinTempsQuizz)
END_MESSAGE_MAP()

puis dans la méthode FinTempsQuestion :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
LRESULT CQuizzDlg::FinTempsQuestion(WPARAM,LPARAM)
{
	if(p != Mc.GetTailPosition())
		{
			Mc.GetNext(p);
			Afficher();
		}
	return 0;
}

L'image du programme pour mieux comprendre :


1er probleme débile : Comment faire en sorte que la barre de progression arrive jusqu'au bout ? (pourtant j'ai bien défini les mini et maxi avec Setrange...

2e probleme : a la fin de la 1ere barre de progression (celui du haut) il envoie un message a l'ihm pour que le thread du compteur reparte a zero et change de question. Or mon probleme, la barre s'arrete a la fin et ne redemarre pas sauf la 2e barre de progression (or sa devrait etre l'inverse)? Kesaco ?

Merci a celui qui me remettra sur le bon chemin avec les Sendmessages...

[3DArchi]

Bonjour,
Pour le dernier élément, en général, c'est parce qu'on ne laisse pas le temps à l'IHM de se rafraîchir avec le dernier tick.
Pour ta deuxième question: n'aurais-tu pas inversé les deux éléments (question bête mais des fois... )?

Sinon, je ne suis pas sur d'avoir tout compris. Tu as CQuizzDlg qui possède deux barres et Sablier est un CWinThread? Ou est le second thread dont tu parles? Peux-tu préciser la dynamique que tu veux mettre en oeuvre entre tes différents éléments?

[Cyr69]

je procède comme cela pour les threads :

Code :

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

1
2
3
4
5
6
Sablier1 = new Sablier(WM_USER,&C1,this);
	Sablier2 = new Sablier(WM_USER+1,&C2,this);
 
	//créée les 2 threads
	T1 = AfxBeginThread(Install_thread,(LPVOID)Sablier1,0,0,CREATE_SUSPENDED,0); //crée le thread 1
	T2 = AfxBeginThread(Install_thread,(LPVOID)Sablier2,0,0,CREATE_SUSPENDED,0); //crée le thread 2

Egalement la classe Sablier hérite d'une classe abstraite Thread.

Pour te répondre j'ai déja essayé d'inversé ça ne change rien...

[3DArchi]

1/Pour aller jusqu'au bout : ton while s'arrête à <maxi
2/ Pour réinitialiser, faire un setpos(0)

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
UINT Sablier::Execute()
{
	value->SetRange(mini,maxi);
	value->SetPos(0);
        int i=0;
        do
	{
		value->SetPos(i);
		Sleep(100);
		i++;
 
	}while(i<=maxi);
 
	ihm->PostMessage(valmess,0,0);
	
	return 0;
}

Par contre, je reste assez sceptique d'utiliser les méthodes de CProgressCtrl dans un autre thread. Je ne suis pas sur qu'elles soient thread-safe.

[Cyr69]

deja j'ai reussi a corrigé le 1er probleme avec ton setpos(0);

Par contre thread-safe : j'ai pas compris?

On peut tres bien gerer 2 thread avec 2 adresses differentes de CProgressCTrl

grace au passage d'adresse de l'objet :

Code :

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

1
2
3
4
5
6
7
8
 
Sablier::Sablier(UINT mess,CProgressCtrl* valueEdit,CDialog *IHM)
{
     value = valueEdit; // met l'adresse du CProgressCtrl concernée dans la variable value pour le thread
     ihm = IHM;
     valmess = mess;
 
}

Si tu vois toujours pas dit le moi.

Sinon mon 2e probleme persiste encore...

[3DArchi]

deja j'ai reussi a corrigé le 1er probleme avec ton setpos(0);

Euh, ce devait plutôt corrigé le second ! Le premier est corrigé par la modification du test en while. Enfin, c'est ce que je pensais faire...

Par contre thread-safe : j'ai pas compris?

Tu créés ton contrôle dans ton thread principal. Tu utilises un CProgressCtrl*créé dans le thread principal dans ton thread secondaire. Ca je suis pas sûr que ce soit très catholique... Perso, quand je fais ce genre de chose, le thread secondaire envoie une notification au thread principal et à charge de celui-ci de mettre à jour l'IHM. Bref: un seul thread intervient sur l'IHM (le thread principal, qui hérite de CWinApp et qui l'a créé).

On peut tres bien gerer 2 thread avec 2 adresses differentes de CProgressCTrl

Oui, comme j'ai essayé de le dire ci-dessus, le pb c'est plus entre les threads secondaires et le thread principal.

Sinon mon 2e probleme persiste encore...

Faudrait que tu montres CQuizzDlg::FinTempsQuestion et CQuizzDlg::FinTempsQuizz. Sablier::Execute() est tel que tu l'as recopié ou il y a autre chose?