程序员最近都爱上了这个网站  程序员们快来瞅瞅吧！  it98k网:it98k.com

我有一种返回产品集合的方法：

Collection<Product> getProducts() { ... }  

每个产品可能都有保证。但这不是必需的。

interface Product {
    Optional<Guarantee> getGuarantee();
}

现在，我需要检查所有产品，并检查隔离区是否已过期。未过期的应收集到列表中。

这是我的工作：

List<Optional<Guarantee>> optionalGar = getProducts().stream()
      .map(f -> f.getGuarantee()).collect(Collectors.toList());

List<Guarantee> gar = optionalGar.stream()    
      .map(op -> op.orElse(null))             
      .filter(Objects::nonNull)
      .filter(g -> !g.hasExpired())
      .collect(Collectors.toList());

有什么方法可以避免使用.orElse(null)？

（op.get()如果可选为空，则替换为会导致异常）

PS：我可以在Java 8和Java 9之间自由选择，因此欢迎两种解决方案（不确定是否会有所不同）

解决方案

Java 8

List<Guarantee> expiredGuarantees = getProducts().stream()
                                                 .map(Product::getGuarantee)    
                                                 .filter(Optional::isPresent)
                                                 .map(Optional::get)
                                                 .filter(not(Guarantee::hasExpired))
                                                 .collect(toList());

Java 9

Java9已经有了Optional::stream。因此，您可以使用single替换过滤和映射flatMap：

List<Guarantee> expiredGuarantees = getProducts().stream()
                                                 .map(Product::getGuarantee)    
                                                 .flatMap(Optional::stream)
                                                 .filter(not(Guarantee::hasExpired))
                                                 .collect(toList());

笔记

Java 8没有Predicates.not方法。仅从第11版开始包含。

通过将以下方法添加到您的项目中，您将可以将其与上述解决方案一起使用。

public static <T> Predicate<T> not(Predicate<T> predicate) { 
    return predicate.negate();
}

更新

尽管这不是CodeReview社区，但以下是有关代码的一些注意事项：

• 通过将两个管道合并为一个，您的代码将更加清晰（在这种情况下）。
• 尽可能优先method reference于alambda
• 给您的变量起个适当的名字，这样您的代码将更易于维护