Java 8のオプションが存在する場合はメソッドが返されます

Question

Java 8では、Optionalが存在する場合にメソッドを強制的に返す方法はありますか？ 私は答えがノーだと思います。しかし、次のコードでisPresent()とget()を使用しないようにするには？

public Bar myMethod(){ 
    if(condition){ 
     return getOptionalFoo() 
       .map(Foo::getBar) 
       .orElseGet(() -> getOtherBar()); 
    } 

    return getOtherBar(); 
} 

しかし、今、私たちは二度getOtherBarへの呼び出しを記述するので、それは良いアイデアのように見えるしていません：

public Bar myMethod(){ 
    if(condition){ 
     Optional<Foo> foo = getOptionalFoo(); 
     if(foo.isPresent()){ 
      return foo.get().getBar(); 
     } 
    } 

    return getOtherBar(); 
} 

一つの解決策は、書くことだろう。これを避ける方法はありますか？

編集：私はまた

public Bar myMethod(){ 
    //Or a ternary operator but there are hard to read when they make long lines 
    Optional<Foo> foo; 
    if(condition){ 
     foo = getOptionalFoo(); 
    } else { 
     foo = Optional.empty(); 
    } 

    return foo.map(Foo::getBar) 
     .orElseGet(() -> getOtherBar()); 
} 

考えるしかし、ここで問題は、我々はそれに役に立たない空Optionalとチェーンのメソッドを作成することです。条件が時間の99％に満たない場合は、元のコードよりも効率が悪くなります（特に、複数の場合はmap()とfilter()）。

注：実際に私たちのコードでは、他の呼び出し（fooに依存する）の結果に依存するより多くの条件があります。

Answer 1

これはいかがですか？以下のコードの形態は可読以上であるが、それはあなたより遅い、conditionが満たされない場合ので、getOtherBar()もOptionalに追加操作（map & orElseGet）を必要とします。重複の多くは、上記のように好きであるかどう

public Bar myMethod() { 
    Optional<Foo> source = condition ? getFoo() : Optional.empty(); 

    return source.map(Foo::getBar) 
       .orElseGet(this::getOtherBar); 
} 

しかし、その後は、たとえば、抽出方法を自分自身を解放することができます。

private Optional<Foo> foo(){ 
    return condition ? getFoo() : Optional.empty(); 
} 

public Bar myMethod() { 
    return foo().map(Foo::getBar).orElseGet(this::getOtherBar); 
} 

public Other otherMethod() { 
    return foo().map(Foo::getOther).orElseGet(this::getOther); 
} 

Answer 2

これはかなり簡単にあなたに得れば一緒に落ちますOptionalの考え方を使用し、3値演算子を使用して条件を処理します。

public Bar myMethod(){ 
    return (condition ? getOptionalFoo() : Optional.<Foo> empty()) 
     .map(f -> f.getBar()) 
     .orElseGet(() -> getOtherBar()) 
} 

Answer 3

conditionとは何ですか？それはインスタンスメソッドに配置されているため、インスタンスの状態に応じた条件でなければなりません。これはあなたのクラスのフィールドであることを前提と簡単にするために、あなたは

public Bar myMethod() { 
    return Optional.of(this) 
     .filter(obj -> obj.condition) 
     .flatMap(ClassContainingMyMethod::getFoo) 
     .map(Foo::getBar) 
     .orElseGet(this::getOtherBar); 
} 

を書くことができgetFoo()は高価ではありませんし、副作用を持たない場合、あなたはするつもりはない場合でも、常にそれを呼び出して受け入れることができますその結果をconditionに応じて使用してください。コードを簡略化します

public Bar myMethod() { 
    return getFoo() 
     .filter(foo -> condition) 
     .map(Foo::getBar) 
     .orElseGet(this::getOtherBar); 
} 

元のコードには意味の違いがあります。 getBarがnullを返した場合、元のコードはnullを返しますが、Optional.mapの結果を返す関数を使用するコードはすべてgetOtherBar()に戻ります。