MFC应用程序挂起在发出终止信号的线程中

Question

我正在MFC中编写一个带有后台工作线程（通过_beginthreadex创建）和UI线程的应用程序。从UI线程中单击一个按钮以开始和结束工作线程。如果m_threadRunning标志为false，则启动后台线程，如果为true，则停止后台线程。我停止线程的方法是将m_threadRunning标志设置为false，并调用WaitForSingleObject让后台线程完成它正在做的事情。

我的应用程序有四种不同的状态。我让前三个州正常工作，加上第四个州就是我的问题所在。对于第四种状态，我希望能够对桌面进行采样，并将平均RGB值发送到COM端口进行处理。当处于前三种状态中的任何一种时，如果我想停止执行向COM端口发送数据，它将正常终止，没有问题。如果我处于第四种状态并单击"停止"，应用程序将挂起，因为我对WaitForSingleObject的呼叫没有时间。

我还有一个自定义的CEdit框CColorEdit，它显示了当前的RGB值。当我处于状态3或状态4时，我会从后台线程更新它（因为它们都会动态更改颜色）。我已经将问题缩小到当我设置调用Invalidate或RedrawWindow的颜色时的调用。

我已经想出了一些解决方案，但我不喜欢其中的任何一个，我宁愿了解问题的原因，因为我在MFC中写这篇文章的目的是学习和理解MFC。以下是解决问题的方法：

1. 我在我的工作线程中调用Sleep（）的速度已经是每秒60个样本。将其更改为较低的值，例如每秒30个样本，在大多数情况下解决了问题
2. 我在我的工作线程中轮询m_threadRunning，以检查该线程是否应该终止。如果我在对屏幕进行采样之后但在更新编辑控件之前对其进行轮询，则大多数情况下都可以解决问题
3. 我在调用WaitForSingleObject时会超时5秒，当线程等待失败时，我会调用TerminateThread手动终止线程，这一直解决了问题。这是我目前的解决方案

以下是相关的代码位（我锁定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;
}

非常感谢您能提供的任何帮助！

Answer 1

当工作线程试图访问在主线程中创建的控件，而主线程在WaitForSingleObject中挂起时，会出现死锁。只有当主线程接受控件的关联消息时，才能从工作线程更新控件。

从工作线程中删除对控件的所有访问。相反，PostMessage将自定义消息发送到主线程中的窗口。这里有一个例子：

http://vcfaq.mvps.org/mfc/12.htm

同样的技术可以用于通知主线程工作线程已经完成，因此可以避免使用WaitForSingleObject。