Java11下SonarQube提示Replace this lambda with a method reference如何解决

代码调整方案 #

你给出的代码中SonarQube触发提示的是filter(e -> e.isPrimaryAcc())这部分lambda逻辑，该逻辑仅调用AccountMember类的实例方法，无额外处理，可直接替换为实例方法引用，调整后的完整代码如下：

// 注意原代码存在两处可优化点：1. 类型拼写错误 Sting -> String 2. findFirst返回Optional类，需补全取值逻辑
Optional<AccountMember> firstPrimaryAccount = accountMemberList.stream()
        .filter(AccountMember::isPrimaryAcc) // 替换原有lambda为方法引用
        .sorted(Comparator.comparingInt(AccountMember::getSubMemberNo))
        .findFirst();
// 按业务需求补全取值逻辑即可得到对应String类型结果，示例如下：
String abc = firstPrimaryAccount.map(AccountMember::getAccountNo).orElse("");

方法引用是否为推荐最佳实践 #

• 绝大多数场景下是推荐做法：当lambda仅做透传参数、调用单个已有方法的操作时，方法引用的写法更简洁、语义更明确，也能减少不必要的匿名函数声明，是Java 8+版本官方推荐的编码风格，Sonar的这条规则也是基于通用规范制定的。
• 无需强制替换的场景：如果lambda中包含额外的参数处理逻辑（比如判空、数值转换、多方法组合调用），或者替换方法引用后会因为方法重载等问题提升理解成本时，可保留lambda写法，不用为了符合规则强行调整。

内容的提问来源于stack exchange，提问作者Pallavi Singh