嵌套 jQuery 循环 - 如何更好地实现这一点？

Question

我正在整理我继承的一个库，并遇到了这个小花絮：

queries = urlParams.query.split("&");

// Split URL Queries
$.each(queries, function (i, val) {
    val = val.split("=");
    args[val[0]] = val[1];  //Assign query args into args object using their name (ie: test=123) as the key
});

// Loop through arguments
$.each(args, function (i, val) {
// Loop through affiliates to compare url arguments against those of the affiliates
    $.each(self.affiliates, function (inc, value) {
        if (value.urlTag === i) {
        self.setAffiliateCookies(i, val, 1);    //Set affiliate cookies
            gotAff = true;
            return false;
        }
    });
});

上面发生的事情的要点是它正在解析查询字符串并将元素分解为键值对。很容易。

之后发生的事情是，它循环遍历该新数组，然后测试 args 的值是否存在于 self.affiliates.urlTag 的对象文字值中。如果是这样，它会设置一个 cookie，将 gotAff 设置为 true，然后返回 false 以终止 $.each。

这对我来说似乎不太有效。我一直在使用递归函数，但我不太了解，而且我不确定我是否走错了路。我也不确定用 return false 杀死 $.each 是否是最有效的方法。

有什么想法吗？有什么建议吗？这种模式在多个地方重复出现，我很想知道如何更好地实现它。

Answer 1

如果我正确理解数据结构，这可能会更清晰一些：

$.each(self.affiliates, function (inc, value) {
    if (args.hasOwnProperty(value.urlTag)) {
        self.setAffiliateCookies(value.urlTag, args[value.urlTag], 1);    //Set affiliate cookies
        gotAff = true;
        return false;
    }
});

Answer 2

我非常确定，无论它是什么 n² 操作，因为您必须遍历每个参数，并查看它是否存在于列表中。在这种情况下，你能做的最好的事情就是以某种方式澄清代码，但我不确定你是否有正确的函数在 jQuery 中执行此操作。

作为示例，我将在 MooTools 中执行此操作：

// Create array of affiliate URL tags.
var affiliateURLs = self.affiliates.map(function(affiliate) {
    return affiliate.urlTag;
});

// Filter args list to those with affiliates.
// This is the n² part.
var matchedArgs = Object.filter(args, function(arg, argURL) {
    return affiliateURLs.contains(argURL);
});

// Create cookie for each matched arg.
Object.each(matchedArgs, function(arg, argURL) {
    self.setAffiliateCookies(argURL, arg, 1);
});

// Note that gotAff is simply
// (Object.getLength(matchedArgs) > 0)