修复duilib库UISlider控件的4个bug

转载级请注明原出处，谢谢~· 

        昨天封装好一个音频类，我在为dulib做音频播放demo时发现了一些问题，由CSliderUI控件导致的，进而发现了这个控件的好几样不足，他无法满

足我们做一个播放器的进度控件和音量控制控件，具体问题如下：

//=====================================================================================================

         2014.7.28 redrain修改，QQ：491646717，有疑问或者bug2请联系我

        修改的代码的DoEvent部分的逻辑借鉴和复制了网友 快樂每一天（群昵称 花落半歌，QQ：848861075 ）的代码  

        问题说明：我正在制作仿酷狗播放器，做到音乐播放的部分时用到CSliderUI控件，后台的音频类回去控制CSliderUI的行为  CSliderUI的行为与酷狗的很不一样，有几样缺陷：

        问题1：只能通过点击CSliderUI的某个位置才能触发valuechanged消息，无法通过滑动滑块去触发，这个bug最严重
        问题2：点击CSliderUI的某个位置，当鼠标弹起时滑块才改变位置，而其他软件都是鼠标按下时就改变了
        问题3：后台有代码一直调用SetValue函数改变滑块的位置时，会和鼠标土洞滑块冲突，表现在滑块会一直来回跳动

        问题4：滑块滑动过程中无法通知主窗体正在改变，这点用在音量改变时，通常我们是一边滑动一边就改变了音量，而不是滑动完成后再改， 为

此我们添加一个新的消息DUI_MSGTYPE_VALUECHANGED_MOVE，把这个消息的定义放到UIDefine.h文件中

        #define DUI_MSGTYPE_VALUECHANGED_MOVE      (_T("movevaluechanged"))

        同时出于效率考虑，要让CSliderUI发出这个消息，应该设置属性sendmove为真，默认为假


        我修改的代码可以通过搜索字符串“2014.7.28 redrain”，来查找，方便大家查看源码

        此次修改不会影响控件原有的属性，我个人水平有限，如果有任何问题，可以联系我

//=====================================================================================================

        问题的描述结束了，其中的大部分问题都是由于在DoEvent函数中对一些逻辑的判断的不足导致的。

问题1的解决：只能通过点击CSliderUI的某个位置才能触发valuechanged消息，无法通过滑动滑块去触发这是由于原来的UIEVENT_BUTTONUP

消息处理不当造成的，原代码为：

if( event.Type == UIEVENT_BUTTONUP )
{
	int nValue;
	if( (m_uButtonState & UISTATE_CAPTURED) != 0 ) {
		m_uButtonState &= ~UISTATE_CAPTURED;
	}
	if( m_bHorizontal ) {
		if( event.ptMouse.x >= m_rcItem.right - m_szThumb.cx / 2 ) nValue = m_nMax;
		else if( event.ptMouse.x <= m_rcItem.left + m_szThumb.cx / 2 ) nValue = m_nMin;
		else nValue = m_nMin + (m_nMax - m_nMin) * (event.ptMouse.x - m_rcItem.left - m_szThumb.cx / 2 ) / (m_rcItem.right - m_rcItem.left - m_szThumb.cx);
	}
	else {
		if( event.ptMouse.y >= m_rcItem.bottom - m_szThumb.cy / 2 ) nValue = m_nMin;
		else if( event.ptMouse.y <= m_rcItem.top + m_szThumb.cy / 2  ) nValue = m_nMax;
		else nValue = m_nMin + (m_nMax - m_nMin) * (m_rcItem.bottom - event.ptMouse.y - m_szThumb.cy / 2 ) / (m_rcItem.bottom - m_rcItem.top - m_szThumb.cy);
	}
	if(m_nValue !=nValue && nValue>=m_nMin && nValue<=m_nMax)
	{
		m_nValue =nValue;
		m_pManager->SendNotify(this, DUI_MSGTYPE_VALUECHANGED);
		Invalidate();
	}
	return;
}

        在最后的判断出可以看到，只有当m_nValue !=nValue时才会发送DUI_MSGTYPE_VALUECHANGED消息，而我们滑动滑块时，通过上面的代码

不难分析出，

        这两个值一直是相等的，所以导致他没有发出消息，所以我们把这个判断注释掉就行了

问题2的解决：

        点击CSliderUI的某个位置，当鼠标弹起时滑块才改变位置，而其他软件都是鼠标按下时就改变了，这是由于原来的UIEVENT_BUTTONDOWN 消

息的处理不全面造成的，原代码为：

 

if( event.Type == UIEVENT_BUTTONDOWN || event.Type == UIEVENT_DBLCLICK )
 		{
 			if( IsEnabled() ) {
 				RECT rcThumb = GetThumbRect();
 				if( ::PtInRect(&rcThumb, event.ptMouse) ) {
 					m_uButtonState |= UISTATE_CAPTURED;
 				}
 			}
 			return;
 		}

        可以看到原代码没有做对于控件的外观的任何修改，我们把代码修改如下：

