救命：MFC怎么防止非模态对话框被重复创建？？

[avigdkwynw]

很多书上创建非模态对话框的代码都是这样，在某个菜单的响应命令中添加：
CMyDialog *dlg = new CMyDialog;
dlg->Create(IDD_DIALOG1,this);
dlg->ShowWindow(SW_SHOW);

可是这样的话，每次执行这个菜单命令都会创建一次这个非模态对话框，于是我将它改为：
CMyDialog *dlg = new CMyDialog;
if(!IsWindow(dlg->m_hWnd)) //如果m_hwnd存在，就不创建
{
dlg->Create(IDD_DIALOG1,this);
dlg->ShowWindow(SW_SHOW);
}
发现效果还是一样，原来是dlg指针又重新分配了一次地址，于是我又把它改为静态的，防止重新分配：
static CMyDialog *dlg = new CMyDialog;
if(!IsWindow(dlg->m_hWnd))
{
dlg->Create(IDD_DIALOG1,this);
dlg->ShowWindow(SW_SHOW);
}
这样就不会重新创建了，可是问题又来了，当我关闭对话框，然后再打开时，程序就出错了，
因为在销毁非模态对话框时需要销毁这个对象：
void CMyDialog::PostNcDestroy()
{
delete this;
CDialog::PostNcDestroy();
}
我调试了一下，发现代码执行到"dlg->Create(IDD_DIALOG1,this);"
时出错的，错误提示为：assess violation。
唉，实在搞不懂了，请各位高手救救小弟吧，55555！！！

[ZWHLOVEPRO]

if(dlg->GetSafeHanle() == NULL)
{
plg->Create(...);
....
}

[yc_8301]

来晚了，作为成员变量，在new之前判断是否为NULL就OK了。

[miao120550]

把这个指针保存为成员变量就好办了。创建之间先判断句柄是否有效，否则就创建，然后再显示，退出的时候删除指针，OK。

[sixbusy]

写一个单例模式，就可以了！
构造函数私有！参数可以设置为父类的ID。

[baoyz]

[Quote=引用 2 楼 AAa_tnT 的回复:]
手动添加一个BOOL 变量就行了。。
创建时候 进行判断 为0就创建 然后写1
为1就什么也不做
[/Quote]
对，这个最可靠，最简单。

关闭对话框时再写0。

[congyu888]

4楼的方法，控制dialog的创建和释放都比较方便！

[用户 昵称]

用一个全局变量，当对话框成功创建后，变量在对话内部变成1，在销毁时变成0.

[StartDay]

在非模态对话框的PostNcDestroy里面，给显示这个非模态对话框的窗体send一个自定义消息，
之后delete this；

消息处理函数里面把指向非模态对话框的指针赋值为NULL。

每次show的时候记得判断指向非模态对话框的指针是否为空。

[wow415411408]

我自己做的时候麻烦了点 多弄了个BOOL变量控制
声明
private:
BOOL bT;
CDialog *dlg;

构造里
bT=FALSE;

然后
void CMyView::OnMyWindow()
{
if(!bT)
{
dlg=new Dialog_Setsize_Result;
dlg->Create(IDD_DIALOG1);
dlg->ShowWindow(SW_SHOW);
bT=TRUE;
}else{
if(dlg!=NULL)
{
dlg->DestroyWindow();
delete dlg;
dlg=NULL;
bT=FALSE;
OnMyWindow();
}
}
}

[effective100]

前段时间项目遇到类似的问题，不过比这复杂多的去了，最后的做法和2楼说的一致
[Quote=引用 2 楼 AAa_tnT 的回复:]
手动添加一个BOOL 变量就行了。。
创建时候 进行判断 为0就创建 然后写1
为1就什么也不做
[/Quote]

[lhsxsh]

做一个状态判断一下就行了

[effective100]

秀一段用消息来做的方法：
CTestDlg dlg; //CTestDlg 使我们用来测试的Dlg类；
//在处理WM_CLOSE消息时做如下动作：
void CTestDlg::OnClose()
{
// TODO: Add your message handler code here and/or call default
GetParent()->PostMessage(WM_SETCREATDLGFLAG);//别忘了定义WM_SETCREATDLGFLAG消息；
CDialog::OnClose();
}
//CNoModeDlg_DemoDlg是父窗口类，别忘了在消息泵中注册WM_SETCREATDLGFLAG消息，如下:
BEGIN_MESSAGE_MAP(CNoModeDlg_DemoDlg, CDialog)
...
ON_MESSAGE(WM_SETCREATDLGFLAG, OnSetCreateDlgFlag)
...
END_MESSAGE_MAP()

