ダイアログのＯＫボタンを押してウィンドウを生成する方法は？

質問

ダイアログのＯＫボタンを押すとウィンドウが生成する方法を教えてください。
MFCでプログラミングしてメインウィンドウをダイアロボックスにしています

//アプリケーション
class CMyApp : public CWinApp
{
public:
virtual BOOL InitInstance();
};

//メインダイアログ
class CMyDlg : public CDialog
{
public:
CMyDlg(CWnd *pParentWnd=NULL);
protected:
virtual void OnOK();
virtual void OnCancel();
};

#include <afxwin.h>
#include <afxdlgs.h>
#include <string.h>

#include "dlg.h"
#include "resource.h"

CMyApp myApp;

BOOL CMyApp::InitInstance(){

m_pMainWnd = new CMyDlg;
m_pMainWnd ->ShowWindow(m_nCmdShow);
m_pMainWnd ->UpdateWindow();
return TRUE;
}

CMyDlg::CMyDlg(CWnd* pParent /*=NULL*/)
{
Create("MYDLG");
}
としています。
ダイアログはできるのですが、ここからウィンドウを生成する方法を教えてください

回答No.3

machongola · ベストアンサー

こんばんは。補足頂きました。

　CMyDlgクラスのコンストラクタで、m_pMyWndをNULL初期化してください。
　恐らく、未初期化でポインタに不正値が入り込んでいる為、条件判定に失敗するのだと思います。

CMyDlg::CMyDlg(CWnd* pParent /*=NULL*/)
: m_pMyWnd(NULL)//イニシャライザでポインタをNULLに初期化
{
Create("MYDLG");
}

お礼

どうもありがとうございます
: m_pMyWnd(NULL)//イニシャライザでポインタをNULLに初期化
をつけたとたん、ウィンドウが開きました。感動でした。
ただ、closeとキャンセルはウィンドウズエラーが発生します
そこらへんは自分で研究したいと思いますのですが、なにか気になるところがあるようでしたら、ご意見いただきたいと思います
ありがとうございました。

回答No.2

machongola

補足頂きました。

>>”//CMyDlgのメンバに追加”というのがこれでいいのかよくわかりません。
　其処は合っています。ただ、まだ追加すべき事は残っています。

///////////////////////////////////////////////////
// dlg.h

//アプリケーション
class CMyApp : public CWinApp
{
public:
virtual BOOL InitInstance();
};

//ウィンドウ
class CMyWnd : public CFrameWnd
{
public:
protected:
DECLARE_MESSAGE_MAP()
//(1)virtual void OnClose();此れを追加
};

//メインダイアログ
class CMyDlg : public CDialog
{
public:
CMyDlg(CWnd *pParentWnd=NULL);
CMyWnd* m_pMyWnd;//此れは合っている。ただ、余程の理由が無い限り、直接アクセス出来ない様に、privateにした方が良い
//(2)void DoneFromMyWnd();此れを追加
protected:
//(3)void Done();此れを追加
virtual void OnOK();
virtual void OnCancel();
//(4)virtual void OnClose();此れを追加
};

///////////////////////////////////////////////////
// Test.cpp

#include <afxwin.h>
#include <afxdlgs.h>
#include <string.h>

#include "dlg.h"
#include "resource.h"

CMyApp myApp;

BOOL CMyApp::InitInstance(){

m_pMainWnd = new CMyDlg;
m_pMainWnd ->ShowWindow(m_nCmdShow);
m_pMainWnd ->UpdateWindow();
return TRUE;
}

BEGIN_MESSAGE_MAP(CMyWnd, CFrameWnd)
END_MESSAGE_MAP()

//(1)CMyWndが「×」で閉じられる時
void CMyWnd::OnClose()
{
//親ダイアログを取る
CMyDlg* pMyDlg = static_cast<CMyDlg*>(this->GetParent());

//親ダイアログに終了する事を伝える
pMyDlg->DoneFromMyWnd();

CFrameWnd::OnClose();
}

CMyDlg::CMyDlg(CWnd* pParent /*=NULL*/)
{
Create("MYDLG");
}

//(2)CMyWndが一方的に終了した際に送り返される
void CMyDlg::DoneFromWyWnd()
{
m_pMyWnd = 0;
}

//(3)此方の意思で終了させる
void CMyDlg::Done()
{
if(m_pMyWnd)
{
m_pMyWnd->DestroyWindow();
m_pMyWnd = 0;
}
}

//ココは合ってる
void CMyDlg::OnOK()
{

if(!m_pMyWnd)
{
const RECT rc = {10, 10, 320, 240};
m_pMyWnd = new CMyWnd();
m_pMyWnd->Create(NULL, "MyWnd", WS_OVERLAPPEDWINDOW, rc, this);
m_pMyWnd->ShowWindow(SW_SHOW);
}

}

void CMyDlg::OnCancel()
{
//×DestroyWindow();此れだと自分が閉じてしまう
Done();
}

//(4)CMyDlgが「×」で閉じられる時
void CMyDlg::OnClose()
{
Done();
CMyDlg::OnClose();
}

補足

たびたびどうもすみません。ご指摘とても参考になりました。ただコンパイルは通るのですが、動作がうまくいきません。ウィンドウはOKで開かず、キャンセルやクローズもウィンドウズエラーが出てしまいます。たびたびすみませんが、参考意見御願いします

///////////////////////////////////////////////////
// dlg.h

//アプリケーション
class CMyApp : public CWinApp
{
public:
virtual BOOL InitInstance();
};

//ウィンドウ
class CMyWnd : public CFrameWnd
{
public:
protected:
virtual void OnClose();
DECLARE_MESSAGE_MAP()
};

