如何修复SonarQube‘重构代码使用Stream管道’重大问题？

这个报错对应SonarQube规则S3958，触发原因是低版本SonarQube没有适配Java 16+新增的Stream.toList()方法，静态分析时会误判这段代码的流只有filter、map中间操作，没有终端消费动作，不符合流流水线的使用要求。
如果你的SonarQube版本已经在9.8以上，大概率是链式调用的格式问题导致分析识别异常，调整换行格式即可。

方案1：代码调整（无需升级Sonar，推荐） #

把流末端的toList()替换为collect终端操作，这是Java 8开始就存在的标准流收集写法，所有版本Sonar都能正确识别。
如果要保留原代码返回不可变列表的特性，使用Collectors.toUnmodifiableList()；如果不需要不可变约束，用Collectors.toList()即可。
调整后代码如下：

import java.util.List;
import java.util.Objects;
import java.util.stream.Collectors;

public List<String> getUserroles(List<UserRoleDTO> userRoles) {
    // 可选：增加入参非空校验、空元素过滤，规避NPE，符合Sonar防御式编程要求
    if (userRoles == null) {
        return List.of();
    }
    return userRoles.stream()
            .filter(Objects::nonNull)
            .filter(UserRoleDTO::getRight)
            .map(UserRoleDTO::getActionId)
            .collect(Collectors.toUnmodifiableList());
}

如果不想加额外的非空判断，最简改法只需要替换最后一行的.toList()为.collect(Collectors.toList())即可通过校验。

方案2：升级SonarQube版本 #

将SonarQube升级到9.8及以上正式版，新版本已经完成Java 16+语法适配，可以正确识别Stream.toList()为合法的流终端操作，原有代码不需要修改就能通过检测。

注意事项 #

• Stream.toList()返回的是不可变列表，不支持增删操作，如果业务逻辑需要修改返回的角色集合，请使用Collectors.toList()代替Collectors.toUnmodifiableList()，避免运行时抛出UnsupportedOperationException。
• 如果代码上下文中已经保证入参userRoles非空、集合内不存在null元素，可以省略对应的非空校验逻辑，不会影响Sonar规则的校验结果。

改完后重新触发构建，Sonar不会再报流流水线相关的违规问题。如果同时加了非空判断，还能顺带解决潜在的空指针风险，提升代码健壮性。

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