ウィンドウを動かすとメモリ使用量が急激に増え、プログラムが停止してしまう。

Question

.bat形式のあるデータ（球体の半径など）を読み取り、タイマー関数を用いてビットマップ形式で描画しようとしています。ビルドも無事にでき、実行したのですが、ウィンドウを素早く動かしたり一定時間経ってから動かすとプログラムが停止します。そこで、メモリの使用量をみながら実行したのですが、やはりウィンドウを動かすとメモリ使用量が急激に大きくなり、プログラムが停止していることがわかりました。この改善策として何かアドバイスをいただきたいです。開発環境はVisual Studio C++です。まだまだ初心者で理解不足や説明不足があるかもしれませんが、ご了承ください。よろしくお願いいたします。

Comments

[TaroToyotomi]

ソースコードを掲示すると回答が付きやすくなりますよ。

[TRON1216.]

どこのコードを見せればよいのか検討がつかなくて、、、。

[yumetodo]

迷ったときはすべてgithubとかにあげることをおすすめします。(公開して良いコードなら)。まあとりあえずメモリーリークとか無限ループとかその辺を探してみてはどうでしょう

Answer 1

これって
ハンドルされない例外、書き込み中にアクセス違反。タイマー関数を使いビットマップ形式で球を描画したいのだが、描画途中で描画されなくなる
で質問された同じプログラムですよね？

であれば、そちらの方に書きましたが、malloc()での確保の仕方（確保場所）がまずいです。

Comments

[TRON1216.]

その通りです。では、どこに確保すべきなのでしょうか？また、どうしてまずいのか教えていただけると助かります汗よろしくお願いいたします。

Answer 2

こんにちは。

メモリ使用量が急激に大きくなり、プログラムが停止していることがわかりました。

メモリのとりすぎ、もしくは、メモリ解放忘れのどちらかと思います。
メモリをどの程度使用するのか、見積もってみましょう。それより遥かに多くのメモリをだんだんと消費するようならメモリ解放忘れです。獲得したメモリをどこで開放しているのか確認してみましょう。

見積もったメモリ使用量がPCに搭載しているメモリ量に比べて多いようなら、仮想メモリか物理メモリを増やす、もしくは、メモリ使用量を減らすようプログラムを工夫するなどを試みましょう。

ところで、「.bat形式のあるデータ（球体の半径など）」って珍しいですね。拡張子batのファイルは大抵バッチ・ファイルです。シェルスクリプトの一種でコマンド・プロンプトで実行します。

Comments

[TRON1216.]

丁寧な回答ありがとうございます。ウィンドウを動かすことと、メモリの使用量は関係ないということでしょうか、？また、メモリの解放のタイミングについてなのですが、この質問の一つ前の質問（いまもなお受付中）で載せているコードについて、解放のタイミングは正しいかどうかよければ確認していただきたいです、、。

拡張子は.datなんですけど中身はcsv形式で書かれているデータです。珍しいのかもわからずやってます（笑）

[Chironian]

> ウィンドウを動かすことと、メモリの使用量は関係ないということでしょうか、？

私はそのようなことは書いていないですよ。どこからその結論を導かれたのか不思議です。

> 解放のタイミングは正しいかどうか

WndProc_bitmapの全体が記載されていませんので断定できませんが、症状からするとWndProc_bitmap内で開放のタイミングが間違っている可能性は非常に高いです。（なお、ここにあったら他にはないということにはならないのでご用心。バグはいくらでも作れます。もちろん私には他にもあるかどうかは分かりません。ご自身で確認するしかないです。）

さて、WndProc_bitmapについて「獲得したメモリをどこで開放しているのか確認してみましょう。」
見ても良く分からないようでしたら、WndProc_bitmapの頭にブレークポイントを張ってWndProc_bitmap内をステップ実行してみましょう。WndProc_bitmapを抜けたらGoして良いです。辛抱強く頑張れば分かる筈ですよ。

[TRON1216.]

ウィンドウを動かすとどうしてメモリを食うのかわかりましたら教えていただきたいです。それとも様々な要因が重なってその現象が起きるのでしょうか？また、解放はWM_PAINT内だけだと足りないのですね汗ブレークポイントを張ってステップ実行してみます。そういう方法すら知りませんでした。丁寧にありがとうございます。助かります。

[Chironian]

> ウィンドウを動かすとどうしてメモリを食うのかわかりましたら教えていただきたいです。

ウィンドウを動かすと通常はそのウィンドウへWM_MOVEメッセージが通知されます。後は分かるのでは？
他にも様々なメッセージが投げられますよ。

[TRON1216.]

アドバイスありがとうございます。直したことは２つで、g_imgとそのメンバーの初期化をWM_PAINT内に書きました。また、各WMメッセージにfree(g_img.lpBmpData)を書きました。これで実行すると、ウィンドウを動かしてもメモリ使用量は増えず、無事最後のデータまで描画できました。