【问题标题】:Synchronous loop in Promise allPromise all 中的同步循环
【发布时间】:2018-07-08 10:19:16
【问题描述】:

我想在我的部分代码中做一个同步循环。 函数 saveInDatabase 检查数据库中是否已存在项目标题(字符串)。这就是为什么它不能并行解决,否则条件永远不会适用(并且会产生重复)。

Promise.all(arr.map(item => {
     saveInDatabase(item).then((myResult) => ... );
}));

我尝试将这个函数封装到单独的promise中,也尝试使用npm包(synchronous.js,sync),但它似乎不适合我的代码。

也许这个解决方案完全是愚蠢的。 您认为将 promise.all 替换为同步循环(例如 foreach)是一个更好的主意吗?问题是我需要每次迭代的结果...

我使用的是节点 6.11.2。你能给我一些处理这个问题的技巧吗? 先感谢您。

【问题讨论】:

  • Promise.all 可能是这里工作的错误工具。您需要一次运行一个,用 .then 链表示比传递给 promise.all 的数组更好。如果你有一个数组,[].reduce 可以为你设置该链
  • let looper = (current, max) => { saveInDatabase(arr[current]).then( r => { looper(current + 1, max) }) }; looper(0, arr.length); 只是给你一个想法,你知道...... Promise.all 在概念上对我来说无论如何都是错误的(在这种情况下)
  • synchronous loop 异步代码永远不是解决方案。 Promise.all 不一定是错误的,如果你想在最后一次迭代完成后访问所有结果。事实上,您使用 .map 而没有从 .map 回调中返回任何内容,这意味着您已经在使用 .map,就好像它是 .forEach 一样!

标签: javascript node.js mongoose promise


【解决方案1】:

不使用await(不在node.js v6.11.2 中,但会使这更简单),序列化一堆返回承诺的异步操作的经典模式是使用reduce() 循环,如这个:

arr.reduce(function(p, item) {
    return p.then(function() {
        return saveInDatabase(item).then((myResult) => ... );
    });
}, Promise.resolve()).then(function() {
    // all done here
}).catch(function(err) {
    // error here
});

如果要保存所有结果,可以使用.then(myResult => ...) 处理程序将.push() 结果放入一个数组中,完成后可以访问该数组。

这将序列化所有对saveInDatabase(item) 的调用,使其在调用第二个之前等待第一个完成,在调用第三个之前等待第二个完成,等等...

如果saveInDatabase(item) 拒绝,这里的默认实现将停止。如果你想继续(你没有在你的问题中说),即使它给出了一个错误,那么你可以添加一个.catch() 来将拒绝的承诺变成一个履行的承诺。


在 node.js v7+ 中,您可以在常规的 for 循环中使用 await

async function myFunc() {
    let results = [];
    for (let item of arr) {
        let r = await saveInDatabase(item).then((myResult) => ... );
        results.push(r);
    }
    return results;
}

myFunc().then(function(results) {
    // all done here
}).catch(function(err) {
    // error here
});

如果您可以并行运行所有请求,那么您可以这样做:

Promise.all(arr.map(item => {
     return saveInDatabase(item).then((myResult) => ... );
})).then(function(results) {
    // all done here
}).catch(function(err) {
    // error here
});

在其中任何一个中,如果您不希望它在被拒绝时停止,则将 .catch() 添加到您的 saveInDatabase() 承诺链中,以将拒绝变成具有某些已知值或错误值的已解决承诺可以检测到。

【讨论】:

  • 感谢您的精彩解释。不过,有一些事情让我很困扰。如果我将所有结果放入一个数组中,如果有任何重复(根据其标题),这将无法解决检查数据库的问题。我想我会修改我的函数 saveInDatabase 的目的:它只会返回所有项目的列表,而不是在数据库中“保存”。然后我可以检查最终数组中的重复条件。是的,我忘了告诉一个项目可以返回子项目,等等......
猜你喜欢
  • 1970-01-01
  • 2021-10-30
  • 2020-05-31
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2023-04-06
相关资源
最近更新 更多