■MFC相談室 mfc19d.dll■

■MFC相談室 mfc19d.d ..

697:デフォルトの名無しさん
08/11/29 17:15:04
>>693 >>694
モードレスの場合はダイアログの作成と破棄のタイミングが違うのでメンバに抱えてnewを使う

698:デフォルトの名無しさん
08/11/29 18:14:42
>>697
モードレスとモーダルを同じダイアログで使いまわす状況なんてないんじゃね？

699:デフォルトの名無しさん
08/11/29 18:30:38
何れにしても、わざわざnew/deleteして使わなくてもいいだろ。
ダイアログが閉じられたからといって、
クラスのインスタンスを即座に破棄しなければならない理由は無い。
別々の派生クラスを時と場合に応じて使い分ける、とかいうなら話は解るけど。

700:デフォルトの名無しさん
08/11/29 20:37:50
>>699
じゃあモードレスの場合にダイアログが作成されてるか破棄されてるかはどうやって判別すんの？

701:デフォルトの名無しさん
08/11/29 21:03:49
画面に線だけを書きたいときは領域を塗りつぶしてから書いたほうがいいですか？それともなんかコントロールを置いてそれに書いたほうがいいですか？

702:デフォルトの名無しさん
08/11/29 21:08:11
領域を塗りつぶしてから書いたほうがいい

703:デフォルトの名無しさん
08/11/29 21:17:29
>>700
用語が曖昧だからなんとも言えないけれど、モードレスダイアログが実際に表示されているのかされていないのか、
そのダイアログ自身以外に一体全体誰が気にしなければいけないというの?
そのダイアログのインスタンスを持っている側にとっては、出ていて欲しいときに出せと要求するだけで充分でしょ。

704:デフォルトの名無しさん
08/11/29 21:17:48
塗りつぶして線を引いた、領域と同じサイズの絵を描画した方がいい

705:デフォルトの名無しさん
08/11/29 21:18:19
>>701
用途と状況と目的によりけりかと。

706:!699
08/11/29 21:22:14
>>700
質問元のモードレスダイアログは、処理中ダイアログみたいだから、
判別する必要ねえんじゃね？

ユーザーによるオペレーションで表示/非表示するようなやつの場合、
俺は（複数表示する必要なかったので）シングルトン的なものにして対応したんで、
作成されている/いないを判別するは必要なかった。

707:706
08/11/29 21:24:28
リロードせずにレスした。　反省した。　次はリロードする。

708:デフォルトの名無しさん
08/11/29 22:06:13
>>706
処理中ダイアログって考えはなかったわ
でも１ハンドラ内に>>691の記述でモードレス中ダイアログを表示できたっけ？

709:デフォルトの名無しさん
08/11/29 22:07:10
>>703
表示/非表示じゃなくダイアログの作成/破棄な　CDialog::Create()とCWnd::DestroyWindow()
インスタンスの生成/破棄(new/delete)とも別だよ

710:デフォルトの名無しさん
08/11/29 22:20:51
>>708
PeekMessageでもはさむんじゃね？

711:デフォルトの名無しさん
08/11/29 22:48:54
>>709
事情は変わらんだろ。ダイアログ自身が管理していれば、利用者はそれを意識する必要はない。

712:デフォルトの名無しさん
08/11/29 22:52:27
俺はそもそも
モードレスとモーダルをいっしょのクラスで分岐させる無意味な汎用性と
Createされてる状況でCreate関数を再び呼ぶ必要を考慮してるクラスの仕様のが
気持ち悪くていまの話題についていけないけどね

713:デフォルトの名無しさん
08/11/29 22:56:54
いっそのこと新しいC++向けのMS製フレームワークを提供してほしいわ
.NETは相性悪すぎる

せっかくDirect2DとDirectWriteもC++向けに提供するんだし、アプリケーションハンガリアンでエレガントなクラス群を是非

714:デフォルトの名無しさん
08/11/29 23:01:17
>>711
モードレスダイアログのサンプル書いてみてよ

715:デフォルトの名無しさん
08/11/29 23:03:47
>>714
CWaitDlg wait;
wait.Create(this);
// 処理
wait.DestroyWindow();

716:デフォルトの名無しさん
08/11/29 23:14:20
>>715
処理中ダイアログじゃなくてOKボタン押すまで表示されてるモードレスダイアログのサンプル書いてみてよ

717:デフォルトの名無しさん
08/11/29 23:15:52
>>716
意味がわかんない

718:デフォルトの名無しさん
08/11/29 23:18:19
>>716
CWaitDlg wait;
wait.Create(this);

while(1){
PeekMessage
GetMessage
TranslateMessage
DispatchMessage

if(ボタンが押されたら)break;

//処理

}

wait.DestroyWindow();

719:デフォルトの名無しさん
08/11/29 23:32:06
>>718
モードレスのサンプルになってないよ
その用途ならモーダル使うのが普通じゃん

720:デフォルトの名無しさん
08/11/29 23:36:43
>>719
プログレスバーが表示できるじゃん
質問元が思いっきりそういう意図じゃん

