Javaの8オプションあまりにも多くのorElses

Question

に対処する方法のは、ラムダなしの例を見てみましょう：

Credentials credentials = CredentialService.get(id); 
if (credentials != null && credentials.isActive()) { 
    User user = UserService.get(credentials.getUserId()); 
    if (user != null) 
     return Status.ok(user); 
} 
return Status.bad(); 

あなたが見ることができるように、Status.ok()ユーザーがnullない場合にのみ返されます。それ以外の場合はStatus.bad()が返されます。今ラムダで（サービスのメソッド今Optional<T>を返す）：

return CredentialService.get(id) 
     .filter(Credentials::isActive) 
     .map(credentials -> UserService.get(credentials.getUserId()) 
      .map(Status::ok) 
      .orElse(Status.bad())    
     ).orElse(Status.bad()); 

今私は（実際のコードでは、およそ4-5）Status.bad() 2回を返すことがあります。それは一度Status.bad()を返す方法ですか？

Answer 1

私はUserServiceのを推測することができます::あなたがより良いflatMapを使用するには、この場合のオプションを返す取得：私はこれらのコード行の詳細についてのより多くの情報を持っていない

CredentialService.get(id) 
     .filter(Credentials::isActive) 
      .flatMap(credentials -> UserService.get(credentials.getUserId()) 
      .map(Status::ok) 
      .orElse(Status.bad())