如何重构代码满足SonarQube认知复杂度从21降至15要求

老哥，你这SonarQube的认知复杂度警告我太熟了！这段代码里重复的空值检查、一堆嵌套属性访问还有零散的条件赋值，都是把复杂度拉到21的元凶。咱们一步步重构，把数字压到15以下，同时代码还能更易读。

重构核心思路 #

• 干掉重复判断：原代码里反复写this.data && this.data.deviceInfo，换成可选链this.data?.deviceInfo一次性提取，少做几次分支判断。
• 简化属性赋值：把一堆a || b换成更精准的空值合并运算符??，配合可选链?.处理嵌套属性，减少零散的小条件累加。
• 合并冗余操作：原代码先给this.deviceDetails赋值空对象，后面又覆盖，直接合并成一次赋值，去掉无意义的中间步骤。
• 提前解构核心对象：把需要频繁访问的device、driver提前处理，减少重复的属性查找，让代码更清爽。

重构后的代码 #

// 提前提取deviceInfo，用可选链替代重复的&&判断
const deviceInfo = this.data?.deviceInfo;

// 统一给deviceDetails赋值，避免先空对象再覆盖的冗余操作
this.deviceDetails = deviceInfo ? {
  name: deviceInfo.device?.name ?? '',
  manufacturer: deviceInfo.device?.manufacturer ?? '',
  deviceType: deviceInfo.device?.deviceType ?? '',
  model: deviceInfo.device?.model ?? '',
  description: deviceInfo.device?.description ?? '',
  managerId: deviceInfo.device?.deviceManager?.managerId ?? null,
  locationId: deviceInfo.device?.location?.locationId ?? null,
  active: deviceInfo.device.active,
  connectionType: deviceInfo.connectionType ?? null,
  driver_id: deviceInfo.driver?.driverId ?? null,
  ipAddress: deviceInfo.ipAddress ?? '',
  port: String(deviceInfo.port) ?? '',
  connectionStatus: deviceInfo.active,
} : {};

if (deviceInfo) {
  this.getSessionInfo();
  this.oldDeviceDetails = {...this.deviceDetails};
  this.deviceLocation = deviceInfo.device?.location?.locationId ?? null;
}

为啥这么改能降复杂度？ #

1. 去掉了一次重复的this.data && this.data.deviceInfo分支判断，直接减少复杂度计数。
2. 把对象赋值里的十几个||条件换成可选链+空值合并，每个属性访问从“条件判断+赋值”变成“直接读取+默认值”，大幅减少认知复杂度的累加。
3. 合并了deviceDetails的两次赋值，消除了无意义的中间状态，让逻辑更线性，不用来回切换上下文。
4. 提前提取deviceInfo后，后续所有访问都更简洁，减少了大脑需要处理的嵌套层级。

这样改完，认知复杂度肯定能降到15以下，而且代码的可读性和维护性也上了一个台阶。

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