ファイル読込における例外処理

Question

以下のようなコードを書いたら、コードレビューで例外処理に問題があるので書き直すように指示されました。
どこに問題があるのかわかりません。

    BufferedReader reader = null;
    try {
        reader = new BufferedReader(new FileReader(file));
        String line;
        while ((line = reader.readLine()) != null) {
            // ...
        }
    } catch (IOException e) {
        logger.severe(e.getMessage());
    } finally {
        try {
            reader.close();
        } catch (Exception e) {
        }
    }

Comments

[maisumakun]

読むべきfileは、どのように指定される場面でしょうか？

[退会済みユーザー]

他のメソッドから引数で渡されます。

Answer 1

try-with-resource を使え
reader.closeでヌルポ
例外メッセはわかるが発生箇所がわからん（例外オブジェを渡してスタック表示できん？）
ログレベルがシステム停止を指すと思われるのに処理が続行されている

詳細化（第一弾)

BufferedReader reader = null;
try {
	reader = new BufferedReader(new FileReader(file));
	String line;
	while ((line = reader.readLine()) != null) {
	    // ...
	}
} catch (IOException e) {
	// exit を行わないので warn レベル以下とすべきである
	// 例外情報は ログフォーマットにて制御させるため渡す必要あり
	// ここ以外は多分許容されただけだと思われる。
	// また ログレベルに関しては適切なものを選んでください。
	log(Level.WARNING, e.getMessage(), e); 
} finally {
	try {
		// NullPointerException は発生しても握りつぶすのではなく
		// 発生しないように対策をする。（例外コスト削減）
		if (reader != null) reader.close();
	} catch (IOException e) { // 無視する例外は IOException  だけと明示する
		// nop // 処理を入れてないことを明示する
	}
}

あたりになるでしょう。
また [try-with-resource ] をりようすると

try (BufferedReader reader = new BufferedReader(new FileReader(file))) {
	String line;
	while ((line = reader.readLine()) != null) {
	    // ...
	}
} catch (IOException e) {
	log(Level.WARNING, e.getMessage(), e); 
}

で上記コードとほぼ同一になります。

Comments

[退会済みユーザー]

try-resourceって何ですか？　ヌルポとは？　スタック表示とは何のことでしょうか？　処理が続行されている問題は理解しました。

[退会済みユーザー]

リーダーが読み失敗しても closeするので ヌルポ
（キャッチしてるからいいとおもってるようだが例外処理コストが無駄

[退会済みユーザー]

try resourceもといtry whit resource

https://docs.oracle.com/javase/jp/7/technotes/guides/language/try-with-resources.html

[退会済みユーザー]

スタック情報=通常 printStacktrace で出力されるクラス情報

スタック情報をトレースし出力するから上記メソッド

[退会済みユーザー]

log4jでいうところの第二引数に例外オブジェ

[KSwordOfHaste]

> ヌルポとは？

NullPointerExceptionの日本での俗称です。英語だと略称としてNPEがよく使われる気がします。