判断一个字符串是否是回文

Question

我想知道我的逻辑哪里出了问题。将字符串 (str) 作为参数（字符串可以是小写、大写、逗号和句点）并设置为等于 var string。然后将其反转，如果它等于原始字符串，则返回值为true。如果不等于返回值false，则显示。为什么它评估一切都是真实的？

 function palindrome(str) {
     var string =str.toLowerCase().replace(/\s/g, '').replace(/,/g , '').replace(/./g , '');

     if (string==string.split("").reverse().join("")) {
         return true;
     }
     else{
         return false;
     }      
}

palindrome("nope");

Answer 1

tl;dr 将最后一个正则表达式中的 . 替换为 \. 以去除文字 .s;现在，您正在删除 所有 个字符。

问题

在正则表达式中，. 匹配任何字符（除了换行符，\n，至少在默认情况下）。所以，这段代码替换了 any 字符：

replace(/./g , '')

因此，您正在清空字符串。空字符串的反转是一个空字符串，所以你总是得到一个返回值true。

解决方案

要匹配文字句点，您必须通过编写 \. 来转义 .。所以，把那部分改成

replace(/\./g , '')

一些重构让你的代码更棒

您也可以只返回比较结果；这里不需要if/else 块。这将使您的整个代码（为便于阅读而格式化）：

function palindrome(str) {
    var string =str.toLowerCase()
                    .replace(/\s/g, '')
                    .replace(/,/g , '')
                    .replace(/\./g , '');

    return string==string.split("").reverse().join("");
}

palindrome("nope");

您可以更进一步，只删除所有非字母字符，这样您的代码就可以更简单：

function palindrome(str) {
    var string =str.toLowerCase().replace(/[^a-z]/g, '');

    return string==string.split("").reverse().join("");
}

palindrome("nope");