同步forEach循环(等待它结束）

Synchronous forEach loop (wait for it to end)

本文关键字：结束 等待 forEach 循环 同步      更新时间：2023-09-26

Node.js中有一个函数，它接收一个数组并在其中循环，对每个元素进行一些耗时的计算。

这是一个超级简化版本的功能：

var analyze_data = function (data) {
    data.forEach(function (elm) {
        if (elm.myProp == true) {
            return true;
        }
    });
    return false;
}

本质上，如果元素的任何属性myProp等于true，我希望函数返回true。如果没有任何元素满足此条件，则函数应返回false。

但是，代码从不等待forEach循环完成。换句话说，如果数组中的第100个元素满足条件，则函数应返回true。相反，它会跳到return false;，并在forEach循环结束之前返回false。

有解决办法吗？

编辑

所以我意识到我过于简化了我的问题——我实际上使用的是Node.js包es6-promise，我的代码看起来更像这样：

var analyze_data = function (data) {
    return new Promise(function (resolve, reject) {
        data.forEach(function (elm) {
            if (elm.myProp == true) {
                resolve(elm);
            }
        });
        resolve(false);
    });
}

所以实际上，我没有在forEach函数中返回值的问题，而不是在外部函数中返回。此外，请注意函数如何解析为元素本身，而不是true，否则为false。如果其中一个元素通过了条件，我实际上想返回一些相关的数据，否则为false，表示它们都失败了。

现在有什么想法吗？：p此外，感谢所有原创答案！

这不是同步/异步执行的问题。您所做的是在for-each回调中返回true（这对于analyze_data函数是不可见的），然后在for-ech完成后返回false。

您需要使用Array.prototype.some:

var analyze_data = function (data) {
    return data.some(function (elm) {
        return elm.myProp == true;
    });
}

问题不在于forEach是异步的，因为它不是。但因为你把一个回报误认为另一个回报。

您从回调返回true，而不是从主函数返回。

forEach调用forEach，因为它对数组中的每个元素都执行，所以不能在在中间停止。以下是文档摘录：

注意：没有办法停止或中断forEach循环。解决方案是使用Array.every或Array.some.

问题是您从内部函数返回true值，但没有在外部函数

var retVal = false;
data.forEach(function (elm) {
    if (elm.myProp == true) {
        retVal = true;
    }
});
return retVal;

现在你有两个功能：

var analyze_data = ***function (data)*** {
  data.forEach(***function (elm)*** {
    if (elm.myProp == true) {
        return true; //returns out of function(elm)
    }
  });
  //the true value is gone - you didn't do anything with it
  return false; //always returns false out of function(data)
}

编辑：

data.forEach(function (elm) {
    if (elm.myProp == true) {
        resolve(elm);
    }
});
resolve(false);

您现在（可能）正在解析为elm，但之后总是解析为false。我不能100%确定这种行为，但我猜后面的行为会覆盖第一个行为。所以，你需要再次检查：

is_found = false;
data.forEach(function (elm) {
    if (elm.myProp == true) {
        resolve(elm);
        is_found = true;
    }
});
if (!is_found) {
    resolve(false);
}

当然，这似乎是你应该做的：

if (!is_found) {
   reject(false);
}

那么你可以做：

promise.then(function(result) {
    // do stuff since it found stuff :)
}, function(err) {
    // do stuff since if didn't find anything :(
});

相关文章：