如何在另一个线程中关闭MFC Modal对话框并获得对话框的返回值？

Question

我有一个MFC对话框项目。 主对话框中有一个下载按钮，单击该按钮将提示进度条并开始下载。 下载完成后，我希望它自动关闭。

void CProgressBarTest::DoDataExchange(CDataExchange* pDX)
{
    static auto funDownload = [&]() {
        m_downloadRetValue = ::SomeDownloadAPI(funDownloadCallback);

        //When download finished, close the current dialog (progress bar). Here are two options:
        //EndDialog(IDYES); // causes crash
        //::PostMessage(this->GetSafeHwnd(), WM_CLOSE, 0, 0);// doesn't crash, but without return valud IDYES.
    };

    m_thDownload = std::thread(funDownload);
}

这是关闭进度条的两种方法：

EndDialog(IDYES) ：它导致崩溃。

::PostMessage(this->GetSafeHwnd(), WM_CLOSE, 0, 0) ：它可以关闭窗口而不会崩溃，但是也没有返回值（IDYES）。

我想在外面做这样的检查，

void CGUIThreadTestDlg::OnBnClickedButton3()
{
    CProgressBarTest dlg(this);
    INT_PTR nRet = dlg.DoModal();
    switch (nRet)
    {
    case -1:
        AfxMessageBox(_T("Dialog box could not be created!"));
        break;
    case IDYES:
        AfxMessageBox(_T("Yes!"));
        break;
    case IDOK:
        // Do something 
        break;
    case IDCANCEL:
        AfxMessageBox(_T("IDCANCEL!"));
        break;
    default:
        // Do something 
        break;
    }
}

Answer 1

将应用程序定义的消息从下载线程发布到您的主GUI线程，如下所示：

BOOL CProgressBarTest::OnInitDialog()
{
    CDialog::OnInitDialog();

    auto funDownload = []( HWND hwnd ){
        auto const downloadRetValue = ::SomeDownloadAPI(funDownloadCallback);

        ::PostMessage( hwnd, WM_APP_DOWNLOAD_FINISHED, static_cast<WPARAM>( downloadRetValue ), 0 );
    };

    m_thDownload = std::thread(funDownload, GetSafeHwnd());

    return TRUE;
}

注意1：我使用OnInitDialog()启动线程，因为DoDataExchange()确实是一个糟糕的选择，因为它将被多次调用。 OnInitDialog()将仅被调用一次。

注意2：无捕获的lambda用于更好地将下载线程与GUI线程解耦。 将this从GUI对话框传递给工作线程是造成灾难的秘诀，因为它很容易只写GUI线程变量而忽略了所需的同步。 除此之外，更少的耦合和更少的依赖关系总是一件好事。

什么是WM_APP_DOWNLOAD_FINISHED ？ 这是我的应用程序定义的消息ID ，通常这样定义：

enum {
    WM_APP_0 = WM_APP,
    WM_APP_DOWNLOAD_FINISHED
    // for future extension...
};

添加消息映射条目：

BEGIN_MESSAGE_MAP(CProgressBarTest, CDialog)
    ON_MESSAGE( WM_APP_DOWNLOAD_FINISHED, &CProgressBarTest::OnDownloadFinished )
END_MESSAGE_MAP()

并定义消息处理程序以从对话框中返回一个取决于下载结果的值：

LRESULT CProgressBarTest::OnDownloadFinished( WPARAM wp, LPARAM lp ) 
{    
    m_thDownload.join();

    auto const downloadRetValue = wp;
    EndDialog( downloadRetValue == ERROR_SUCCESS ? IDYES : IDCANCEL );

    return 0; 
}

确保像我之前那样join()线程，以避免std::thread析构函数崩溃，这要求线程已加入。

Answer 2

也许这是一种解决方法。

我可以从对话框的DoModal函数返回自定义值吗？

不要使用DoModal的返回值。 只需为CProgressBarTest添加一个成员函数。