Discussion :

[coincoin73]

Bonjour,

Mon problème est le suivant :
J'ai un programme (MFC) qui fait des calculs assez lourd et qui consomment pas mal de temps CPU.
Or j'ai besoin d'afficher un état de mes calculs au travers de voyant (CStatic avec 2 bitmaps que je switch quand nécessaire).

Le problème vient de l'affichage de ces voyants qui ne se rafraichissent pas.

J'ai fait un thread avec la classe CWinThread qui gère le toggle des voyants.

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
41
42
43
44
45
46
47
48
49
50
51
52
53
 
CMyClass::CMyClass()
{
	m_NbEvent = 0;
	m_MyEvent = NULL;
	m_bStatus=THR_STOPPED;					// Thread arreté
}
 
CMyClass::~CMyClass()
{
	if (m_MyEvent ) {
		for (int EvtI = 0 ; EvtI < m_NbEvent ; EvtI++)
			CloseHandle(m_MyEvent[EvtI]);
		delete m_MyEvent;
	}
}
 
void CMyClass::ToggleBitmap(int nIndex)
{
	::SetEvent(m_MyEvent[nIndex]);
}
 
void CMyClass::SetNbEvent(int iMaxEvent)
{
	m_NbEvent = iMaxEvent;
	m_MyEvent = new HANDLE[m_NbEvent];
	for (int EvtI = 0 ; EvtI < m_NbEvent ; EvtI++)
		m_MyEvent[EvtI] = CreateEvent(NULL, FALSE, FALSE, NULL);
}
 
BOOL CMyClass::InitInstance()
{
	m_bStatus= THR_RUNNING;
	DWORD Ret;
	int nEvent;
 
	// Debut boucle principale
	while (m_bStatus==THR_RUNNING) {
 
		Ret = ::WaitForMultipleObjects(m_NbEvent, m_MyEvent , FALSE, INFINITE);
		nEvent = Ret - WAIT_OBJECT_0;
		::ResetEvent(m_MyEvent[nEvent]);
 
		if (m_MainDlg) {
			if (m_MainDlg->m_cLight[nEvent])
				m_MainDlg->m_cLight[nEvent]->SetBitmap(m_MainDlg->m_bGreen);
		}
	};
	m_bStatus= THR_STOPPED;
	while (m_bStatus!=THR_STOPPED_ACK) Sleep (2000);
 
	return TRUE;
}

Le thread est créé avec une priorité normale (par défaut).

[Médinoc]

Franchement, ce que je conseillerais, ce serait réduire la priorité du thread de travail.

Et peut-être aussi, éviter d'utiliser une attente bloquante dans ton thread d'UI. Pourquoi ne pas plutôt le prévenir avec un PostMessage() (asynchrone) ou un SendMessage() (synchrone) ?

[coincoin73]

Je ne peux pas réduire la priorité du thread de travail car c'est du quasi temps réel.
Il me semblait que le PostMessage n'est possible qu'avec une fenêtre, me trompe-je ?

[Médinoc]

Ça n'est possible que vers une fenêtre, mais ton thread d'UI est supposé en avoir une, ou ça ne serait pas un thread d'UI.

Déjà, rien qu'un CStatic, c'est déjà une fenêtre.

[3DArchi]

Salut,
Quelques remarques :
1/ Comment sors-tu de la boucle ? m_bStatus est modifiée ailleurs? Alors il te faudrait peut être vérifié l'accès concurrent (Mutex, section critique).
2/ Ton thread dérive de CWinThread ? Normalement, tu ne devrais pas faire tout le boulot dans InitInstance mais comme dans le thread principal.
3/ Ta gestion de la fin du thread (while (m_bStatus!=THR_STOPPED_ACK) Sleep (2000); ) est limite. Tu devrais utiliser un mécanisme de synchronisation spécifique pour terminer ton thread (un évènement de fin de thread par expl).
4/ Normalement, il n'est pas recommandé d'échanger des MFC entre les threads mais d'échanger leur handle. Donc si m_MainDlg est l'instance créé dans le thread principal, alors tu devrais plutôt échanger le HWND et faire ensuite un CWnd::FromHandle ou CWnd::FromHandlePermanent.
5/Post/SendMessage prennent en paramètre un HWND. Mais rien n'empêche de créer une fenêtre 'boîte aux lettres' : Message-Only Windows.
5 bis/ Pour rappel, le comportement de SendMessage entre thread différents est un peu tordu.
6/ C'est le thread de travail qui pianote sur le tableau d'évènements? Pourquoi avoir fait un thread séparé du thread principal? Tu pouvais rester en écoute dans le thread principale avec MsgWaitForMultipleObjectsEx ? Ou alors faire du 'polling' dans OnIdle?

