Discussion :

[elodie35]

Bonjour,

J'ai un problème, je développe un logiciel en c++ avec visual c++. J'ai créé un combo box d'identifiant IDC_COMBO1 et je veux y mettre des valeurs.
J'ai ce code :

CComboBox* pCbo = (CComboBox*)GetDlgItem(IDC_COMBO1);
ASSERT(pCbo);
if (pCbo){
pCbo->AddString("toto");
pCbo->AddString("titi");
pCbo->SetCurSel(0);
}

Et ca plante au niveau du GetDlgItem, il aime pas le ASSERT(::IsWindow(m_hWnd)) :

CWnd* CWnd::GetDlgItem(int nID) const
{
ASSERT(::IsWindow(m_hWnd));
if (m_pCtrlCont == NULL)
return CWnd::FromHandle(::GetDlgItem(m_hWnd, nID));
else
return m_pCtrlCont->GetDlgItem(nID);
}

Pourtant, en appelant la fonction GetDlgItem avec la combobox en paramètre, ca devrait marcher non?

[hiko-seijuro]

est ce que tu es dans une méthode de la classe représentant la boite de dialogue contenant le combobox ?

[elodie35]

oui, la classe représentant la boite de dialogue contenant le combobox s'appelle COptionsBDD.
Ce bout de code se trouve dans COptionsBDD.cpp, dans le constructeur.

/////////////////////////////////////////////////////////////////////////////
// COptionsBDD dialog

COptionsBDD::COptionsBDD(CIfe2central* pIfe2C, CWnd* pParent /*= NULL*/)
: CDialog(COptionsBDD::IDD, pParent), postgreActif(0)
{

m_pIfe2C = pIfe2C;
//{{AFX_DATA_INIT(COptionsBDD)
m_driver = _T("");
m_nomBDD = _T("");
m_serveur = _T("");
m_port = _T("");
m_id = _T("");
m_mdp = _T("");
//}}AFX_DATA_INIT
SetParam();
CComboBox* pCbo = (CComboBox*)GetDlgItem(IDC_COMBO1);
ASSERT(pCbo);
if (pCbo){
pCbo->AddString("toto");
pCbo->AddString("titi");
pCbo->SetCurSel(0);
}
}

[olive_le_malin]

ben justement, moi je le mettrais apres l'appel du constructeur

[hiko-seijuro]

place ce genre d'informations dans OnIniutDialog car les composants ne sont pas initialiser dans le constructeur !

[olive_le_malin]

vi vi

[elodie35]

humm je l'ai bien mis dans le OnInitDialog comme ceci :

BOOL COptionsBDD::OnInitDialog()
{
CDialog::OnInitDialog();
CComboBox* pCbo = (CComboBox*)GetDlgItem(IDC_COMBO1);
ASSERT(pCbo);
if (pCbo){
pCbo->AddString("toto");
pCbo->AddString("titi");
pCbo->SetCurSel(0);
}
return TRUE;
}

Mais ca plante au niveau de l'ASSERT(pCbo)...
Je suis desesperée

[ricky78]

Bonjour

je mz trompe peut etre mais je suis pas sur que tu peux utiliser le ASSERT sans comparaison

essaye plutot

Code :

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

ASSERT(pCbo != NULL);

Cordialement

[elodie35]

Merci pour ta réponse mais hélas, ca ne change rien, cela me donne cette erreur :

Exception non gérée à 0x0040bdb8 dans ife2.exeoint d'arrêt utilisateur.

[farscape]

