for文内でのtry-catchブロック

Question

前提・実現したいこと

フォルダ内にある複数のファイルに関して、
同じ処理を行いたい。
また、例外が発生した場合は処理を抜ける

発生している問題・エラーメッセージ

抽象的で申し訳ないのですが、
for文の中にtry-catchブロックを書くことになんとなく違和感があるので、
以下のソースはjavaのお作法としてどうなのか、
ご教示いただけたらと思ってます。

該当のソースコード

File[] files = new File("フォルダパス").listFiles();

        for (File file : files)
        {
            try (FileInputStream in = new FileInputStream(file))
            {
                // 処理            }
            catch (FileNotFoundException e)
            {
                e.printStackTrace();
            }
            
        }

試したこと

try-with-resourcesではなく、
try-catchブロックを外出しにした方がいいのかで悩んでます。
外部からの呼び出しメソッド内です。

まだ自分の中で違和感がないのは以下です

FileInputStream fileInputStream = null;
        try
        {
            for (int i = 0; i < files.length; i++)
            {
                fileInputStream = new FileInputStream(files[i]);
                
            }
        }
        catch (FileNotFoundException e)
        {
            // 処理
        }
        finally
        {
            if (fileInputStream != null)
            {
                try
                {
                    fileInputStream.close();
                }
                catch (IOException e)
                {
                    // 
                }
            }
        }

以下は通らないとわかっていますが、
これがなんとなくすっきりするというパターンです
※KSwordOfHasteさんの例5から

File[] files = new File("フォルダパス").listFiles();

        for (File file : files)
        {
            try (FileInputStream in = new FileInputStream(file))
            {
                // 処理            }            
        }

補足情報（FW/ツールのバージョンなど）

Java8

Comments

[azuapricot]

ん～。例外が起きても処理を繰り返したいのか、そこで終わりたいのか次第な気がしますけどう～ん。

[stdio]

うむ、時と場合によるけど...
ファイル関連ならありかな？とは思う。

[m.ts10806]

Exception捕捉して何をしたいかによりますね。
文法上間違いでなければ個人的な違和感は関係なく「あり」ですし。

[pontaq]

返信ありがとうございます。
・例外が起きた場合、処理は中断したい
・ファイル読み込み関連
・文法上間違いはない(デバッグで確認済み)
となります。まとめてで申し訳ありません

[pontaq]

すみません、上の文法上間違いを訂正で、
処理上の間違いはない、が正しいです。
Javaの文法に自信がありません・・・

[m.ts10806]

コンパイル通れば文法に間違いはない　ということにはなります。

[Q71]

反対に、どのような書き方なら、違和感がないのでしょうか。
テラテイルを始め、多くの技術系QAサイトは、技術的知見の蓄積こそがメインです。あなたの「違和感がある」というのは、良い情報だと思います。そこに、「こうだと違和感がない」というのがあれば、情報の価値がさらに上がると思います。ご一考ください。

[pontaq]

デバッグしたわけではないのですが、

try
{
for (int i = 0; i < files.length ; i++)
{
FileInputStream in = new FileInputStream(files[i]);
// 処理
}
}
catch (FileNotFoundException e)
{
// 処理
}
finally
{
if (in != null)
{
in.close();
}
}

こういう感じならまだ違和感がないです。

[m.ts10806]

それはそれで、質問本文に追記してください。

[pontaq]

すみません、多分実際はfinallyのif (in != null)でこけると思うので、
またあとでちゃんと考えてから追記します。

[m.ts10806]

スコープ外ですしね。

[Q71]

C#とJavaの「例外」に関しては、このあたりの記事かな。
http://d.hatena.ne.jp/j5ik2o/20091017/1255799827
https://blogs.msdn.microsoft.com/nakama/2008/12/29/net-part-1/

[pontaq]

Q71さん
ありがとうございます！nakamaさんの記事内の
>try-catch ブロックは、例外が発生しうる『1 行』のみを囲む。
これがもしかしたら頭の中にあって、混合していたのかもしれません(try-catch構文とtry-with-resources構文について)。
for文の中というよりは、読み込んだファイルについての処理を複数行にわたり記述することに違和感がよりあったのだと思いました。

何か違和感があるけど、これはJava初心者だからなのか、
そうではないのかがすっきりしなくて、今回質問させていただき、
その回答と自身の理解が得られてとても勉強になりました。
テラテイルの運営方針と違うかもしれませんが、大変助かりました。
今後も勉強続けていきます。

[pontaq]

ここに記載することじゃないとはわかってるのですが、言い訳させてください。
言いたいこと伝えきれずで申し訳ありませんでした。その中で全部の情報を得て答えが得られたことに本当に感謝いたします。

Answer 1

想像ですがC#の書き方と比べて違う感じがするから「違和感」とおっしゃっているのではないでしょうか？

C#の構文は

・例外をハンドリングする構文=>try/catch/finally
・IDisposableなものをスコープから離れたときに自動的にDisposeする構文=>using

に分離されています。一方Javaの構文は

・例外をハンドリングする構文=>try/catch/finally
・AutoClosableなものをスコープから離れたときに自動的にCloseする構文=>try with resource

となりtryに両方の構文機能がmixされているわけです。

そのためC#で

例1:

try {
  for (var filename : ...) {
    using (FileStream fs = new FileStream(filename, FileMode.Read)) {
      ...
    }
  }
} catch (...) {
  ...
}

をそのままJavaに直すには

例2:

try {
  for (var filename : ...) {
    try (FileInputStream fs = new FileInputStream(filename)) {
      ...
    } // (A)
  }
} catch (IOExceptio e) { // (B)ここにcatch節を書けば(A)には一々書かなくてよい
  e.printStackTrace();
}

と書けばよいことになります。しかしusingと違いJavaのtry-with-resourceは例外のハンドリングを行うtry/catch/finally構文の一つのバリエーションなので
例3:

  for (var filename : ...) {
    try (FileInputStream fs = new FileInputStream(filename)) {
      ...
    } catch (...) {
      e.printStackTrace();
      break;
    }
  }

とも書けます。forループのちょうど外側にtry/catchがあるケースでは内側のcatch節でbreakと書けば外側のtry/catchを書かずに済ますこともできます。ループの内側にのみtry/catchを書くか、外側にもtry/catchを書くかはやりたいことを自然に記述できるかどうかで判断すればよいと思います。上の例2/例3はエラーの発生をにぎりつぶす例ですのでcatch節にbreakを書いてますが・・・あまりよい例とは言えません。例外を握りつぶすのは普通やっちゃだめなことですよね。

自分が考える典型的パターンは「途中で何かまずいことがおきたらそのメソッド全体を失敗とする」ことなので、リソースの解放漏れさえなくせれば他に必要な例外処理はないことが多く、外側にtry/catchを書いたりメソッドにthrows句を書くより

例4:

void method() {
  for (var filename : ...) {
    try (FileInputStream fs = new FileInputStream(filename)) {
      ...
    } catch (IOException e) {
      throw new UncheckedIOException(e);
    }
  }
}

こう書くことを好みます。Javaの検査例外(IOExceptionなど)はかならず外側でtry/catchで検査例外をスローしないようにハンドリングするかメソッドにthrows句を書くかどちらかを選択する必要があることに注意してください。Java以外の言語に親しんでいるプログラマーにとって「非常にうっとしいもの」に感じると思います。例4はC#でusingだけを書いた例

例5:

void method() {
  for (var filename : ...) {
    using (FileStream fs = new FileStream(filename, FileMode.Read)) {
      ...
    }
  }
}

と同様の雰囲気のものになります。例4は検査例外IOExceptionを非検査例外UncheckedIOExceptionにすげかえているだけですが、このすげかえが必要ない(必要としない考え方の)言語(C#などJava以外のほとんど全て)で例5のように書くので、例4のような配慮が必要なJavaでは「例外のすげかえがちょっとうっとうしい」と感じる人が多いかも知れません。

Comments

[pontaq]

>>C#の書き方と比べて違う感じがするから「違和感」
このとおりなんだと思います。
また、今回に関しては
>>途中で何かまずいことがおきたらそのメソッド全体を失敗とする
でコードを書いておりました。
例4の提示ありがとうございます。参考にさせてください。

また、他の回答いただけた方々もありがとうございました。

Answer 2

swordoneさんの言う通り、処理が変わってしまうため問題はないです。

処理を見る限りファイル読み込み関連なため、どこのファイルでエラーが出るのかを知りたければ、catchにログ仕込んでおくといいかもしれません。

Answer 3

tryを外出しした場合、1つの処理で例外が発生した時点で、forからも抜けてしまいます。
作法がどうとか以前に、処理の内容が変わってしまうわけですので問題ないです。

Answer 4

※書いたのに投稿しわすれていたので、遅ればせながら……せっかくなので投稿しておきます。

違和感とかなんとかではなく、このコードだと例外発生時はスタックトレース情報を出力するだけで、処理は続行されますよね？なので

• 例外発生後も繰り返しファイル処理を可能な限り続けたい …… Forの中
• 例外発生したら即時ループから飛びだしてファイル処理を終わらせたい …… Forの外

この基準で、どう書かという話だけです。違和感とかより先に「何を作りたいのか」を明確にして、まずはそれをそのままプログラムとして書く。違和感があっても、それで狙った動きをするのであれば「一応は完成」なので、まずは何をやりたいかを明確にして、それが実現できるかどうかを考えましょう。

違和感とかはその先の話です。

Comments

[pontaq]

コード不足申し訳ないです。
実際はcatchの中にreturn null;が入ります。
思った動きはしてくれているのですが、上にも書いてあるとおり、
ただ漠然と違和感がある、と思って質問してみました。
こちらの運用方針からそれる質問でしたら申し訳ありません。

[backyard]

そういうことでしたら、やはり「あなたがそうする理由を述べられるのであれば、違和感を感じる必要は無い」ですね。二つの同等の選択肢があり、どっちを選ぶか、という話であれば好みや違和感で選択を決めますが、ループ内の例外処理を出すか、入れるかはたいていの場合「そうしなければならない理由」とともに決められる物だと思うので、違和感を感じようが感じまいが「書くべきように書く」ということになるかと。

プログラムにしても設計にしてもすべては「そうした理由」があるべきで、作り手はそれをきちんと説明できないと行けません。違和感があるならば、その違和感の原因をきちんと考えるべきかと。少なくとも私は違和感を感じません。

お望みの答えではないかもしれませんが何かの参考になれば。

Answer 5

「for文の中にtry-catchブロックを書くこと」がお作法的に NG ということはないと思います。

質問者さんの「なんとなく違和感」の理由がわからないと何とも言えません。

Comments

[pontaq]

回答ありがとうございます。

C#のusingのつもりでtry-with-resourcesを使用していたのですが
(そもそもこの感覚が間違いならすみません)、
ループ文の中にtry構文を書くことへの違和感としかいえないです。
個人的主観とJava初心者なので、
Javaの有識者からのご意見が欲しく、質問しました