![](http://oshiete.xgoo.jp/images/v2/pc/qa/question_title.png?e8efa67)
C++にあるCopyFile関数でフォルダをコピーする関数を作成しました。
コピー関数を使うプログラムは常駐プログラムです。
Aフォルダに決められたファイルが作成されたら、A1フォルダを作成してそのなかにAフォルダのファイルをコピーしていきます。
A1の次はA2、A3…というふうにどんどんファイルをコピーしていくのですが、メモリが増えていくだけで、減りません。
このままいくとメモリリークとなってしまうと思うのですが、なぜ増えて行くのでしょうか?
下がプログラムです。
int FoldersCopy(CString from, CString to)
{
HANDLE hSearch;
WIN32_FIND_DATA fd;
CString FindPath;//検索用のパス
FindPath = from;//コピー
if (FindPath.Right(1) != L"\\")
{
FindPath += L"\\";
}
FindPath += L"*.*";
hSearch = FindFirstFile(FindPath, &fd);
while(TRUE)
{
if (!FindNextFile(hSearch, &fd))
{
break;
}
if (fd.cFileName != L"." || fd.cFileName != L"..")
{
CString fFilepath, tFilepath;
fFilepath = from + fd.cFileName;
tFilepath = to + fd.cFileName;
CopyFile(fFilepath, tFilepath, TRUE);
fFilepath.Empty();
tFilepath.Empty();
}
}
FindClose(hSearch);
return 0;
}
解放はしっかりやっていると思うのですが、何かいけない場所はありますか?
そもそも常駐プログラムでファイルコピーをしたらダメなのでしょうか?
よろしくお願い致します。
A 回答 (4件)
- 最新から表示
- 回答順に表示
No.4
- 回答日時:
直接の回答ではないですけどメモリリークはメモリ不足の意味ではないですよ。
その原因の一つではありますけど。また「メモリが増えていくだけで減りません」ということですがタスクマネージャなどで確認されたのだと思いますが、一般的にOS上で動作するC,C++のライブラリではOSに確保してもらったメモリはプログラムが終了するまでOSには返さず再利用に備えるようになってます。
No.3
- 回答日時:
No.2さんが指摘されているようにデバッカで確認したほうがいいと思います。
仮にメモリーリークしていてたとしてもこの関数じゃないないかも知れません。
あと質問とは関係ないですが、
このコードだと最初にFindFirstFileでヒットしたファイルが無視されて
FindNextFileでヒットする二番目以降のファイルしかコピーされない気が…
わざとだったらごめんなさい。
No.2
- 回答日時:
残るなら既にメモリリークなわけだが・・・。
説明がないけどVisualC++でMFCとWin32APIですよね?
多分この関数は問題無いです。
で、デバッグ起動での終了時にリークしているならリーク情報が出力ウィンドウに表示されるはずですが。
No.1
- 回答日時:
ランタイムなどは、確保したメモリが不要になっても、すぐに返却せずに再利用する場合がありますから、際限なく増えるのでなければ、問題ないように思います。
コード上で確保しているメモリは、それほど多くはないと思うのですが。
CopyFileはWIN32APIですよね。ここでメモリが使われているのであれば、アプリ側の問題ではないでしょう。試にCopyFileをコメントにしてみて、使用メモリが増え続けるか確認してみては?
CopyFileがメモリを消費しているのであれば、SHFileOperationを使ってみるとか。
あと、本題とは関係ありませんが、
> if (fd.cFileName != L"." || fd.cFileName != L"..")
これ、おかしくないですか?
そもそも、ポインタの比較になっていて文字列の比較をしていないようですし、ORで結ぶと常にtrueじゃないかと思います。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
関連するカテゴリからQ&Aを探す
おすすめ情報
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
VBAの配列サイズとメモリに関して
-
【C言語】再帰が時間がかかる...
-
ファイルマッピング関数で失敗
-
エクセルのメモリ使用状況/Appl...
-
メモリのセグメント違反の解決...
-
「memcpy」と「strcpy」について
-
mmap()
-
Apacheでバーチャルホストの最...
-
C言語で、メモリを解放しないで...
-
ReadEventLogについて
-
C言語 昇順・降順 ソート
-
メモリ不足
-
HEWでのビルド
-
H8について教えて下さい。
-
C言語における再帰呼び出しの...
-
プログラマのためのPC選び(パ...
-
fortranプログラムについて
-
デバッグツール,Efenceでのmma...
-
メモリアロケーション異常の発...
-
C#におけるexeファイルのサイズ...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
C言語で、メモリを解放しないで...
-
VBAの配列サイズとメモリに関して
-
これて逆じゃないですか?
-
【C言語】再帰が時間がかかる...
-
エクセルのメモリ使用状況/Appl...
-
メモリのセグメント違反の解決...
-
エクセルVBA 大容量CSVファイル...
-
メモリ不足
-
メモリが不足しています(VBA)
-
C言語における再帰呼び出しの...
-
「ヒープサイズの設定」て何?
-
エクセル キャッシュメモリー...
-
EXCEL-VBAにてADOのレコードセ...
-
バッチファイルでの実行EXEのメ...
-
closeとメモリの開放について
-
ExcelのVBAでメモリ解放できない
-
C言語初心者です。debug assert...
-
動的メモリとexit(C言語)
-
大容量のメモリ確保をスワップ...
-
メモリの解放の仕方
おすすめ情報