VisualC++2008ExpressEditionにてプログラミングしています。
C++のconstメソッド(関数)の基本的な働きについては理解しているつもりです。
次のようなキャッシュされたテクスチャを取り出すクラスメソッドがありまして、メインループの中でテクスチャを表示したいときに呼び出されている、とします。
const LPDIRECT3DTEXTURE9 Renderer::getCachedTexture(const string& name) {
if (_cachedTextures.find(name) != _cachedTextures.end()) {
return _cachedTextures[name];
}
return NULL;
}
このRenderクラスはメインループからは参照経由で、texture = render.getCachedTexture(name)のような形で利用されています。
実はこのままのメソッドで利用するとなぜかメモリリークするのですが、const LPDIRECT3DTEXTURE9 Renderer::getCachedTexture(const string& name) const; のようにconstメソッドにするとリークしないことを確認しました。_cachedTexturesはPocoというライブラリの Poco::HashMapクラスなんですが、簡単にいうとstd::map<string,LPDIRECT3DTEXTURE9>インターフェースです。ちなみに、constの有無に関わらずメソッドの実装はかえずともコンパイルでき、実行できます。変わるのはconstが無いとリークする、有るとしない、ということだけなのですが、constの有無で実装を変えることができるのでしょうか?(_cachedTexturesの find()メソッドの実装内容が変わる?)
ちなみに、_CrtSetDbgFlag()を使ってアプリ起動から終了までのダンプを取ったりしてみましたが、const有り無しでは変わらず、いずれも全体ではリーク無しで終了できます。つまり、アプリを長期間実行したままにしておくと溜まってしまう状況で、それがこのメソッドのconst無しのときだけ発生するので、どのようなメカニズムで生じているのか、知りたいと考えています。尚、メモリリークについてはメインループにて、1ループ毎に GetProcessMemoryInfo()のPROCESS_MEMORY_COUNTERS.WorkingSetSizeにて確認しています。
No.3ベストアンサー
- 回答日時:
とりあえず Poco のソースを拾ってきてちょろっと読んでみましたが, 少なくとも今のものについてはリークしていないようにも見えます. まあ, 私の目は節穴だったりするのでよく見落としますが. でも, 意味的には「find でリークする」ことは考えにくいんだよね.... 簡単にできる (かもしれない) こととしては, このあたりでしこたまログを吐くくらい?
いずれにしても
return _cachedTextures[name];
は無駄っぽいので, その前の _cachedTextures.find(name) で得られるイテレータを覚えておくとよいでしょう.
あと, 「const の有無」は「シグネチャの違い」と認識していいと思います.
C++ では「引数の数やその型」が違えば同名の関数を 1つの名前空間で同時に定義することができるのはご存じだと思います. この「引数やその型」を「シグネチャ」と呼びます. つまり, 「シグネチャが違えば同じ名前の関数を定義することができる」わけです.
で, (static でない) メンバ関数には「どのオブジェクトを通しで実行されたか」という情報も this ポインタとして与えられるわけですが, これは実質的に「関数に対する引数」として作用します. そして, 「const と宣言されたかどうか」によって this ポインタの型が変わります (クラス T のメンバ関数において, const なメンバ関数なら this の型は const T *, const でなければ T *). したがって, 「const かどうか」は「シグネチャの違い」とみなすことができるので「const なメンバ関数と const でないメンバ関数」は同時に定義できます. もちろんその実装も変えることができます (が意味は同じであることが普通... というか, 意味まで違うと使う人が混乱する).
なお, シグネチャには「返り値の型」は含まれないので, 「返り値の型だけが違う」関数は宣言できません.
この回答への補足
私も調べてみました。どうやらoperator[]でのアクセスですとリークするようです。ですので、_cachedTextures[name]を_cachedTextures.find(name)->secondにしたところ、リークしなくなりました。find()などはTacosanのおっしゃるとおり、特に気になるところはなかったので、Tacosanの回答のような実装にしておこうと思います。Poco::HashMapのoperator[]もやはりconst有り無しがあり、const版の方は特に気にならなかったのですが、無し版の方はちょっと気になる感じだったので、もうちょっと調べてみます。
補足日時:2009/07/27 13:28No.4
- 回答日時:
確認しました. 確かにバグです.
ただし, operator [] のバグではありません. ここで使っている手法は
「sentinel (番人, 番兵などと訳す)」というもので, これを使って「要素がな
かったときに追加する」というコストを削減しています (実際にはあまり減りま
せんが).
内容を完全に理解したわけではないので具体的に「ここ」とは言いきれないので
すが, おそらくバグがあるのは HashMap::opeartor [] の内部で呼び出している
LinearHashTable::insert です.
LinearHashTable::insert は最初に LinearHashTable::split を呼び出していま
すが, その結果として「insert を呼ぶたびにハッシュ表が (1つずつ) 大きくな
る」ことになります. ところが, 実際に insert で要素が追加されることはない
(と思っていい) ので, 結果として「表が無駄に大きくなっている」といえます.
これを「リーク」と呼んでよいかどうかはちょっと微妙.
回答ありがとうございます。私もそこかなと思いまして、質問のconstの有無が直接影響してはないな、ということもわかりましたので、締め切らせていただきました。ご協力ありがとうございました。
参考までに、このメソッドは1/60秒当たり3~5回呼ばれているので、1秒で30回ほど呼ばれることになります。実際に実行していて、1分弱で1MBくらい増えていて、数時間で落ちてしまっていました。24時間365日動作させたいアプリケーションなので、1バイトでも無駄にしたくなかったもので。
いずれにせよ、明確に原因がわかりましたので、完全な対処ができました。ありがとうございました。
No.2
- 回答日時:
>constの有無で実装を変えることができるのでしょうか?
「実装を変えることができるかどうか?」と言う質問の答えは「出来る」です。
以下の話は「コンパイラに依存し、コンパイル環境により異なる話」としてお読み下さい。
1つのクラスに「同じメソッド名のメソッド」が複数存在出来るのはご存知ですよね?
例えば
void func1(void);
void func1(int arg1);
int func1(int arg1,String arg2);
とか、戻り値や引数の個数が違うなら、同名のメソッドを定義できます。
これは「ユーザーの目に触れない、内部的な関数名が異なる」からこそ、可能なのです。
あるコンパイラでは、上記の3つのメソッドの「関数の実体の名前」は、以下のようになります。
・ソースでの表記
void func1(void);
void func1(int arg1);
int func1(int arg1,String arg2);
↓
・関数の実体の名前
ClassName.VSD.func1@V
ClassName.VSD.func1@I4
ClassName.I4SD.func1@I4SU
このように「戻り値や引数の個数が違う同名のメソッドを定義すると、関数の実体の名前が異なってコンパイルされる」のです。
これは
int func2(void);
const int func2(void);
のように「constの有無」でも同じで「constが有るのと無いのでは、関数の実体の名前が異なる」のです。
言い換えれば「関数の実体の名前が異なるので、constが有るのと無いのが、同時に存在出来る」と言う事であり、つまり「constの有無で実装を変えることができる」と言う事です。
丁寧な説明ありがとうございます。
実際にPoco::HashMapも確認してみたところ、find()やend()もそれぞれconst有り無しの実装がありました。
No.1
- 回答日時:
えぇと....
「メモリ使用量の増加」は「メモリリーク」に直結しないんだけど, その辺は理解できていますでしょうか?
あと, その _cachedTextures というのは Renderer のメンバーなんでしょうか? また, Poco::HashMap クラスには const な find と const でない find の両方があったりしませんか? もしそうなら, Renderer::getCachedTexture(const string& name) が const かどうかで呼び出す find が違うので, そこでなにかがあるのかもしれません.
この回答への補足
2~3時間するとメモリを使い切ってヒープが壊れてる、みたいなダイアログが出てアプリが落ちてしまうので、メモリリークと判断しています。
_cachedTextures は Renderer のprivateなメンバーです。また、確かに、Poco:HashMapにはconstのfind()/end()とそうでないfind()/end()がありました。試しに存在チェックのところをなくしていきなり_cachedTextures[name]を返すようにしてみましたが、それでも同様にconstを外すとメモリが増えていくようです。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- C言語・C++・C# C++プログラミングコードにポリモーフィズムを取り入れ方を教えてください。 2 2023/06/09 11:17
- JavaScript Javascriptが機能せず原因が分からないので教えて頂きたいです 3 2023/06/04 14:50
- JavaScript sessionStorageを調べています。 1 2023/06/20 12:41
- Excel(エクセル) VBAの指示の内容 昨日こちらでご教示頂いたのですが初心者な為、一つ一つの指示が何をやっているのかわ 2 2022/10/25 18:08
- JavaScript フォームが空欄の時にフォームの外をクリックすると、エラーが出るコードを調べています。 1 2023/06/25 11:51
- C言語・C++・C# const char** p;のとき、free(p)でC4090エラーとなるのはなぜですか 3 2023/03/31 16:28
- HTML・CSS ボタンをクリックした時に、入力フォームのすぐ下部に、「入力欄が空白です」というテキストメッセージが表 1 2022/04/27 16:25
- Excel(エクセル) エクセル VBA For Next 繰り返しの書き方を教えてください 6 2022/09/01 14:11
- JavaScript clear機能を失わずにファイルアップロード機能を作成したい 3 2023/06/10 16:12
- Visual Basic(VBA) モードレスでユーザーフォームが開け(表示)ません。 4 2022/09/09 11:05
関連するカテゴリからQ&Aを探す
おすすめ情報
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
レコード件数の表示
-
Strutsでチェックボックスの値...
-
abstract と static を一緒に付...
-
Google Apps Script で getRang...
-
public static void main (Stri...
-
なぜprotected overrideなのか
-
String.containsの反対機能はあ...
-
別クラスのmainメソッドの実行
-
Jakarta POI での、excelのシ...
-
Java初級 引数に適用できません
-
親の親のメソッドを呼ぶには?
-
VB.Net: クラスの共有メソッド...
-
C# でメソッドに送られてきたOb...
-
JDBC Connectionを開放し続ける...
-
不要なstaticメソッドの見分け...
-
JUnitのテストメソッドごとのメ...
-
メソッド/ルーチン
-
メソッド宣言の戻り値の型にク...
-
メソッドにより複数の情報を得...
-
C# 点の描き方をおしえてくだ...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
Java初級 引数に適用できません
-
abstract と static を一緒に付...
-
なぜprotected overrideなのか
-
C# 点の描き方をおしえてくだ...
-
レコード件数の表示
-
String.containsの反対機能はあ...
-
Google Apps Script で getRang...
-
StringBufferからStringへキャ...
-
public static void main (Stri...
-
YYYYMMDD書式の日付に対する適...
-
クラスを作るとメソッドの数が...
-
C# でメソッドに送られてきたOb...
-
C# の ByVal と ByRef について
-
Fileの読み取り専用の解除
-
メソッド宣言の戻り値の型にク...
-
Strutsでチェックボックスの値...
-
メソッドの引数に指定されてい...
-
別クラスのmainメソッドの実行
-
contextってなんですか?
-
javaに"search"という関数 or ...
おすすめ情報