普通だったらモードレスなんてつかわねぇよ
正直いってバグりやすいもん
大部分がこんな感じで済むんだよ

721:706
08/11/29 23:49:45
>>708
1ハンドラでの処理は考えてなかった。　てか忘れてた。
new/deleteの話いらないって話なんだから、1ハンドラだね。
そういうわけで、>>706はなかったことに。
（>>710の通りやればできそうだけど、確認したくないという事情によります）

まったく持ってごめん。　反省した。　次は、、次も、、とり頭だからやってしまうかもしれん。

722:デフォルトの名無しさん
08/11/29 23:58:54
>>718を>>711のようにするにはどうすればいいの？

723:デフォルトの名無しさん
08/11/30 00:00:13
>>722
何がやりたいのか具体的にいいたまへ

724:デフォルトの名無しさん
08/11/30 00:13:23
>>716

どっかのクラス
{
　　CWaitDlg m_wait;
};

どっかのクラス::モーダレス表示するハンドラ()
{
　　wait.Create(AfxGetMainWnd());
}

これで、親クラスが消滅するかダイアログの「×」ボタン「OK」または
「Cancel」ボタンがクリックされるまで表示されたままになる。

>>719
モードレスだが？

>>722
スレッドを起こせばいい。

725:デフォルトの名無しさん
08/11/30 00:15:04
>>724
>　　wait.Create(AfxGetMainWnd()); ... 誤

　　m_wait.Create(AfxGetMainWnd()); ... 正

726:デフォルトの名無しさん
08/11/30 00:30:19
>>723
ダイアログ自身が管理する形で書かれていないので
CWaitDlgが管理するようにしたいんだけど

727:デフォルトの名無しさん
08/11/30 00:38:36
>>724
m_wait.Create(AfxGetMainWnd());が２回呼ばれたらマズくない？
m_wait.DestroyWindow();は必要ないの？

728:デフォルトの名無しさん
08/11/30 00:39:43
>>726
何を管理する？ちゃんと日本語で書けよバカSEよ。それとも低級な魔法
使い気取りか？

729:デフォルトの名無しさん
08/11/30 00:51:56
>>727
> m_wait.Create(AfxGetMainWnd());が２回呼ばれたらマズくない？

概念の説明だけでなく、エラー処理まで手取り足取り見なきゃならん
のか？

if(m_wait.m_hWnd=NULL)
　　m_wait.Create(AfxGetMainWnd());

とでもしておけばいい。

> m_wait.DestroyWindow();は必要ないの？

MFCの場合、明示的に呼ばなくても、m_waitのデストラクタ内で
DestroyWindow()が呼び出される仕様。

CFile::Close()等も同様。

730:デフォルトの名無しさん
08/11/30 01:12:45
>>729
呼ばれなくね？
とりあえず俺の環境では

Warning: calling DestroyWindow in CDialog::~CDialog --
OnDestroy or PostNcDestroy in derived class will not be called.

って警告がでちゃうよ

731:デフォルトの名無しさん
08/11/30 01:27:56
だよね。質問元の>>684にもそう書いてあるし

732:デフォルトの名無しさん
08/11/30 02:18:51
>>730-731

MFCのソースも読めないし、トレースの仕方も知らんと恥を晒している
ようだな。　まさに宝の持ち腐れ。　豚に真珠ってヤツだ。

その警告メッセージは、MFC(デバッグ版ライブラリ)が出している。

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(); // ← ここで呼んでるだろうが、ボケ!!!
　　}
}

ついでに、英語も理解できないおまえらに、低学歴の漏れが警告メッ
セージの意味を解説してやると、

『テメェが忘れてるから、デストラクタ内（CDialog::~CDialog()）で
ケツ拭きでDestroyWindow()呼んでやるが、今からじゃ手遅れで派生
クラスのOnDestroy()やPostNcDestroy()は呼ばれねぇけど、間違っても
Windowsのバグとか、ゲイツ様のせいにすんじゃねぇぞ、ゴルァ！』

ってこった。

733:デフォルトの名無しさん
08/11/30 02:49:47
そんなことは、皆了解済みの上で話していると思うのだがどうか。
わざわざトレース文が入っているようなお情け処理を、
正式な仕様として扱う勇気は俺には無いな。

734:デフォルトの名無しさん
08/11/30 05:34:25
>>688 >>716
今までの話を整理したらこうなった

void CPropertyDialogSampleView::OnEditProperty()
{
　　m_propertyDlg.Show();
}

CModelessDialogBase::~CModelessDialogBase()
{
　　if (CWnd::GetSafeHwnd() != NULL)
　　　　CWnd::DestroyWindow();
}

void CModelessDialogBase::Show(CWnd* pParentWnd = NULL)
{
　　if (CWnd::GetSafeHwnd() == NULL)
　　　　CDialog::Create(GetTemplateID(), pParentWnd);
　　CWnd::ShowWindow(SW_SHOW);
　　CWnd::BringWindowToTop();
}

