[MFC] thread d'affichage

Version imprimable

Bonjour,
J'essaye de faire un "thread d'affichage" mais il ne fonctionne pas!
Code:

1 2 3 4 5 6 7 8 9 UINT CImageControllerDlg::display( LPVOID p_param ) { CImageControllerDlg* dlg = (CImageControllerDlg*)p_param; dlg->_processed_img.SetWindowText("0"); dlg->_avnct_gnl_haut->SetWindowText("Calcul du nombre total d'images à analyser"); AfxEndThread(0); return 0; }
dès que le code passe à l'instruction "dlg->_processed_img.SetWindowText("0");", il se bloque! savez vous pourquoi?

19/05/2005, 17h28
farscape

salut,
voir faq:
http://c.developpez.com/faq/vc/?page...d#WorkerThread
:D

Je ne comprend pas tout dans la faq...

J'apelle mon thread d'affichage ainsi :
Code:

1 2 _thread_display = AfxBeginThread( display, GetSafeHwnd(), THREAD_PRIORITY_NORMAL );
Comment faire la fonction display?
Code:

1 2 3 4 5 6 7 8 UINT CImageControllerDlg::display( LPVOID p_param ) { HWND* dlg = (HWND*)p_param; _progress_bar = (CProgressCtrl*)GetDlgItem(IDC_PROGRESS_DETAIL); _processed_img.SetWindowText("0"); }
... :(

19/05/2005, 17h49
farscape
soit tu travailles en win32
Code:

1 2 3 HWND hProgress=::GetDlgItem(dlg,IDC_PROGRESS_DETAIL); // etc..
soit tu fais un message privé dans ta dialogue qui s'occupe de l'affichage.
comme indiqué dans la faq:
voir aussi pour le message privé:
http://www.developpez.net/forums/vie...567222#1567222
:D
19/05/2005, 17h55
Joeleclems
Code:

1 2 3 HWND _progress=::GetDlgItem(dlg,IDC_PROGRESS_DETAIL); _progress->SetPos(100);
ça ne fonctionne pas...
le seul attribu que j'ai dans _progress est "unused"...
19/05/2005, 18h14
farscape

normal HWND n'est pas une classe mais un handle de fenetre...

fait une fonction message prive suivant les indications du post cité
met ton code d'affichage dedans et dans ton thread envoi le message comme indiqué dans la faq.

:D
19/05/2005, 18h17
Joeleclems

lorsque le thread à envoyé le message, comment faire pour le recuperer (c'est pas écris dans la faq, ni dans le message)?
19/05/2005, 18h21
farscape

8O c'est tout ecris la dedans lol
http://www.developpez.net/forums/vie...567222#1567222
dans la deuxieme partie tu as un exemple pour un thread ....
:D
20/05/2005, 09h47
Joeleclems

j'ai fais exactement toutes les étapes décrite et pourtant je ne reçoit pas le message "coucou"...
20/05/2005, 09h51
farscape

tu as mis un point d'arret en debug dans le thread et dans la fonction du message privé ?
:D

il passe bien dans le thread

Code:

1
2
3
4
5
6
7
8
9
 
UINT CImageControllerDlg::display( LPVOID p_param )
{
	//HWND dlg = (HWND)p_param;
	HWND hWnd= static_cast< HWND >( p_param) ; 
	 ::PostMessage(hWnd,WM_TEST,0,0) ; 
	return 0;
 
}

mais ne passe pas dans la fonction associée au message

Code:

1
2
3
4
5
6
7
8
9
10
 
ON_MESSAGE(WM_TEST, OnReceiveTest)  // message privé 
END_MESSAGE_MAP()
 
 
long CImageControllerDlg::OnReceiveTest(WPARAM wparam, LPARAM lparam) 
{ 
   AfxMessageBox("coucou"); 
   return 0L; 
}

...

EDIT:
j'ai bien

Code:

1
2
 
#define WM_TEST WM_USER+100

dans le .h de la boite de dlg ainsi que

Code:

1
2
 
long OnReceiveTest(WPARAM wparam,LPARAM lparam); // fonction de réponse pour le message privé

en protected...

20/05/2005, 10h04
Joeleclems

ne faut il pas faire quelque chose dans le proc pere pour récuperer le message?
EDIT :
quand je fais un afxMessageBox après avoir lancé le thread dans le proc principal, alors je passe bien dans la fonction et la boite de dlg "coucou" apparait!
20/05/2005, 10h18
farscape

tu es sur du handle de fenetre passe dans ton thread ?
:D
20/05/2005, 10h31
Joeleclems
je ne sais pas, j'apelle la fonction GetSafeHwnd:
Code:

1 2 AfxBeginThread( display, GetSafeHwnd(), THREAD_PRIORITY_NORMAL );

J'ai essayé de faire comme ça :

Dans le père :
Code:

1 2 AfxBeginThread( display, this, THREAD_PRIORITY_NORMAL );
Dans le thread :
Code:

1 2 3 4 5 6 7 UINT CImageControllerDlg::display( LPVOID p_param ) { CImageControllerDlg* dlg = (CImageControllerDlg*)p_param; ::PostMessage(dlg->GetSafeHwnd(),WM_TEST,GetCurrentThreadId(),0); return 0; }
ça me donne exactement le même resultat : si je lance une afxmessagebox dans le père après avoir lancé le thread, ça affiche bien "coucou", sinon il ne rentre pas dans la fonction "OnReceiveTest"...
farscape au secours!

20/05/2005, 11h52
Brouzouf

Essaie dans ton .h de préfixer la fonction de réception du message par afx_msg.

EDIT : ça ane dvrai rien chager en fait. Tu ne pourrais pas envoyer tes sources si elles ne sont pas volumineuses ?
20/05/2005, 12h04
Joeleclems

ça fait exactement la même chose :piou:

mes sources completes sont volumineuses... mais voici les extraits "utiles"
:
ImageController.h

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
25
26
27
28
29
30
31
 
#pragma once
#include "afxwin.h"
#include "Report.h"
#include <iostream>
 
#define WM_TEST WM_USER+100 
class CImageControllerDlg : public CDialog
{
 
public:
enum { IDD = IDD_IMAGECONTROLLER_DIALOG };
 
(...)
 
protected:
	HICON m_hIcon;
 
	virtual void DoDataExchange(CDataExchange* pDX);	// Prise en charge DDX/DDV
	// Fonctions générées de la table des messages
	virtual BOOL OnInitDialog();
	afx_msg void OnSysCommand(UINT nID, LPARAM lParam);
	afx_msg void OnPaint();
	afx_msg HCURSOR OnQueryDragIcon();
	afx_msg long OnReceiveTest(WPARAM wparam,LPARAM lparam); // fonction de réponse pour le message privé 
	DECLARE_MESSAGE_MAP()
 
	virtual void OnRangeUpdateCheck(UINT nID);
 
(...)
};

ImageController.cpp

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
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
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
 
#include "stdafx.h"
#include "ImageController.h"
#include "ImageControllerDlg.h"
#include ".\imagecontrollerdlg.h"
#include "Images.h"
#include "FindFile.h"
#include "FolderDlg.h"
 
#ifdef _DEBUG
#define new DEBUG_NEW
#endif
 
//static int Save_histo;
//static UINT StartThread (LPVOID param);	//controlling function header
//static UINT computeTime (LPVOID param);	//controlling function header
 
// boîte de dialogue CAboutDlg utilisée pour la boîte de dialogue 'À propos de' pour votre application
class CAboutDlg : public CDialog
{
public:
	CAboutDlg();
 
// Données de boîte de dialogue
	enum { IDD = IDD_ABOUTBOX };
 
	protected:
	virtual void DoDataExchange(CDataExchange* pDX);    // Prise en charge DDX/DDV
 
// Implémentation
protected:
	DECLARE_MESSAGE_MAP()
};
 
CAboutDlg::CAboutDlg() : CDialog(CAboutDlg::IDD)
{
}
 
void CAboutDlg::DoDataExchange(CDataExchange* pDX)
{
	CDialog::DoDataExchange(pDX);
}
 
BEGIN_MESSAGE_MAP(CAboutDlg, CDialog)
END_MESSAGE_MAP()
 
 
// boîte de dialogue CImageControllerDlg
 
BEGIN_MESSAGE_MAP(CImageControllerDlg, CDialog)
	ON_WM_SYSCOMMAND()
	ON_WM_PAINT()
	ON_WM_QUERYDRAGICON()
	ON_MESSAGE(WM_TEST, OnReceiveTest)  // message privé 
	//}}AFX_MSG_MAP
	ON_BN_CLICKED(B_GO, OnBnClickedGo)
	ON_BN_CLICKED(B_CANCEL, OnBnClickedCancel)
	ON_BN_CLICKED(B_STOP, OnBnClickedStop)
	ON_BN_CLICKED(B_BROWSE_SOURCE, OnBnClickedBrowseSource)
	ON_BN_CLICKED(B_BROWSE_DEST, OnBnClickedBrowseDest)
	ON_BN_CLICKED(B_BROWSE_REPORT_DEST, OnBnClickedBrowseReportDest)
	ON_BN_CLICKED(C_BLUR, OnBnClickedBlur)
	ON_BN_CLICKED(C_MONOTONE, OnBnClickedMonotone)
	ON_BN_CLICKED(C_IDENTICAL_FOLDER, OnBnClickedIdenticalFolder)
	ON_BN_CLICKED(C_COPY_IMAGES, OnBnClickedCopyImages)
	ON_BN_CLICKED(C_IDENTICAL_FOLDER_REPORT, OnBnClickedIdenticalFolderReport)
	ON_EN_CHANGE(EC_SENSIBILITE, OnEnChangeSensibilite)
	ON_BN_CLICKED(C_COPY_MONOTONE, OnBnClickedCopyMonotone)
	ON_BN_CLICKED(C_CUT_MONOTONE, OnBnClickedCutMonotone)
	ON_EN_CHANGE(EC_NB_THREAD, OnEnChangeNbThread)
	ON_EN_CHANGE(EC_DEST_FOLDER, OnEnChangeDestFolder)
END_MESSAGE_MAP()
 
 
long CImageControllerDlg::OnReceiveTest(WPARAM wparam, LPARAM lparam) 
{ 
   AfxMessageBox("coucou");
   return 0L; 
}

fonction qui lance le programme

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
//Lancement du processus de verification
void CImageControllerDlg::OnBnClickedGo()
{
	_stop_thread = false;
 
	if (_is_process == true)
	{
		AfxMessageBox ("Processus actuellement en cours !");
		return;
	}
	else
	{
		execute();
		return;
	}
}

fonction execute

Code:

1
2
3
4
5
6
7
8
 
int CImageControllerDlg::execute(void)
{
(...)
_thread_display = AfxBeginThread( display, this, THREAD_PRIORITY_NORMAL );
_thread_time = AfxBeginThread( computeTime, this , THREAD_PRIORITY_NORMAL);
(...)
}

fonction associée au thread display :

Code:

1
2
3
4
5
6
 
UINT CImageControllerDlg::display( LPVOID p_param )
{
	CImageControllerDlg* dlg = (CImageControllerDlg*)p_param;
	::PostMessage(dlg->GetSafeHwnd(),WM_TEST,GetCurrentThreadId(),0);
return 0;

... si tu veux d'autres "bouts" dis le moi, se sera avec plaisir :-)

20/05/2005, 13h44
Brouzouf

Que fais-tu apès dans la méthode execute ?

Edit : est-ce que tu n'attenderais pas un évènement par hasard ?
20/05/2005, 13h51
Joeleclems
Je parcours le dossier passé en paramètre par l'utilisateur en recursif. Dans chaque dossier, j'analyse la liste de toutes les images en effectuant differents traitement. Je sais c'est un peu vague, mais le code est vraiment très long!
ha si, ce qui est peut-être interressant, c'est que je lance d'autres threads : un thread par image! sans dépasser le nombre de 4 threads. Le principe est des qu'un thread a fini de traiter une image, je lui donne une nouvelle image à traiter, jusqu'à ce qu'il n'y ait plus d'images à traiter.
enfin, je ne pense pas que le problème vienne du code qui suit, parce que si dans la fonction "execute", juste après :
Code:

1 2 thread_time = AfxBeginThread( computeTime, this , THREAD_PRIORITY_NORMAL);
je fais :
Code:

1 2 afxMessageBox("toto");
alors j'ai les deux messages box qui apparaissent : "coucou", puis "toto".
20/05/2005, 14h10
Brouzouf

En fait, la seule raison pour que tu ne récupères pas ton message, c'est que ton thread principal se retrouve bloqué et qu'il ne puisse pas traité le message que tu essaie de lui envoyé.

Essaie en mettant en commentaire tout ce qu'il y a après dans ta fonction execute.
20/05/2005, 14h24
Joeleclems

merci :D
Tu as raison, en mettant tout ce qui suit en commentaire, ça passe!
20/05/2005, 15h55
Joeleclems

ce qui bloquait c'était le "WaitForSingleObject(_threads[i]->m_hThread, 100);" : le problème c'est que mon prog principal doit attendre la fin des thread, mais doit aussi réagir aux messages que les threads lui envoient! et même si un thread envoie un message par PostMessage, alors le prog principal se bloque et attend : il ne rentre pas dans la fonction associé au message! comment faire?
20/05/2005, 17h17
farscape

et bien message privé :
quand le thread a fini il envoie un message pour le signaler au prog principal.
:D
20/05/2005, 17h24
Joeleclems

et ce n'est pas PostMessage qui sert à ça? j'utilise déjà un post message, mais la fonction qui lui es associé n'est jamais appelée...
23/05/2005, 09h47
Joeleclems
Je ne sais pas exactement pourquoi, mais lorsque mon application lance un thread des le départ et que ce même thread envoie d'autre thread, alors le systeme des messages fonctionne... mais maintenant, les threads que j'envoie font une boucle, et au second tours, ils se bloquent! en mettant visual en pause, je m'arrete sur la ligne soulignée:
Code:

1 2 3 4 5 6 7 BOOL AFXAPI AfxInternalPumpMessage() { _AFX_THREAD_STATE *pState = AfxGetThreadState(); [u]if (!::GetMessage(&(pState->m_msgCur), NULL, NULL, NULL))[/u] (...)
cette partie de code appartient au fichier thrdcore.cpp... est ce que quelqu'un peut m'éclairer? je ne comprends pas d'ou cela peut venir!
23/05/2005, 09h50
Brouzouf

Rien ne t'empêche de faire plusieurs Postmessage dans tes threads.
WaitForSingleObject est une fonction bloquante, donc quand tu l'utilises, tu bloques ton thread un certain temps.

Tu peux à la fin d'un thread poster un message signalant la fin du thread, il suffit que tu passes en paramètre l'id du thread se terminant.

Si ton thread pricipal n'est pas bloqué, normalement tu dois recevoir le message.
23/05/2005, 10h03
Joeleclems

c'est ce que je fais!
mon thread est un while(bool){}, lorsqu'il arrive à la fin de la boucle, alors le thread envoie un message au thread principal, et se bloque. Le thread principal verifie s'il a encore besoin du thread : si c'est la cas, alors il le relance, sinon il met le bool de la boucle while à false et le relance (ce qui termine le thread).
Le problème c'est que le premier passage dans la boucle se fait correctement, mais au second, il plante (cf message precedent). Au secours!
23/05/2005, 11h31
Brouzouf

Comment est-ce que tu relances ton thread dans les deux cas ?
23/05/2005, 11h48
Joeleclems

dans les deux cas, c'est la fonction associé au PostMessage qui fait un "ResumeThread"

Tu t'ais fait une classe dérivée de cwinthread ?

Je te conseille la méthode suivante :

Tu as deux events, un pour faire travailler ton thread, un pour l'arrêter.

Tu fais un waitformultipleobjects sur ces event dans ton thread, tu le met dans une boucle while (1).

Quand tu veux faire travailler ton thread, tu set l'event correspondant, il travaille puis retourne sur le wait.

Quand tu veux qu'il se termine, tu set l'event correspondant, il sort du wait et la tu fais un break pour qu'il sorte du while.

Ne pas oublier de faire des event uniques pour chaque thread.

Voici un exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 HANDLE TabEvent[2]; DWORD dwRet; TabEvent[0] = hEvtDoWork; TabEvent[1] = hEvtSortie; while (1) { dwRet = ::WaitForMultipleObjects(2,TabEvent,FALSE,ININITE) if (dwRet == WAIT_OBJECT_0) { //traitement } else if (dwRet == WAIT_OBJECT_0+1) { //on sort break; } }
En espérant que ça t'aide.

23/05/2005, 14h47
Joeleclems

Je vais essayer ta méthode, pour l'instant je ne travaille qu'avec des PostMessage et pas d'event. Pour répondre à ta question, je travaille avec la classe de base de thread : je n'ai pas de classe à moi.