如何简化JavaScript函数中的嵌套循环代码

你这段代码首先存在几个基础问题需要先修正：

• 语法错误：数组遍历方法拼写错误，正确写法是forEach（驼峰命名），同时函数体、遍历回调的闭合大括号缺失，也没有返回最终生成的Map结果
• 性能缺陷：循环内嵌套find的写法时间复杂度为O(n*m)，n为IDS数组长度、m为objectList数组长度，数据量较大时性能损耗非常明显

方案1：消除嵌套循环，性能最优 #

核心思路是把重复遍历objectList的逻辑提前，只做一次遍历构建id和对象的映射关系，后续遍历IDS时直接通过Map的O(1)复杂度取值，总时间复杂度降到O(n+m)，代码也更简洁：

function getMap(objectList) {
  // 单次遍历objectList，构建id到实例的映射
  const idToObj = new Map(
    objectList.map(item => [item.getId(), item])
  );
  // 遍历IDS直接取值，匹配不到时赋值null
  return new Map(
    IDS.map(id => [id, idToObj.get(id) ?? null])
  );
}

方案2：保留嵌套循环结构，仅精简代码 #

如果受场景限制不能调整循环结构，可以直接删掉冗余的if/else分支，修正语法问题即可：

function getMap(objectList) {
  const objectMap = new Map();
  IDS.forEach(id => {
    // find匹配不到时返回undefined，用空值合并运算符直接转成null
    objectMap.set(id, objectList.find(obj => obj.getId() === id) ?? null);
  });
  return objectMap;
}

注意事项 #

• 不要用||运算符代替??：如果getId()可能返回0、空字符串这类合法假值，||会把对应正常对象误判为null，??只会在左侧值为undefined/null时才会走右侧赋值
• 如果运行环境不支持??运算符，可以换成|| null的写法，前提是确认id不会出现上面说的假值场景

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