//创建非模态对话框，m_bCreateDlg 初始化位false；
CTestDlg dlg;
void CNoModeDlg_DemoDlg::OnBnClickedButton1()
{
// TODO: Add your control notification handler code here
if ( !m_bCreateDlg )
{
dlg.Create(IDD_DIALOG_ONE,this);
dlg.ShowWindow(SW_SHOW);
m_bCreateDlg = true;
}
}

LRESULT CNoModeDlg_DemoDlg::OnSetCreateDlgFlag(WPARAM wParam, LPARAM lParam)
{
m_bCreateDlg = false;
dlg.DestroyWindow();
return 0;
}

[txk1451]

办法真的是太多
类成员变量、全局对象....

[avigdkwynw]

1、类成员变量不好销毁吧？
2、全局对象尽量少用吧？
3、还是harry330说得好，每次要创建之前通过FindWindow查找是否已经存在,存在就BringWindowToTop，或者SetForgroundWindow将窗口带到前台窗口。
4、我总结一下：

//第一步，响应菜单命令，创建与显示对话框。
void CXXView::OnXXX() 
{
  // TODO: Add your command handler code here
  CWnd* pWnd=FindWindow(0,"XXX");
  if(!pWnd)
  {
    CMyDlg* dlg=new CMyDlg;
    dlg->Create(IDD_DIALOG1,this);
    dlg->ShowWindow(SW_SHOW);
  }
  else
  {
    pWnd->BringWindowToTop();
  }
}

//第二步，在所有关闭对话框的地方调用DestroyWindow，如OnOK,OnCancel,OnClose等
void CMyDlg::OnCancel() 
{
  // TODO: Add extra validation here
  DestroyWindow();
  //CDialog::OnOK();
}

//第三步，覆盖对话框的虚函数PostNcDestroy，即在销毁非模态对话框后,释放对话框指针
void CMyDlg::PostNcDestroy() 
{
  // TODO: Add your specialized code here and/or call the base class
  delete this;
  //CDialog::PostNcDestroy();
}

[hestephen]

楼主啊,你总结的有个问题:那就在else
{
pWnd->BringWindowToTop();
}
里如果要得到这个已经创建的对话框的指针该怎么操作呢?

[zhenxisuiyuan]

天剑诗仙 方法最好，牛人！！

[unwritewolf]

[Quote=引用 18 楼 avigdkwynw 的回复:]

1、类成员变量不好销毁吧？
2、全局对象尽量少用吧？
3、还是harry330说得好，每次要创建之前通过FindWindow查找是否已经存在,存在就BringWindowToTop，或者SetForgroundWindow将窗口带到前台窗口。
4、我总结一下：
C/C++ code

//第一步，响应菜单命令，创建与显示对话框。
void CXXView::OnXXX()
{
……
[/Quote]
delete this；要放在在窗口销毁之后

[lhdzwkk4]

可能被n个人搜索这个文章,我来解释下楼主的代码
把static CMyDialog *dlg = new CMyDialog;
定义在类成员变量里,别static就没问题了,
汗一个
其实二楼是正解
最简单的测试代码:
CDLG1 *pdlg1a;
//--------------------------------
pdlg1a=new CDLG1;
if(!IsWindow(pdlg1a->m_hWnd))
{
pdlg1a->Create(IDD_DIALOG1);
pdlg1a->ShowWindow(SW_SHOW);
pdlg1a->DestroyWindow();
//pdlg1a->CenterWindow();
}

[lhdzwkk4]

以上做法错误,
应该是定义static CMyDialog *dlg = new CMyDialog;在cpp文件中,即函数外却不是.h中
然后一切照旧
if(!IsWindow(pdlg1a->m_hWnd))
....

去掉static 则可以打开n个,可以用if(!(FindWindow(0,"XXX"))判断

[bolischou]

这个。。。。你把
if(!IsWindow(dlg->m_hWnd)) //如果m_hwnd存在，就不创建
{
dlg->Create(IDD_DIALOG1,this);
dlg->ShowWindow(SW_SHOW);
}
改成
if(!IsWindow(dlg->m_hWnd)) //如果m_hwnd存在，就不创建
{
dlg->Create(IDD_DIALOG1,this);
}
dlg->ShowWindow(SW_SHOW);
应该就行了吧。。。