for循环中使用JavaScript .replace()方法无效如何解决

代码不生效来自两个核心错误：

• JavaScript 中字符串是不可变类型，.replace() 方法不会修改原始字符串，执行后会返回替换后的新字符串。原代码在循环中调用replace后，没有将返回结果重新赋值给数组对应位置的元素，所有替换结果都被直接丢弃，最终拼接的仍然是原始字符。
• 函数声明关键字书写错误：JavaScript 中声明普通函数的关键字为小写的function，原代码大写开头的Function是内置的函数构造函数，不符合普通函数的声明语法。

最小改动修正（保留原有逻辑） #

仅修正函数声明关键字，同时将replace的返回值回写到数组元素即可：

function correct(string) {
  let array = string.split('')
  for (let i = 0; i < array.length; i++) {
    array[i] = array[i].replace('5', 'S')
    array[i] = array[i].replace('0', 'O')
    array[i] = array[i].replace('1', 'I')
  }
  return array.join('');
}

简洁实现（推荐） #

无需拆分数组遍历，直接通过全局正则匹配做批量替换，代码更简洁、执行效率更高：

function correct(string) {
  return string.replace(/5/g, 'S')
               .replace(/0/g, 'O')
               .replace(/1/g, 'I')
}

如果后续需要新增替换规则，可以用映射表+正则回调的写法，维护成本更低：

function correct(string) {
  const replaceMap = {
    '5': 'S',
    '0': 'O',
    '1': 'I'
  }
  return string.replace(/[501]/g, match => replaceMap[match])
}

注意：使用正则替换时必须添加g全局匹配修饰符，否则只会替换字符串中第一个匹配到的目标字符，无法覆盖全部符合规则的内容。

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