problem d'affichage de boîte de dialog

Version imprimable

01/10/2007, 14h42
mamwrya

problem d'affichage de boîte de dialog

Bonjour,
Je crée une boîte de dialog avec une nouvelle classe dédier à celle-ci. Dans cette boîte j'ai un simple Combobox. Mon projet est un SDI et il se compile correctement. Quand j'instencie un objet de cette classe dans un des membres de mon CDocument, et que je veux afficher la fenêtre avec DoModal(). J'ai la fenêtre de (MS VC++ Debug Library, Debug Assertion Faild File: afxwin2.inl)? J'ai aussi cette fenêtre d'Assetion Failed quand je veux rajouter un membre à ma liste de Combobox?
Avez-vous une idée pourquoi ça m'arrive?
Quand je regard l'instance dans une session de débugage le hWnd est à 0x00000000, est-ce normal?

Merci d'avance pour votre aide.
01/10/2007, 14h49
David Fouejio

bonjour
Rassures toi que tu travailles avec le combo box après l'apppel de CDialog::OnInitDialog, car c'est dans cette fonction de la classe de base que tous les controles sont crées.
01/10/2007, 15h22
mamwrya

Bonjour,

En effet, il n y va pas dans le OnInitDilaog à l'instanciation, pourtant dans la classWizard de ma class de dialog le Message WM_INITDIALOG est bien activé, et il est bien dérigé vers la fonction membre OnInitDialog. A ton avis pourquoi?
01/10/2007, 16h50
Spout

Etant donné que

Citation:

Envoyé par MSDN

This message is sent to the dialog box during the Create, CreateIndirect, or DoModal calls, which occur immediately before the dialog box is displayed.

Ce n'est pas normal si il ne passe pas dans le OnInitDialog.
Ta classe hérite bien de CDialog?
Montre nous ton code qu'on puisse essayer de voir et/ou reproduire l'erreur.

Oui biensur que c'est de CDialog.
voici mon .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
32
33
34
35
36
37
38
39
 
class CDeviceMonitor : public CDialog
{
// Construction
public:
	CDeviceMonitor(CWnd* pParent = NULL);   // standard constructor
 
// Dialog Data
	//{{AFX_DATA(CDeviceMonitor)
 
	enum { IDD = IDD_DIALOG1 };
	CComboBox	m_devicelist;
 
	//}}AFX_DATA
 //   CWMTempsReelDoc* GetDocument();
    void Initialize(void);
// Overrides
	// ClassWizard generated virtual function overrides
	//{{AFX_VIRTUAL(CDeviceMonitor)
	protected:
	virtual void DoDataExchange(CDataExchange* pDX);    // DDX/DDV support
	//}}AFX_VIRTUAL
 
// Implementation
protected:
 
	// Generated message map functions
	//{{AFX_MSG(CDeviceMonitor)
		virtual BOOL OnInitDialog();
	//}}AFX_MSG
	DECLARE_MESSAGE_MAP()
};
 
 
 
//{{AFX_INSERT_LOCATION}}
// Microsoft Visual C++ will insert additional declarations immediately before the previous line.
 
#endif // !defined(AFX_DEVICEMONITOR_H__DF0C2128_CA9D_4330_946D_CF8B10C3E761__INCLUDED_)

et le .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
class CDeviceMonitor : public CDialog
{
// Construction
public:
	CDeviceMonitor(CWnd* pParent = NULL);   // standard constructor
 
// Dialog Data
	//{{AFX_DATA(CDeviceMonitor)
 
	enum { IDD = IDD_DIALOG1 };
	CComboBox	m_devicelist;
 
	//}}AFX_DATA
 //   CWMTempsReelDoc* GetDocument();
    void Initialize(void);
// Overrides
	// ClassWizard generated virtual function overrides
	//{{AFX_VIRTUAL(CDeviceMonitor)
	protected:
	virtual void DoDataExchange(CDataExchange* pDX);    // DDX/DDV support
	//}}AFX_VIRTUAL
 
// Implementation
protected:
 
	// Generated message map functions
	//{{AFX_MSG(CDeviceMonitor)
		virtual BOOL OnInitDialog();
	//}}AFX_MSG
	DECLARE_MESSAGE_MAP()
};
 
 
 
//{{AFX_INSERT_LOCATION}}
// Microsoft Visual C++ will insert additional declarations immediately before the previous line.
 
