“filter（）”和“map（）”可以互换

Question

我有一个简单的流如下：

List<Long> r = l.stream()
                .filter(a->a.getB() % 2 == 0)
                .map(A::getB)
                .collect(Collectors.toList());

但Intellij建议我：

“filter（）”和“map（）”可以互换。检查信息：报告流API调用链可以简化。它允许在遍历集合时避免创建冗余的临时对象。例如

• collection.stream（）→collection.for每个（）
• collection.stream（）。

Intellij给出的例子很容易理解，但我不明白为什么它建议我使用map（）。过滤器（）。

我查看了ReferencePipeline的来源，但没有找到任何线索：map（）。filter（）或filter（）。map（）对于与流实现相关的临时对象（filter（））没有影响。如果A.b是一个让我更困惑的原语，map（）的自动装箱功能就会减少）。

那么，我是否错过了一些流点实现，或者这是Intellij的错误警报？

Answer 1

a.getB（）被调用两次——一次在过滤器内部，它也是映射函数，因此与其这样做两次，不如先使用getB映射它，然后过滤掉它

<代码>列表

编辑

如果getB返回long，则可以使用mapToLong来避免中间装箱操作。

List<Long> r = l.stream()
                .mapToLong(A::getB)
                .filter(b->b % 2 == 0)
                .boxed()
                .collect(Collectors.toList());

样本输出

使用静态计数器计算get方法的调用次数：

class A {
    public static int count = 0;
    private long b;

    public long getB() {
        count++;
        return b;
    }
}

List<A> list= List.of(new A(1L), new A(3L), new A(4L));

list.stream() 
    .filter(a -> a.getB()%2 == 0)
    .map(A::getB)
    .collect(Collectors.toList());
System.out.println(A.count); // returns 4

然而

list.stream()
    .mapToLong(A::getB)
    .filter(b->b % 2 == 0)
    .boxed()
    .collect(Collectors.toList());
System.out.println(A.count); // returns 3