可能な（チェックされていない）すべての例外を適切に処理していますか？

Question

が

現在、私のサインアップは次のようになります。最初のために

public void signup(User newUser) throws Exception { 
    log.info("Sign up: " + newUser.getEmail()); 
    if (restService.emailAlreadyExists(newUser.getEmail())) { 
     throw new Exception("Email already in use."); 
    } 
    List<Role> roles = new ArrayList<Role>(); 
    roles = roleRepository.findAllOrderedByName(); 
    roles.add(roleRepository.findByName("user")); 
    newUser.setRoles(roles); 
    newUser.setPassword(restService.getHashedValue(newUser.getPassword())); 
    try { 
     em.persist(newUser); 
    } catch (Exception e) { 
     throw new Exception("Just noobs use apps with less bugs. Try again. Now!"); 
    } 
    log.info(newUser.toString()); 
    userEvent.fire(newUser); 
} 

私は、ユーザーのために（FacesMessageになります）2つのメッセージにだけ興味があります。ユーザーの他の隠れたメッセージを防ぐために、私はさらにtryブロックを役割まで拡張する必要があります。

これは悪い習慣だと思います。また、一般的なExceptionのにおいを使用して、彼らは言う。しかし：

• IllegalStateException
• IllegalArgumentException
• EntityExistsException
• TransactionRequiredException
• ObserverException

でも8については話していない：私は、このコードの小片でException秒を記録し、次の検出（！）Exception s方法のgetSingleResult()はjavax.persistence.TypedQueryである。

この例ではすべてExceptionを実際に処理する必要がありますか、またはいくつかをスキップしてもかまいません（および/または上記のような一般的なExceptionを使用することもできます）。

Answer 1

ベストプラクティスは、すべての例外を個別にキャッチし、できるだけカスタムログメッセージを作成することです。障害のある状況を理解し、それに従って行動する方が簡単です。

典型的なビジネスアプリケーションの現実は、あなたの例外をグループ化しようとします（例えば、あなたのメソッド入力によって引き起こされる可能性のある例外すべてをグループ化し、dbによってスローされるすべての例外をグループ化してください）。悪名高いcatch (Exception e)か、一般的なメッセージで一般的な例外を再開してください。そうすれば、人々は象のように成長しているログについてあなたに電話し始めるでしょう。