[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