SONAR：用方法引用替换此lambda。

Question

当我在sonar中检查时，结果是：

Replace this lambda with a method reference. 

它实际上指的是：

.filter(s -> aIDetailsDto.getResult().getIdNo().equals(s))

我的代码如下所示：

AIDetailsDto aIDetailsDto = aaaService
        .getDetailsByUserId(userId)
    if (!ObjectUtils.isEmpty(aIDetailsDto)) {
      List<String> kvpValues = callService.getKVPCodes(NewConstants.REMOVED)
          .stream()
          .filter(s -> aIDetailsDto.getResult().getIdNo().equals(s))

我试图改变它，但我得到了一个错误。有人知道如何改变它吗？

Answer 1

想想你把这个叫做aidetailsdo。getResult（）。getIdNo（）对于每一个s，让它更好、让sonar（和你）高兴的是，将它向上移动一点（并且只调用一次）：

String idNo = aIDetailsDto.getResult().getIdNo();
... stream()
    .filter(idNo::equals)

Answer 2

您可以替换。过滤器-

lambda表达式将计算术语aIDetailsDto。getResult（）。每次求值谓词函数时，getIdNo（），而方法引用将在创建谓词实例时求值，并捕获结果值，以便在每次后续谓词求值时对同一对象调用等于。另请参见“System.out：：println的等效lambda表达式是什么”。

如果表达式aIDetailsDto.getResult（）. getIdNo（）每次都应该计算相同的结果，则没有任何区别，并且由于不重复计算它，方法引用可能会更有效。

另请注意备选方案。filter（谓词.isEqual（aIDetailsDto.getResult（））。getIdNo（）），它也将只对参数表达式求值一次，但它是唯一一个处理其求值为null，然后只接受null元素的变量（请参见Predicate.isEqual（…））。但是，如果假定此值永远不为null，则最好使用其他变体的抛出行为。

Answer 3

您应该使用。过滤器（aIDetailsDto.getResult（）。getIdNo（）：：等于）。