Discussion :

[jonzuzu]

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 :

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

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 :

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

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 :

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
 
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.

[la drogue c'est mal]

tu fais :

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


à quel moment tu fais un SetItemData ?

[jonzuzu]

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

[la drogue c'est mal]

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 :

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

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 :

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

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

[jonzuzu]

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

[jonzuzu]

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

[jonzuzu]

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

Code :

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

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 :

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

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...

[jonzuzu]

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

[la drogue c'est mal]

essai ca

Code :

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

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);
 
   }

[Jasmine]

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