在运行 for 循环之前检查是否 - javascript

Question

在 Typescript (ES6) 中，我有一个以 1 毫秒为间隔运行的队列，我想知道哪种方法对性能更好。

1.

setInterval(() => {
  //if (this._queue.filter(a => !a.running && a.cbs.length) { // incorrect
  if (this._queue.filter(a => !a.running && a.cbs.length).length) { //edit
    for (let i = 0; i < this._queue.filter(a => !a.running && a.cbs.length).length; i++) {
      ...
    }
  }
}, 1);

setInterval(() => {
  for (let i = 0; i < this._queue.filter(a => !a.running && a.cbs.length).length; i++) {
    ...
  }
}, 1);

在方法 #1 中，它显然有一行额外的代码，但我很确定 if 会在每次间隔迭代中花费更少的 CPU 计算。 这样对吗？

在方法#2 中，它必须定义 i，然后运行过滤器，然后尝试迭代。

这可能是性能差异如此之低，这可能无关紧要，但我仍然很感兴趣。

Answer 1

你的 if 语句不正确

if (this._queue.filter(a => !a.running && a.cbs.length) {
// this always resolves as true
// should be
if (this._queue.filter(a => !a.running && a.cbs.length).length) {

只需重用迭代

setInterval(() => {
  const arr = this._queue.filter(a => !a.running && a.cbs.length)
  if (arr.length) {
    for (let i = 0; i < arr.length; i++) {
      ...
    }
  }
}, 1);

Answer 2

在这两种情况下，您都在每次迭代中评估过滤器表达式。 您能做的最好的事情就是对过滤器进行一次评估：

setInterval(() => {
  const queueLength = this._queue.filter(a => !a.running && a.cbs.length).length;
  for (let i = 0; i < queueLength;  i++) {
    ...
  }
}, 1);

Answer 3

注意确保您在 for 之外评估过滤器的值：否则您将在每次迭代时评估过滤器的值，即使不依赖于迭代 i：

您可以通过在过滤器 iteretee 中添加console log来验证这一点。

setInterval(() => {
 for (let i = 0; i < this._queue.filter(a => {console.log(a); reurn !a.running && 
 a.cbs.length} ).length; i++) 
 {
 ...
 }
 }, 1);

所以最好在循环外定义一个具有值this._queue.filter(a => !a.running && a.cbs.length)的this._queue.filter(a => !a.running && a.cbs.length)