[MFC] CList et CComboBox

Bonjour,

j'ai une liste d'objets créée avec la classe CList et je l'affiche dans une combo box.
Je souhaite aussi pouvoir supprimer des éléments de cette liste en les sélectionnant dans la combo box mais ca ne marche pas.
Voilà mon bout de code pour supprimer un élément:

Code:

---

1 2 3 4 5 6 7 8

void CListeMessagesDlg::OnDeleteMsg() { int iIndex = m_pListMsg->GetCurSel(); CMessage* pMessageSel = (CMessage*) m_pListMsg->GetItemData(iIndex); ((CMainFrame*) m_pMainFrame)->SupprimerMessage(pMessageSel); AfficherListeMessages(); }

---

code de la méthode SupprimerMessage(pMessageSel):

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

void CMainFrame::SupprimerMessage (CMessage* pMessageASupprimer) { if( pMessageASupprimer == NULL) { return; } else { POSITION pos=m_pListeMessages->Find(pMessageASupprimer); m_pListeMessages->RemoveAt(pos); } }

---

et enfin le code de la méthode AfficherListeMessages():

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

void CListeMessagesDlg::AfficherListeMessages() { m_pListMsg = (CComboBox*) GetDlgItem(IDC_COMBO_LIST_MSG); m_pListeMessages = ((CMainFrame*) m_pMainFrame)->GetListeMessages(); if (m_pListeMessages == NULL) { return; } UINT uiNbMessages = ((CMainFrame*) m_pMainFrame)->GetNbMessages(); POSITION pos = m_pListeMessages->GetHeadPosition(); for (UINT ui = 0; ui < uiNbMessages/*(int) uiNbMessages*/ ; ui++) { m_pMessage = (CMessage*) m_pListeMessages->GetNext(pos); CString sNomMessage = m_pMessage->GetNomMessage(); m_pListMsg->AddString(sNomMessage); } }

---

Voilà le problème vient du fait que quelque soit l'élément sélectionné le pointeur pMessageSel de la méthode OnDeleteMsg() est toujours à NULL et donc ses champs sont vides...

Si vous voyez d'ou cela peut venir merci de me l'indiquer.

Bien sur la méthode AfficherListeMessages() est appelée dans le InitDialog() de la dialog box donc les variables m_pListMsg et m_pListeMessages sont définies.

tu fais :

CMessage* pMessageSel = (CMessage*) m_pListMsg->GetItemData(iIndex);


à quel moment tu fais un SetItemData ?

je n'en fais pas...
je vois pas pourquoi je dois en faire un surtout

bah si tu fais un GetItemData, c'est pour recuperer ce que tu as mis dans SetItemData. Donc si tu fais jamais de SetItemData, il est normal que tu recupere NULL lorsque tu fais GetItemData.

Comment le COMBO peut savoir ce que tu veux recuperer dans

Code:

---

1 2 3 4 5

int iIndex = m_pListMsg->GetCurSel(); CMessage* pMessageSel = (CMessage*) m_pListMsg->GetItemData(iIndex); ((CMainFrame*) m_pMainFrame)->SupprimerMessage(pMessageSel); AfficherListeMessages();

---

en sachant que tu ne fais que

Code:

---

1 2	m_pListMsg->AddString(sNomMessage);

---

tu demandes au COMBO d'ajouter une chaine de caratere et il est censé deviner à quel CMessage associé ???


un COMBOBOX est une armoire à tiroir. Le AddString ne fait que mettre une etiquette sur un tiroir, si tu mets rien a l'interieur et que tu lui demandes de sortir ce qu'il y a dans le tiroir 1, il va te dire qu'il y a rien

CQFD

c'est bon ca marche merci
j'ai rajouté un SetItemData apres le AddString().
Encore merci

En fait non ca marche tjs pas.
Il vaut une valeur maintenant, mais forcément c toujours la valeur de mon dernier élément entré...
il ne se remet jamais a jour

voilà en fait ce que j'ai modifié:

Code:

---

1 2 3 4 5 6 7 8

for (UINT ui = 0; ui < uiNbMessages ; ui++) { m_pMessage = (CMessage*) m_pListeMessages->GetNext(pos); m_pListMsg->SetItemDataPtr(ui,(CMessage*) m_pMessage); CString sNomMessage = m_pMessage->GetNomMessage(); m_pListMsg->AddString(sNomMessage); }

---

dans la méthode AfficherListeMessages()

et dans la méthode OnDeleteMsg():

Code:

---

1 2 3 4 5

int iIndex = m_pListMsg->GetCurSel(); m_pMessage = (CMessage*) m_pListMsg->GetItemDataPtr(iIndex); ((CMainFrame*) m_pMainFrame)->SupprimerMessage(m_pMessage); AfficherListeMessages();

---

Mais là encore le pointeur reste a NULL sauf si l'élément sélectionné correspond au dernier élément ajouté a la liste...

non en fait il reste a NULL quelque que soit l'élément sélectionné.
désolé

essai ca

Code:

---

1 2 3 4 5 6 7 8 9 10 11

for (UINT ui = 0; ui < uiNbMessages ; ui++) { m_pMessage = (CMessage*) m_pListeMessages->GetNext(pos); m_pListMsg->SetItemDataPtr(ui,(CMessage*) m_pMessage); CString sNomMessage = m_pMessage->GetNomMessage(); int pos = m_pListMsg->AddString(sNomMessage);   m_pListMsg->SetItemData(pos, (DWORD)m_pMessage);   }

---

C'est vrai ça la drogue c'est mal