Discussion :

[loupdeau]

Salut,

Voici le contexte : depuis un thread exécuté à partir d'un CProcessDlg (modal) je poste un message dans ce même CProcessDlg pour commander l'affichage d'un CAskUserDlg (modal).

CProcessDlg et CAskUseDlg sont dérivés de CDialog.

voici le code dans CProcessDlg permettant d'afficher le CAskUserDlg :

void CProcessDlg::askUser(WPARAM, LPARAM lParam)
{
m_askUserText = *reinterpret_cast<CString*>(lParam);

CAskUserDlg dialog;
dialog.setText(m_askUserText);
dialog.DoModal();
}

Tout fonctionne très bien en mode DEBUG. En mode RELEASE ca se gate...

En mode RELEASE :
Le CAskUserDlg s'affiche correctement (celui ci contient un CStatic, une CProgressbar et 2 CButton).
Mais dès que je passe ma souris soit sur la CProgressbar soit sur l'un des CButton l'application plante.

The memory could not be read

Je comprends rien, pourquoi ca marche en DEBUG, pourquoi ca marche pas en RELEASE...

[farscape]

le message qui genere askUser c'est quoi posmessage ou sendmessage ?

[loupdeau]

C'est PostMessage.

Je viens d'essayer avec SendMessage... mais ca change rien.

Je pensais aussi au début que l'erreur provenais du "postage" de message... mais je ne pense plus. Le texte que je poste apparait correctement dans tout les cas.

L'erreur se produit vraiment seulement quand je passe la souris sur un des boutons ou sur la progressbar.


voila tout le contexte si ca peut aider :

Dans le thread :

Code :

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

1
2
3
4
5
6
 
void CDialogCommand::execute()
{
  ::PostMessage(*m_pvParam,m_msg,0,(LPARAM)&m_text) ;
  AfxGetThread()->SuspendThread();
}

Hors du thread :

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
void CProcessDlg::askUser(WPARAM, LPARAM lParam)
{
 
	m_askUserText = *reinterpret_cast<CString*>(lParam);
 
	CAskUserDlg dialog;
	dialog.setText(m_askUserText);
	int i = dialog.DoModal();
 
	if(i==1)
	{
		m_pThread->ResumeThread();
	}
                else
                {
                               TerminateThread(m_pThread,0);
                }
 
}

[Caine]

Tu es certain de ne pas avoir de code conditionnel au mode débug qui devarait aussi apparaître en mode Release?

[loupdeau]

Perso, je n'ai ajouté aucune ligne de code conditionnelle DEBUG/RELEASE.

La seul chose qu'on peut trouver c'est ce que Visual C++ ajoute tout seul dans chaque fichier :

Code :

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

1
2
3
4
5
#ifdef _DEBUG
#define new DEBUG_NEW
#undef THIS_FILE
static char THIS_FILE[] = __FILE__;
#endif

...

désespérant...

[farscape]

salut,

dialog.setText(m_askUserText);
int i = dialog.DoModal();

j'espere que setText ne fais pas de traitement graphique ?

[Trap D]

C'est aussi souvent un problème de variables non initialisées, en mode debug le compilo les initialise gentiment, mais pas en mode release.

[loupdeau]

Non ne t'inquiètes pas (j'avoue j'ai failli me faire avoir... )

Voila ce que fait setText

Code :

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

1
2
3
4
void CAskUserDlg::setText(CString text)
{
   m_text = text;    // m_text est un CString
}

Un CStatic prendra la valeur de m_text dans OnInitDialog...


Je viens de trouver ca :
Visual C++ 6 - Release builds crash but Debug build do not crash

ce qui est dit, c'est qu'en général, quand ca marche en debug et pas en release, c'est qu'il y a un problème avec les messagemap.

Vu que j'utilise des messages également, je pense que mon erreur peu venir de la... seulement je vois pas trop où est mon erreur (si il y en a une à ce niveau)

Ma message map :

Code :

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

1
2
3
4
5
6
BEGIN_MESSAGE_MAP(CProcessDlg, CDialog)
	//{{AFX_MSG_MAP(CProcessDlg)
	//}}AFX_MSG_MAP
ON_THREAD_MESSAGE(WM_ASK_USER,askUser)
ON_THREAD_MESSAGE(WM_EDIT_EMAIL,editEmail)
END_MESSAGE_MAP()

la déclaration de ma fonction :

Code :

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

1
2
3
4
void CProcessDlg::askUser(WPARAM wParam, LPARAM lParam)
{
...
}

[farscape]

hum ce qui me plait pas trop c'est ton utilisation de ON_THREAD_MESSAGE
pour une dialogue ...
cette macros s'utilise avec une classe CWinThread .
remplace la par ON_MESSAGE.
voir ce post pour les messages prives :
http://www.developpez.net/forums/vie...567222#1567222

[loupdeau]

Je viens de remplaer ON_THREAD_MESSAGE par ON_MESSAGE...
ca fait toujours la même chose que tout à l'heure.
Ca plante en release, quand je passe au dessus d'un bouton, ou progressbar...

[loupdeau]

J'ai trouvé d'où vient le plantage (mais je sais toujours pas pourquoi).

Dans CAskUserDlg j'avais mis en place un timer qui faisait avancer la progessbar.

Si je vire tout ce qui est relatif à ce timer, ca ne plante plus en release...
mais pourquoi ca plante ???

dans le fichier .hpp

Code :

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

1
2
 
UINT m_timer;

dans le fichier .cpp

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
 
BEGIN_MESSAGE_MAP(CAskUserDlg, CDialog)
//{{AFX_MSG_MAP(CAskUserDlg)
ON_BN_CLICKED(IDSTOP, OnStop)
ON_BN_CLICKED(IDWEITER, OnWeiter)
//}}AFX_MSG_MAP
ON_MESSAGE(WM_TIMER,onTimer)
END_MESSAGE_MAP()
 
BOOL CAskUserDlg::OnInitDialog() 
{
 ... 
 m_timer = SetTimer(1,1000,0);
 ...	
return TRUE;  
}
 
 
void CAskUserDlg::onTimer(UINT nIDEvent)
{
    switch(nIDEvent)
    {
        case 1:updateProgressBar();  
        default:break;
    }    
    // Call base class handler.
    CDialog::OnTimer(nIDEvent);
}*/

: : :

[farscape]

tu as le droit d'avoir un timer ,
c'esr peut etre ta fonction updateProgressBar() qui met le bazare.

[loupdeau]

j'ai testé avec une fonction OnTimer qui fait rien du tout... le plantage à lieu...

Code :

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

1
2
3
4
5
6
void CAskUserDlg::OnTimer(UINT nIDEvent)
{
 
 
 
}