IndexOf与多维数组问题：代码替换结果不符合预期求助

哦，我一眼就揪出问题所在啦！你的代码里**indexOf的条件判断逻辑出错了**，这才导致"home"没被成功替换成"interior"。

问题根源 #

String.indexOf()方法的返回规则是：

• 找到匹配的子字符串时，返回它的起始索引位置（比如"home"在你的forbidden字符串里的索引是0）
• 找不到匹配时返回-1

而你写的if(forbidden.indexOf(items[i][0]))，会把返回的索引值当成布尔值判断：

• 当索引是0的时候（比如匹配"home"），0在JS里会被判定为false，直接跳过替换逻辑
• 只有索引大于0的时候（比如"decor"在索引5，"learn"在索引16），才会进入替换

这就导致最后一个替换规则["home", "interior"]根本没执行，所以结果里还留着"home"。

修复后的代码 #

把条件判断改成检查返回值不等于-1（表示找到匹配），或者用更直观的includes()方法：

方案1：修正indexOf判断

var forbidden = "home decor learn"; 
var items = [ 
  ["decor", "decoration"], 
  ["learn", "learning"], 
  ["home", "interior"], 
]; 
for(i = 0; i<items.length; i++){ 
  // 判断是否找到匹配，而不是直接用indexOf的返回值
  if(forbidden.indexOf(items[i][0]) !== -1){ 
    forbidden = forbidden.replace(items[i][0], items[i][1]); 
  } 
} 
document.write(forbidden); // 现在会输出"Interior decoration learning"

方案2：用includes()更直观（ES6+）

var forbidden = "home decor learn"; 
var items = [ 
  ["decor", "decoration"], 
  ["learn", "learning"], 
  ["home", "interior"], 
]; 
for(i = 0; i<items.length; i++){ 
  if(forbidden.includes(items[i][0])){ 
    forbidden = forbidden.replace(items[i][0], items[i][1]); 
  } 
} 
document.write(forbidden);

额外小提示 #

如果你的items里的替换词可能出现多次，replace()默认只会替换第一个匹配项。如果需要全局替换，可以用正则表达式，比如：

forbidden = forbidden.replace(new RegExp(items[i][0], 'g'), items[i][1]);

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