关于无模式对话框

[Kernel_Don]

在《Visual C++ 技术内幕》第四版中介绍无模式对话框时，提到：
--------------------------------------------------------------------
对于无模式对话框，一定要注意不要调用CDialog::OnOK或CDialog::OnCancel
函数，也就是说必须在派生类中重载这些函数；否则，当使用Esc、回车或用
鼠标单击某按钮时，就会激发对相应基类函数的调用，进而导致调用Windows
的EndDialog函数，而EndDialog函数只适合模式对话框。对于无模式对话框，
必须调用DestroyWindow函数。
--------------------------------------------------------------------

我对这做了试验，用new...Create...ShowWindow...建立了一个无模式的对话
框，然后用CDialog::OnOK()关闭该对话框，好像一切都正常，没什么不对呀...

为什么这本经典的书上说：一定要注意不要调用CDialog::OnOK或CDialog::OnCancel
函数?

[西江残月]

hehe,你关闭后再启动一次看看!会出现内存错误的!

[akiko]

MSDN上的解释,最后一句说明问题了:
If you implement the OK(Cancel) button in a modeless dialog box, you must override the OnOK(OnCancel) member function and call DestroyWindow from within it. Don’t call the base-class member function, because it calls EndDialog, which makes the dialog box invisible but does not destroy it.

[丁淇石头]

你做的实验一切都正常，那可能是你的实验不全面，呵呵。还是不要随便怀疑MSDN的好。

[闻缺陷则喜何志丹]

hehe,你关闭后再启动一次看看!会出现内存错误的!
============================================
同感

[闻缺陷则喜何志丹]

void CTestDialog::OnOK()
{
CDialog::OnOK();
}
在无模式对话框中也是不行的

[bcpl]

EndDialog并不立即销毁对话框，只有在模态对话框机制下才在后续执行中销毁，
如果在非模态对话框中调用，窗口并没有被真正销毁，如果用同一个MFC的CDialog对象再次
Create，将引发一系列的ASSERT。

[Kernel_Don]

我在CADlg中：
class CADlg : public CDialog
{
public:
CXDialog * xDlg;
// ...
};

CADlg::~CADlg()
{
// ...
if (xDlg != NULL)
delete xDlg;
// ...
}

BOOL CTest030821Dlg::OnInitDialog()
{
//...
xDlg = NULL;
//...
}

//在对话框CADlg中有一按钮Button1,按下该按钮后弹出一个无模式对话框：
void CADlg::OnButton1()
{
// TODO: Add your control notification handler code here
if (xDlg != NULL)
delete xDlg;
xDlg = new CXDialog;
xDlg->Create(IDD_DIALOG1, this);
xDlg->ShowWindow(SW_SHOW);
}
/////////////////////////////////////////
//在CXDialog对话框中也有一按钮Button1,该按钮的执行代码如下：
void CXDialog::OnButton1()
{
// TODO: Add your control notification handler code here
CDialog::OnOK();
}

//按下CXDialog::Button1后该无模式对话框关闭，按下CADlg::Button1后该无模式对
话框又打开，反复操作无出错。

[bcpl]

这样倒是不会出错，因为你动态分配/销毁对话框对象，
但不表示这样做就是对的，ms要你别用EndDialog关闭非模态对话模式，
你就乖乖听话吧

[yanzai8848]

首先必须明白所有的Win窗口使用API函数CreateWindow(或*Ex)创建, 使用DestroyWindow撤消(删除全部, 释放分配的内存)
然后看以下源代码:
1,以下说明了为什么使用delete是正确的
CDialog::~CDialog()
{
if (m_hWnd != NULL)
{
TRACE0("Warning: calling DestroyWindow in CDialog::~CDialog --\n");
TRACE0("\tOnDestroy or PostNcDestroy in derived class will not be called.\n");
DestroyWindow();
}
}

2,以下说明了为什么使用EndDialog不正确:

void CDialog::EndDialog(int nResult)
{
ASSERT(::IsWindow(m_hWnd));

if (m_nFlags & (WF_MODALLOOP|WF_CONTINUEMODAL))
EndModalLoop(nResult);

::EndDialog(m_hWnd, nResult);
}

//::EndDialog的MSDN说:
//EndDialog does not close the dialog box immediately. Instead, it sets a flag that directs the dialog box to close as soon as the current message handler returns.

[闻缺陷则喜何志丹]

void CADlg::OnButton1()
{
xDlg = new CXDialog;
xDlg->Create(IDD_DIALOG1, this);
xDlg->ShowWindow(SW_SHOW);
}

void CADlg::OnButton()
{
xDlg->DestroyWindow();
delete xDlg;

}

[闻缺陷则喜何志丹]

上面的方法不安全
构造函数中
xDlg = NULL;
BOOL CADlg::OnInitDialog()
{
}不作处理

void CADlg::OnButton1()
{
if(NULL != xDlg)
return ;
xDlg = new CXDialog;
xDlg->Create(IDD_DIALOG1, this);
xDlg->ShowWindow(SW_SHOW);
}

void CADlg::OnButton()
{
if(NULL == xDlg)
return;
xDlg->DestroyWindow();
delete xDlg;
xDlg = NULL;

}

[闻缺陷则喜何志丹]

不明白之处
请给我发短消息
不要忘记了加帖子的地址

[flinming]

我到是没有遇到过，我都没有销毁也不会错误
xDlg = new * CXDialog;
xDlg->Create(IDD_DIALOG1);
xDlg->ShowWindow(SW_SHOW);

[flinming]

上面写错了是这样
CXDialog *xDlg = new CXDialog;
xDlg->Create(IDD_DIALOG1);
xDlg->ShowWindow(SW_SHOW);

[Kernel_Don]

class CADlg : public CDialog
{
public:
CXDialog * xDlg;
// ...
};


CADlg::~CADlg()
{
// ...
if (xDlg->GetParent() != NULL)
xDlg->DestroyWindow();
if (xDlg != NULL)
delete xDlg;
// ...
}

BOOL CADlg::OnInitDialog()
{
//...
xDlg = new CXDialog;
//...
}

//在对话框CADlg中有两个按钮Button1和Button2
按下Button1按钮后弹出一个无模式对话框：
void CADlg::OnButton1()
{
xDlg->Create(IDD_DIALOG1, this);
xDlg->ShowWindow(SW_SHOW);
}
按下Button2按钮后消毁该无模式对话框：
void CADlg::OnButton()
{
xDlg->DestroyWindow();
}
/////////////////////////////////////////

为什么按下BUTTON1产生对话框，再按BUTTON2消毁该对话框后，第二次按下BUTTON1时会出错？

[Kernel_Don]

回复人： flinming(flinming) ( ) 信誉：100 2003-08-27 20:25:00 得分:0


我到是没有遇到过，我都没有销毁也不会错误
CXDialog *xDlg = new CXDialog;
xDlg->Create(IDD_DIALOG1);
xDlg->ShowWindow(SW_SHOW);

/////////////////////////
这样虽然无出错提示，但每次按下按钮都new一次，会有内存泄漏...