finallyを利用してファイルを閉じる

Question

前提・実現したいこと

csvファイルを読み込むプログラムです。
例外発生時にはメッセージが表示されるようにしたいです。

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

finallyを利用して、どんな場合でもクローズ処理が行われるようにしたかったですが、
finallyを追加したら、メッセージが表示されなくなりました。

「処理されない例外の型 FileNotFoundException」
「処理されない例外の型 IOException」

というエラーメッセージが出てきます。
例外処理と、finallyを利用したクローズ処理を同時にしたいですが、
どのように修正したらいいかがわかりませんでした。
ご教授お願いします。

該当のソースコード

		File csv = new File(PATH);
		FileReader fr = new FileReader(csv);
		BufferedReader br = new BufferedReader(fr);

		try {
			String line;
			while ((line = br.readLine()) != null){
				String[] info = line.split(",", -1);

				for (String output : info){
					System.out.print("[" + output + "]");
				}
				System.out.println();
			}
		}catch (FileNotFoundException ex){
			System.out.print("ファイルが存在しません。");

		}catch (IOException ex){
			System.out.print("ファイルの読み込み中にエラーが発生しました。");

		}catch (Exception ex){
			System.out.print("システムエラー");

		}finally{
			br.close();
			fr.close();
		}
	}
}

Comments

[m.ts10806]

「初心者アイコン」をつけておいてください。
できれば質問タイトルや本文は要件を書くことに終始してもらいたいので
「初心者です」などのあいさつはなくていいかなと個人的には思ってます。

[firetruck]

初心者アイコンがあることを知りませんでした。ありがとうございます。

Answer 1

close()がIOExceptionを出す可能性があるため、これをfinallyの中でtry-catchで囲う必要があります。

        }finally{
            try {
                br.close();
                fr.close();
            }
        }

これでもbr.close()に失敗したらfr.close()が実行できないので、別々に囲うべきでしょうが…

Java7以降であれば、try-with-resources構文を使うとclose処理を明記せずに済みます。

Comments

[Y.H.]

FileNotFoundExceptionについては、FileReaderの変数宣言とインスタンス生成を分離して、インスタンス生成をtry句のブロック内に移動する必要もありますね。（引きずられる形で、BufferedReader も同様）

Answer 2

Java SE7以降であればtry-with-resources 文を使ってください。

try-with-resources文はtryまたはcatchブロックを抜けた時点でtry()の括弧内に書いたリソースを解放してくれます。そのため、わざわざfinallyブロックを用意してclose()を呼び出す必要がなくなります。

        File csv = new File(PATH);

        try　(
            FileReader fr = new FileReader(csv);
            BufferedReader br = new BufferedReader(fr);
        ) {
            String line;
            while ((line = br.readLine()) != null){
                String[] info = line.split(",", -1);

                for (String output : info){
                    System.out.print("[" + output + "]");
                }
                System.out.println();
            }
        }catch (FileNotFoundException ex){
            System.out.print("ファイルが存在しません。");

        }catch (IOException ex){
            System.out.print("ファイルの読み込み中にエラーが発生しました。");

        }catch (Exception ex){
            System.out.print("システムエラー");

        }
    }
}

もしtry-with-resources文が使えない場合はfinallyブロックの中でtry-catch文を書き、そこでbr.close();やfr.close();を呼び出してください。