void CModelessDialogBase::OnOK()
{
　　CWnd::ShowWindow(SW_HIDE);
}

void CModelessDialogBase::OnCancel()
{
　　CWnd::ShowWindow(SW_HIDE);
}

735:デフォルトの名無しさん
08/11/30 07:02:30
>>733
でもそこなんでトレースいれるんだろうね
少なくとも俺の感覚としちゃそこでDestroyWindowを呼ぶのは当然の処理で
警告なんて出す意味がわからないんだけど

736:デフォルトの名無しさん
08/11/30 07:11:07
CreateとDestroyWindowが対だからじゃね？
コンストラクタでCreateしないからデストラクタでもDestroyWindowしない方針

737:デフォルトの名無しさん
08/11/30 08:00:16
>>736
しろよな
しない方針にすることでなにかいいことあるのか？って感じ

Create関数のヘルプにもそんなこと書いてねーし
DestroyWindow関数のヘルプには
MFC側で呼んだり呼ばなかったりするからユーザー側でうまくやれよマジで
とかかなりむかつく仕様だしで作った奴マジ嫌な感じだな

738:732
08/11/30 10:06:48
>>733
> そんなことは、皆了解済みの上で話していると思うのだがどうか。

理解したり、(実際に呼ばれていることを)知っていたら『呼ばれなくね？』
なんて発言は、マヂでありえなくね。(w

C++の基本を理解し、警告メッセージの意味を正しく理解していれば、

> わざわざトレース文が入っているようなお情け処理を、
> 正式な仕様として扱う勇気は俺には無いな。

なんて発言も出てこないハズ。

739:732
08/11/30 10:08:09
>>735
CDialogクラスのデストラクタで呼び出しているので、(この時点では
派生クラスのメッセージマップや仮想関数テーブルを参照できない、
もしくは存在自体を知りえないため)派生クラスのOnDestroy()やPost
NcDestroy()は呼ばれないだけ。　当然ながら、CDialog::OnDestroy()
やCDialog::PostNcDestroy()の呼び出しは行われる。

派生クラスのOnDestroy()やPostNcDestroy()で独自の実装(例:閉じた
時のウィンドウサイズを取得してレジストリやINIファイルに保存する
等)を呼び出していなければ、デストラクタに処理を任せてもなんら
問題ない。

だからこそ「Error:」じゃなく、あくまで「Warning:」ってなってる。

CFileクラスのデストラクタや、CGdiObjectクラス(CPen,CBrush等の基本
クラス)のデストラクタでも同様のことをやっているけど、これらはメッセ
ージループを廻す必要がないので、TRACE記述はない。

740:732
08/11/30 10:13:43
>>736-737
こういう文句を言う連中は、MFCはおろかC++の基本的な仕様(派生クラスと
基本クラスのコンストラクタ/デストラクタが呼び出される順序や、仮想
関数など)さえロクに理解していないんだろうナァ。

方針なんて関係ない。強いて言えばデザインパターンってやつか？

コンストラクタでデフォルト値を代入してメンバ変数の初期化忘れを防ぐ
とか、デストラクタでメモリやハンドルの解放忘れを防ぐというのは、
基本中の基本。

インスタンスの消滅で確実にウィンドウを破棄し、なおかつ派生クラス
のOnDestroy()やPostNcDestroy()も呼び出されるようにしたければ、MFC
内部の実装に関係なく、派生クラスのオブジェクトが消滅するタイミング
でDestroyWindow()を呼び出せばいいだけ。

すなわち、自分が作るCDialog派生クラスで、デストラクタ関数を定義し、
m_hWndが有効ならDestroyWindow()を呼び出してやればいい。

class CMyDialog : public CDialog
{
　　virtual ~CMyDialog();
};

CMyDialog::~CMyDialog()
{
　　if(m_hWnd!=NULL)
　　　　DestroyWindow();
}

ただそれだけのことだ。

741:デフォルトの名無しさん
08/11/30 10:14:42
>>738
俺とそいつは別人だ
ちなみに「よばれてなくね？」は俺

>>739
警告の必要なくね？

742:デフォルトの名無しさん
08/11/30 10:18:33
>>740
でもヘルプでだってどういうときに
DestroyWindowを呼ばなきゃいけないのか書いてないわけだし
当然MFCの中身の都合なんてこっちはしったこっちゃないわけで
もし、自分でこういうもんを作るとしたら解放はフレームワークのほうで自動でやってほしいかな？

743:デフォルトの名無しさん
08/11/30 11:15:41
>>740のコードがまずいというのもわかりにくいしな

744:デフォルトの名無しさん
08/11/30 15:54:00
何この流れ
どうせお互いまったく譲り合わないんだから、これ以上書いても無駄だよ

745:デフォルトの名無しさん
08/11/30 18:51:38
Win7ってペイントとかワードパッドにリボン採用してるよな
今後作るソフトはリボンで作った方が惹きやすいのだろうか・・・2008SP1は持ってるが