关于自封装的消息循环函数的改进问题

[Fireway2008]

各位高手， 本人利用AfxBeginThread开辟了一个UI线程，并自写了一个消息循环函数，希望以后可以封装入DLL内。

上网搜索了不少例子，但都感觉讲的不够详细，在此把自己写的贴出来，当然这个函数存在一定的缺陷，感觉有点冗繁，希望大家指正。

改进后也方便其他人学习参考。
如下代码：


BOOL CTestDlg::DoLoop()
{
MSG msg;
BOOL bRet;
while(1)
{
if(PeekMessage(&msg, NULL, 0, 0, PM_NOREMOVE))// 从Windows消息队列中取出消息
{
bRet = GetMessage(&msg, NULL, 0, 0);
if(bRet != 0)
{
if(bRet == -1)//判断返回值是否有问题
{
AfxMessageBox(_T("Handle the error and possibly exit!"));
break;
}else{//以下分情况讨论获取到的消息对应什么操作
switch(msg.message)
{
case WM_CLOSE:
DestroyWindow();
break;
case WM_DESTROY:
delete this;
PostQuitMessage(0);
break;
default:
TranslateMessage(&msg);
DispatchMessage(&msg);
break;
}
}
}// end of inner if

if(!AfxGetApp()->PreTranslateMessage(&msg))// 如果无法预处理消息
{
::TranslateMessage(&msg);// 转换消息
::DispatchMessage(&msg); // 发送消息
}
}
AfxGetApp()->OnIdle(0);
AfxGetApp()->OnIdle(1);
}// end of while
return TRUE;
}

[Fireway2008]

[Quote=引用 71 楼 wangask00 的回复:]
我感觉你的思维很乱，我觉得你应该找本mfc的书系统的看一下，重点搞清楚windows的消息机制，你对windows的消息机制还没搞清楚

不过我也解释不清楚你的问题

你的PreTranslateMessage函数处理鼠标左键，跟按键消息是没错

后来又，只是无法响应鼠标消息
却可以相应 键盘消息……
[/Quote]

说的是阿，我这个方面的知识很是欠缺……

[paerxiushi]

回69楼:
我想实现的是：

1.在新UI线程中的模态对话框

or

2.在新UI线程中的非模态对话框
－－－－－－－－－－－－－－－－－－－－－
如果说是按照MFC的方式，创建UI线程：
1.定义一个线程类，让它继承CWinThread,比如:CUIThread
2.在这个类中的InitInstance函数中使用Create创建非模态对话框，使用DoModal创建模态对话框，然后将对话框指针赋给类的成员m_pMainWnd
3.将要创建UIThread线程时，使用
CWinThread* m_pUIThread=AfxBeginThread(RUNTIME_CLASS(CUIThread),THREAD_PRIORITY_BELOW_NORMAL,0,0,NULL);

如果说是按照SDK方式，创建UI线程：
1.使用CreateThread创建线程
2.线程回调函数中，使用CreateDialog创建非模态对话框，使用DialogBox创建模态对话框，对于非模态对话框，需要在回调函数中加入消息循环，模态的，则不需要



动用AfxBeginThread + 对话框对象X + create 就是创建了一个非模态的对话框，然后在X
内添加DoLoop就可以实现其UI线程的循环
－－－－－－－－－－－－－－－－－－

如果说是AfxBeginThread真正创建了UI线程，那么为了创建一个非模态的对话框，DoLoop函数显得多余了。




LZ把源代码发给我看一下，我的邮箱是:[e-mail]

[simon031187]

LZ贴了大把代码，但是没看到AfxBeginThread是怎么调用的，有没有把子对话框完全包在子线程里
如果DoLoop函数在主线程里面调用，就抢占了主对话框的消息调度，主对话框八成要假死
还有贴代码时请加上代码样式，别欺负俺眼神不好

BOOL CTestDlg::DoLoop()
{
MSG msg;
BOOL bRet; 
while(1) 
{ 
if(PeekMessage(&msg, NULL, 0, 0, PM_NOREMOVE))// 从Windows消息队列中取出消息
{ 
bRet = GetMessage(&msg, NULL, 0, 0);
if(bRet != 0) 
{
if(bRet == -1)//判断返回值是否有问题 
{ 
AfxMessageBox(_T("Handle the error and possibly exit!")); 
break;
}else{//以下分情况讨论获取到的消息对应什么操作
switch(msg.message) 
{   
case WM_CLOSE: 
DestroyWindow(); 
break; 
case WM_DESTROY:   
delete this;
PostQuitMessage(0); 
break; 
default: 
TranslateMessage(&msg);
DispatchMessage(&msg);
break;
}
}
}// end of inner if

if(!AfxGetApp()->PreTranslateMessage(&msg))// 如果无法预处理消息
{
::TranslateMessage(&msg);// 转换消息
::DispatchMessage(&msg); // 发送消息
}
}
AfxGetApp()->OnIdle(0);
AfxGetApp()->OnIdle(1);
}// end of while
return TRUE;
}

[simon031187]

居然没有自动缩进

[Fireway2008]

[Quote=引用 72 楼 cnzdgs 的回复:]
看了一下DoLoop的代码，下面这段代码有些问题，先注释掉看看。
if(!AfxGetApp()->PreTranslateMessage(&msg))// 如果无法预处理消息
{
::TranslateMessage(&msg);// 转换消息
::DispatchMessage(&msg); // 发送消息
}
}
AfxGetApp()->OnIdle(0);
AfxGetApp()->OnIdle(1);
[/Quote]