//メインダイアログ
class CMyDlg : public CDialog
{
private:
CMyWnd* m_pMyWnd;
public:
CMyDlg(CWnd *pParentWnd=NULL);
void DoneFromMyWnd();
protected:
void Done();
virtual void OnOK();
virtual void OnCancel();
virtual void OnClose();
};

///////////////////////////////////////////////////
// Test.cpp

#include <afxwin.h>
#include <afxdlgs.h>
#include <string.h>

#include "dlg.h"
#include "resource.h"

CMyApp myApp;

BOOL CMyApp::InitInstance(){

m_pMainWnd = new CMyDlg;
m_pMainWnd ->ShowWindow(m_nCmdShow);
m_pMainWnd ->UpdateWindow();
return TRUE;
}

BEGIN_MESSAGE_MAP(CMyWnd, CFrameWnd)
END_MESSAGE_MAP()

void CMyWnd::OnClose()
{
//親ダイアログを取る
CMyDlg* pMyDlg = static_cast<CMyDlg*>(this->GetParent());

//親ダイアログに終了する事を伝える
pMyDlg->DoneFromMyWnd();

CFrameWnd::OnClose();
}

CMyDlg::CMyDlg(CWnd* pParent /*=NULL*/)
{
Create("MYDLG");
}

void CMyDlg::DoneFromMyWnd()
{
m_pMyWnd = 0;
}

void CMyDlg::Done()
{
if(m_pMyWnd)
{
m_pMyWnd->DestroyWindow();
m_pMyWnd = 0;
}
}

void CMyDlg::OnOK()
{

if(!m_pMyWnd)
{
const RECT rc = {10, 10, 320, 240};
m_pMyWnd = new CMyWnd();
m_pMyWnd->Create(NULL, "MyWnd", WS_OVERLAPPEDWINDOW, rc, this);
m_pMyWnd->ShowWindow(SW_SHOW);
}

}

void CMyDlg::OnCancel()
{
Done();
}

void CMyDlg::OnClose()
{
Done();
CMyDlg::OnClose();
}

回答No.1

machongola

こんにちは。
　取り敢えず、CFrameWndを継承してCMyWndを作成して、そのポインタをCMyDlgが持つようにすれば良いのでは？
　OnOK()「OKボタン」が呼ばれたらCMyWndを開いて、その他何かのアクションでCMyWndを閉じ、CMyWndが自ら一方的に終了する際には、向こうから、CMyDlgに向かって通知させれば良さそうです。
　大体こんな感じではないのでしょうか？

//CMyDlgのメンバに追加
CMyWnd* m_pMyWnd;

//OKボタンが押された
void CMyDlg::OnOK()
{
// TODO: この位置にその他の検証用のコードを追加してください
//適当に開く
if(!m_pMyWnd)
{
const RECT rc = {10, 10, 320, 240};
m_pMyWnd = new CMyWnd();
m_pMyWnd->Create(NULL, "MyWnd", WS_OVERLAPPEDWINDOW, rc, this);
m_pMyWnd->ShowWindow(SW_SHOW);
}
}

//此方の意思で終了させる
void CMyDlg::Done()
{
if(m_pMyWnd)
{
m_pMyWnd->DestroyWindow();
m_pMyWnd = 0;
}
}

//CMyWndが一方的に終了した際に送り返される
void CMyDlg::DoneFromWyWnd()
{
m_pMyWnd = 0;
}

/////////////////////////////////////////////////////////////////////////////
// CMyWnd メッセージ ハンドラ

void CMyWnd::OnClose()
{
// TODO: この位置にメッセージ ハンドラ用のコードを追加するかまたはデフォルトの処理を呼び出してください
//親ダイアログを取る
CMyDlg* pMyDlg = static_cast<CMyDlg*>(this->GetParent());

//親ダイアログに終了する事を伝える
pMyDlg->DoneFromMyWnd();

CFrameWnd::OnClose();
}

お礼

回答どうもありがとうございます
まだ勉強不足で、”//CMyDlgのメンバに追加”というのがこれでいいのか
よくわかりません。下記のプログラムでうまく動作しません。

///////////////////////////////////////////////////
// dlg.h

//アプリケーション
class CMyApp : public CWinApp
{
public:
virtual BOOL InitInstance();
};

//ウィンドウ
class CMyWnd : public CFrameWnd
{
public:
protected:
DECLARE_MESSAGE_MAP()
};

//メインダイアログ
class CMyDlg : public CDialog
{
public:
CMyDlg(CWnd *pParentWnd=NULL);
CMyWnd* m_pMyWnd;
protected:
virtual void OnOK();
virtual void OnCancel();
};

///////////////////////////////////////////////////
// Test.cpp

#include <afxwin.h>
#include <afxdlgs.h>
#include <string.h>

#include "dlg.h"
#include "resource.h"

CMyApp myApp;

BOOL CMyApp::InitInstance(){

m_pMainWnd = new CMyDlg;
m_pMainWnd ->ShowWindow(m_nCmdShow);
m_pMainWnd ->UpdateWindow();
return TRUE;
}

BEGIN_MESSAGE_MAP(CMyWnd, CFrameWnd)
END_MESSAGE_MAP()

CMyDlg::CMyDlg(CWnd* pParent /*=NULL*/)
{
Create("MYDLG");
}

void CMyDlg::OnOK()
{

if(!m_pMyWnd)
{
const RECT rc = {10, 10, 320, 240};
m_pMyWnd = new CMyWnd();
m_pMyWnd->Create(NULL, "MyWnd", WS_OVERLAPPEDWINDOW, rc, this);
m_pMyWnd->ShowWindow(SW_SHOW);
}

}

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

どうか間違いを教えてください