如何提升多字段对象数组映射转换的性能？

先给你揪出原代码里最致命的问题——你把products = []放在for循环内部了！每次循环都会重置这个数组，最后你只能得到最后一条record的匹配结果，这不仅是性能浪费，逻辑完全不对，得先把这个核心问题解决掉。

接下来针对性能和可维护性给你几个优化方向：

1. 用映射表替代重复的if判断 #

你的代码里写了一堆独立的if，字段越多越难维护，而且重复的条件判断也没必要。我们可以把字段名和对应的产品字符串提前存在一个映射对象里，然后遍历这个映射表来做判断，代码更简洁，性能也更稳定。

2. 缓存数组长度（小细节但有用） #

传统for循环里，每次判断i < records.length都会读取数组的length属性，虽然现代JS引擎会优化，但提前缓存起来能避免不必要的属性查找，尤其是数组很大的时候。

3. 用块级作用域变量替代var #

var存在函数作用域和变量提升的问题，换成let/const不仅更安全，也符合现代JS的编码规范。

优化后的代码示例（传统for循环，性能最优） #

// 先定义字段与产品名称的映射表，新增/修改字段只需要改这里
const fieldProductMap = {
  Custom_BA1__c: 'Custom BA1',
  Custom_Web_Sim__c: 'Custom Web Sim',
  Finance_for_Non_Financials__c: 'Finance for Non Financials',
  Custom_Board__c: 'Custom Board',
  // 剩下的100多个字段都按这个格式加进来
};

// 把products数组声明移到循环外面！
const products = [];
// 缓存数组长度，避免每次循环都读取
const recordsLen = records.length;

for (let i = 0; i < recordsLen; i++) {
  const currentRecord = records[i];
  // 遍历映射表的键值对
  for (const [fieldName, productName] of Object.entries(fieldProductMap)) {
    if (currentRecord[fieldName] === 'true') {
      products.push(productName);
    }
  }
}

更简洁的写法（用flatMap，可读性更好） #

如果你的环境支持ES6+，可以用flatMap来一步完成转换和合并，代码更简洁，性能和for循环差距不大：

const fieldProductMap = {
  // 同上的映射表
};

const products = records.flatMap(record => {
  const matchedProducts = [];
  for (const [field, name] of Object.entries(fieldProductMap)) {
    if (record[field] === 'true') {
      matchedProducts.push(name);
    }
  }
  return matchedProducts;
});

为什么这样优化？ #

• 映射表的优势：把字段和产品名称的对应关系集中管理，不用写几十上百个if，后期维护成本极低，而且遍历映射表的性能和一堆if差不多，但代码整洁度提升几个档次。
• 修复逻辑错误：把products移到循环外才是正确收集所有匹配项的方式，原代码的写法完全是逻辑bug。
• 缓存长度：减少了不必要的属性访问，在处理超大数组时能看到细微的性能提升。

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