仿酷狗音乐播放器开发日志十一——CTreeNodeUI的bug修复
由于做播放列表控件，我的CMusicLength控件继承了CTreeVieWUI控件，在向分组控件中添加播放项目时，发现代码无法正常工作，调用CTreeNodeUI控件的Add方法后无反应，导致我的播放器无法动态增加项目。
在对duilib源码调试后发现，CTreeNodeUI类的AddAt方法存在bug，AddAt的原型为
long CTreeViewUI::AddAt( CTreeNodeUI* pControl, int iIndex )；
其中的iIndex参数代表希望插到的位置，在函数中做了多个判断来确保插入的正确性，但其中有一行代码存在bug，bug代码如下：
CTreeNodeUI* pParent = static_cast<CTreeNodeUI*>(GetItemAt(iIndex));
if(!pParent)
return -1;
函数用来在分组中插入子元素，插入时做判断，在指定的位置的是否是可以容纳元素的分组项目，但是分组项目的位置应该是将要被插入的位置的上一个位置。原函数直接检测在被插入位置是否是分组项目，理所当然会返回NULL，导致函数最终返回-1，而插入失败。
修复的方法很简单，把 CTreeNodeUI* pParent = static_cast<CTreeNodeUI*>(GetItemAt(iIndex));改为 CTreeNodeUI* pParent = static_cast<CTreeNodeUI*>(GetItemAt(iIndex -1 ));
这样可以正确检索到分组项目，程序可以正常运行了。这个此次软件项目中发现duilib的第三个bug。mark。
函数用来在分组中插入子元素，插入时做判断，在指定的位置的是否是可以容纳元素的分组项目，但是分组项目的位置应该是将要被插入的位置的上一个位置。原函数直接检测在被插入位置是否是分组项目，理所当然会返回NULL，导致函数最终返回-1，而插入失败。
修复的方法很简单，把 CTreeNodeUI* pParent = static_cast<CTreeNodeUI*>(GetItemAt(iIndex));改为 CTreeNodeUI* pParent = static_cast<CTreeNodeUI*>(GetItemAt(iIndex -1 ));
这样可以正确检索到分组项目，程序可以正常运行了。这个此次软件项目中发现duilib的第三个bug。mark。
2014.7.19 13:42 Redrain