Discussion :

[Alice9]

Bonjour,


j'ai une application MDI dans laquelle j'ai mis des CDialogBar.

En mode debug, quand je fais un showControlBar d'une de ces boîtes, cette boîte apparait ou disparait correctement.

En mode release, quand je veux faire disparaitre une de ces boîtes avec un ShowControlBar, pas de problème.
Mais quand je veux faire apparaître une de ces boîtes avec un ShowControlBar , l'ordi me répond que cette action engendre un problème mémoire.

Auriez vous une idée du problème ?



Alice

[r0d]

salut,

j'ai récemment appris, à mes dépends, qu'en mode debug, le compilo initialise les variables à 0 (==NULL). Alors qu'en mode release, il n'en fait rien et du coup, ça peut engendrer des problèmes de pointeur. Vérifie donc l'initialisation de ta CDialogBar.

[farscape]

pb typique de debordement memoire ou de mauvaise initialisation de pointeur.
note pour r0d:
les variables ne sont pas initialisées en DEBUG:
une variable declarée non initialisée prend la valeur :0xcdcdcdcd ou 0xCCCCCCCC

[Alice9]

pour eviter ce problème je réinitialise à null le pointeur sur ma futur boîte de dialog (avant sa création)

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
m_dlgBar_Gamme = NULL ;
m_dlgBar_Saisie = new CDlgBar_Saisie ;
 
m_dlgBar_Saisie->m_pMainFrame=this; 
 
if (!m_dlgBar_Saisie->Create(
				pilot3D_listType,
				pilot3D_listRef,
				pilot3D_dataDir,
				this, IDD_DLGBAR_SAISIE,
				CBRS_TOP | CBRS_GRIPPER |CBRS_TOOLTIPS | CBRS_FLYBY,
				IDD_DLGBAR_SAISIE))
{
	writeInLog("---> Création de la boîte de dialogue flottante Saisie :") 
	writeInLog(" impossible de créer cette boîte\n") ;
	return -1;		// fail to create
}
// caractéristiques : se colle sur tous les bords de la fenêtre
m_dlgBar_Saisie->EnableDocking(CBRS_ALIGN_ANY);
EnableDocking(CBRS_ALIGN_ANY);
DockControlBar(m_dlgBar_Saisie);
ShowControlBar(m_dlgBar_Saisie,FALSE,FALSE);  // on la désactive

et quand je veux la faire apparaître :

Code :

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

1
2
3
4
5
if (m_dlgBar_Saisie->IsWindowVisible() != TRUE)
	{
		ShowControlBar(m_dlgBar_Saisie,TRUE,FALSE); 
	}
	else ShowControlBar(m_dlgBar_Saisie,FALSE,FALSE);

-> tous les membres de la classe CDlgBar_Saisie sont eux aussi initialisés.

d'où mon interrogation : où y a t il une erreur qui puisse engendrer un tel débordement ? (sachant que le problème apparait à partir du moment où l'on souhaite visualiser la boîte)

[r0d]

une variable declarée non initialisée prend la valeur :0xcdcdcdcd ou 0xCCCCCCCC

exact...
[désolé]

[r0d]

Code :

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

ShowControlBar(m_dlgBar_Saisie,TRUE,FALSE);

as-tu vérifié la valeur du pointeur m_dlgBar_Saisie à ce niveau du programme (en debug)?

[Alice9]

en mode debug :

au début mon pointeur contient : 0x000000
puis après le new : 0x00452730
après le OnCreate de ma boîte : 0x00452730
et à la ligne contenant la commande maudite : 0x00452730

-> a priori il pointe correctement en mémoire.


Alice

[farscape]

essaye de rajouter avant l'appel :

Code :

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

1
2
 
ASSERT(::IsWindow(mypdlgbar-> GetSafeHwnd()));

ou mypdlgbar est ta dialoguebar.

[Alice9]

Avec cette ligne à la place de la cmde ShowControlBar, pas de problème que se soit en debug ou en relise
Avec la cmde ShowControlBar + cette ligne de code juste avant ou non , quand je demande voir la boîte voici les 2 messages successifs d'insultes :

l'instrucion à "0x0012fc86" emploie l'adresse mémoire "0x00000001". La mémoire ne peut pas être "read"
cliquez sur ok pour terminer le programme
Cliquez sur annuler pour déboguer le programme

puis

l'instruction à "0x73d211c7" emploie l'adresse mémoire "0x00000004". La mémoire ne peut pas être "read".
cliquez sur ok pour terminer le programme

Je pencherai aussi pour un pointeur fou dans l'appli au niveau de ces boîtes mais pour l'instant je n'ai pas de pistes valides .....



Alice

[Alice9]

J'ai réalisé un nouveau test en ne mettant pas la cmde ShowControlBar de manière ce que ma boîte soit directement visible :

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
m_dlgBar_Gamme = NULL ;
m_dlgBar_Saisie = new CDlgBar_Saisie ;
 
m_dlgBar_Saisie->m_pMainFrame=this;
 
if (!m_dlgBar_Saisie->Create(
            pilot3D_listType,
            pilot3D_listRef,
            pilot3D_dataDir,
            this, IDD_DLGBAR_SAISIE,
            CBRS_TOP | CBRS_GRIPPER |CBRS_TOOLTIPS | CBRS_FLYBY,
            IDD_DLGBAR_SAISIE))
{
   writeInLog("---> Création de la boîte de dialogue flottante Saisie :")
   writeInLog(" impossible de créer cette boîte\n") ;
   return -1;      // fail to create
}
// caractéristiques : se colle sur tous les bords de la fenêtre
m_dlgBar_Saisie->EnableDocking(CBRS_ALIGN_ANY);
EnableDocking(CBRS_ALIGN_ANY);
DockControlBar(m_dlgBar_Saisie);

