メモリ

質問

これを見て直感的にどこがおかしいか分かる人いませんか？
VC++6.0のMFCです。
ViewクラスとDialogクラスの２つあります
(1)
1回目の処理

ファイルオープンダイアログを出現させ開くを押した場合のみ次に進む(Viewクラス)
↓
構造体を作ってメモリを取得し、ファイルから構造体に値を入れる(Viewクラス)
↓
Dialogクラスにその構造体の一部の値を渡しダイアログを出現させ
値を表示する。(作図してOKなら) okボタンを押しokボタンを
押したときのみ次の処理(作図)。(Dialogクラス)
↓
Viewクラスに戻り作図(Viewクラス)
↓
viewクラスのデストラクトにて構造体のメモリ解放(Viewクラス)
↓
終了

２回目も同じ処理

結果：エラーなし

(2)
ファイルオープンダイアログを出現させ開くを押した場合のみ次に進む(Viewクラス)
↓
構造体を作ってメモリを取得し、ファイルから構造体に値を入れる(Viewクラス)
↓
Dialogクラスにその構造体の一部の値を渡しダイアログを出現させ
値を表示する。(作図してOKなら) okボタンを押しokボタンを
押したときのみ次の処理(作図)。(Dialogクラス)
↓
Viewクラスに戻り作図(Viewクラス)
↓
viewクラスのデストラクトにて構造体のメモリ解放(Viewクラス)
↓
終了

２回目はファイルオープンダイアログにて開くを押さない

結果：エラー

(3)
１回目２回目ともにファイルオープンエラにて開くを押さない

エラーなし

(4)
エラーとなるのは(2)ですがデストラクタにて
メモリを開放しなければエラーはでませんでした。

どこがおかしいのか分かる人いれば教えて下さい。

回答No.1

chie65536（@chie65535） · ベストアンサー

＞２回目はファイルオープンダイアログにて開くを押さない
＞結果：エラー

予想１
ダイアログで「開く」を押してない時も、Viewクラスのオブジェクトをコンストラクトしてしまっていて、コンストラクトしただけで何もしてない。

そして、デスクトラクタに来た時に「１回目で使い終わってメモリを開放しちゃってる構造体を、メモリ確保してないのに再度メモリ開放しようとして、エラーになっている。

予想２
２回とも「開く」を押さなかったとき、構造体を示すポインタ変数は「初期化時にNULLか何かになっている」ので、デスクトラクタに来た時にポインタ変数がNULLなので、何もエラーが起きない。

原因予想
「開くを押したかどうか？」を判断する前に、Viewクラスのオブジェクトをコンストラクトしているのが原因。

予想される修正方法
「開くを押したかどうか？」を判断する前は、一切の変数、オブジェクトをコンストラクトしないようにする。

駄目な例
関数(～～～)
{
　ViewClass View; //ここでコンストラクトすると…
　構造体の変数 = NULL;
　オープンダイアログ表示
　if (開いたを押したか？) {
　　構造体をメモリ確保
　　(中略)
　}
//この位置で、つまり関数を抜ける時にデクストラクタが呼ばれ、確保してない構造体を開放しようとしてしまう。
}

良い例
関数(～～～)
{
　構造体の変数 = NULL;
　オープンダイアログ表示
　if (開いたを押したか？) {
　　ViewClass View; //ここでコンストラクトすると…
　　構造体をメモリ確保
　　(中略)
　　//この位置、if文を抜ける時にデクストラクタが呼ばれ、構造体を確保した時だけしか開放しない。
　}
}

で、デクストラクタでは
　if (構造体の変数 != NULL) {
　　構造体を開放
　　構造体の変数 = NULL;
　}
と書いて「NULL以外の時だけ開放する」「開放直後は必ずNULLにする（開放した後のゴミの値を残さない）」ようにしておく。

もちろん「最初に１回しか通らない所で、必ずNULLで初期化しておく」のも忘れずに。

補足

勉強になりました。ありがとうございました。