salut,
si ça plante dans OnInitDialog c'est que ton identifiant IDC_COMBO1 n'est pas valide (n'existe pas dans la fenetre).
ou alors tu as mis des proprietés incorrectes dans ta combobox genre owner draw.

[elodie35]

Ben tout est ok pourtant, voici les propriétés de l'objet comboBox :

Apparence :
...

Comportement:
Accept files : False
Auto : False
Data :
Disabled : False
Help ID : False
Owner draw : No
Sort : True
Visible : True

Divers :
Group : False
ID :IDC_COMBO1
Tabstop : True

Je ne comprends pas comment l'identifiant ne serait pas valide
Désolée, je ne vois vraiment pas ce qui ne va pas.

[Gabrielly]

COptionsBDD::COptionsBDD(CIfe2central* pIfe2C, CWnd* pParent /*= NULL*/)
: CDialog(COptionsBDD::IDD, pParent), postgreActif(0)
{

m_pIfe2C = pIfe2C;
//{{AFX_DATA_INIT(COptionsBDD)
m_driver = _T("");
m_nomBDD = _T("");
m_serveur = _T("");
m_port = _T("");
m_id = _T("");
m_mdp = _T("");
//}}AFX_DATA_INIT
SetParam();
CComboBox* pCbo = (CComboBox*)GetDlgItem(IDC_COMBO1);
ASSERT(pCbo);
if (pCbo){
pCbo->AddString("toto");
pCbo->AddString("titi");
pCbo->SetCurSel(0);
}
}

Salut ton erreur d'assertion s'explique!!!
L'appel de ton combobox ne doit pas se faire dans le constructeur mais dans la méthode candidate "OnInitDialog() préparée pour cela.

Le constructeur de la boite de dialogue construit simplement l'objet C++, mais la méthode CDialog::OnInitDialog() prépare le même objet C++ sur le plan graphique (GUI).

Et donc tant que le m_hWnd est null dans

Code :

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

1
2
 
ASSERT(::IsWindow(m_hWnd));

cela veut dire que sur le plan GUI tes objets ne sont pas encore près. Il faut donc les initialiser.

En d'autres termes il y a deux étapes avant qu'un objet soit opérationnel

d'abord on prépare l'objet C++ en appelant son constructeur
ensuite on prépare l'entité concrète représentée par cet objet C++ en appelant ses méthodes du genre Create(), OnInitDialog(), OnInitialUpdate(), Initialize(), etc.

Si tu remarques bien la plupart des objets MFC et surtout des objets fenêtres, disposent d'une méthode Create(). Cela veut dire que sur le plan GUI tu ne peux utiliser aucun objet tant que Create() n'est pas appelé mais les constructeurs sur ces derniers sont déjà appelés.

[Gabrielly]

bon je vois que tu l'as placé dans OnInitDialog().
Essaie de supprimer ton combo et de le replacer sur ta dialogue.
Et assigne lui un autre IDC

[elodie35]

ca change rien

[Gabrielly]

Montre nous ton fichier resource.h

[elodie35]

//{{NO_DEPENDENCIES}}
// Microsoft Visual C++ generated include file.
// Used by ife2.rc
//
#define IDD_ABOUTBOX 100
#define IDR_NOUVEAU_FILTRE_TMPL 103
#define IDR_MAINFRAME 128
#define IDR_IFE2TYPE 129
#define IDD_NouveauFiltre 130
#define IDD_BDD 131
#define IDC_EDIT1 1000
#define IDC_EDIT2 1001
#define IDC_EDIT3 1002
#define IDC_EDIT4 1003
#define IDC_EDIT5 1004
#define IDC_EDIT6 1005
#define IDC_EDIT7 1006
#define IDC_EDIT8 1007
#define IDC_EDIT9 1008
#define IDC_RADIO1 1009
#define IDC_RADIO2 1010
#define IDC_RADIO3 1011
#define IDC_COMBO1 1013
//#define combo1 1013
#define ID_FILTRE_SAUV 1014
#define ID_IMPORTER 32772
#define ID_LATEX 32773
#define ID_BDD 32774
#define ID_ERASE 32775
#define ID_OPTIONS_BDD 32775
#define ID_WORD 32776
#define ID_ML 32777
#define ID_ERASED 32778
#define ID_COM 32779
#define ID_GXML 32780
#define ID_CSV 32781
#define ID_NET 32782
#define ID_VIDER 32784
#define ID_FILTRE_NOUVEAUFILTRE 32790
#define ID_FILTRE_OUVRIRFILTRE 32791
#define ID_FILTRE_ANNULERFILTRE 32795

// Next default values for new objects
//
#ifdef APSTUDIO_INVOKED
#ifndef APSTUDIO_READONLY_SYMBOLS
#define _APS_3D_CONTROLS 1
#define _APS_NEXT_RESOURCE_VALUE 133
#define _APS_NEXT_COMMAND_VALUE 32777
#define _APS_NEXT_CONTROL_VALUE 1016
#define _APS_NEXT_SYMED_VALUE 101
#endif
#endif

J'ai renommé le combo IDC_COMBO1, ce qui explique la ligne en commentaires.

[farscape]

salut ,
de toute facon il n'y a pas 36 solutions:
si GetDlgItem echoue sur un identifiant c'est que le controle correspondant n'existe pas sur la fenetre.
si tu ne mets pas ces lignes , la fenetre s'affiche avec la combo à l'ecran ?
verifie sur la ressource l'identifiant utilisé .

[elodie35]

Salut,

Alors oui, la fenetre s'affiche avec le combo à l'écran si je mets les lignes en commentaires.
Sinon je suis allée voir le fichier ressource :

IDD_BDD DIALOGEX 0, 0, 194, 281
STYLE DS_SETFONT | DS_MODALFRAME | WS_POPUP | WS_CAPTION | WS_SYSMENU
CAPTION "Options de la base de données"
FONT 8, "MS Sans Serif", 0, 0, 0x0
BEGIN
COMBOBOX IDC_COMBO1,86,10,96,30,CBS_DROPDOWN | CBS_SORT |
WS_VSCROLL | WS_TABSTOP
LTEXT "Driver :",IDC_STATIC,13,8,57,14,SS_CENTERIMAGE
LTEXT "Nom de la base :",IDC_STATIC,13,37,63,14,SS_CENTERIMAGE
EDITTEXT IDC_EDIT2,86,36,96,14,ES_AUTOHSCROLL
LTEXT "Serveur : ",IDC_STATIC,13,66,57,14,SS_CENTERIMAGE
EDITTEXT IDC_EDIT3,86,64,96,14,ES_AUTOHSCROLL
LTEXT "Port : ",IDC_STATIC,13,95,25,14,SS_CENTERIMAGE
EDITTEXT IDC_EDIT4,86,95,96,14,ES_AUTOHSCROLL
LTEXT "Identifiant : ",IDC_STATIC,13,124,57,14,SS_CENTERIMAGE
EDITTEXT IDC_EDIT5,86,124,96,14,ES_AUTOHSCROLL
LTEXT "Mot de passe : ",IDC_STATIC,13,153,54,14,SS_CENTERIMAGE
EDITTEXT IDC_EDIT6,86,153,96,14,ES_PASSWORD | ES_AUTOHSCROLL
LTEXT "Système de gestion de base de données :",IDC_STATIC,19,
184,133,8
CONTROL "PostgreSql",IDC_RADIO1,"Button",BS_AUTORADIOBUTTON |
WS_GROUP,35,207,50,10
CONTROL "MySql3",IDC_RADIO2,"Button",BS_AUTORADIOBUTTON,113,206,
39,10
CONTROL "MySql5",IDC_RADIO3,"Button",BS_AUTORADIOBUTTON,69,230,
39,10
DEFPUSHBUTTON "OK",IDOK,12,253,50,14
PUSHBUTTON "Cancel",IDCANCEL,132,253,50,14
END

Ma comboBox se situe bien dans la boite de dialogue IDD_BDD.

Pour info :
Avec ce code :

BOOL COptionsBDD::OnInitDialog()
{
CDialog::OnInitDialog();
CComboBox* pCbo = (CComboBox*)GetDlgItem(IDC_COMBO1);
ASSERT(pCbo);
if (pCbo){
pCbo->AddString("toto");
pCbo->AddString("titi");
pCbo->SetCurSel(0);
}
return TRUE;

l'erreur se situe ici, au niveau de l'assert(FALSE) :

HWND CDataExchange:repareCtrl(int nIDC)
{
ASSERT(nIDC != 0);
ASSERT(nIDC != -1); // not allowed
HWND hWndCtrl;
COleControlSite* pSite = NULL;
m_pDlgWnd->GetDlgItem(nIDC, &hWndCtrl);
if (hWndCtrl == NULL)
{
// Could be a windowless OCX
pSite = m_pDlgWnd->GetOleControlSite(nIDC);
if (pSite == NULL)
{
TRACE(traceAppMsg, 0, "Error: no data exchange control with ID 0x%04X.\n", nIDC);
ASSERT(FALSE); ////////////////////////////////////////////////////////////////////////////////////////////////////////// ICI
AfxThrowNotSupportedException();
}
}
m_idLastControl = nIDC;
m_bEditLastControl = FALSE; // not an edit item by default

return hWndCtrl;
}

[farscape]

reprenons :
GetDlgItem(IDC_COMBO1) renvoie NULL c'est sur ?
GetDlgItem(1013) renvoie quoi ?
tu peux le verifier en debug ?

[elodie35]

Alors non, GetDlgItem(IDC_COMBO1) ne renvoie pas NULL mais je ne sais pas comment faire pour voir ce qu'il renvoie. J'ai juste fait :

CComboBox* pCbo = (CComboBox*)GetDlgItem(IDC_COMBO1);
if (pCbo != NULL) afficher un message

Et ca fait le meme type d'erreur en mettant 1013 en paramètre de GetDlgItem.