转载请说明出处,谢谢~~:http://blog.csdn.net/zhuhongshu/article/details/42264947
这个bug我在仿酷狗开发日志里提到过,不过后来发现修复的不够好,后来重新修改了代码,并记录到博客。
问题描述:
在仿酷狗程序中,动态添加音乐项目到播放列表里,这是必须的功能,动态添加的功能已经做好。但是我发现,当一个分组本来在收缩状态下,这是给它动态添加音乐项目就会发生很搞笑的问题:分组是收缩状态,而动态添加的项目却显示了出来,如图:
可以看到,一共有20个音乐项目,原本的10个已经在收缩状态下所以没有显示,而新添加的项目却直接显示了出来。这里应该有两种处理逻辑:
1)动态添加子项目时,发现分组处于收缩状态后,新添加的项目自动隐藏而不显示出来
2)动态添加子项目时,发现分组处于展开状态后,新添加的项目自动展开
修复这个bug只要,在CTreeNodeUI添加元素时进行判断,控制新加元素的显示隐藏属性就可以了。修改CTreeNodeUI的AddAt和AddChildNode函数。修改后的函数如下:
bool CTreeNodeUI::AddAt( CControlUI* pControl, int iIndex )
{
if (!pControl)
return false;
//省略多余代码····
else
{
//parent TreeNode not bind TreeView just insert to parent TreeNode
bRet = mTreeNodes.InsertAt(iIndex, pControl);
}
if(bRet) //add by redrain 2014.11.7
{
pControl->SetVisible(GetFolderButton()->IsSelected());
}
return bRet;
}
bool CTreeNodeUI::AddChildNode( CTreeNodeUI* _pTreeNodeUI )
{
if (!_pTreeNodeUI)
return false;
if (_tcsicmp(_pTreeNodeUI->GetClass(), _T("TreeNodeUI")) != 0)
return false;
_pTreeNodeUI = CalLocation(_pTreeNodeUI);
bool nRet = true;
if(pTreeView){
CTreeNodeUI* pNode = static_cast<CTreeNodeUI*>(mTreeNodes.GetAt(mTreeNodes.GetSize()-1));
if(!pNode || !pNode->GetLastNode())
nRet = pTreeView->AddAt(_pTreeNodeUI,GetTreeIndex()+1) >= 0;
else nRet = pTreeView->AddAt(_pTreeNodeUI,pNode->GetLastNode()->GetTreeIndex()+1) >= 0;
}
if(nRet)
{
_pTreeNodeUI->SetVisible(GetFolderButton()->IsSelected()); //add by redrain 2014.11.7
mTreeNodes.Add(_pTreeNodeUI);
}
return nRet;
}
总结:
bug的修复代码已经提交到我自己的Duilib库。
我的Duilib库代码下载地址:点击打开链接
Redrain 2014.12.30