Discussion :

[Emyleet]

Bonjour à tous!

Ca fait pas mal de temps que je tourne en rond sur ce problème alors j'ai besoin de votre aide...

J'ai un thread qui permet de recevoir des valeurs et de les afficher dans des edit control en continu.
Je pensais faire ca:

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
 
long WINAPI thread14(long lParam)
{
	CString str;
 
	while(1)
	{
	Card.getLeftPositionActual(NULL);
	Card.getRightPositionActual(NULL);
 
 
	CEdit* editFG=(CEdit*) GetDlgItem(NULL,IDC_EDIT3);
	CEdit* editFD=(CEdit*) GetDlgItem(NULL,IDC_EDIT4);
	CEdit* editHG=(CEdit*) GetDlgItem(NULL,IDC_EDIT7);
	CEdit* editHD=(CEdit*) GetDlgItem(NULL,IDC_EDIT8);
 
	str.Format(_T("%.1i"),Card.m_PosDInst);
	editFG->SetWindowText(str);
 
	str.Format(_T("%.1i"),Card.m_PosGInst);
	editFD->SetWindowText(str);
 
	str.Format(_T("%.1i"),Card.m_PreDInst);
	editHD->SetWindowText(str);
 
	str.Format(_T("%.1i"),Card.m_PreGInst);
	editHD->SetWindowText(str);
 
 
 
	}
}

Le projet compile mais à léxécution j'ai l'erreur suivante lorsque j'arrive à ce thread la:
dans la fonction void CWND::SetWindowText(LPCTSTR lpzString) de winocc.cpp
la ligne suivante est surlignée
ENSURE(::IsWindow(m_hWnd) || (m_pCtrlSite != NULL));

je pense que ca vient du hwnd que je ne définis pas mais comment faire pour le définir correctement?

merci d'avance

[PetitPapaNoël]

Bonjour,

Effectivement, ça doit venir du fait que tu ne passes pas le handle de ta fenêtre à la méthode GetDlgItem. As-tu un pointeur ou une variable sur la fenêtre concernée ?

[Emyleet]

non pas vraiment... je ne vois pas comment faire ?
Avec la fonction GetSafeHwnd() je récupére le HWND de la fenetre?

[PetitPapaNoël]

Ça, c'est pour avoir le handle. Mais avant d'avoir le handle, il faut avoir la fenêtre, sous forme de pointeur ou de variable. Où est-elle créée, cette fenêtre ?

[Emyleet]

euhhh j'ai utilisé les toolbox je ne l'ai pas créé manuellement mais par contre je l'appelle à partir de la fenêtre précédente de la manière suivante:

Code :

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

1
2
3
4
5
 
InstantResult *pDlg;
    pDlg = new InstantResult();
    pDlg->Create(InstantResult ::IDD,AfxGetMainWnd());
    pDlg->ShowWindow(SW_SHOW);

le constructeur de cette fenêtre est alors le suivant:

Code :

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

1
2
3
4
5
 
InstantResult::InstantResult(CWnd* pParent /*=NULL*/)
	: CDialog(InstantResult::IDD, pParent)
{
}

[PetitPapaNoël]

Et ce pointeur pDlg, tu le connais à l'endroit où tu fais tes GetDlgItem ?

[farscape]

salut ,
tu fais vraiment n'importe quoi sur cette ligne!:

Code :

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

1
2
CEdit* editFG=(CEdit*) GetDlgItem(NULL,IDC_EDIT3);

tu castes violemment un handle de fenêtre HWND en un objet CEdit .
comme si tu voulais passer d'une carotte a un radis.
il ne faut pas s'étonner si ça plante..
de plus comme tu utilises l'api Win32 pour GetDlgItem tu lui passes null en argument ce qu'elle ne vas pas apprécier puisque c'est sensé être la fenêtre dont l'identifiant IDC_EDIT3 fait partie.
il y a un post dans la faq pour le travail avec un thread de travail et la mise a jour d'un interface MFC.

[PetitPapaNoël]

Si tu disposais d'un pointeur sur ta fenêtre tel que pDlg, tu pourrais faire ceci :

Code :

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

CEdit* editFG = (CEdit*) pDlg->GetDlgItem(IDC_EDIT3);

[farscape]

l'accès direct aux objets GUI MFC fonctionnera en release pas en debug ...

[PetitPapaNoël]

l'accès direct aux objets GUI MFC fonctionnera en release pas en debug ...

Ah bon ! Pour quelle raison ?

[Emyleet]

PetitPapaNoel merci pour ta réponse.
Cependant si je fais ca:

Code :

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

1
2
3
4
5
6
7
8
9
 
InstantResult *pDlg;
pDlg = new InstantResult();
 pDlg->Create(InstantResult ::IDD,AfxGetMainWnd());
 
CEdit* editFG=(CEdit*) pDlg->GetDlgItem(IDC_EDIT3);
 
str.Format(_T("%.1i"),Card.m_PosDInst);
editFG->SetWindowText(str);

je vais avoir une nouvelle fenêtre InstantResult qui va s'ouvrir ... et si je laisse que InstantResult *pDlg, le pointeur n'est pas initialisé.

[farscape]

Ah bon ! Pour quelle raison ?

Explications :
Dans un processus win32 tous les threads travaillent dans le même espace mémoire et peuvent accéder aux variables globales.
En multi-thread les MFC stockent les tables sur les fenêtres et objets GDI localement sur le thread principal grâce au système TLS :thread-local-storage, justement pour éviter des accès intempestifs des différents threads.

Donc bien retenir la règle suivante :
On ne peut pas partager des objets MFC liés aux fenêtres ou d'objets GDI entre plusieurs threads de service.
On passera le handle de fenêtre en paramètre dans la fonction AfxBeginThread pour établir le lien avec la fenêtre de traitement.

a cause du stockage TLS, de manière officielle il n'est pas conseillé d'accèder aux objets MFC en release a partir d'un autre thread..
en debug suivant les accès effectués ça plante.

[PetitPapaNoël]

Ok, je ne savais pas. Il faut dire que je n'ai encore jamais travaillé sur un projet multithread...
Merci, Farscape pour cet éclaircissement !

[Emyleet]

C'est bon j'ai réglé mon pb :
Je déclare un pointeur sur ma fenêtre

Code :

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

1
2
 
InstantResult* fen;

dans mon OnInitDialog, j'initialise mon pointeur:

Code :

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

1
2
 
fen = this;

et enfin dans mon thread:

Code :

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

1
2
3
4
5
 
CEdit* editFG=(CEdit*) fen->GetDlgItem(IDC_EDIT3);
 
str.Format(_T("%.1i"),Card.m_PosDInst);
editFG->SetWindowText(str);

Voila merci !!!!

[PetitPapaNoël]

Mais Farscape vient de dire qu'il ne fallait pas faire comme ça !

[Emyleet]

Mais comment je peux faire alors pour récupérer un pointeur vers ma fenêtre courante ?
je vois pas :/