已经修改过了，可是主窗口M 无法响应鼠标的情况还是会出现……
其他类型的书写方法也尝试过了，但是还是不行……

目前DoLoop 按如下代码书写：
BOOL CHoleDlg::DoLoop()
{
MSG msg;
BOOL bRet=0;
while(1)
{
if(PeekMessage(&msg,NULL,0,0,PM_REMOVE))
{
if(msg.message == WM_QUIT)
{
MessageBox(_T("已调用过PostQuitMessage(0)"),_T("注意了"),MB_OK);
break;
}
TranslateMessage(&msg);
DispatchMessage(&msg);
}
else
{
AfxGetApp()->OnIdle(0);
AfxGetApp()->OnIdle(1);
}
}// 可以保证，关闭对话框X后，退出了while 循环
return TRUE;
}

我只是打算在新UI线程中创建一个非模态对话框 而已，虽然用继承CWinThread的方法已经实现，但是用上了
工作线程+ 非模式对话框类的消息循环
却出现了这么多问题，

不知各位在实现这个功能的时候，消息循环是如何书写的？
应该不会很复杂吧。

[wangask00]

paerxiushi
真是个热心的好人啊，想问下paerxiushi是从事哪个方面工作的，工作多长时间了

[Fireway2008]

[Quote=引用 76 楼 simon031187 的回复:]
LZ贴了大把代码，但是没看到AfxBeginThread是怎么调用的，有没有把子对话框完全包在子线程里
如果DoLoop函数在主线程里面调用，就抢占了主对话框的消息调度，主对话框八成要假死
还有贴代码时请加上代码样式，别欺负俺眼神不好
[/Quote]

呵呵，这位朋友，
我在20楼已经贴好了调用的代码。

[Fireway2008]

To paerxiushi ：
代码已经打包发送，本人邮箱[e-mail]
请查收！

To wangask00 ：

本人非IT专业工作者，但对编程很有兴趣，学习VC 2年。



感谢2位对我帖子的关注。

[cnzdgs]

看了一下DoLoop的代码，下面这段代码有些问题，先注释掉看看。
if(!AfxGetApp()->PreTranslateMessage(&msg))// 如果无法预处理消息
{
::TranslateMessage(&msg);// 转换消息
::DispatchMessage(&msg); // 发送消息
}
}
AfxGetApp()->OnIdle(0);
AfxGetApp()->OnIdle(1);

[Fireway2008]

原打算封装好一个函数留以后方便使用，
没想到有这么多的禁忌和漏洞，
创建窗体后问题多多……


希望大家继续关注此帖。

[cnzdgs]

从消息循环上已经看不出有什么问题，估计与窗口的创建及消息响应有关，建议你另做一个简单的测试程序来分析这个问题。

[Fireway2008]

[Quote=引用 85 楼 cnzdgs 的回复:]
从消息循环上已经看不出有什么问题，估计与窗口的创建及消息响应有关，建议你另做一个简单的测试程序来分析这个问题。
[/Quote]

OK, I will try .
Thank you !

[paerxiushi]

你发过来的附件是0字节的，重新发一遍，就用rar压缩包的格式好了。

[paerxiushi]

你的每个子窗体的确是在新的UI线程中的，这点没错。

1.主窗体无法响应鼠标消息是因为CtestDlg重载了OnNcLButtonDown函数,代码改成：

void CTestDlg::OnNcLButtonDown(UINT nHitTest, CPoint point) 
{
	m_bDig = TRUE;	
	CDialog::OnNcLButtonDown(nHitTest,point);   //新添加的代码
}

2.当子窗体显示之后，主窗体无法响应消息是因为AfxGetApp()->OnIdle()占用了CPU，代码改成:

BOOL CHoleDlg::DoLoop()
{
	MSG msg; 
	BOOL bRet=0;  
	while(1)  
	{   
	    
		if(PeekMessage(&msg,NULL,0,0,PM_REMOVE))
		{
			if(msg.message==WM_QUIT)
				break;
			if(!AfxGetApp()->PreTranslateMessage(&msg))//如果无法预处理消息 
			{ 
				::TranslateMessage(&msg);//转换消息 
				::DispatchMessage(&msg);//发送消息 
			} 
		}
		else
		{
			//在这里添加空闲任务处理
			Sleep(100);

		}	
	

	}
	return TRUE;
}

Sleep函数会将当前线程挂起，不影响其他 UI线程的运行。
而AfxGetApp()->OnIdle()阻塞了整个进程，在这个方法执行时，其他线程无法响应消息。当UI线程数量超过2个时，会引起死机。每个UI线程占用CPU为100%

[paerxiushi]

我已经把改好后的源代码发到你的邮箱里了。

[Fireway2008]

非常感谢！
看来不能随便乱用AfxGetApp()->OnIdle()


问题解决，感谢各位朋友的参与！！

[pysjp]

帮楼主顶起，好帖子

[Fireway2008]

补充一下，本人在处理窗口消息的时候，重载了DefWindowProc
应该重载WindowProc才是正确的。

详细请见下边的帖子：WindowProc和DefWindowProc的区别

[clever101]

这个贴一定要顶的！楼主的好学精神以及下面众位高手的耐心指导都值得学习啊！

[StonesHeart]

不错，学习了。