1

我在javascript中使用了一种递归方法，它工作得很好，直到我把它放在一个命名空间中。该函数从数组中返回一个元素，该元素具有给定的quoteproduct id 作为它的id 属性。这是一个嵌套数组，这就是函数递归的原因。这是函数声明：

QuoteProductService.getQuoteProduct = function (quoteproductid) {
    var founditem = null;
    $.each(QuoteProductService.QuoteProductConfigurations, function (index, item) {
        if(item.id == quoteproductid) {
            founditem = item;
            return false; // break the $.each if an item is found
        } else {
            founditem = QuoteProductService.getQuoteProduct(item.children, quoteproductid);
            if(founditem != null) return false; // break the $.each if an item is found
        }
    });
    return founditem;
}

这就是我声明命名空间的方式：

var QuoteProductService = QuoteProductService || {};

这是我在函数中使用的命名空间中的数组：

QuoteProductService.QuoteProductConfigurations = [];

该数组在页面加载时填充。

现在，每当我调用该函数时，都会出现“递归过多”错误。我究竟做错了什么 ？同样，在我将函数和数组放入命名空间之前，这个函数就起作用了。

4

1 回答 1

3

我刚刚用更简单的变量名重写了您的代码：

var a = {
    b: = [{id: 1}, {id: 2}, {id: 3}]
};
a.get = function( searchId ) {
    var match = null;

    $.each(a.b, function(key, value) {
        if ( value.id === searchId ) {
            // Yes we found the match, break and everything

            match = value;
            return false;
        }
        else {
            match = a.get();

            if ( match ) {
                return false;
            }
        }
    });
    return match;
};

a.get(1) // will return {id: 1}
a.get(2) // will throw recursive error

为什么？

由于您的结构，正如您始终指向的$.each那样a.b。

因此它是这样的：

循环a.b：ab[0].id === searchId？
好的一切都很好返回第一个值

如果不是 ab[0].id === searchId
循环a.b
ab[0].id === searchId ？
好的，一切都很好
，如果不是 ab[0].id === searchId 则返回第一个值
循环a.b
......

希望你能理解：

要解决此问题，您需要指定我们必须循环的数组：

QuoteProductService.getQuoteProduct = function (quoteproductid, loopArray) {
    var founditem = null;

    // if (loopArray) {loopArray = loopArray} else { loopArray=Quote...QuteConfig.. }
    loopArray = loopArray || QuoteProductService.QuoteProductConfigurations;

    $.each(loopArray, function (index, item) {
        if(item.id == quoteproductid) {
            founditem = item;
            return false; // break the $.each if an item is found
        } else {
            founditem = QuoteProductService.getQuoteProduct(quoteproductid, item.children);
            if(founditem != null) return false; // break the $.each if an item is found
        }
    });
    return founditem;
}
于 2012-11-16T15:15:40.783 回答