Java 8 使用过滤器 如果 boolean 为真

Question

我有对象列表，以及我想查找的其他 object。

让我们称之为ProductDto和Product列表

我想按名称和类型过滤 ProductDtos。 如果存在域代码- 也按它过滤。

在简单的 java 代码中，它将如下所示：

ProductDto  find(List<ProductDto> productDtos, Product product) {
   for(ProductDto dto : productDtos) {
      if(dto.getName.equals(product.getName) && dto.getType.equals(product.getType)) {
         boolean isCodePresent = dto.getCode() != null && product.getCode() != null;
         if(!isCodePresent) return dto;
         else if(isCodePresent && dto .getCode.equals(product.getCode)) return dto;
      }
   }
   return null;
}

它在并行 stream 中的外观如何？

productDtos.parralelStream()
    .filter(i -> i.getName().equals(product.getName) && // check if type is equal)
    // use filter if isCodePresent 
    .map(...)
    .collect(Collectors.toList());

Answer 1

许多 null 测试混淆了清晰度。

ProductDto find(List<ProductDto> productDtos, Product product) {
   return ProductDts.parallelStream()
      .filter(dto -> dto.getName.equals(product.getName)
          && dto.getType.equals(product.getType)
          && ((dto.getCode() == null || product.getCode() == null)
              || 
              (dto.getCode() != null && product.getCode() != null
                  && dto.getCode.equals(product.getCode))
              ))
      .findAny().orElse(null);
}

没有orElse会更安全、更短：

Optional<ProductDto> find(List<ProductDto> productDtos, Product product) {
   return ProductDts.parallelStream()
      .filter(dto -> dto.getName.equals(product.getName)
          && dto.getType.equals(product.getType)
          && (dto.getCode() == null) == (product.getCode() == null)
          && (product.getCode() == null || dto.getCode.equals(product.getCode)))
    .findAny();
}

过滤条件也有点简化。

消除不变子表达式：

Optional<ProductDto> find(List<ProductDto> productDtos, Product product) {
   boolean productCodeNull = product.getCode() == null;
   return ProductDts.parallelStream()
      .filter(dto -> dto.getName.equals(product.getName)
          && dto.getType.equals(product.getType)
          && (dto.getCode() == null) == productCodeNull 
          && (productCodeNull || dto.getCode.equals(product.getCode)))
    .findAny();
}