转载请说明原出处,谢谢

今天本来打算把仿酷狗播放列表的子控件拖动插入功能做一下,但是仔细使用播放列表控件时发现了几个逻辑错误,由于我的播放

列表控件是基于CTreeViewUI和CTreeNodeUI做得,所以产生这几个bug的原因还在于他们两个,在《仿酷狗音乐播放器开发日志十一

——CTreeNodeUI的bug修复》中已经修复过一个动态添加控件的相关bug,这属于第二次修复了。关于第一次bug的修复,后来

Duilib扩展群的 joe 又进行过比我更全面的修复,我现在使用的CTreeViewUI就是joe修复过的,文章结尾会把第二次修复后的源文件免

费提供给大家。

我先来描述一下这个bug。

        第一个BUG:

含有子节点的CTreeNodeUI节点的选中和未选中状态是相反的,这个在我给播放列表添加音乐项目后可以看到,如图:

这一点让人觉得很不舒服,看了一下源码,负责展开与收缩的函数是CTreeViewUI的OnFolderChanged函数和OnDBClickItem,

CTreeNodeUI在收缩按钮被单击或者整体被双击时就会去分别调用这两个函数,看了这两个函数的源码,发现了一些逻辑错误。首

先看OnFolderChanged函数的源码:

bool CTreeViewUI::OnFolderChanged( TNotifyUI* pTNotifyUI,LPARAM lParam,WPARAM wParam )
{
	if(pTNotifyUI->sType == DUI_MSGTYPE_SELECTCHANGED)
	{
		CCheckBoxUI* pFolder = (CCheckBoxUI*)pTNotifyUI->pSender;
		CTreeNodeUI* pItem = (CTreeNodeUI*)pFolder->GetParent()->GetParent();
		pItem->SetVisibleTag(!pFolder->GetCheck());
		SetItemExpand(!pFolder->GetCheck(),pItem);
		return true;
	}
	return false;
}

问题出在pItem->SetVisibleTag(!pFolder->GetCheck());和SetItemExpand(!pFolder->GetCheck(),pItem);上,收缩按钮默认状态下

是未选中状态,当我点击了收缩按钮后,CCheckedUI控件会自动设置自己为选中状态,这时pFolder->GetCheck()会返回真,而

OnFolderChanged函数却取反,得到了假,然后再调用SetVisibleTag和SetItemExpand函数,恰好得到了相反结果,所以在这里把

取反去掉就可以了。

       第二个BUG:

接着看看OnDBClickItem函数的源码:

bool CTreeViewUI::OnDBClickItem( TNotifyUI* pTNotifyUI,LPARAM lParam,WPARAM wParam )
{
	if(pTNotifyUI->sType == DUI_MSGTYPE_ITEMDBCLICK)
	{
		CTreeNodeUI* pItem		= static_cast<CTreeNodeUI*>(pTNotifyUI->pSender);
		CCheckBoxUI* pFolder	= pItem->GetFolderButton();
		pFolder->Selected(!pFolder->IsSelected());
		pItem->SetVisibleTag(!pFolder->GetCheck());
		SetItemExpand(!pFolder->GetCheck(),pItem);
		return true;
	}
	return false;
}

这个的bug同样也是那两句代码,在双击了包含子节点的了CTreeViewUI控件后进入到此函数内,第一步先获取收缩按钮的句柄

并且将他的选中状态取反,pFolder->Selected(!pFolder->IsSelected());,这句代码没问题。接下来收缩按钮的状态已经为正确的状态

了,此时调用pFolder->GetCheck()后得到的就是正确值,不应该再取反了。所以后面的两句代码又把逻辑搞反了,应该把取反去

掉。

        第三个BUG:

在仿酷狗程序中,动态添加音乐项目到播放列表里,这是必须的功能,动态添加的功能已经做好。但是我发现,当一个分组本

来在收缩状态下,这是给它动态添加音乐项目就会发生很搞笑的问题:分组没有自动展开,而动态添加的项目却显示了出来,如

图:

可以看到,一共有20个音乐项目,原本的10个已经在收缩状态下所以没有显示,而新添加的项目却直接显示了出来。这里应该

有两种处理逻辑:

         1)动态添加子项目时,发现分组处于收缩状态后,新添加的项目自动隐藏而不显示出来

         2)动态添加子项目时,发现分组处于收缩状态后,把分组先展开,然后再添加

