我正在 MFC 中编写一个带有后台工作线程(通过创建_beginthreadex
)和 UI 线程的应用程序。从 UI 线程单击一个按钮以开始和结束工作线程。如果m_threadRunning
标志为假,则启动后台线程,如果为真,则停止后台线程。我停止线程的方法是将m_threadRunning
标志设置为 false 并调用WaitForSingleObject
以让后台线程完成它正在做的事情。
我的应用程序有四种不同的状态。我让前三个状态正常工作,添加第四个状态是导致我的问题的原因。对于第四种状态,我希望能够对桌面进行采样并将平均 RGB 值发送到 COM 端口进行处理。当处于前三种状态中的任何一种时,如果我想停止向 COM 端口发送数据的执行,它将正常终止并且没有问题。如果我处于第四个状态并单击“停止”,应用程序将挂起,因为我没有时间来调用WaitForSingleObject
.
我还有一个显示当前 RGB 值的自定义CEdit
框。CColorEdit
当我处于状态 3 或 4 时,我从后台线程更新它(因为它们都动态改变颜色)。当我设置调用Invalidate
或的颜色时,我已将问题缩小到调用RedrawWindow
。
我提出了一些解决方案,但我不喜欢其中的任何一个,并且宁愿了解导致问题的原因,因为我在 MFC 中编写此问题的目标是学习和理解 MFC。以下是解决问题的方法:
- 我已经在我的工作线程中以大约 60 个样本/秒的速度调用 Sleep()。将其更改为较低的值,例如 30 个样本/秒,大部分时间都可以解决问题。
- 我在我的工作线程中轮询
m_threadRunning
以检查是否应该终止线程。如果我在对屏幕进行采样之后但在更新编辑控件之前对其进行轮询,这在大多数情况下都可以解决问题。 - 我在调用
WaitForSingleObject
和调用 TerminateThread 时超时 5 秒以在线程等待失败时手动终止线程,这始终解决了问题。这是我目前的解决方案。
以下是相关的代码位(我锁定了 outBytes 的任何使用):
void CLightControlDlg::UpdateOutputLabel()
{
CSingleLock locker(&m_crit);
locker.Lock();
m_outLabel.SetColor(outBytes[1], outBytes[2], outBytes[3]); //the call to this freezes the program
CString str;
str.Format(L"R = %d; G = %d; B = %d;", outBytes[1], outBytes[2], outBytes[3]);
m_outLabel.SetWindowText(str);
}
这部分代码用于终止工作线程
m_threadRunning = false;
locker.Unlock(); //release the lock...
//omitted re-enabling of some controls
//normally this is just WaitForSingleObject(m_threadHand, INFINITE);
if(WaitForSingleObject(m_threadHand, 5000) == WAIT_TIMEOUT)
{
MessageBox(L"There was an error cancelling the I/O operation to the COM port. Forcing a close.");
TerminateThread(m_threadHand, 0);
}
CloseHandle(m_threadHand);
CloseHandle(m_comPort);
m_threadHand = INVALID_HANDLE_VALUE;
m_comPort = INVALID_HANDLE_VALUE;
我派生的编辑控件中更新文本颜色的代码:
void SetColor(byte r, byte g, byte b)
{
_r = r;
_g = g;
_b = b;
br.DeleteObject();
br.CreateSolidBrush(RGB(r,g,b));
Invalidate(); //RedrawWindow() freezes as well
}
最后,我的线程过程的代码:
unsigned int __stdcall SendToComProc(void * param)
{
CLightControlDlg *dlg = (CLightControlDlg*)param;
while(1)
{
if(!dlg->IsThreadRunning())
break;
switch(dlg->GetCurrentState())
{
case TransitionColor: //state 3
dlg->DoTransition();
dlg->UpdateOutputLabel();
break;
case ScreenColor: //state 4
dlg->DoGetScreenAverages();
//if(!dlg->IsThreadRunning()) break; //second poll to IsThreadRunning()
dlg->UpdateOutputLabel();
break;
}
dlg->SendToCom();
Sleep(17); // Sleep for 1020 / 60 = 17 = ~60samples/sec
}
return 0;
}
非常感谢您提供的任何帮助!