JavaScript withoutString函数无法移除指定字符串问题求助

我来帮你梳理下代码里的几个关键问题，以及对应的修复方案：

代码里的核心问题 #

• 变量名错误：你在代码里用了strArray.length，但这个变量根本没定义，应该是removeArray.length（你之前定义的要移除的字符串数组）。
• 数组引用比较失效：JavaScript里数组是引用类型，直接用===比较两个数组，只会判断它们是不是同一个内存引用，而不是内容是否相同。所以sentenceArray.slice(i, i+removeLength) === removeArray永远会返回false，自然不会触发删除操作。
• splice用法错误：splice的参数格式是(起始索引, 删除的元素数量)，你写的i+removeLength是结束位置，完全不符合API要求。而且splice会直接修改原数组，它的返回值是被删除的元素集合，你把sentenceArray赋值为这个返回值，最后得到的会是被删掉的内容，而不是剩下的数组。
• 循环索引逻辑漏洞：当你删除数组元素后，数组长度会变短，后面的元素会往前移动，如果继续递增i，会跳过下一个本该检查的元素。

修复后的完整代码 #

let sentencePrompt = prompt("Please enter any sentence: ");
let removePrompt = prompt("Please enter a string of letter(s) to be removed from your original sentence: ");

function withoutString(sentence, str) {
  let sentenceArray = sentence.split("");
  let removeArray = str.split("");
  const removeLength = removeArray.length;
  
  // 处理要移除的是空字符串的边界情况
  if (removeLength === 0) return sentenceArray.join("");

  for(let i = 0; i <= sentenceArray.length - removeLength; ) {
    // 把数组片段转成字符串，这样就能正确比较内容
    const currentSub = sentenceArray.slice(i, i + removeLength).join("");
    const targetStr = removeArray.join("");
    
    if(currentSub === targetStr) {
      // 从当前索引开始，删除removeLength个元素
      sentenceArray.splice(i, removeLength);
      // 删除后不递增i，因为后面的元素往前移了，需要留在当前位置检查新内容
    } else {
      // 没匹配到，正常递增索引
      i++;
    }
  }
  
  // 把数组转回字符串返回，符合"移除字符串"的预期
  return sentenceArray.join("");
}

let newSentence = withoutString(sentencePrompt, removePrompt);
console.log("原语句:", sentencePrompt);
console.log("要移除的字符串:", removePrompt);
console.log("处理后的语句:", newSentence);

更简洁的替代方案 #

如果不需要刻意练习数组操作，其实JavaScript的字符串原生方法replaceAll可以一行解决问题：

function withoutString(sentence, str) {
  return sentence.replaceAll(str, "");
}

这个方法会把原字符串中所有匹配str的子串都替换为空，完美实现你的需求。

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