Java ファイル出力ファイル名操作

###前提
何度もすみません。以前（https://teratail.com/questions/36024）で質問させていただいたのですが、先輩からのコードレビューの結果、修正を求められました。

理由としては主に以下でした、
・whileの中でインスタンス生成をしているため、出力フォルダの同名ファイルの枝番の数だけ無駄にメモリを消費することとなる。
・なんだか無限ループにみえる
・出力フォルダの同名ファイルの枝番の数だけループさせるの？

うまく言い返せませんでした。
どのように最適化していけばいいでしょうか。。

###処理仕様
ファイル出力をする際に、対象のフォルダ内に同名ファイルがあれば、
拡張子の前に枝番（例：20160525215810_access_1.log）を付与する。
枝番のついた同名ファイルがあれば枝番を+1して（例：20160525215810_access_2.log）出力する

そのほかの条件、
・ _(ｱﾝﾀﾞｰﾊﾞｰ)と拡張子の間に数字以外の値が入っているファイルは無視
・枝番が飛んでいた場合(例：_1と_3がある)は、抜けている枝番のファイルを作成。
・同名ファイルがなければそのまま出力（例：20160525215810_access.log）

###現在のソースコード

Java
1// 出力ファイルの設定 
2 String outputPath;  //保存場所のパス(パス区切り文字で終わっている）
3String outputFileName;  //保存ファイル名
4 File outputFile = new File(outputPath + outputFileName);
5
6// 拡張子取得 
7 int extensionIndex = outputFileName.lastIndexOf("."); 
8 String extension = outputFileName.substring(extensionIndex); 
9
10//拡張子なしファイル名
11String filename = outputFileName.substring(0, extensionIndex);
12
13// 同名ファイルが存在する場合、出力ファイルに枝番を付与する処理 
14 int branch = 0; 
15 while (outputFile.exists()) { 
16     branch++; 
17     outputFile = new File(outputPath + filename + "_" + branch + extension); 
18 } 
19

退会済みユーザー

2016/05/27 13:48

あなたの疑問に答えるのは、先輩の仕事では？

swordone

2016/05/27 14:57 編集

質問とは直接関係ないですが、これだとoutputFileNameの文字列が拡張子がついたままになり、whileでのファイル名生成時に拡張子のあとに枝番がついて更に拡張子が付く形になってしまいます。あとファイル出力処理がないままになっていますね。

Hyonta

2016/05/28 02:35

ご指摘ありがとうございます。拡張子なしファイル名取得を追加しました。ファイル出力処理は省略しています。

行動規範の内容に同意します

回答2件

あなたの仕事はここで答えを聞いて先輩に伝えることですか？

そもそも自分で考えていないので、言い返せないのは当たり前ですよね。
また、コードレビューの結果に対して言い返すと言う表現を使うこと自体が誤りです。

投稿2016/05/27 14:13

yona

総合スコア18155

Hyonta

2016/05/28 02:26

厳しいお言葉ありがとうございます。できるだけ自分で考えて議論できるようにがんばります。

行動規範の内容に同意します

ベストアンサー

File.exists()を使って判定していましたが、保存ディレクトリのFileオブジェクトに対してlist()でファイル名文字列配列を取得し、枝番付きのファイル名を順次作成し、そのファイル名が配列内にあるかどうかチェックする、という方法もあるかもしれません。ファイル名の配列をArrays.asListでListにすると、入っているかどうかのチェックがcontains()で一発で済みます。
ただし、逐次配列内をループして探索することになるので、ループ数は増えると思います。

その先輩の論に反論するならば…

・whileの中でインスタンス生成をしているため、出力フォルダの同名ファイルの枝番の数だけ無駄にメモリを消費することとなる。

変数が参照するFileオブジェクトが逐次入れ替わるため、古いFileはガベージコレクションの対象になる。むしろ上記の方法の様に存在するファイルを表すインスタンスを生成するという方法は、確実にファイルの数だけインスタンスを生成することになるが、この方法であれば、「そのディレクトリに存在するファイルすべてが入力したファイル名またはその枝番付きで、欠けのない連番になっている」状態の時のみ、ファイル数+1回のインスタンス生成になるが、途中で存在しないファイル名があればそこでインスタンス生成は打ち切られるため、平均的にはインスタンス生成数は少ない。