'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().forEach() → collection.forEach()
• collection.stream().collect(toList/toSet/toCollection()) → 新的 CollectionType(collection)

Intellij给出的例子很容易理解，但我不明白为什么它建议我map().filter()。

我查看了ReferencePipeline 的来源，但没有找到任何线索：map().filter() 或filter().map() 在涉及与流实现相关的临时对象时没有区别（filter().map() 如果A.b 将有更少的自动装箱@是一个让我更困惑的原始人）。

那么，我是否遗漏了一些流实现，或者这是 Intellij 的误报？

Answer 1

a.getB() 被调用两次 - 一次在过滤器内部，它也是映射函数，因此与其执行两次，不如先使用getB 映射它，然后过滤掉它

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

编辑

如果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