#endif // !defined(AFX_DEVICEMONITOR_H__DF0C2128_CA9D_4330_946D_CF8B10C3E761__INCLUDED_)

J'espère que ça aide, Merci

pardon mon .cpp est

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
 
 
// CDeviceMonitor dialog
 
 
CDeviceMonitor::CDeviceMonitor(CWnd* pParent /*=NULL*/)
	: CDialog(CDeviceMonitor::IDD, pParent)
{
	//{{AFX_DATA_INIT(CDeviceMonitor)
	//}}AFX_DATA_INIT
}
 
 
void CDeviceMonitor::DoDataExchange(CDataExchange* pDX)
{
	CDialog::DoDataExchange(pDX);
	//{{AFX_DATA_MAP(CDeviceMonitor)
	DDX_Control(pDX, IDC_COMBO1, m_devicelist);
	//}}AFX_DATA_MAP
}
 
 
BEGIN_MESSAGE_MAP(CDeviceMonitor, CDialog)
	//{{AFX_MSG_MAP(CDeviceMonitor)
		// NOTE: the ClassWizard will add message map macros here
	//}}AFX_MSG_MAP
END_MESSAGE_MAP()
 
 
/////////////////////////////////////////////////////////////////////////////
// CDeviceMonitor message handlers
 
BOOL CDeviceMonitor::OnInitDialog() 
{
	CDialog::OnInitDialog();
 
 
	return TRUE;
}

01/10/2007, 18h28
Spout

Et l'appel à ta classe?

J'ai essayé de 2 manière
1- en créeant un nouvel objet
Code:

1 2 3 4 5 6 7 8 9 10 11 12 void CWMTempsReelDoc::OnPaDevices() { CDeviceMonitor *audio_devices = new CDeviceMonitor; ASSERT_VALID(audio_devices); int ii; for(ii=0; ii<DeviceIndex; ii++) audio_devices->m_devicelist.AddString(DeviceList[ii]); .....
en sachant que DeviceList[ii] est un tableau de CString définit avant. même l ASSERT_VALID passe.
L'affichage plante à la ligne (audio_devices->m_devicelist.AddString(DeviceList[ii]) ). C'est pareil quant c'est un audio_devices->DoModal();

2- et aussi l'appel de la classe
Code:

1 2 3 4 5 6 7 8 9 10 11 void CWMTempsReelDoc::OnPaDevices() { CDeviceMonitor audio_devices; // = new CDeviceMonitor; // ASSERT_VALID(audio_devices); int ii; for(ii=0; ii<DeviceIndex; ii++) audio_devices.m_devicelist.AddString(DeviceList[ii]);
Les deux plantent au meme endroit.

Merci,

02/10/2007, 14h04
stephdim

tu essayes d'ajouter des chaines a une ComboBox qui n'existe pas encore !!!

le ComboBox est créé (et detruit) uniquement dans DoModal()

l'endroit ou doit etre initialisé ton ComboBox est dans OnInitDialog() et pas avant (car inexistant)

tu dois passer par un tableau, un CStringArray par ex pour stocker tes chaines.

Puis dans OnInitDialog(),faire un transfert de ton CStringArray vers ton Combo par AddString()

@+
02/10/2007, 16h51
Spout

stephdim a raison, c'est pas en déclarant un pointeur sur un nouveau DeviceMonitor que la ComboBox est créée. Tu dois forcément passer par Create, CreateIndirect, ou DoModal.

Merci infiniment, ça a marché,

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
BOOL CDeviceMonitor::OnInitDialog() 
{
 
    //int ii;
	CDialog::OnInitDialog();
 
		m_devicelist.AddString("device0");
		m_devicelist.AddString("device1");
		m_devicelist.AddString("device2");
 
 
 
        UpdateData(false);
	return TRUE;
}

puis le DoModal comme ceci

Code:

1
2
3
4
5
6
7
8
9
10
11
 
void CWMTempsReelDoc::OnPaDevices() 
{
 
 
	CDeviceMonitor *audio_devices = new CDeviceMonitor;
	ASSERT_VALID(audio_devices);
 
 
	audio_devices->DoModal();
....

02/10/2007, 18h58
stephdim

le UpdateData(false) est superflu (inutile)

CDeviceMonitor peut etre déclaré sur la pile en variable locale puisque tout est confiné dans DoModal(), ça evite les fuites mémoires par un delete oublié ...

sinon c'est bien comme ça qu'il faut faire :king:

@+