ユーザセッションの期限が切れたとき、またはnullを返すときに例外をスローする

Question

この質問は、設計とクリーンコードの詳細です。チェックされたexpectionがスローされるべきとき、またはnull値を返してチェックする必要があるときこの値がnullの場合はクライアント上で

例外的な状況が発生する可能性がありますが、私たちはそれを防ぐことはできません。また私は、制御フローとして例外を使用すべきではないことを読んだ。

私の場合、多くのユーザーがリクエストを行い、ユーザーセッションが期限切れになっているかどうかを確認する方法があります（そして10分後に期限が切れます）。私はこの状況を防ぐことができないので、私はそこで例外をスローする必要がありますが、その状況が10分ごとに発生する場合、反対側では例外が高価ですし、nullを返して、クライアントはパフォーマンスの点で優れています。この場合、あなたは何をしますか？

これは私の方法である：

@Override 
public Integer getUserIdBySessionKey(String sessionKey) { 
    User user = userSessions.get(sessionKey); 
    boolean isUserSessionValid = isUserSessionValid(user); 
    return isUserSessionValid ? user.getUserId() : null; 
} 

Answer 1

あなたは、呼び出し元のコードに考慮に入れなければならない重要な例外的な状況を通知したいときは、チェック例外を使用する必要があり、そしてそのために意味のあるアクションが可能撮影。たとえば、ユーザーを認証する場合、パスワードが間違っているか、アカウントが期限切れになっているため、確認済みのAuthenticationExceptionまたはAccountExpiredExceptionが妥当である可能性があります。クライアントコードはこれらの例外をキャッチし、それらを適切に処理します。

一方、妥当な措置が取られていない場合や、一般的な方法で処理できるほど一般的である場合は、チェックされていない例外があります。すべてのコーディングエラー（NullPointerException、IllegalStateException、データベースエラー、ネットワークエラーなど）はこのカテゴリに分類されます。

問題は、呼び出しコードがセッションの期限切れについて何かすることを期待していますか？このような例外を捕捉してログインページにリダイレクトするこのような状況（たとえばトップレベルのServletFilterなど）を処理するための一般的なメカニズムがある場合は、未確認の例外にします。これに呼び出しコードから特に注意が必要な場合は、チェックします。

あなたの場合、私はチェックを外し、トップレベルで一般的な方法で処理します。