从javascript函数返回true或false

Question

我正在对函数中的字符串进行正则表达式检查:

function ValidateZipCodeString(listOfZipCodes) {
    var regex = /^([, ]*\d{5})+[, ]*$/,
    matches = regex.exec(listOfZipCodes);

    if (regex.exec(listOfZipCodes) === null) {
        console.log('validation failed');
        return false;
    } else {
        console.log('validation passed');
        return true;
    }
}

正则表达式正确检测有效/无效的邮政编码列表.

我用这个调用函数:

console.log('zip code: ' + listOfZipCodes);
if (ValidateZipCodeString(listOfZipCodes)) {
    $tr.find('label#lblCoverageEditError').text('There is invalid text in the list of zip codes. Only 5-digit zip codes allowed.').show();
} else {
    console.log('validate function returned true');
}

问题是当验证函数中的控制台输出显示"验证失败"时,上面的if/else转到else子句.所以我一定不能正确地称这个功能.

做我正在做的事情的正确方法是什么？

Answer 1

您的功能可以大大简化为:

function ValidateZipCodeString(listOfZipCodes) {
    var regex = /^([, ]*\d{5})+[, ]*$/;

    if (regex.test(listOfZipCodes)) {
        console.log('validation passed');
        return true;
    } else {
        console.log('validation failed');
        return false;
    }
}

...要么:

function ValidateZipCodeString(listOfZipCodes) {
    var regex = /^([, ]*\d{5})+[, ]*$/;
    return regex.test(listOfZipCodes);
}

......甚至只是:

function ValidateZipCodeString(listOfZipCodes) {
    return /^([, ]*\d{5})+[, ]*$/.test(listOfZipCodes);
}

......但真正的问题(正如Teemu 指出的那样)不在你的功能中,而在于它的使用.你的函数回答了一个问题,"这是一个有效的邮政编码字符串吗？",但你对它的使用是说:"如果我的函数说的话,说这是无效的."

Answer 2

实际上true,验证失败时,验证功能不会返回.你只是错误地检查了它,它应该是:

if (!ValidateZipCodeString(listOfZipCodes)) {
    $tr.find('label#lblCoverageEditError').text('There is invalid text in the list of zip codes. Only 5-digit zip codes allowed.').show();
} else {
    console.log('validate function returned true');
}