Question

我正在研究一些使用MFC的UI线程来实现经理线程工作者线程机制的遗留代码。该代码用于在MFC GUI应用程序下运行，但现在位于单独的dll中，并且可以从GUI应用程序和控制台应用程序运行。

管理器线程，工作线程和主应用程序通过线程消息进行通信（工作线程实际上不需要向管理器线程发送消息，但这是它最初实现和工作的方式，所以你去了）。

现在，当我从控制台应用程序运行代码时，处理从主线程发送到管理器线程的消息并调用我的处理程序。只有当我尝试从管理器线程向工作线程发送消息时，我才会遇到问题。对PostThreadMessage的调用成功，但从不调用处理程序。在普通的控制台应用程序以及Win32控制台应用程序（包括带有所有MFC好东西的预编译头文件）中都重现了这种行为。

我发现这篇旧的微软文章：http://support.microsoft.com/kb/142415，但我不得不承认我并不是真的理解它。我尝试覆盖PreTranslateMessage函数并在那里显式处理我的自定义消息，但是在调用PostThreadMessage之后从未调用过该函数

我尝试在下面的示例中重现问题，在我的示例中，即使向管理器线程发送的消息也从未被处理过，这证实了我怀疑我做错了什么。

编辑：我将缺少的InitInstance和ExitInstance添加到我的示例代码中缺失的重载ManagerThread，正如MarsRover建议的那样，现在确实有ManagerThread条消息被抽取，但是WorkerThread消息不是，它准确地再现了我在原始代码中遇到的问题。示例代码：

//Common.h

//update the progress message
#define WM_START_RUN (WM_USER + 1)

//update the progress message
#define WM_JOB_DONE (WM_USER + 2)

//run thread has finished
#define WM_RUN    (WM_USER + 3)

// ManagerThread.h
class ManagerThread : public CWinThread
{
    DECLARE_DYNCREATE(ManagerThread)
protected:
    ManagerThread(){}           // protected constructor used by dynamic creation
    virtual ~ManagerThread();
    BOOL InitInstance();
    int ExitInstance();
    std::vector<WorkerThread*> m_WorkerThreads;
    int numOfJobs;
    DECLARE_MESSAGE_MAP()
    afx_msg void OnStartRun(WPARAM wParam, LPARAM lParam);
    afx_msg void OnJobDone(WPARAM wParam, LPARAM lParam);
    afx_msg void OnQuit(WPARAM wParam, LPARAM lParam);
};

//WorkerThread.h
class WorkerThread : public CWinThread
{
    DECLARE_DYNCREATE(WorkerThread)

protected:
    WorkerThread(){}        // protected constructor used by dynamic creation
    virtual ~WorkerThread(){}
    virtual BOOL InitInstance();
    virtual int ExitInstance();

public:
    void SetManager(CWinThread* pManager) {m_Manager = pManager;}
    void SetID(int _id) {id = _id;}
protected:
    int id;
    CWinThread* m_Manager;
    DECLARE_MESSAGE_MAP()
    afx_msg void OnRun(WPARAM wParam, LPARAM lParam);
    afx_msg void OnQuit(WPARAM wParam, LPARAM lParam);
};

// ManagerThread.cpp

IMPLEMENT_DYNCREATE(ManagerThread, CWinThread)

ManagerThread::~ManagerThread() {
    while(!m_WorkerThreads.empty()) {
        std::vector<WorkerThread*>::iterator it = m_WorkerThreads.begin();
        (*it)->PostThreadMessage(WM_QUIT, 0, 0);
        m_WorkerThreads.erase(it);
    }
}

BOOL CFilterManagerThread::InitInstance()
{
    return CWinThread::InitInstance();
}


int CFilterManagerThread::ExitInstance()
{
    return CWinThread::ExitInstance();
}

BEGIN_MESSAGE_MAP(ManagerThread, CWinThread)
    ON_THREAD_MESSAGE(WM_START_RUN, OnStartRun)
    ON_THREAD_MESSAGE(WM_JOB_DONE, OnJobDone)
    ON_THREAD_MESSAGE(WM_QUIT, OnQuit)
END_MESSAGE_MAP()

