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。因此，您可以使用单个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
为变量指定适当的名称，以使代码易于维护

如何从可选参数的集合中获取数据？

1 个答案: