Java 8オプションadd.result.isPresentの場合にのみ返す戻り値

Question

私は、インターフェイスにオプションのreturnメソッドがあり、何かを返すためにそれを実装するクラスのいくつかがありますが、それ以外のコードはありません。この華麗な「ヌルキラー」を採用する努力において

は、ここで私が試したものです：

public interface Gun { 
    public Optional<Bullet> shoot(); 
} 

public class Pistol implements Gun{ 
    @Override 
    public Optional<Bullet> shoot(){ 
     return Optional.of(this.magazine.remove(0)); 
    }//never mind the check of magazine content 
} 

public class Bow implements Gun{ 
    @Override 
    public Optional<Bullet> shoot(){ 
     quill--; 
     return Optional.empty(); 
    } 
} 

public class BallisticGelPuddy{ 
    private Gun[] guns = new Gun[]{new Pistol(),new Bow()}; 
    private List<Bullet> bullets = new ArrayList<>(); 
    public void collectBullets(){ 
     //here is the problem 
     for(Gun gun : guns) 
      gun.shoot.ifPresent(bullets.add(<the return I got with the method>) 
}} 

私はこの例がどのように愚かなことをお詫び申し上げます。
私はちょうど持って帰って確認して、オプションの場合はそれを使用して追加することができますか？

P.S.任意の本当の有用性はありますか？（X！= null）はできませんでしたか？

Answer 1

飛行機（Bulletよりもクラス名が良いかもしれません）がBallisticGelPuddyを通過したとき、それが止まってしまいます。滞留した場合は、BallisticGelPuddyに累積します。

for(Gun gun: guns) { 
    final Bullet bullet = gun.shoot(); 
    if(bullet != null) { 
     bullets.add(bullet); 
    } 
} 

を非常に簡単、右：

は、私たちが代わりにnullチェックを使用していた場合のコードを書き直してみましょうか？それが存在する場合、我々はそれを追加したい

のはに戻って、オプションのスタイルを追加してみましょう：。Optionalアプローチはterserですが、効果的にこれら二つのことは、同じことを実現

for(Gun gun: guns) { 
    gun.shoot().ifPresent(bullets::add); 
} 

。

このシナリオでは、常に存在を確認しようとしているため、2つの方法の違いはありません。 Optionalは、nullを処理する際の間違いを防ぐためのもので、express a more fluid call chainを許可しますが、このシナリオではOptionalを使用することの実用性を考慮してください。この場合は完全にのとは思われません。あなたが行うことができますストリームAPIで

Answer 2

私は、あなたがしたいと思う：

gun.shoot().ifPresent(bullets::add); 

それとも、あまりにも（符号化）ループで済ますことができます。

guns.stream() 
    .map(Gun::shoot) 
    .filter(Optional::isPresent) 
    .map(Optional::get) 
    .forEach(bullets::add); 

しかし、それは醜悪です。

Answer 3

、：

List<Bullet> bullets = Arrays.stream(guns) 
      .map(Gun::shoot) 
      .flatMap(this::streamopt) // make Stream from Optional! 
      .collect(Collectors.toList()); 

は残念ながら、Javaの8で、ストリームするOptionalsを変換方法はありませんので、あなたはそれを自分で記述する必要があります。 を参照してくださいUsing Java 8's Optional with Stream::flatMap

Answer 4

今後の参考のために、私のような問題に陥っている人のために投稿したいと思います。

public class Bullet{ 
    private int weight = 5; 
    public int getWeight(){ return weigth;} 
} 
public interface Gun { 
    public Optional<Bullet> shoot(); 
} 

public class Pistol implements Gun{ 
    @Override 
    public Optional<Bullet> shoot(){ 
     return Optional.of(this.magazine.remove(0)); 
    }//never mind the check of magazine content 
} 

public class Bow implements Gun{ 
    @Override 
    public Optional<Bullet> shoot(){ 
     quill--; 
     return Optional.empty(); 
    } 
} 

public class BallisticGelPuddy{ 
    private Gun[] guns = new Gun[]{new Pistol(),new Bow()}; 
    private List<Bullet> bullets = new ArrayList<>(); 
    private int totWeigth = 0; 
    public void collectBullets(){ 
     // IF YOU WANT TO ONLY ADD WHAT YOU HAVE FOUND IN A COMPATIBLE CLASS 
     // thanks to makoto and bohemian for the answers 
     for(Gun gun : guns) 
      gun.shoot.ifPresent(bullets::add) 
     //IF YOU WANT TO ACCESS THE RETURNED OBJECT AND ADD IT TOO 
     for(Gun gun : guns) 
      gun.shoot.ifPresent(arg -> {totWeight += arg.getWeigth(); 
             bullets.add(arg);}); 
}}