如何解决Java中条件判断的SonarLint认知复杂度问题？

SonarLint标记这类问题的核心原因是嵌套的空判断、多层&&和三元运算符让代码分支过多，阅读时需要频繁跳转逻辑，提升了认知负担。以下是几种实用的重构方案：

1. 使用Java标准库Optional重构链式空判断 #

针对长链式的空判断场景，用Optional的map方法逐步提取属性，替代嵌套的&&和三元运算符，逻辑更线性：

原代码：

AppointmentDetailsInfo appointmentDetails =(newAppointmentDetailsDTO !=null &&newAppointmentDetailsDTO.getData()!=null && newAppointmentDetailsDTO.getData().getAppointmentDetails()!=null) ?newAppointmentDetailsDTO.getData().getAppointmentDetails():null;

重构后：

AppointmentDetailsInfo appointmentDetails = Optional.ofNullable(newAppointmentDetailsDTO)
    .map(AppointmentDetailsDTO::getData)
    .map(Data::getAppointmentDetails)
    .orElse(null);

2. 简化嵌套三元运算符并复用空判断逻辑 #

针对嵌套三元表达式（如gosDTO != null ? gosDTO.getIsNewPatient() ? 1 : 0 : null），可以拆分逻辑或提取通用方法减少重复代码：

方法一：用Optional拆分嵌套逻辑 #

原代码：

isNewPatient(gosDTO != null ? gosDTO .getIsNewPatient() ? 1 : 0 : null);
referredProviderName(gosDTO != null ? gosDTO .getReferredProviderName() : null);
isMobile(gosDTO != null ? gosDTO .getIsMobile() : null);

重构后：

// 拆分isNewPatient的嵌套判断
Integer newPatientFlag = Optional.ofNullable(gosDTO)
    .map(GosDTO::getIsNewPatient)
    .map(isNew -> isNew ? 1 : 0)
    .orElse(null);
isNewPatient(newPatientFlag);

// 其他属性直接用Optional提取
referredProviderName(Optional.ofNullable(gosDTO).map(GosDTO::getReferredProviderName).orElse(null));
isMobile(Optional.ofNullable(gosDTO).map(GosDTO::getIsMobile).orElse(null));

方法二：提取通用辅助方法避免重复空判断 #

如果多个属性都需要对gosDTO做空判断，写一个通用方法复用逻辑：

// 通用辅助方法：安全提取GosDTO的属性
private <T> T getGosProperty(GosDTO gosDTO, Function<GosDTO, T> propertyGetter) {
    return Optional.ofNullable(gosDTO).map(propertyGetter).orElse(null);
}

调用Builder时：

isNewPatient(getGosProperty(gosDTO, dto -> dto.getIsNewPatient() ? 1 : 0));
referredProviderName(getGosProperty(gosDTO, GosDTO::getReferredProviderName));
isMobile(getGosProperty(gosDTO, GosDTO::getIsMobile));

3. （可选）使用第三方工具类简化空安全访问 #

如果项目已引入Apache Commons Lang或Spring Core等工具库，也可以用它们的空安全工具类：

• Apache Commons Lang的ObjectUtils.chain：

AppointmentDetailsInfo appointmentDetails = ObjectUtils.chain(newAppointmentDetailsDTO)
    .get(AppointmentDetailsDTO::getData)
    .get(Data::getAppointmentDetails)
    .orElse(null);

• Spring的ObjectUtils.nullSafeGet（适合简单属性提取）：

referredProviderName(ObjectUtils.nullSafeGet(gosDTO, GosDTO::getReferredProviderName));

重构后的优势 #

重构后代码的逻辑线性化，每个步骤只处理一个空判断或属性提取，减少了分支嵌套，不仅解决了SonarLint的认知复杂度问题，还大幅提升了代码可读性和可维护性。

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