JavaScript替换数组对象指定字符并新增firstname字段实现问题

你的代码存在三个核心问题：

1. 逻辑冗余：需求仅需要处理Subname字段，不需要遍历对象的所有属性
2. 字段名赋值错误：item['firstname'[i]]的写法会把firstname作为字符串按下标i取单个字符，当i为0时取'f'、i为1时取'i'、i为2时取'r'，这就是输出中出现f、i、r三个异常字段的直接原因
3. 方法误用：Array.map用于生成新数组，你没有接收返回值也没有在回调中返回数据，属于无意义调用，仅遍历修改原数组用Array.forEach更合适

如果你可以接受直接修改原数组，用以下代码即可得到预期结果：

var dataList = [
{ 
    "Name": "Max", 
    "Age": "42",
    "Subname": "Ünger Pes"
},{ 
    "Name": "Bertha", 
    "Age": "53",
    "Subname": "Kl. Fr"
}];

dataList.forEach(item => {
  item.firstname = item.Subname.toLowerCase()
    .replaceAll('.', '')
    .replaceAll(' ', '')
    .replaceAll('ü', 'ue');
});

如果你不想修改原数组，需要生成新的数组，用map的正确写法如下：

const newDataList = dataList.map(item => {
  return {
    ...item,
    firstname: item.Subname.toLowerCase()
      .replaceAll('.', '')
      .replaceAll(' ', '')
      .replaceAll('ü', 'ue')
  }
});

你当前的链式replace写法有两个明显缺陷：

• 传入字符串作为匹配规则时，只会替换第一个命中的字符，存在多个空格/点的场景下会替换不全
• 多次调用replace会多次遍历整个字符串，数据量大时性能损耗明显

更高效的实现方式是预定义替换映射表，用正则一次完成全局匹配替换，后续新增替换规则也不需要修改逻辑，只需要更新映射表即可：

// 替换规则映射，新增规则直接在此处添加即可
const replaceMap = {
  '.': '',
  ' ': '',
  'ü': 'ue'
};
// 生成全局匹配正则
const replaceReg = new RegExp(`[${Object.keys(replaceMap).join('')}]`, 'g');

// 业务逻辑
dataList.forEach(item => {
  item.firstname = item.Subname.toLowerCase()
    .replace(replaceReg, match => replaceMap[match]);
});

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