Sonar:用方法引用替换这个lambda。什么时候应该使用“stream()”?[重复]

Question

我想知道这两段代码有什么区别：

underlyingConnectors.values().stream().forEach(connector -> connector.start());

underlyingConnectors.values().forEach(connector -> connector.start());

underlyingConnectors.values().forEach(Connector::start);

第二行编译得很好，但我的 Ecplipse IDE 抱怨“声纳：用方法引用替换这个 lambda”。

如何选择要使用的适当代码？这些都有具体的用例吗？

Answer 1

如果你只是通过 .forEach（） 对每个元素执行一个操作，则可以省略 .stream（）。调用 .stream（） 是多余的。

underlyingConnectors.values().stream().forEach(connector -> connector.start());
underlyingConnectors.values().forEach(connector -> connector.start());

第2行是正确的。

如果您只是对每个条目应用一个操作，那么只需使用方法引用，无需编写lambda。这基本上是一种速记。

underlyingConnectors.values().forEach(connector -> connector.start());
underlyingConnectors.values().forEach(Connector::start);

第2行是正确的。

Answer 2

第一行包含对stream（）的不必要调用，后面的两个是见仁见智的问题。如果您想编写最短、最简洁的代码，您可以使用最后一个。所有这些都以等效的方式工作，主要是关于样式。

stream（）是不必要的，因为values（）返回一个Collection，它扩展了具有forEach（Consumer

Answer 3

第二个简单地是第一个的速记符号，第三个是第二个的速记符号。经验法则是，如果一个方法没有像第二个一样的参数，你可以简单地用像第三个一样的符号替换它。

更一般地说，如果lambda表达式右侧（RHS）的表达式是lambda表达式左侧（LHS）的方法调用，不需要额外的参数，或者是只接受表达式左侧作为参数的方法调用，那么lambda表达式可以替换为方法引用-一旦RHS接受两个参数，它就不像Java那样“漂亮”，目前不支持方法的柯里化作为语言特性，但可以替换为包装器：

package com.example;

import java.util.Arrays;
import java.util.List;
import java.util.function.Consumer;

public class Application {

  private static final String LINEBREAK = System.lineSeparator();

  public static void main(String[] args) {
    final List<String> strings = Arrays.asList("Hello", "World", "!");

    strings.forEach(string -> System.out.print(string + LINEBREAK)); // 1

    strings.forEach(string -> Application.printWithLineBreakParameter(LINEBREAK).accept(string)); // 2

    strings.forEach(Application.printWithLineBreakParameter(LINEBREAK)); // 3
  }

  private static Consumer<String> printWithLineBreakParameter(final String lineBreak) {
    return (string) -> System.out.print(string + lineBreak);
  }
}

这里，所有三个调用都是相等的，但是2可以用3代替，这样看起来比1更简洁。