为此我先测试了酷狗播放器的行为,发现给它添加项目时,它使用了第二种逻辑,那我也使用了同样的逻辑。修复这个bug只要

在CTreeNodeUI添加元素时进行判断,发现自己出于收缩状态,就通知CTreeViewUI,让自己先展开。修改CTreeNodeUI的AddAt和

AddChildNode函数,在适当位置添加判断即可,需要添加的代码为:

if (!GetFolderButton()->IsSelected())
{
	m_pManager->SendNotify(this, DUI_MSGTYPE_ITEMDBCLICK);
}

修改后完整代码为:

bool CTreeNodeUI::AddChildNode( CTreeNodeUI* _pTreeNodeUI )
{
	if (!_pTreeNodeUI)
		return false;

	if (_tcsicmp(_pTreeNodeUI->GetClass(), _T("TreeNodeUI")) != 0)
		return false;

	if (!GetFolderButton()->IsSelected())       //add by:Redrain   2014.8.8
	{
		m_pManager->SendNotify(this, DUI_MSGTYPE_ITEMDBCLICK);
	}

	_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)
		mTreeNodes.Add(_pTreeNodeUI);

	return nRet;
}
bool CTreeNodeUI::AddAt( CControlUI* pControl, int iIndex )
{
    if (!pControl)
        return false;

    if(_tcsicmp(pControl->GetClass(), _T("TreeNodeUI")) != 0)
        return false;

	if (!GetFolderButton()->IsSelected())    //add by:Redrain   2014.8.8
	{
		m_pManager->SendNotify(this, DUI_MSGTYPE_ITEMDBCLICK);
	}

    //filter invalidate index
    int iDestIndex = iIndex;
    if (iDestIndex < 0)
    {
        iDestIndex = 0;
    }
    else if (iDestIndex > GetCountChild())
    {
        iDestIndex = GetCountChild();
    }
//省略·····

        总结:

在开发仿酷狗播放器的过程中已经修复了好几个bug,估计以后还有会,duilib的作者不维护,只能靠我们这些使

用者自己修复了。此次修复只是我临时修改,如果有不妥的地方,请联系我,我会及时修复代码。在代码里搜

索“//add by:Redrain   2014.8.8”和“//edit by:Redrain   2014.8.8 ” 可以找到我修改的地方。此源码是第二次修复

bug,在这之前已经修复了无法动态添加节点的bug。

源码下载地址:点击打开链接

Redrain  2014.8.8  20:29    QQ:491646717

仿酷狗音乐播放器开发日志十九——CTreeNodeUI的bug修复二(附源码)的更多相关文章

  1. 仿酷狗音乐播放器开发日志二十七 用ole为窗体增加文件拖动功能(附源码)

    转载请说明原出处,谢谢~~ 中秋到了,出去玩了几天.今天把仿酷狗程序做了收尾,已经开发完成了,下一篇博客把完结的情况说一下.在这篇博客里说一下使用OLE为窗体增加文件拖拽的功能.使用播放器,我更喜欢直 ...

  2. 仿酷狗音乐播放器开发日志二十四 选项设置窗体的实现(附328行xml布局源码)

    转载请说明原出处,谢谢~~ 花了两天时间把仿酷狗的选项设置窗体做出来了,当然了只是做了外观.现在开学了,写代码的时间减少,所以整个仿酷狗的工程开发速度减慢了.今天把仿酷狗的选项设置窗体的布局代码分享出 ...

  3. 仿酷狗音乐播放器开发日志二十六 duilib在标题栏弹出菜单的方法

    转载请说明原出处,谢谢~~ 上篇日志说明了怎么让自定义控件响应右键消息.之后我给主窗体的标题栏增加右键响应,观察原酷狗后可以发现,在整个标题栏都是可以响应右键并弹出菜单的.应该的效果如下: 本以为像上 ...

  4. 仿酷狗音乐播放器开发日志二十五 duilib右键事件的不足的bug修复

    转载请说明原出处,谢谢~~ 虽然仿酷狗的各个菜单早就写好了,但是一直没有附加到程序里.今天把菜单和播放列表控件关联时发现了问题. 和播放列表相关的菜单有三个,分别是每个音乐项目控件相关的菜单.分组的菜 ...

  5. 仿酷狗音乐播放器开发日志二十三 修复Option控件显示状态不全的bug(附源码)

    转载请说明原出处,谢谢~~ 整个仿酷狗工程的开发将近尾声,现在还差选项设置窗体的部分,显然在设置窗体里用的最多的就是OptionUI控件,我在写好大致的布局后去测试效果,发现Option控件的显示效果 ...

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

    由于做播放列表控件,我的CMusicLength控件继承了CTreeVieWUI控件,在向分组控件中添加播放项目时,发现代码无法正常工作,调用CTreeNodeUI控件的Add方法后无反应,导致我的播 ...

  7. 仿酷狗音乐播放器开发日志三——修复CEditUI的bug2

    无意中发现了CEditUI控件的另一个bug,当我给播放器的搜索栏获取焦点时,这时再改变窗体大小,原本搜索栏应该对应着也改变大小,却发现CEditUI内嵌的edit控件没有跟着改变(如下图),跟着调试 ...

  8. Redrain仿酷狗音乐播放器开发完毕,发布测试程序

    转载请说明原出处,谢谢~~ 从暑假到现在中秋刚过,我用duilib开发仿酷狗播放器大概经历了50天.做仿酷狗的意图只是看原酷狗的界面比较漂亮,想做个完整一些的工程来练习一下duilib.今天把写好的程 ...

  9. 项目源码--Android类似酷狗音乐播放器

    下载源码 知识技能概要: 1.音乐文件的扫描与管理 2.音频流的解码 3. UI控件的综合使用 4.播放列表方式管理 5.随机播放方式 6.源码带详细的中文注释 ...... 详细介绍 1. 音乐文件 ...

随机推荐

  1. linux中find批量删除空文件夹

    空文件夹 列出用find 删除管道即可 find -type d -empty | xargs -n 1 rm -rf 注意最后不能rm -f,这样删不了目录,必须-r

  2. SqlServer基础之(触发器)

    概念:   触发器(trigger)是SQL server 提供给程序员和数据分析员来保证数据完整性的一种方法,它是与表事件相关的特殊的存储过程,它的执行不是由程序调用,也不是手工启动,而是由事件来触 ...

  3. sublime text3 less2css rem

    1.下载sublime text3 官网地址:https://www.baidu.com/link?url=2kr0ijQXVL1_6oXdPByYh7ecMl7OUAYVx5fyTNjMrYVdtq ...

  4. java高薪之路__007_反射

    参考地址: 1. http://www.cnblogs.com/rollenholt/archive/2011/09/02/2163758.html2. http://www.cnblogs.com/ ...

  5. FFT教你做乘法(FFT傅里叶变换)

    题目来源:https://biancheng.love/contest/41/problem/C/index FFT教你做乘法 题目描述 给定两个8进制正整数A和B(A和B均小于10000位),请利用 ...

  6. TSP问题

    之前写过一道类似的题目,Uva 1347. http://www.cnblogs.com/TreeDream/p/5981535.html 这个题目和TSP问题已经很接近了,只是描述的奇奇怪怪的,从最 ...

  7. 动态切换采用 CSplitterWnd 静态划分的视图布局(MFC)

    标题读起来有些拗口,具体是什么情况,我们来看: 一.问题的提出 一个采用MFC开发的软件,其窗体视图采用CSplitterWnd三分,效果如下图所示: 图1 软件的默认视图布局 该MFC开发的软件功能 ...

  8. 李洪强iOS学习交流群-iOS大神群

    iOS学习大神群-群号:483959373

  9. BZOJ 3925 ZJOI2015 地震后的幻想乡

    假设我们用了边权前i小的边使得图连通,那么对答案的贡献为i/m+1 又因为期望的线性性质,我们只需要求用了i条边就可以了 不妨设g(S)(i)表示用了i条边使得点集S连通的概率 设f(S)(i)表示用 ...

  10. Node.js权威指南 (6) - 在Node.js中操作文件系统

    6.1 同步方法与异步方法 / 856.2 对文件执行读写操作 / 86 6.2.1 文件的完整读写 / 86 6.2.2 从指定位置处开始读写文件 / 916.3 创建与读取目录 / 97 6.3. ...