Discussion :

[vanitom]

Bonjour,

Je suis en train de me tirer les cheveux sur un problème à la con avec un CTreeCtrl.

Alors, je suis en train de créer une interface se composant d'un arbre à gauche et d'une liste à droite. Il faut remplir la liste de droite avec certains éléments de l'arbre de gauche.
Le principe est qu'au moment d'un double clic sur un élément de l'arbre, cet élément passe dans la liste de droite.
Le principe est tout simple mais j'ai un gros problème technique.

Le problème apparait lors de la suppression de l'élément sur lequel je viens de double-cliquer. Ce n'est pas un crash mais juste une boite de dialogue qui rend inutilisable le soft.
Au bout d'un certain nombre de "DeleteItem" ( 2 ou 3 voir même des fois 1 seul ), j'ai le message suivant qui apparait

User breakpoint called from code at 0x7c911230

et je peux en avoir 3-4 de suite. Ce message vient de NTDLL!. Si je remonte dans la pile j'arrive au CTreeCtrl::eleteItem.

Voici mon code de création de l'élément :

Code :

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

1
2
3
4
5
6
7
8
9
 
	TVINSERTSTRUCT tvInsert = {0};
	tvInsert.hParent = TVI_ROOT; //hParent;
	tvInsert.hInsertAfter = TVI_SORT;
	tvInsert.item.mask = TVIF_PARAM | TVIF_TEXT;
	tvInsert.item.pszText = (LPTSTR)(LPCTSTR)sName;
	tvInsert.item.cchTextMax = strlen(tvInsert.item.pszText);
	tvInsert.item.lParam = nIndex;
	HTREEITEM hItem = m_treeUnused.InsertItem ( &tvInsert );

Voici mon code de suppression :

Code :

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

1
2
3
 
	HTREEITEM hItem = m_treeUnused.GetSelectedItem ( );
	m_treeUnused.DeleteItem ( hItem );

Personnellement ça fait un moment que je gallère dessus sans en voir le bout alors si quelqu'un à une idée, je suis preneur.

Merci d'avance

[farscape]

salut,
vérifies que hitem est valide (!=NULL) avant l'appel de deletitem...

[vanitom]

Voici le code exacte de la fonction.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
	HTREEITEM hItem = NULL;
	hItem = m_treeUnused.GetSelectedItem ( );
	if ( hItem == NULL )
		return;
 
 
	if ( m_treeUnused.ItemHasChildren ( hItem ) == TRUE )
		return;
 
 
	m_treeUnused.DeleteItem ( hItem );

[vanitom]

Je viens de faire un test.

J'ai remplassé le double clic par le simple clic et maintenant je n'ai plus aucun problème. A part que ce n'est plus le comportement que je veux.

[farscape]

ah double clic, tu dois avoir un probleme de reentrance dans le code ...

[vanitom]

Ba la en fait j'ai rien compris.

Qu'est ce que tu entends par "avoir un probleme de reentrance dans le code" ?

[farscape]

tu ne rentres pas deux fois dans la fonction qui fait le deleteitem ?
met un trace pour le vérifier:

Code :

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

1
2
 
TRACE("\nHitem %x",hItem);

[vanitom]

Je n'ai pas eu le temps de tester les traces mais j'ai géré l'expanding.

Et quand je fais le double clique, je passe dedans. Alors je me dis que ça pourrais venir de là.

[vanitom]

Je viens de faire le test et je n'ai pas de "reentrance" dans le code.
Mais en tout cas, il y a peut être deux indice.

1 - Quand je double clique je passe à chaque fois par le OnItemexpandingTreeUnused.
2 - Au moment du message d'erreur j'ai une trace que je n'avais pas vu

Voici ce que j'ai :

Hitem 20e950
Hitem 20e5f8
Hitem 20e660HEAP[Admin-DC_d.exe]: HEAP: Free Heap block 20e5f0 modified at 20e610 after it was freed

[Gabrielly]

Salut.

Hitem 20e660HEAP[Admin-DC_d.exe]: HEAP: Free Heap block 20e5f0 modified at 20e610 after it was freed

Effectivement lors que tu double clic sur ton item il y a deux actions qui se produisent.
Dans le gestionnaire interceptant le NM_DBLCLK, tu expédies ton ton item vers la listview et tu fais un delete dans ce même gestionnaire,
et entre temps il y a une autre paire de message TVN_ITEMEXPANDING, TVN_ITEMEXPANDED qui sont lancés simplement à cause du double click. Ce qui est un comportement standard du TreeCtrl.

Et donc ton item cible n'existe plus et le hItem est certainement invalide dans ton OnItemexpandingTreeUnused()

Ce que tu pourrais faire s'est simplement expédié ton item vers ta listview dans le gestionnaire du NM_DBLCLK et dans celui du TVN_ITEMEXPANDING tu fais le delete de ton item cible.

En d'autres termes, si tu prends en charge les TVN_ITEMEXPAND cherche à eviter à supprimer l'item cible dans ton NM_DBLCLK