Question

我有一个函数，它获取一个点分隔的字符串并将其解析为数组。我想循环这些数组元素并检查一个大于255的值并返回false，如果没有继续执行函数语句并返回true作为函数结束。但它永远不会停止循环..并且总是返回true。

这是代码：

checkipAddress = function(value){//instance value: 999.999.999.999 result:true
        debugger
        var array = value.split('.');
        $.each(array, function(index,val){
            debugger
            if(parseInt(val)>255)
                return false; // it should end the loop and exit function with return false.
        });
        return true;
    }

Answer 1

从一个函数返回并不会神奇地使调用它的那个函数结束，更不用说使用特定的返回值了。

如果你想这样做，你必须设置一个外部函数将使用的变量：

checkipAddress = function(value){
    var rv = true; // <=== Default return value
    var array = value.split('.');
    $.each(array, function(index,val){
        if(parseInt(val)>255)
            return rv = false; // <=== Assigns false to `rv` and returns it
    });
    return rv; // <=== Use rv here
}

附注：您的功能将很乐意允许"0.-234343.-1.0"和"1foo.2foo.3foo.4foo"等IP字符串。你可能会考虑：

checkipAddress = function(value){
    var rv = true; // <=== Default return value
    var array = value.split('.');
    $.each(array, function(index,str){
        str = $.trim(str);
        var val = +str;
        if(!str || val < 0 || val > 255)
            return rv = false; // <=== Assigns false to `rv` and returns it
    });
    return rv; // <=== Use rv here
}

这有点好，但它也不会检查IP是否只有四个部分，并且允许使用"1.1e2.3.4"（指数表示法）等值。当然，所有这一切都是针对IPv4的，而我们正在进入IPv6世界......

坚持使用IPv4，如果您的目标是确保它是正常形式的四部分IPv4地址，那么我会为正则表达式做好准备：

checkipAddress = function(value){
    var rv;
    var match = /^(\d{1,3})\.(\d{1,3})\.(\d{1,3})\.(\d{1,3})$/.exec(value);
    var n, part;
    if (match) {
        rv = true;
        for (n = 1; rv && n < 5; ++n) {
            part = +match[n]; // We know it's not blank from the regex
            if (part > 255) { // We know it's not negative or fractional from the regex
                rv = false;
            }
        }
    } else {
        rv = false;
    }
    return rv;
}

或者在现代浏览器上（或使用体面的Array#every垫片）：

checkipAddress = function(value){
    var match = /^(\d{1,3})\.(\d{1,3})\.(\d{1,3})\.(\d{1,3})$/.exec(value);
    rv = !match ? false : Array.prototype.slice.call(match, 1).every(function(entry) {
        // We know it's not blank, negative, or fractional from the regex
        return +entry <= 255;
    });
    return rv;
}

jquery每个循环返回false而不是结束函数

1 个答案: