如何修复多分支If-Else导致的SonarLint认知复杂度错误

你的代码中冗长的if-else链式判断是导致SonarLint提示认知复杂度过高的核心原因——每一个else if都会增加代码的决策分支深度，即使换成switch-case也无法解决，因为switch更适合离散值匹配，而非范围判断。

可以通过以下方式重构代码，彻底降低认知复杂度：

步骤1：完善枚举类 #

给枚举添加分数的getter方法，并预计算按分数升序排列的非UNKNOWN枚举列表，避免每次调用时重复排序：

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

public enum ExampleEnum {
    UNKNOWN(-1),
    D0_D100(0),
    D100_D200(100),
    D200_D300(200),
    D300_D400(300),
    D400_D500(400),
    D500_D600(500),
    D600_D700(600),
    D700_D800(700),
    D800_D900(800),
    D900_D1000(900);
    
    private final int score;
    
    ExampleEnum(int score) {
        this.score = score;
    }
    
    public int getScore() {
        return score;
    }
    
    // 预排序的范围枚举列表，避免重复计算
    private static final List<ExampleEnum> SORTED_RANGE_ENUMS = Arrays.stream(values())
            .filter(e -> e != UNKNOWN)
            .sorted(Comparator.comparingInt(ExampleEnum::getScore))
            .collect(Collectors.toUnmodifiableList());
}

步骤2：重构getScoreType方法 #

使用循环遍历预排序的枚举列表，找到匹配的范围。由于列表是升序排列，一旦遇到大于输入分数的枚举值，即可提前终止循环：

public static ExampleEnum getScoreType(Integer score) {
    if (Objects.isNull(score) || score < 0) {
        return ExampleEnum.UNKNOWN;
    }
    
    ExampleEnum matchedType = ExampleEnum.UNKNOWN;
    for (ExampleEnum enumVal : ExampleEnum.SORTED_RANGE_ENUMS) {
        if (score >= enumVal.getScore()) {
            matchedType = enumVal;
        } else {
            // 后续枚举分数更大，无需继续判断
            break;
        }
    }
    return matchedType;
}

替代方案：使用Stream API #

如果你更喜欢函数式风格，也可以用Stream来实现，代码更简洁：

public static ExampleEnum getScoreType(Integer score) {
    if (Objects.isNull(score) || score < 0) {
        return ExampleEnum.UNKNOWN;
    }
    
    return ExampleEnum.SORTED_RANGE_ENUMS.stream()
            .filter(e -> score >= e.getScore())
            .max(Comparator.comparingInt(ExampleEnum::getScore))
            .orElse(ExampleEnum.UNKNOWN);
}

这两种方式都彻底消除了冗长的条件链，认知复杂度大幅降低，同时逻辑和原代码完全一致：

• 处理null或小于0的分数返回UNKNOWN
• 匹配对应区间的枚举值
• 大于等于900的分数返回D900_D1000

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