「ファイルを保存するコモンダイアログ」と「クラス」

質問

c++を勉強しています。APIを使っているのですが、ファイル保存コモンダイアログの表示は出来ているのですが、ちょっと疑問点があります。どなたかご教授ください。よろしくお願いいたします。
分かりにくかったらごめんなさい。

とりあえず必要かなと思うところだけ編集して、ソースを張っています。

Save関数の中で
static　で宣言するとよいのですが、static無ではダイアログが動きません。

なぜstaticが必要なのですか？
Save関数の中に入ると、クラスを作り、抜けると破棄するというようにと思っていたのですが
static　を付けてしまうと、ずっと保持されてしまいますよね？
どうすれば、よいのでしょうか？

開発環境
vista
visual studio 2010 c++

/********************/
class FNameDlg　　//コモンダイアログのためのもの
{
　　public:
　　　　int SaveFile( winproc proc );　　//ダイアログを開く
　　　　void ofnSet( winproc proc, wchar_t *Filter, wchar_t *Title, wchar_t *Ext );
private:
　　　　TCHAR m_strName_full[MAX_PATH];
　　　　TCHAR m_strName[MAX_PATH];
　　　OPENFILENAME m_ofn;
};

/************/
class WaveSave 　： public FNameDlg　//wavファイルを保存するため
{
　　public:
　　　　void SaveMain( winproc proc, Audio_Format hWave, Save cSave );
　　private:
　　　　void WriteFile(LPTSTR lpszFileName, LPWAVEFORMATEX lpwf, LPBYTE lpWaveData, DWORD dwDataSize);

};

/*********/
void WaveSave：：SaveMain( winproc proc, Audio_Format hWave, Save cSave )
{
　　　wchar_t *FilterW = _T("wav files( *.wav )\0*.wav\0All files(*.*)\0.*.*\0\0");
　　　wchar_t *TitleW = _T("名前を付けて保存");
　　　wchar_t *ExtW = _T("wav");

　　　ofnSet( proc, FilterW, TitleW, ExtW );　　//OPENFILENAMEに値を入れます
　　　if( SaveFile( proc ) == -1 )　//ここでGetSaveFileName( &m_ofn )を実行しています。
　　　　　　return ;　　　　　　　　　　//staticを付けてないと返されてしまいます。

　　　WriteFile( GetFull(), &hWave.m_Wf, cSave.m_pbSave, cSave.m_iDataSize );
}

/*************/
void Save(　//　省略します　// )
{
　　　static WaveSave cWSave; //staticを付けるとよい　ないとだめ　

　　　cWSave.SaveMain( proc, hWave, cSave );

}

/*************/
LRESULT CALLBACK WindowProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam)
{
　　　　//　　省略します　　//

　　　　Save(　//　省略します　// )；

　　　return DefWindowProc(hwnd, uMsg, wParam, lParam);
}

回答No.1

noname#164673 · ベストアンサー

肝心のSaveFileの中身が無いので、ハッキリとは言えませんが、変数の初期化が絡んでいる様な気がします。

プログラムを拝見すると、FNameDlgのメンバ変数の初期化コードがありません。
通常は、FNameDlgのコンストラクタで初期化するのですが、コンストラクタが省略されているようです。

staticでクラスを宣言すると、静的領域にインスタンスが生成されます。
静的領域は、プログラム起動時にnullで初期化されるのが普通です。

一方、auto変数としてインスタンスを構築すると、スタック領域にインスタンスが配置されます。

スタック領域は、他の関数や割り込み処理が上書きしては戻すので、グチャグチャな状態です。
つまり何が書かれているか分からない初期化が必要な領域です。

static を記述した場合は、キレイなメモリ状態から動作し、static 無しつまりauto変数の場合は、グチャグチャなメモリ状態からプログラムが起動します。

それが原因では無いでしょうか？

解決策は、FNameDlgのコンストラクタでメンバ変数を初期化する事だと思います。

割愛されたプログラムを拝見した、想像した回答です。
間違っているかもしれませんので、ご注意下さい。

お礼

回答ありがとうございます。

確かに、コンストラクターで初期化を行うことで、static宣言なしでダイアログが表示されました。

＞staticでクラスを宣言すると、静的領域にインスタンスが生成されます。
＞静的領域は、プログラム起動時にnullで初期化されるのが普通です。

なるほど、静的領域はNULLで初期化されるのですね。だから、staticで宣言しないときは初期化が必要で、ダイアログが動かなかったと！！
大変勉強になりました。ありがとうございました。