给函数参数赋值是否安全？复用函数参数变量是不良实践吗？

你给出的示例场景是否安全 #

你贴的代码中修改的是字符串类型的参数，这种场景本身功能上完全安全，可以正常运行：
JS中基础类型（字符串、数字、布尔值、Symbol、BigInt、null、undefined）都是按值传递的，你给函数参数重新赋值，只会修改当前函数作用域内的变量绑定，不会影响外部传入的原始变量，不会产生隐式副作用。

为什么业内普遍不推荐这种写法 #

这一规范主要是为了规避潜在风险、降低代码维护成本，核心原因有3个：

• 可读性损耗：如果函数逻辑较长，后续维护者无法第一时间判断变量是原始入参还是已经被修改过的值，排查问题时需要额外回溯变量的修改链路，增加理解成本。绝大多数团队的ESLint规则（如no-param-reassign）默认禁用参数重赋值，就是为了统一风格降低维护负担。
• 引用类型的隐式副作用风险：如果修改的是对象、数组这类引用类型的参数，风险会非常高。引用类型是按共享引用传递的，如果你直接修改参数对象的属性、给参数数组增删元素，外部传入的原始变量也会被同步修改，这种隐式的副作用极难排查，非常容易引发意料之外的线上bug。
• 影响JS引擎优化：部分JS引擎的性能优化逻辑依赖函数参数的稳定性，频繁修改参数可能会导致函数被标记为“不可优化”，极端场景下会出现不必要的性能损耗。

最佳实践建议 #

• 如果你的函数逻辑非常短、修改的是基础类型参数，且团队没有明确禁用相关写法，这种用法不会出现功能问题，可以正常使用。
• 只要参数是引用类型，或者函数逻辑较长，建议重新声明新变量承接修改后的值，写法更清晰也没有额外性能开销，比如你的示例可以调整为：

const reformat = (phone)=>{
    const formattedPhone = phone.replace("-", ",")
    // ...后续统一使用formattedPhone操作
    return formattedPhone;
}

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