0

我一直在编写一个 javascript 函数，如果该值与大约 4 个值之一匹配（在下面的示例中只有 3 个），则该函数返回 true。问题是，当我只有两个值时，函数可以正常工作，但添加第三个值会破坏代码。

我对javascript很陌生，我猜有更好的方法吗？我已经尝试过搜索，但目前还没有找到任何东西。

任何帮助深表感谢。

function isValid(elem, helperMsg){

                var sn6 = /[sS][nN]6/;
                var sn5 = /[sS][nN]5/;
                var sn38 = /[sS][nN]38/;

                if(elem.value.match(sn6 || sn5 || sn38)){
                        //do stuff
                        return true;
                }else{

                        return false;
                }
            }

编辑：

这是我对数组的第二次尝试：

    function isLocal(elem, helperMsg){
        var validPostcodes=new Array();
            validPostcodes[0]= /[wW][rR]12/;       
            validPostcodes[1]= /[cC][vV]35/;
            validPostcodes[2]= /[sS][nN]99/;
            validPostcodes[3]= /[sS][nN]6/;
            validPostcodes[4]= /[sS][nN]5/;
            validPostcodes[5]= /[sS][nN]38/;
            validPostcodes[6]= /[oO][xX]29/;
            validPostcodes[7]= /[oO][xX]28/;

        var i = 0;

        for (i = 0; i < validPostcodes.length; ++i) {
            if(elem.value.match(validPostcodes[i])){
                    // do stuff
                    return true;
            }else{
                    alert(helperMsg);
                    elem.focus();
                    return false;
            }
        }
    }
4

1 回答 1

4 
a || b || c

是一个计算结果为布尔值的表达式。这意味着您正在运行match(true)或match(false)。你必须把它写成：

match(a) || match(b) || match(c)

另一种选择是将它们存储在一个数组中并循环遍历它。这意味着如果模式的数量增加，您不必更改模式列表以外的代码。另一种方法虽然仅限于这种情况，但可能是将模式更改为等同于将三个选项组合在一起的模式（未经测试，我对正则表达式有点生疏）：

elem.value.match(/[sSnN][6|5|38]/)

基于数组的示例：

var patterns = [/../, /.../];
for (var i = 0; i < patterns.length; ++i) {
    if (elem.value.match(patterns[i])) { return true; }
}

在实际代码中，我可能会像这样格式化它：

function isValid(elem, helperMsg){
    var patterns = [/../, /.../],
        i = 0;
    for (i = 0; i < patterns.length; ++i) {
        if (elem.value.match(patterns[i])) {
            return true;
        }
    }
}

这只是一种习惯，因为 JavaScript 会将变量提升到其作用域的顶部。绝不需要像这样声明变量。

于 2012-05-14T08:17:06.160 回答