邮件正文变量替换仅最后一项生效的问题排查与解决

你遇到的问题是：在替换邮件正文模板变量时，只有variableNames数组的最后一项（这里是NAME4）被成功替换，其他变量都没生效。相关代码如下：

var variableNames = [ 'NAME1', 'NAME2', 'NAME3', 'NAME4'];
var regex = '';
var key2 = '';
var position = '';
for (var key in variables) {
 if (variableNames.includes(key)) {
 position = variableNames.indexOf(key);
 key2 = variableNames[position];
 regex = new RegExp("###" + key2 + "###", "g");
 template.content = template.body.replace(regex, variables[key2]); //only replace the last one
 }
}

问题根源 #

你每次循环都直接用**原始的template.body**来执行替换操作，而不是基于上一次替换后的结果（也就是template.content）。这意味着每次循环都会覆盖之前的替换结果：

• 第一次循环替换NAME1，结果存在template.content里；
• 第二次循环又拿未修改的template.body去替换NAME2，直接覆盖了template.content的内容；
• 以此类推，最后只有最后一次替换NAME4的结果会保留在template.content里，前面的操作全白做了。

修复方案 #

只需要把替换的基准从template.body改成每次更新后的template.content，同时记得先初始化template.content为原始模板内容：

var variableNames = [ 'NAME1', 'NAME2', 'NAME3', 'NAME4'];
var regex = '';
var key2 = '';
var position = '';
// 先初始化content为原始模板内容
template.content = template.body;
for (var key in variables) {
 if (variableNames.includes(key)) {
 position = variableNames.indexOf(key);
 key2 = variableNames[position];
 regex = new RegExp("###" + key2 + "###", "g");
 // 基于上一次替换后的content继续替换
 template.content = template.content.replace(regex, variables[key2]);
 }
}

额外优化建议 #

其实代码里的position和key2是多余的，因为key已经是variableNames里的有效项了，可以直接简化代码：

var variableNames = [ 'NAME1', 'NAME2', 'NAME3', 'NAME4'];
template.content = template.body;
for (var key in variables) {
 if (variableNames.includes(key)) {
 const regex = new RegExp(`###${key}###`, "g");
 template.content = template.content.replace(regex, variables[key]);
 }
}

如果variables的键和variableNames数组是一一对应的，还可以直接遍历数组，避免遍历variables里的多余键，逻辑更直观：

var variableNames = [ 'NAME1', 'NAME2', 'NAME3', 'NAME4'];
template.content = template.body;
variableNames.forEach(key => {
 if (variables[key]) { // 确保变量存在对应值
 const regex = new RegExp(`###${key}###`, "g");
 template.content = template.content.replace(regex, variables[key]);
 }
});

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