JavaScript对象属性更新函数中重复空值检查的必要性分析

问题背景 #

我们有如下JavaScript对象：

const recordCollection = {
  2468: {
    albumTitle: '1999',
    artist: 'Prince',
    tracks: ['1999', 'Little Red Corvette']
  },
  1245: {
    artist: 'Robert Palmer',
    tracks: []
  },
  5439: {
    albumTitle: 'ABBA Gold'
  }
};

为更新该对象的属性，有两个实现函数：

函数实现1 #

function updateRecords(records, id, prop, value) {
  if (value === "") {
    delete records[id][prop];
  } else if (prop !== "tracks" && value !== "") {
    records[id][prop] = value;
  } else if (prop === "tracks" && value !== "") {
    if (records[id].hasOwnProperty("tracks") === false) {
      records[id][prop] = [];
    }
    records[id][prop].push(value);
  }
  return records;
}

let result = updateRecords(recordCollection, 2468, "tracks", "");
console.log(result);

函数实现2 #

function updateRecords(records, id, prop, value) {
   if (value === ""){ 
       delete records[id][prop];
   } else if(prop === "tracks" && value !== ""){
   records[id][prop] = records[id][prop] || [];
    records[id][prop].push(value);
  } else if(prop !== "tracks" && value !== "") {
    records[id][prop] = value;
  }
  return records;
}

let result = updateRecords(recordCollection, 2468, "tracks", "lala");
console.log(result);

两个函数都先通过if (value === "")处理空值删除逻辑，但后续else if条件中重复添加了&& value !== ""的判断，针对这一情况解答以下问题：

问题解答 #

1. 重复的空值检查是否必要？能否安全移除？ #

完全没有必要，可以安全移除所有后续else if中的&& value !== ""判断，不会引入bug或改变代码预期功能。

原因很明确：第一个if (value === "")已经拦截了所有value为空字符串的情况，一旦进入这个分支执行删除操作后，后续的else if分支根本不会被触发——只有当value !== ""时，程序才会走到后续的else if逻辑里。所以后续条件里的&& value !== ""是纯粹的冗余判断，移除后逻辑和原代码完全一致。

2. 哪种写法更优？ #

移除冗余判断的写法更优，核心原因有三点：

• 提升可读性：去掉重复的判断逻辑后，代码结构更简洁，读者不需要反复确认相同的空值条件，能更快理解分支的核心逻辑（区分是否为tracks属性）。
• 降低维护成本：如果后续需要调整空值的判断规则（比如要同时处理null或undefined），只需要修改第一个if条件即可，不用逐一修改所有分支的重复判断，避免遗漏。
• 规避逻辑不一致风险：如果未来修改时只改了一处空值判断，没改其他分支的重复判断，会导致逻辑冲突；移除冗余判断后就不存在这个问题。

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