如何从一组选项中获取数据？

Question

我有一个返回产品集合的方法:

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,因此两种解决方案(不确定是否会有所不同)都是受欢迎的

Answer 1

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在lambda可能的情况下
• 为变量指定适当的名称,这样您就可以更轻松地维护代码