ファイル削除の効率的な方法

Question

システムで出力しているログを定期的に削除したいんですが、
効率的な削除方法が無いか悩んでいます。

[仕様]
1.システム起動時に、3ヶ月以上前のログファイルを削除する(3ヶ月分は消さずに置いておく)
2.ログファイル名は「aa_xxxxx_yyyymm.log (例：aa_sample_20190701.log)」
(xxxxxは5文字以上の場合あり、yyyymmはシステム起動時の日付)

[今考えている実装]

string[] files = System.IO.Directory.GetFiles(
    @"E:\work\Log", "*", System.IOSearchOption.AllDirectories);

for(int i=0; i<files.length; i++) {
　System.IO.File.Delete(files[i]);
}
　　　：
　(後続の処理)
　　　：

この場合、
1日1ログファイル生成されるとして、最低90ファイル数(1ヶ月30日として)以上あるので、
メモリで一括保持するのはどうかなと思っています。

他にいい方法はありませんか？

Comments

[退会済みユーザー]

ファイルのパスを保持するだけでメモリを圧迫する可能性があるほど大量ということでしょうか？

[k.matsuda]

参考までに
ファイル名にある日付の数字から抽出するのではなく、ファイルの作成日時や更新日時を取得することも出来ます。

Answer 1

100 ほどのファイル名でメモリが圧迫されるようなシステムであれば、ハードの増強が必須でしょう。この程度のメモリの使用は問題にならないと思います。
しかし、もっと大規模な使用を視野に入れて、無駄なメモリを確保したくないというのであれば、GetFiles の代わりに EnumerateFiles を使ってください。こちらは配列を返すのではなく、列挙子を返します。

例えば次のような使い方をすることによって、foreach ループが一回回るたびに該当ファイルが一つ列挙されます。
これを遅延評価といいます。

[雑記] LINQ と遅延評価

using System;
using System.Linq;
using System.Text.RegularExpressions;

namespace ConsoleApp1
{
	class Program
	{
		static void Main(string[] args)
		{
			var regex = new Regex(@"_(\d{4})(\d{2})(\d{2}).log$");
			var threshould = DateTime.Today.AddDays(-90);
			bool ShouldDelete(string path)
			{
				var match = regex.Match(path);
				if (!match.Success) return false;
				var dateOfLogFile = new DateTime(
					int.Parse(match.Groups[1].Value),
					int.Parse(match.Groups[2].Value),
					int.Parse(match.Groups[3].Value));
				return dateOfLogFile < threshould;
			}
			var files = System.IO.Directory
				.EnumerateFiles("path", "*", System.IO.SearchOption.AllDirectories)
				.Where(ShouldDelete);
			foreach (var file in files)
			{
				System.IO.File.Delete(file);
			}
		}
	}
}

Comments

[haruniku]

低評価とさせて頂きます。理由はわかりますよね？

[Zuishin]

わかりませんね。説明してください。

[Zuishin]

通報しておきましたので、運営が見る前に正当な理由を考えておいてください。

[haruniku]

自分の心の中で考えて見てください^ ^

[Zuishin]

わからないので説明してください。あなたが私よりもこの言語について詳しいのですか？

[Zuishin]

まず、このコードが何をしているのか説明できますか？

[Zuishin]

どうしました権左衛門さん？
一行もわからないのに低評価したのですか？
失笑

[f_tonakai]

Zuishinさん
コードまで載せて頂きありがとうございます。
試してみたいと思います
ありがとうございます

[pepperleaf]

この方法、確かに効率的、とも思ったのですが、遅延評価で、ファイル名取得中に、ファイル削除ってどうなのでしょうか?
.Netの内部処理分かりませんが、Unixみたいに、ディレクトリファイルの場合、ファイル名取得中にファイル名を保持しているファイル(ディレクトリファイル)を更新する事になる気がします。

[Zuishin]

私が試した時にはうまく動きました。確認お願いできますか？

[pepperleaf]

ループの中で、ループ変数(オブジェクト)を変更するのに近いと思った次第。
試してみたいですが、すぐは無理そう。
ソースをみれば、早いでしょうか?

[YAmaGNZ]

リファレンスソースをみると、列挙にはFindFirstFile、FindNextFileを利用しているようです。
フォルダ削除などで使いますし、問題はないのかなと思います。

[Zuishin]

確認ありがとうございます。私の方も特に問題となったという情報は見つからなかったのですが、安心しました。

[pepperleaf]

YAmaGNZさん、確認ありがとうございます。
ソースまで見れてないですが、検索してみると、同様にやっている方もあり、問題なさそうですね。(心配したけど、OKってのもあった)

Answer 2

非力なPCなら、ともかく、Windows10がストレスなく使える PCならば、100ファイル程度のリストの取得は全く問題無いと思います。

今、ログファイルが 100を越えたところで、削除していますが、特に問題はありません。 ただ、テストが面倒なだけ。

Comments

[f_tonakai]

スペック的な問題はありません
削除するときには、消せないログファイルが
常に３ヶ月分(90～100程度)あるので、
単純にそれ以上のファイルリストを
作ることになります。
実装として問題ないのか気になって質問させて頂きました

Answer 3

どうしてもメモリ保持したくないという要件があるとして、xxxxのところが分かると仮定すると。

今日の日付から消すべきファイルのファイル名が計算によって求まります。そのファイル名が存在するかを判定して、存在すれば消せばいいと思います。
運用にもよりますが、数日から1ヶ月分の日付のものを消すことにし、ファイルが存在しなかったときにループを中断すればループ回数も抑制できるかと。

Answer 4

バッチなりシェルなりで消すとか

Comments

[f_tonakai]

bat,vbscript,shellscriptではなく
c#で実装になったので、選択肢がありません
shellscriptが楽だと思ってたのですが

[YAmaGNZ]

C#から、バッチファイルなりを作成し、それを実行後消すとかでもダメなんですか？
まぁ、アプリ起動時だけであれば、列挙して消しても問題ないと思いますけどね。
日付がファイル名にあるので、そのフォルダ全てのファイルを列挙する必要もないでしょうし

[f_tonakai]

日付がファイル名にあるので、そのフォルダ全てのファイルを列挙する必要もないでしょうし

とは
どういうことですか？

[YAmaGNZ]

どのようなスパンでアプリを起動するのか分かりませんが、仮に1日1回は起動するというのであれば、2019/07に起動の場合、2019/03が削除対象になるのですから、*201903*.logと列挙するなりフィルタはできるのではないかと思います。
ただ、正解はZuishinさんの回答だと思いますね

[f_tonakai]

>*201903*.logと列挙するなりフィルタはできるのではないかと思います。
なるほどです。
ありがとうございます。