if( event.Type == UIEVENT_BUTTONDOWN || event.Type == UIEVENT_DBLCLICK )
{
	if( IsEnabled() ) {//2014.7.28 redrain 注释掉原来的代码，加上这些代码后可以让Slider不是在鼠标弹起时才改变滑块的位置
		m_uButtonState |= UISTATE_CAPTURED;

		int nValue;

		if( m_bHorizontal ) {
			if( event.ptMouse.x >= m_rcItem.right - m_szThumb.cx / 2 ) nValue = m_nMax;
			else if( event.ptMouse.x <= m_rcItem.left + m_szThumb.cx / 2 ) nValue = m_nMin;
			else nValue = m_nMin + (m_nMax - m_nMin) * (event.ptMouse.x - m_rcItem.left - m_szThumb.cx / 2 ) / (m_rcItem.right - m_rcItem.left - m_szThumb.cx);
		}
		else {
			if( event.ptMouse.y >= m_rcItem.bottom - m_szThumb.cy / 2 ) nValue = m_nMin;
			else if( event.ptMouse.y <= m_rcItem.top + m_szThumb.cy / 2  ) nValue = m_nMax;
			else nValue = m_nMin + (m_nMax - m_nMin) * (m_rcItem.bottom - event.ptMouse.y - m_szThumb.cy / 2 ) / (m_rcItem.bottom - m_rcItem.top - m_szThumb.cy);
		}
		if(m_nValue !=nValue && nValue>=m_nMin && nValue<=m_nMax)
		{
			m_nValue =nValue;
			Invalidate();
		}
	}
		return;
}

问题3的解决：

        后台有代码一直调用SetValue函数改变滑块的位置时，会和鼠标土洞滑块冲突，表现在滑块会一直来回跳动，这是因为，当我们拖动滑块时会动

态的修改Slider的m_nValue值，并且会刷新控件，而通过读PaintStatusImage函数可知，控件正式通过这个m_nValue变量来决定滑块的绘制位置。而

我在后台让音乐播放类去根据音乐的进度调用SetValue函数，这个函数理所当然的修改了m_nValue值，这就导致了冲突，这个函数是父类的，所以我

们要重写这个函数。当鼠标正在滑动式不让SetValue去改变控件的滑块的位置。

        增加SetValue函数然后重写他，修改的代码为：

void CSliderUI::SetValue(int nValue) 
{
	if( (m_uButtonState & UISTATE_CAPTURED) != 0 ) 
		return;
	CProgressUI::SetValue(nValue);
}

问题4的解决：

        滑块滑动过程中无法通知主窗体正在改变，这点用在音量改变时，通常我们是一边滑动一边就改变了音量，而不是滑动完成后再改变， 为此我们

添加一个新的消息DUI_MSGTYPE_VALUECHANGED_MOVE，把这个消息的定义放到UIDefine.h文件中，这个代码只要在DoEvent的UIEVENT_MOUS

EMOVE消息处理中把DUI_MSGTYPE_VALUECHANGED_MOVE事件传送出去就好了，后来听从网友“不乖打Pp.”的建议，增加了一个属性"sendmove"

，当属行为真时才发送消息出去。

        修改代码为：

if( event.Type == UIEVENT_MOUSEMOVE )
{
	if( (m_uButtonState & UISTATE_CAPTURED) != 0 ) {
		if( m_bHorizontal ) {
			if( event.ptMouse.x >= m_rcItem.right - m_szThumb.cx / 2 ) m_nValue = m_nMax;
			else if( event.ptMouse.x <= m_rcItem.left + m_szThumb.cx / 2 ) m_nValue = m_nMin;
			else m_nValue = m_nMin + (m_nMax - m_nMin) * (event.ptMouse.x - m_rcItem.left - m_szThumb.cx / 2 ) / (m_rcItem.right - m_rcItem.left - m_szThumb.cx);
		}
		else {
			if( event.ptMouse.y >= m_rcItem.bottom - m_szThumb.cy / 2 ) m_nValue = m_nMin;
			else if( event.ptMouse.y <= m_rcItem.top + m_szThumb.cy / 2  ) m_nValue = m_nMax;
			else m_nValue = m_nMin + (m_nMax - m_nMin) * (m_rcItem.bottom - event.ptMouse.y - m_szThumb.cy / 2 ) / (m_rcItem.bottom - m_rcItem.top - m_szThumb.cy);
		}
		if (m_bSendMove)
			m_pManager->SendNotify(this, DUI_MSGTYPE_VALUECHANGED_MOVE);
		Invalidate();
	}


	// Generate the appropriate mouse messages
	POINT pt = event.ptMouse;
	RECT rcThumb = GetThumbRect();
	if( IsEnabled() && ::PtInRect(&rcThumb, event.ptMouse) ) {

		m_uButtonState |= UISTATE_HOT;
		Invalidate();
	}else
	{
		m_uButtonState &= ~UISTATE_HOT;
		Invalidate();
	}
	return;
}

        至此我们就修改了四处地方，还有其他修改的地方大家可以自己看源文件，此次修改不会影响控件原有的属性，我个人水平有限，如果有任何问

题，可以联系我。

        patch的下载地址：http://download.csdn.net/detail/zhuhongshu/7686617

        2014.7.28  13:14  Redrain