Dans ce cas, cela plante direct au niveau de la mémoire. En essayant d'en savoir d'avantage je me suis aperçue que c'est la fonction CDialogBar::Create qui plante l'appli en mode release.... ci dessous le code de la fonction Create de la boîte CDialogBar_Saisie :

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
BOOL CDlgBar_Saisie::Create(CString _listType,	// nom du fichier de la liste des types
							CString _listRef,	// nom du fichier de la liste des référénces du type X
							CString _dataDir,	// répertoire des donnéées fixes
							CWnd* pParent, UINT nIDTemplate, UINT nStyle, UINT nID) 
// création de la boîte de dialogue Saisie
{
	// création de la boîte CDialogBar
	writeInLog("\n---------------------------------\n\n") ;
	writeInLog("           OUVERTURE Boîte DlgBar Saisie\n\n") ;
 
	BOOL bReturn = CDialogBar::Create(pParent, nIDTemplate, nStyle, nID);
	writeInLog("---> Création de la boîte de dialogue flottante Saisie :") ;
	writeInLog("\n---------------------------------\n\n") ;
	// retourne si la boîte à bien été créée ou non
	return bReturn;
}

perso je deviens un peu en recherchant ce bug donc toute nouvelle idée est bienvenue !!!



Alice

[farscape]

re,
comme ça je ne vois rien qui me choque:
essaye de faire:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
BOOL CDlgBar_Saisie::Create(CString _listType,   // nom du fichier de la liste des types
                     CString _listRef,   // nom du fichier de la liste des référénces du type X
                     CString _dataDir,   // répertoire des donnéées fixes
                     CWnd* pParent, UINT nIDTemplate, UINT nStyle, UINT nID)
// création de la boîte de dialogue Saisie
{
   // création de la boîte CDialogBar
    AfxMessageBox("step1");
   BOOL bReturn = CDialogBar::Create(pParent, nIDTemplate, nStyle, nID);
    AfxMessageBox("step2");
   // retourne si la boîte à bien été créée ou non
   return bReturn;
}

ça plante ou exactement apres "step1" ou "step2".

[Alice9]

je ne sais pas exactement ,on a l'impression que ça plante dans la fonction Create.

du coup ce matin je reprends tout tranquillement : je refais l'appli depuis le début sans aller trop vite histoire de comprendre ce qui ne va pas.
-> hier soir sur 3 boîtes de dialogue , j'en avais 1 qui fonctionnai correctement sous release et pas les 2autres alors qu'en comparant les sources il n'y avait ps de différence.

donc je te tiens au courant dès que j'en sais un peu plus



Alice

[Alice9]

ça y est : :trouve: que ce soit en mode debug ou release tout fonctionne correctement !!!!

Pour cela, je ne parle plus de pointeur sur des boîtes de dialogues mais simplement d'instances et surtout le plus important :

Lors de la création d'une boîte de type CDialogBar (voir ), il faut transformer la fonction Create en ajoutant explicitement l'appel à la fonction OnInitDialog qui contient :

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
BOOL CDlgBar_Saisie::OnInitDialog() 
{
	UpdateData(FALSE) ;
	// TODO: Add extra initialization here
	return TRUE;  // return TRUE unless you set the focus to a control
	              // EXCEPTION: OCX Property Pages should return FALSE
}
 
BOOL CDlgBar_Saisie::Create(CWnd* pParent, UINT nIDTemplate, UINT nStyle, UINT nID) 
// création de la boîte de dialogue Saisie
{
	// création de la boîte CDialogBar
 
	BOOL bReturn = CDialogBar::Create(pParent, nIDTemplate, nStyle, nID);
	if(bReturn != false)
	{
		if(!OnInitDialog())		
		{
			return FALSE;	
		}
	}
 
	// retourne si la boîte à bien été créée ou non
	return bReturn;
}

La fonction OnInitDialog n'est pas à implémenter dans le MESSAGE MAP !!!
Deplus la fonction Create doit être transformée dans le header car ce n'est plus une fonction virtuelle mais public.

d'autres part il faut faire super gaffe sur la manière de gérer les évènements de la boîte. Je m'explique :
Après avoirs implémenter de manière traditionnelle les fonctions liées à des évènements (comme dans une CDialog). Il faut rajouter les fonctions OnUpdateTypeDuControl.
Pour les comboBox et les boutons, rajouter à la main la ligne :

Code :

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

1
2
 
	ON_UPDATE_COMMAND_UI(IDC_CB_TYPE, OnUpdateCbType)

ainsi que la fonction OnUpdateXXX :

Code :

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

1
2
3
4
5
6
void CDlgBar_Saisie::OnUpdateCbType(CCmdUI *pCmdUI) 
// obligatoire dans une boîte flottante pour prendre en compte les évènements
{
	// TODO: Add your control notification handler code here
 
}

De plus il est à noter que cette fonction est à intégrer dans la Classe comme une simple fonction protégée et non comme une fonction de type afx_msg !!
Pour les boîtes d'édition (CEdit), rajouter la fonction liée à l'évènement EN_UPDATE (voir AppWizard). De cette manière vous obtiendrez le message suivant dans le MESSAGE_MAP :

Code :

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

ON_EN_UPDATE(IDC_ED_MSN, OnUpdateEdMsn)

Dans ce cas la fonction OnUpdateXXX est générée automatiquement mais ne lui rajoutez pas de paramètres !!

Bien entendu n'oubliez pas de déclarer et de créer dans le programme principale la boîte désirée (voir ).

De cette manière l'application se lance et plus aucun bug mémoire n'apparait (mode Debug ou Release) quand on active ou non les boîtes flottantes !!!

Encore un grand à Farscape et à r0d pour leurs idées !!!

Alice