仿酷狗音乐播放器开发日志十一——CTreeNodeUI的bug修复

26 篇文章 3 订阅
21 篇文章 10 订阅
 由于做播放列表控件,我的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。
2014.7.19  13:42  Redrain 
  • 1
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 4
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 4
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值