JavaScript凯撒密码函数为何无法保留感叹号、逗号与空格？

核心问题1：特殊字符判断逻辑错误 #

你代码里的else if(userInput[i] === "!" || "," || " ")逻辑完全错误——JavaScript的||运算符要求两边都是完整的比较表达式，单独的","或" "属于真值，会导致这个条件永远成立。再加上这段代码嵌套在遍历字母表的内层循环里，每个字符会被重复push26次，最终结果里的特殊字符会大量重复，同时字母的处理也会出现混乱。

核心问题2：代码结构不合理 #

特殊字符不需要参与字母表遍历，应该优先判断：如果是特殊字符直接加入结果，跳过后续循环；只有字母才需要去匹配移位逻辑。

修复后的代码 #

const caesar = function(x, y) {
    const alphaListLower = ["a", "b", "c", "d", "e", "f", "g", "h", "i", "j", "k", "l", "m", "n", "o", "p", "q", "r", "s", "t", "u", "v", "w", "x", "y", "z"];
    const alphaListUpper = ['A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'];

    const userInput = x.split("");
    const result = [];

    for(let i = 0; i < userInput.length; i++){
        const char = userInput[i];
        // 优先处理特殊字符，直接加入结果后跳过后续逻辑
        if (char === "!" || char === "," || char === " ") {
            result.push(char);
            continue;
        }
        // 处理小写字母移位
        const lowerIndex = alphaListLower.indexOf(char);
        if (lowerIndex !== -1) {
            let newIndex = lowerIndex + y;
            // 兼容循环移位（含负数移位场景）
            newIndex = (newIndex % 26 + 26) % 26;
            result.push(alphaListLower[newIndex]);
            continue;
        }
        // 处理大写字母移位
        const upperIndex = alphaListUpper.indexOf(char);
        if (upperIndex !== -1) {
            let newIndex = upperIndex + y;
            newIndex = (newIndex % 26 + 26) % 26;
            result.push(alphaListUpper[newIndex]);
        }
    }

    return result.join("");
};

修复说明 #

1. 调整判断顺序，先处理特殊字符，避免无效的字母表遍历
2. 修复特殊字符的判断逻辑，改为逐个字符的完整比较
3. 用indexOf替代内层循环，简化代码同时提升运行效率
4. 优化移位计算逻辑，兼容负数移位的场景（比如传入y=-3实现左移）
5. 每个字符仅处理一次，彻底解决重复添加的问题

现在调用caesar('Hello, World!', 5)会正确返回Mjqqt, Btwqi!

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