void ManagerThread::OnJobDone( WPARAM wParam, LPARAM lParam) {
    numOfJobs--;
    if (!numOfJobs) {
        OnQuit(0,0);
    }
}

void ManagerThread::OnStartRun(WPARAM wParam, LPARAM lParam) {
    numOfJobs = (int) wParam;
    for (int i = 0; i < numOfJobs; i++) {
        WorkerThread *newThread = (WorkerThread*)AfxBeginThread(RUNTIME_CLASS(WorkerThread), THREAD_PRIORITY_LOWEST, 0, CREATE_SUSPENDED);
        newThread->SetID(i);
        newThread->SetManager(this);
        m_WorkerThreads.push_back(newThread);
        newThread->ResumeThread();
        Sleep(1000); //sleep 1 second before sending message to allow the thread to strat running
        newThread->PostThreadMessage(WM_RUN, 0, 0);
    }
}

void ManagerThread::OnQuit(WPARAM wParam, LPARAM lParam) {
    AfxEndThread(0);
}

// WorkerThread.cpp

IMPLEMENT_DYNCREATE(WorkerThread, CWinThread)

BOOL WorkerThread::InitInstance() {
    // TODO:  perform and per-thread initialization here
    CoInitializeEx(NULL, COINIT_APARTMENTTHREADED);
    return TRUE;
}

int WorkerThread::ExitInstance() {
    // TODO:  perform any per-thread cleanup here

    //uninitialize the COM library
    CoUninitialize();
    return CWinThread::ExitInstance();
}

BEGIN_MESSAGE_MAP(WorkerThread, CWinThread)
    ON_THREAD_MESSAGE(WM_RUN, OnRun)
    ON_THREAD_MESSAGE(WM_QUIT, OnQuit)
END_MESSAGE_MAP()

void WorkerThread::OnRun(WPARAM wParam, LPARAM lParam) {
    cout << id <<endl;
    m_Manager->PostThreadMessage(WM_JOB_DONE, id, 0);
}
void WorkerThread::OnQuit(WPARAM wParam, LPARAM lParam) {
    AfxEndThread(0);
}

和main：

        ManagerThread *manager = (ManagerThread*)AfxBeginThread(RUNTIME_CLASS(ManagerThread), THREAD_PRIORITY_NORMAL, 0, CREATE_SUSPENDED);
        manager->ResumeThread();

        Sleep(1000); //sleep 1 second before sending message to allow the thread to start running

        manager->PostThreadMessage(WM_START_RUN, 10, 0);
        while(true){}

这是一个粗略的样本。当然，在我的原始代码中，我使用比Sleep和while(true)更好的机制来确保同步并避免程序在经理线程结束之前结束。但它重现了我遇到的问题所以我没有看到增加复杂性的重点。

Answer 1

找出问题所在。问题是CoInitializeEx中对WorkerThread::initInstance的调用。显然，该调用阻止了线程的初始化很长时间，甚至超过了示例代码中的Sleep（1000）。所以我在创建消息队列之前发布消息。按照MSDN的说明进行操作：

发布消息的线程必须已创建消息   队列，或者对PostThreadMessage的调用失败。使用以下内容   处理这种情况的方法。

创建一个事件对象，然后创建线程。

使用WaitForSingleObject函数等待设置事件   在调用PostThreadMessage之前发出信号状态。

在邮件将发布到的帖子中，将PeekMessage称为   此处显示强制系统创建消息队列。

PeekMessage（＆amp; msg，NULL，WM_USER，WM_USER，PM_NOREMOVE）

设置事件，以指示线程已准备好接收已发布   消息。

从此previous question开始，我为每个线程类创建了一个成员CEvent，并将InitInstance更改为：

BOOL CFilterWorkerThread::InitInstance()
{
    BOOL worked=CWinThread::InitInstance();
    CoInitializeEx(NULL, COINIT_APARTMENTTHREADED);
    MSG msg;
    PeekMessage(&msg, NULL, WM_USER, WM_USER, PM_NOREMOVE);
    m_ControllerThreadReady.SetEvent();
    return TRUE;
}

为了在我将even设置为true之前强制初始化消息队列。然后在ManagerThread中，在将任何消息发布到WaitForSingleObject之前，我致电WorkerThread。

未调用MFC的CWinThread :: PostThreadMessage处理程序

1 个答案: