个人自用JavaScript代码中使用eval()做属性映射是否需要避免？

方案结论 #

完全不建议使用eval()实现该需求，哪怕是个人私用项目，也存在明显的风险和维护问题，且有更简单稳妥的替代方案。

eval()方案的明确问题 #

• 运行时错误概率更高：如果映射字符串拼写错误、源变量被意外修改/删除，eval()不会在静态检查阶段抛出提示，只有代码执行到对应逻辑才会报错，排查成本远高于普通赋值代码。
• 存在性能损耗：eval()每次执行都需要动态解析JS语法，哪怕是少量循环的场景，性能也比原生赋值低几个数量级，后续映射规则变多后更容易出现卡顿问题。
• 可维护性极差：映射逻辑隐藏在字符串中，后续排查问题、修改规则时需要逐位对照两个字符串的字符位置，非常容易出错。

适配需求的替代实现 #

你的核心需求是灵活修改「目标属性-源变量」的映射关系，完全可以用配置化的方案实现，不需要碰eval()：

1. 先把所有源变量统一收纳到一个源对象中：

// 把你所有要用到的源变量u、v、w、x、y、z都放到这个对象里
const sourceVars = { u, v, w, x, y, z }

2. 用独立的对象声明映射规则，键是目标对象obj的属性名，值是源对象中对应的变量名：

// 要改映射规则只需要修改这个对象即可，增删都非常直观
const mappingRule = {
  a: 'x',
  b: 'y',
  c: 'w',
  d: 'u',
  e: 'v',
  f: 'z'
}

3. 通用赋值逻辑固定不变，不需要跟着映射规则修改：

Object.entries(mappingRule).forEach(([targetKey, sourceKey]) => {
  obj[targetKey] = sourceVars[sourceKey]
})

三种方案的对比 #

长远来看配置化方案是最优选择，既满足你灵活修改映射的需求，也避免了eval()的各种问题，维护成本远低于另外两种方案。

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