当前位置: 首页 > 知识库问答 >
问题:

Java8 peek vs map

华欣荣
2023-03-14

我有以下情况:有一个对象列表-ProductData,其中包含几个字段:

public class ProductData
{
....
  private String name;
  private String xref;

  //getters
  //setters
}

还有一个API,它返回以下对象的列表:

public class RatingTableRow
{
  private String planName;
  private String planXref;
  private int fromAge;
  private int toAge;
  private int ratingRegion;

 //constructor
 //getters
 //setters

}

但它会返回带有空“计划名称”字段的对象,因为在提取该对象时不允许这样做。我需要通过外部参照将产品数据与RatingTableRow链接,以便将计划名称设置到RatingTableRow中,因为我以后需要使用此对象,所以我创建了以下代码来实现这一点:

Map<String, ProductData> productByXref = plans.stream()
        .collect(toMap(ProductData::getInternalCode, Function.identity()));

return getRatingTableRows(...).stream
        .filter(ratingRow -> productByXref.containsKey(ratingRow.getPlanXref()))
        .peek(row -> {
                ProductData product = productByXref.get(row.getPlanXref());
                row.setPlanName(product.getName());
        })....;

我知道java文档说,peek不适合这些需求,但我想听听您对如何以更正确的方式完成这项任务的建议。

共有1个答案

徐佐
2023-03-14

有一个原因是peek被记录为主要用于调试目的。

最终在peek内部处理的内容可能根本不符合终端操作的条件,并且流仅由终端操作执行。

先假设一个微不足道的例子:

    List<Integer> list = new ArrayList<>();
    List<Integer> result = Stream.of(1, 2, 3, 4)
            .peek(x -> list.add(x))
            .map(x -> x * 2)
            .collect(Collectors.toList());

    System.out.println(list);
    System.out.println(result);

一切看起来都很好,对吗?因为在这种情况下,peek将为所有元素运行。但是当您添加过滤器时会发生什么(并且忘记peek做了什么):

 .peek(x -> list.add(x))
 .map(x -> x * 2)
 .filter(x -> x > 8) // you have inserted a filter here

您正在对每个元素执行peek,但不收集任何元素。你确定你想要吗?

这可能会变得更加棘手:

    long howMany = Stream.of(1, 2, 3, 4)
            .peek(x -> list.add(x))
            .count();

    System.out.println(list);
    System.out.println(howMany);

在java-8中,列表是填充的,但在jdk-9中,根本不调用peek。由于您没有使用过滤器平面图,因此您没有修改流的大小,计数只需要它的大小;因此,peek一点也不叫。因此,依赖于peek是一个非常糟糕的策略。

 类似资料:

相关问答

相关文章

相关阅读