[coincoin73]

Ce que je ne comprend pas c'est pourquoi les évènements qui sont levés ne débloque pas le WaitForMultipleObjects ? En fait c'est uniquement à la fin de mes calculs qu'ils font effet.

Ça n'est possible que vers une fenêtre, mais ton thread d'UI est supposé en avoir une, ou ça ne serait pas un thread d'UI.

Déjà, rien qu'un CStatic, c'est déjà une fenêtre.

Qu'est ce que tu entend par thread d'UI, est ce que c'est le thread principal ?

Salut,
Quelques remarques :
1/ Comment sors-tu de la boucle ? m_bStatus est modifiée ailleurs? Alors il te faudrait peut être vérifié l'accès concurrent (Mutex, section critique).

Pour la sortie du thread j'ai pas mis tout le code, mais je suis à peu prés sur de mon coup.

2/ Ton thread dérive de CWinThread ? Normalement, tu ne devrais pas faire tout le boulot dans InitInstance mais comme dans le thread principal.

Tout à fait exact, d'ailleurs je l'ai déplacé dans Run.

3/ Ta gestion de la fin du thread (while (m_bStatus!=THR_STOPPED_ACK) Sleep (2000); ) est limite. Tu devrais utiliser un mécanisme de synchronisation spécifique pour terminer ton thread (un évènement de fin de thread par expl).

J'ai créé le thread en AutoDelete (et franchement cette partie vient d'un copier coller du code d'un collègue et je suis pas un expert).

4/ Normalement, il n'est pas recommandé d'échanger des MFC entre les threads mais d'échanger leur handle. Donc si m_MainDlg est l'instance créé dans le thread principal, alors tu devrais plutôt échanger le HWND et faire ensuite un CWnd::FromHandle ou CWnd::FromHandlePermanent.

m_MainDlg est le pointeur sur ma CDialog.

5/Post/SendMessage prennent en paramètre un HWND. Mais rien n'empêche de créer une fenêtre 'boîte aux lettres' : Message-Only Windows.
5 bis/ Pour rappel, le comportement de SendMessage entre thread différents est un peu tordu.

Je ne savais pas.

6/ C'est le thread de travail qui pianote sur le tableau d'évènements? Pourquoi avoir fait un thread séparé du thread principal? Tu pouvais rester en écoute dans le thread principale avec MsgWaitForMultipleObjectsEx ? Ou alors faire du 'polling' dans OnIdle?

MsgWaitForMultipleObjectsEx est bloquant non ? Du coup je ne pourrais plus faire de traitement dans mon thread principal.
Je ne sais pas ce qu'est le polling.

[3DArchi]

MsgWaitForMultipleObjectsEx est bloquant non ? Du coup je ne pourrais plus faire de traitement dans mon thread principal.

Il est bloquant sur les évènements et sur la réception de message. Donc, si dans ton thread principal tu ne fait rien d'autre que d'attendre des interactions IHM, cela doit pouvoir suffire. A essayer?

Je ne sais pas ce qu'est le polling.

Le polling consiste à demander régulièrement l'état plutôt qu'à se mettre en attente. L'idée dans ton cas est alors: Dans le OnIdle de ton thread principal tu fais un WaitForMultipleObjects non bloquant, et si certains sont positionnés tu fais le traitement, sinon, tu retourne. Régulièrement, OnIdle est appelé sans pour autant bloquer ton appli.