Question

我创建了一个MFC应用程序，其中包含一个Base Dialog（派生自CDialog类），一个Setting Dialog（派生自CBaseDlg和一个App Dialog （也来自CBaseDlg。然后，我创建了一个名为CScrMng（又称屏幕管理器）的类，该类具有ShowDialog()函数（分别对Create和Show对话框）。

基本思想是ScrMng将管理我的所有无模式对话框，并且每当我要打开对话框时，我只需要在CScrMng::ShowDialog()中打开BaseDlg.cpp，对话框就会显示。

这种方法导致资源在这里和那里泄漏。我已经进行了一些有关覆盖PostNcDestroy()的研究，但是我对在何处调用它并不清楚。

我应该使用什么功能来正确关闭这些无模式对话框？

我想从Setting Dialog中打开Base Dialog，然后当我单击Cancel按钮时，它应该返回到Base Dialog屏幕，以便我可以打开另一个对话框。

现在我正在使用EndDialog()。我知道这是错误的，但是调用DestroyWindow()会立即退出程序，这不是我想要的。

源代码

MFCApplication.cpp


#include...

BEGIN_MESSAGE_MAP(CMFCApp, CWinApp)
    ON_COMMAND(ID_HELP, &CWinApp::OnHelp)
END_MESSAGE_MAP()

CMFCApp::CMFCApp()
{
     // support Restart Manager
     m_dwRestartManagerSupportFlags = AFX_RESTART_MANAGER_SUPPORT_RESTART;
}

CMFCApp theApp;

BOOL CMFCApp::InitInstance()
{
    ...
    CWinApp::InitInstance();

    CShellManager *pShellManager = new CShellManager;

    CScrMng::GetInstance()->ShowDialog(IDD_MAINDLG);

    SetRegistryKey(_T("Local Applications"));

    if (pShellManager != NULL)
    {
        delete pShellManager;
    }
    return TRUE;
}

CScrMng.cpp

#include...

CScrMng* CScrMng::m_pInstance = NULL;
CScrMng* CScrMng::GetInstance(){
     if (m_pInstance == NULL)
    m_pInstance = new CScrMng();
    return m_pInstance;
}

CScrMng::CScrMng(){}

void CScrMng::ShowDialog(int ID)
{
    CMainDlg* m_pDlg = NULL;
    switch (ID)
    {
    case IDD_MAINDLG:
        m_pDlg = new CMainDlg();
        theApp.m_pMainWnd = m_pDlg;
        m_pDlg->Create(IDD_MAINDLG);
        m_pDlg->ShowWindow(SW_SHOW);
        m_pDlg->UpdateWindow();
        break;

    case ...
        break;

    case IDD_SETTINGDLG:
        m_pDlg = new CSettingDlg();
        m_pDlg->Create(ID,NULL);
        m_pDlg->ShowWindow(SW_SHOW);
        m_pDlg->UpdateWindow();
        break;
    }

CMainDlg.cpp

#include...

CMainDlg::CMainDlg(CWnd* pParent /*=NULL*/)
    : CDialog(CMainDlg::IDD, pParent) {m_hIcon = AfxGetApp()-> LoadIcon(IDR_MAINFRAME);}

void CMainDlg::DoDataExchange(CDataExchange* pDX) {...}

void CMainDlg::PostNcDestroy() //Added these
{
    CDialog::PostNcDestroy();
    delete this;
}

BEGIN_MESSAGE_MAP(CMainDlg, CDialog)
...
END_MESSAGE_MAP()

BOOL CMainDlg::OnInitDialog() {
    CDialog::OnInitDialog();
    SetIcon(m_hIcon, FALSE);        
    return TRUE; 
}

void CMainDlg::OnPaint() {...}

void CMainDlg::OnBnClickedOpenAppdlg()
{
    CScrMng::GetInstance()->ShowDialog(IDD_APPDLG);
}

void CMainDlg::OnBnClickedOpenSettingdlg()
{
    CScrMng::GetInstance()->ShowDialog(IDD_SETTINGDLG);
}

void CMainDlg::OnBnClickedExit() 
{
     DestroyWindow(); //replaced CDialog::OnCancel() with this.
}

更新：更改SettingDlg.cpp中的代码后，我遇到了Debug Assertion Failed!问题：

void CWnd::MoveWindow(int x, int y, int nWidth, int nHeight, BOOL bRepaint)
{
    ASSERT(::IsWindow(m_hWnd) || (m_pCtrlSite != NULL)); //Breakpoint triggered

    if (m_pCtrlSite == NULL)
        ::MoveWindow(m_hWnd, x, y, nWidth, nHeight, bRepaint);
    else
        m_pCtrlSite->MoveWindow(x, y, nWidth, nHeight);
}

这是我在.cpp文件中所做的更改： SettingDlg.cpp

void CSettingDlg::PostNcDestroy()
{
    CMainDlg::PostNcDestroy();
}

void CSettingDlg::OnBnClickedSettingcancel()
{
    DestroyWindow(); //Using destroyWindow rather than EndDialog();
}

Answer 1

关闭并删除无模式对话框。
正确的方法是：为无模式对话框覆盖PostNcDestroy，OnOk（）和OnCancel（）

void CBaseDlg::PostNcDestroy()
{
    CDialog::PostNcDestroy();
    delete this;
}

。

void CBaseDlg::OnOk()
{
   if(!UpdateData(TRUE))
       return;
    DestroyWindow();
}

。

void CBaseDlg::OnCancel()
{
    DestroyWindow();
}

如何“正确关闭MFC无模式对话框并修复资源泄漏”

1 个答案: