SonarLint提示Spring Boot代码认知复杂度超标，如何重构优化？

核心思路 #

这段代码的问题在于多层嵌套的if判断导致认知复杂度超标，重构的关键是减少嵌套层级、拆分独立逻辑，让代码结构更扁平直观。

步骤1：提前返回，减少嵌套 #

先把所有前置校验逻辑放在最前面，不符合条件直接退出，避免后续代码的多层嵌套：

if (item == null || StringUtils.isBlank(item.getDateService())) {
    return; // 若不需要返回，可改为保持letter原有值，根据业务上下文调整
}
LocalDate dateConverted = MapperUtil.convertToLocalDate(item.getDateService());
if (dateConverted == null) {
    return;
}

步骤2：提取子方法，拆分逻辑 #

把根据installId和年份判断letter的逻辑单独抽成一个子方法，让主方法只负责流程控制，子方法专注于单一判断逻辑：

// 主方法中剩余的核心逻辑
letter = determineLetter(item.getInstallId(), dateConverted.getYear(), year);

// 提取的子方法，专注于letter的规则判断
private String determineLetter(String installId, int dateYear, int targetYear) {
    if (installId != null) {
        return dateYear == targetYear ? "E" : "";
    }
    
    if (dateYear == targetYear) {
        return "A";
    } else if (dateYear == targetYear - 1) {
        return "C";
    }
    
    return ""; // 返回默认值，根据实际业务需求调整
}

步骤3：可选优化：用更简洁的语法替代if-else #

如果使用Java 12+，可以用switch表达式进一步简化子方法的条件判断，让逻辑更清晰：

private String determineLetter(String installId, int dateYear, int targetYear) {
    if (installId != null) {
        return dateYear == targetYear ? "E" : "";
    }
    
    return switch (dateYear) {
        case var y when y == targetYear -> "A";
        case var y when y == targetYear - 1 -> "C";
        default -> "";
    };
}

重构后的完整代码 #

// 主方法逻辑
if (item == null || StringUtils.isBlank(item.getDateService())) {
    return;
}
LocalDate dateConverted = MapperUtil.convertToLocalDate(item.getDateService());
if (dateConverted == null) {
    return;
}
letter = determineLetter(item.getInstallId(), dateConverted.getYear(), year);

// 子方法
private String determineLetter(String installId, int dateYear, int targetYear) {
    if (installId != null) {
        return dateYear == targetYear ? "E" : "";
    }
    
    if (dateYear == targetYear) {
        return "A";
    } else if (dateYear == targetYear - 1) {
        return "C";
    }
    
    return "";
}

效果说明 #

重构后代码的认知复杂度会大幅降低：

• 提前返回消除了原代码的多层嵌套，让主逻辑更线性。
• 子方法拆分让单一职责更明确，阅读时无需同时关注多个逻辑分支。
• 简化的条件表达式减少了心智负担，规则一目了然。

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