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

Question

### 前提・実現したいこと

フォルダ内にある複数のファイルに関して、
同じ処理を行いたい。
また、例外が発生した場合は処理を抜ける


### 発生している問題・エラーメッセージ

抽象的で申し訳ないのですが、
for文の中にtry-catchブロックを書くことになんとなく違和感があるので、
以下のソースはjavaのお作法としてどうなのか、
ご教示いただけたらと思ってます。

### 該当のソースコード

```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ブロックを外出しにした方がいいのかで悩んでます。
外部からの呼び出しメソッド内です。

まだ自分の中で違和感がないのは以下です
```Java
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から
```Java
File[] files = new File("フォルダパス").listFiles();

        for (File file : files)
        {
            try (FileInputStream in = new FileInputStream(file))
            {
                // 処理            }            
        }
```

### 補足情報（FW/ツールのバージョンなど）

Java8

Accepted Answer

想像ですがC#の書き方と比べて違う感じがするから「違和感」とおっしゃっているのではないでしょうか？

C#の構文は

・例外をハンドリングする構文=>try/catch/finally
・IDisposableなものをスコープから離れたときに自動的にDisposeする構文=>using

に分離されています。一方Javaの構文は

・例外をハンドリングする構文=>try/catch/finally
・AutoClosableなものをスコープから離れたときに自動的にCloseする構文=>try with resource

となりtryに両方の構文機能がmixされているわけです。

そのためC#で

例1:
```C#
try {
  for (var filename : ...) {
    using (FileStream fs = new FileStream(filename, FileMode.Read)) {
      ...
    }
  }
} catch (...) {
  ...
}
```
をそのままJavaに直すには

例2:
```java
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:
```java
  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:
```java
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:
```java
void method() {
  for (var filename : ...) {
    using (FileStream fs = new FileStream(filename, FileMode.Read)) {
      ...
    }
  }
}
```

と同様の雰囲気のものになります。例4は検査例外IOExceptionを非検査例外UncheckedIOExceptionにすげかえているだけですが、このすげかえが必要ない(必要としない考え方の)言語(C#などJava以外のほとんど全て)で例5のように書くので、例4のような配慮が必要なJavaでは「例外のすげかえがちょっとうっとうしい」と感じる人が多いかも知れません。

Answer

※書いたのに投稿しわすれていたので、遅ればせながら……せっかくなので投稿しておきます。

違和感とかなんとかではなく、このコードだと例外発生時はスタックトレース情報を出力するだけで、処理は続行されますよね？なので

- 例外発生後も繰り返しファイル処理を可能な限り続けたい …… Forの中
- 例外発生したら即時ループから飛びだしてファイル処理を終わらせたい …… Forの外

この基準で、どう書かという話だけです。違和感とかより先に「何を作りたいのか」を明確にして、まずはそれをそのままプログラムとして書く。違和感があっても、それで狙った動きをするのであれば「一応は完成」なので、まずは何をやりたいかを明確にして、それが実現できるかどうかを考えましょう。

違和感とかはその先の話です。

Answer

swordoneさんの言う通り、処理が変わってしまうため問題はないです。

処理を見る限りファイル読み込み関連なため、どこのファイルでエラーが出るのかを知りたければ、catchにログ仕込んでおくといいかもしれません。

Answer

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

Answer

「for文の中にtry-catchブロックを書くこと」がお作法的に NG ということはないと思います。

質問者さんの「なんとなく違和感」の理由がわからないと何とも言えません。

前提・実現したいこと

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

該当のソースコード

試したこと

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

関連した質問