javascript - 如何重构我的代码以减少嵌套循环的数量？

Question

这是一个辅助函数，它将类似对象的数组转换为实际数组，然后遍历可迭代对象，将列表中的每个值提供给回调函数:

var each = function(iterable, callback) {
  iterable = Array.prototype.concat.apply([], iterable);
  for(var i = 0; i < iterable.length; i++) {
    callback.apply(iterable[i], [iterable[i], i]);
  }
  return iterable;
};

这里我使用前面提到的辅助函数来遍历数组:

var found = [];
each(arguments, function(argument) {
  each(argument.split(","), function(selector) {
    each(handle(selector), function(element) {
      if(found.indexOf(element) < 0) {
        found.push(element);
      }
    });
  });
});

第一个循环遍历参数。第二个循环拆分选择器，第三个循环遍历请求的元素并将它们添加到 found 数组(如果尚未添加)。

注意:handle 函数采用选择器(字符串)并使用 document.querySelectorAll 返回元素列表。

这个脚本可以工作，但问题是可读性和性能。

当有许多参数包含多个(~5-10)逗号分隔的选择器，然后由 handle 函数单独处理时，会出现性能问题。

我通过使用类而不是 id 来解决这个问题。

然后是可读性问题，我试图通过将第二个循环移出父循环来解决这个问题，但这需要创建更多变量，唯一的区别是改变了在哪里 each 循环，这使得可读性更差，因为要阅读的代码更多。

问题:如何重构我的代码以减少嵌套循环的数量？

还有，第一个循环有必要吗？如果我不使用它，我将如何遍历参数以拆分它们以获得每个单独的选择器？我知道 split 方法是针对 String 类型的，不能在数组上调用。

注意:我使用的是 vanilla JavaScript，不包括任何库、框架或外部脚本。

Answer 1

看来您要收集一些出现在某些顺序数据源中的值，例如

["A,B", "C,A,D", "A", "C,E,B"]

进入集合(没有重复项)，例如

{"A", "B", "C", "D", "E"}

您可以在没有任何第三方库的情况下使用三个嵌套循环来做到这一点(无需担心性能或可读性):

const s = new Set();
for (let x of arguments) {
    for (let g of x.split(",")) {
        for (let i of g) {
            s.add(i);
        }
    }
}

按功能编写，您可以将整件事缩减为:

new Set(arguments.join().split(','))

这当然假设您的任何选择器中都没有逗号。

这里的列表并不是真正的嵌套，因为您分三步完全处理了原始列表。分割:

• 原文:[ 'A,B', 'C,A,D', 'A', 'C,E,B' ]
• 加入后:'A,B,C,A,D,A,C,E,B'
• 拆分后:[ 'A', 'B', 'C', 'A', 'D', 'A', 'C', 'E', 'B' ]
• 设置后:设置 { 'A', 'B', 'C', 'D', 'E' }

我认为即使在循环情况下，您也具有线性复杂性，因为您实际上是在对原始数组进行三次遍历。并不是每个元素都在每次传递中都被触及，所以尽管在原始情况下它看起来像三次复杂度，但您应该没问题，但如果情况看起来很糟糕，请考虑进行分析。