为什么/如何扭转消息的顺序?

时间:2015-06-09 01:00:09

标签: c++ multithreading winapi

请考虑以下代码:

CCritialSection listLock;
std::list<CString> messageList;
extern MyApp theApp;    // public inheritance from CWinApp
const int aMessageNumber = WM_APP + 123;

void MyApp::EnqueueMessageForUIThread( const CString message )
{
    CSingleLock lock( &listLock, TRUE );
    messageList.push_back( message );
    theApp.m_pMainWnd->PostMessage( aMessageNumber  );
}

void MyApp:PopupMessageFromNonUIThread( void)
{
    // This function is called via ON_MESSAGE( aMessageNumber, ... )
    CSingleLock lock( &listLock, TRUE );
    bool messagesAvailable( !messageList.empty() );
    while ( messagesAvailable )
    {
        const CString message( messageList.front() );
        messageList.pop_front();
//      lock.Unlock();
        AfxMessageBox( message, MB_ICONINFORMATION );
//      lock.Lock();
        messagesAvailable = !messageList.empty();
    }
}

正如两个函数名称所示,这些函数旨在弹出源自非UI线程的UI线程中的消息 - 当从非UI线程调用UI函数时,Windows上会发生不良事件。

如果两条lock.Unlock()lock.Lock()行没有被注释掉,肯定会更好。这将允许在每个弹出消息等待用户响应时将更多消息排入队列 - 非UI线程不必阻止并等待listLock可用。

但是......至少在我使用这些函数的上下文中,我始终按顺序排列三条消息,然后以相反的顺序将它们弹出给用户。这怎么可能?

如果它很重要(我不相信它),这些功能是“主应用程序”代码的一部分,它与Windows PC上的“帮助应用程序”一起运行。还有另一台(相同的)PC运行相同的两个应用程序。各方之间的通信是通过Windows套接字 - 主要应用程序只与帮助者交谈;帮助者通过网络与其他帮助者交谈。

我按照上面的代码顺序看到的三条消息,但是当我取消注释两条锁定/解锁线时的反向顺序,是“环回”测试的一部分 - 我从一个主应用程序向另一个发送消息,获取第一个(本地)帮助器以确认它,然后第二个(远程)帮助器确认它,然后第二个(远程)主应用程序确认它。

在网络上捕获数据包确认按预期顺序发送/接收数据包。在消息到达时对消息进行编号确认它们是按顺序接收的。只有一个接收线程,在完成之前不可能多次重新进入EnqueueMessageForUIThread()函数。

尽管如此,在使用(编号)消息调用EnqueueMessageForUIThread()函数并弹出PopupMessageForNonUIThread()函数之间的某处,如果我从中移除注释,这些消息将反向弹出lock.Unlock()lock.Lock()

如何?

1 个答案:

答案 0 :(得分:1)

PostMessage是线程安全的,设计可以将消息转发给其他线程。这意味着您遇到的线程不安全在于您的代码。

在这种情况下,您正在打开一个消息框。消息框本身运行消息循环,因此,如果在PopupMessageFromNonUIThread中设置断点,您将发现在第一个消息框完成任何操作之前从队列中提取第二条和第三条消息。

您需要为此实现自己的模式,这不需要锁定消息泵,仅用于传输队列。

bool MyApp::getNextMessage(CString& into)
{
    CSingleLock lock( &listLock, TRUE );
    if ( messageList.empty() )
        return false;
    into = messageList.front();
    messageList.pop_front();
    return true;
}

void MyApp:PopupMessageFromNonUIThread( void)
{
    static bool displaying = false;
    if (displaying)
        return;

    displaying = true;

    // This function is called via ON_MESSAGE( aMessageNumber, ... )
    CString message;
    while ( getNextMessage(message) )
    {
        AfxMessageBox( message, MB_ICONINFORMATION );
    }

    displaying = false;
}

哦,我知道它不是最有效的互斥锁策略......但是你正在显示一个消息框 - 优化你在其周围使用的锁定策略并不重要。