Java例外をスローイントロスペクション

Question

Javaメソッドのイントロスペクション、特に例外に関する質問があります。私は、次のコードを持って言う：

private String getCustomReportResponse(HttpsURLConnection customReportConnection) { 
    int responseCode = 0; 

    try { 
     responseCode = customReportConnection.getResponseCode(); 
     return httpResponseBodyExtractor.extractResponseBodyFrom(customReportConnection); 
    } catch (IOException e) { 
     translateIntoRelevantException(responseCode, e); 
    } 
} 

のは、tryブロックで両方のステートメントを言ってみましょうIOExceptionが投げすることができる - その場合には、translateIntoRelevantExceptionメソッドが呼び出され、次のようになりた：

private void translateIntoRelevantException(int responseCode, IOException e) { 
    if(is5xxResponseCode(responseCode)) { 
     throw new ServerResponseException("Could not generate report - returned response code " + responseCode, e); 
    } 

    throw new ReportException("GeminiReportException: Unable to parse response from HTTP body when requesting custom Gemini report.", e); 
} 

何が起きても、Stringが返されるか、例外がスローされます。 getCustomReportResponseを除いて、メソッドはcatchブロックの後にreturnステートメントを追加せずにコンパイルされません。これは絶対に到達できません。実際、もし私がtranslateIntoRelevantExceptionの内容をキャッチブロックの中に入れたら、それはコンパイルされます。

例外をスローする例外は実行時例外ですが、私は例外をチェックするようにも試みましたが、問題は残っています。

誰かが理由を説明できますか？

Answer 1

これは、 "rethrow"ヘルパーメソッドに直面するcommon problemです。

translateIntoRelevantExceptionというメソッドが返されないことをコンパイラは認識していません（また、表示する方法もありません）。

したがって、tryブロックの後に続くコードパスがあると考えます。ですから、return null（またはthrow new RuntimeException("should never come here")「デッドコード」に配置する必要があります。

あなたはtryブロックの後にそれを置く必要はありません、あなたはcatchの内側にそれを置くことができます。

try { 
    responseCode = customReportConnection.getResponseCode(); 
    return httpResponseBodyExtractor.extractResponseBodyFrom(customReportConnection); 
} catch (IOException e) { 
    translateIntoRelevantException(responseCode, e); 
    throw new RuntimeException("should never come here"); 
} 

がそれですおそらくきれいの実装に依存すべきではありません。そして、あなたは

throw translateIntoRelevantException(responseCode, e); 

Answer 2

コンパイルgetCustomReportResponseの操作を行うことができ、それを投げるだけreturn例外の代わりにヘルパーを持っています複数の理由のため：

• translateIntoRelevantExceptionの実装が使用できない場合があります（それは別のライブラリで、別のクラスにすることができます）。
• translateIntoRelevantExceptionの変更は、すべての呼び出しメソッドを壊す可能性があります。代替案として

あなたは例外を返し、その後、クライアントコードでそれを投げることができます。

private IOException translateIntoRelevantException(int responseCode, IOException e) { 
    if(is5xxResponseCode(responseCode)) { 
     return new ServerResponseException("Could not generate report - returned response code " + responseCode, e); 
    } 

    return new ReportException("GeminiReportException: Unable to parse response from HTTP body when requesting custom Gemini report.", e); 
} 

そして、このようにそれを呼び出す：答えるため

throw translateIntoRelevantException(responseCode, e);