TreeView的OnDeletion中释放结点的Data程序运行出错

[ooolinux]

以下的代码，PData是省的数据，CData是市的数据，在OnDeletion中释放结点的Data，程序关闭时运行出错了。
改成在FormDestroy中遍历TreeView结点释放它的Data，几乎同样的代码，就没有问题。
为啥在TreeView1Deletion中会出错呢？
//---------------------------------------------------------------------------
void __fastcall TForm1::TreeView1Deletion(TObject *Sender, TTreeNode *Node)
{/*
if(0==Node->Level&&Node->Data!=NULL)
{
PData *data=(PData *)(Node->Data);
delete data;
}
else if(1==Node->Level&&Node->Data!=NULL)
{
CData *data=(CData *)(Node->Data);
delete data;
} */
}
//---------------------------------------------------------------------------

void __fastcall TForm1::FormDestroy(TObject *Sender)
{
for(int i=0;i<TreeView1->Items->Count;++i)
{
TTreeNode *Node=TreeView1->Items->Item[i];
if(0==Node->Level&&Node->Data!=NULL)
{
PData *data=(PData *)(Node->Data);
delete data;
}
else if(1==Node->Level&&Node->Data!=NULL)
{
CData *data=(CData *)(Node->Data);
delete data;
}
}
}
//---------------------------------------------------------------------------

[铖邑]

检查一些Node的取值

[ooolinux]

引用 1 楼 SuperDay 的回复:

检查一些Node的取值

嗯

[ooolinux]

引用 3 楼 早打大打打核战争 的回复:

没仔细看，不过一个好的设计应该把每个Node->Data统一管理（如果Data是复杂数据结构的话），比如放在一个TList或者TObjectList中，这样Node->Data只需要存储List中的Index即可

问题是不同层次的结点Data的类型要不同。

[日立奔腾浪潮微软松下联想]

你现在怎么用放在List中也怎么用

[ooolinux]

引用 5 楼 早打大打打核战争 的回复:

你现在怎么用放在List中也怎么用

放在List中应该是可以。
帮我看看这贴：
https://bbs.csdn.net/topics/394550293

[日立奔腾浪潮微软松下联想]

没仔细看，不过一个好的设计应该把每个Node->Data统一管理（如果Data是复杂数据结构的话），比如放在一个TList或者TObjectList中，这样Node->Data只需要存储List中的Index即可

[ooolinux]

原因在这里： https://bbs.csdn.net/topics/394550293