JavaScript函数secretCipher字符替换功能调试求助

问题描述 #

需要实现secretCipher函数，接收字符串sentence和对象cipher，返回替换后的字符串：每个字符替换为cipher中对应的值，若字符不在cipher中则保留原字符。现有代码无法正确匹配字符与cipher的键，导致替换失效。

现有代码的问题分析 #

1. 变量作用域错误：第一个for...in循环里定义的value，在第二个遍历字符串的循环中无法访问，替换时拿不到正确值。
2. 存在性判断错误：sentence[i].includes(cyf)逻辑错误——cyf是键数组，includes是字符串方法（用于判断子串），正确做法是直接检查当前字符是否是cipher的键。
3. 字符串不可变误区：sentence[i] = ...的赋值操作无效，因为JavaScript中字符串是不可变类型。
4. 冗余代码：第一个for...in循环没有实际作用，完全可以去掉。

修正后的代码 #

function secretCipher(sentence, cipher) {
  let result = '';
  // 遍历字符串的每个字符
  for (let i = 0; i < sentence.length; i++) {
    const char = sentence[i];
    // 若字符在cipher中有对应值，就取替换值，否则保留原字符
    result += cipher[char] || char;
  }
  return result;
}

// 测试用例
console.log(secretCipher("lqq me on flcebzzk" , { l : "a", q : "d", z: "o"})); //=> "add me on facebook"
console.log(secretCipher("where are you???" , { v : "l", '?' : "!"})) //=> "where are you!!!"
console.log(secretCipher("twmce" , { m : "n", t : "d", w : "a"})); //=> "dance"

代码说明 #

• 直接遍历字符串的每个字符，用cipher[char]快速获取替换值——如果字符不在cipher中，cipher[char]会返回undefined，此时利用||运算符的短路特性，直接使用原字符。
• 去掉了冗余的循环和无效操作，逻辑更简洁高效。
• 也可以用Array.from结合map、join实现更简洁的写法：

function secretCipher(sentence, cipher) {
  return Array.from(sentence).map(char => cipher[char] || char).join('');
}

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