Node JS Promise.all和forEach


120

我有一个类似结构的数组,它公开了异步方法。异步方法调用返回数组结构,从而返回更多异步方法。我正在创建另一个JSON对象来存储从该结构获得的值,因此我需要注意跟踪回调中的引用。

我已经编写了一个蛮力解决方案,但是我想学习一个更惯用或干净的解决方案。

  1. 对于n级嵌套,该模式应该是可重复的。
  2. 我需要使用promise.all或其他类似的技术来确定何时解析封闭例程。
  3. 并非每个元素都必然涉及进行异步调用。因此,在嵌套promise.all中,我不能仅基于索引对我的JSON数组元素进行分配。不过,我确实需要在嵌套的forEach中使用诸如promise.all之类的内容,以确保在解析封闭例程之前已进行了所有属性分配。
  4. 我正在使用bluebird promise lib,但这不是必需的

这是一些部分代码-

var jsonItems = [];

items.forEach(function(item){

  var jsonItem = {};
  jsonItem.name = item.name;
  item.getThings().then(function(things){
  // or Promise.all(allItemGetThingCalls, function(things){

    things.forEach(function(thing, index){

      jsonItems[index].thingName = thing.name;
      if(thing.type === 'file'){

        thing.getFile().then(function(file){ //or promise.all?

          jsonItems[index].filesize = file.getSize();

这是指向我要改进的工作源的链接。 github.com/pebanfield/change-view-service/blob/master/src/...
user3205931

1
我在示例中看到您正在使用bluebird,在这种情况下,bluebird实际上使(并发)和(顺序)使您的生活更加轻松,还不建议使用-我的答案中的代码显示了如何通过返回 promise 来避免这种情况。承诺都是关于返回值的。Promise.mapPromise.eachPromise.defer
本杰明·格伦鲍姆

Answers:


368

使用一些简单的规则非常简单:

  • 每当您在中创建承诺时then,都将其返回 -您不返回的任何承诺都不会在外面等待。
  • 每当您创建多个promise时,.all它们就会以这种方式等待所有promise,并且其中任何一个的错误都不会消失。
  • 每当嵌套thens时,通常都可以在中间返回 - then链通常最多为1层深。
  • 每当执行IO时,它都应该带有承诺 -应该在承诺中,或者应该使用承诺来表示其完成。

还有一些提示:

  • 映射.map比使用for/push - 更好地完成 -如果要使用函数映射值,则map可以简洁地表达一个动作一一应用并汇总结果的概念。
  • 如果是免费的,并发比顺序执行要好 -并发执行并等待它们Promise.all比先执行又执行-每次都在下一个执行之前要好。

好的,让我们开始吧:

var items = [1, 2, 3, 4, 5];
var fn = function asyncMultiplyBy2(v){ // sample async action
    return new Promise(resolve => setTimeout(() => resolve(v * 2), 100));
};
// map over forEach since it returns

var actions = items.map(fn); // run the function over all items

// we now have a promises array and we want to wait for it

var results = Promise.all(actions); // pass array of promises

results.then(data => // or just .then(console.log)
    console.log(data) // [2, 4, 6, 8, 10]
);

// we can nest this of course, as I said, `then` chains:

var res2 = Promise.all([1, 2, 3, 4, 5].map(fn)).then(
    data => Promise.all(data.map(fn))
).then(function(data){
    // the next `then` is executed after the promise has returned from the previous
    // `then` fulfilled, in this case it's an aggregate promise because of 
    // the `.all` 
    return Promise.all(data.map(fn));
}).then(function(data){
    // just for good measure
    return Promise.all(data.map(fn));
});

// now to get the results:

res2.then(function(data){
    console.log(data); // [16, 32, 48, 64, 80]
});

5
啊,从您的角度来看一些规则 :-)
Bergi 2015年

1
@Bergi某人应该真正列出这些规则以及关于诺言的简短背景。我们可以在bluebirdjs.com上托管它。
本杰明·格林鲍姆

因为我不应该只说谢谢-这个例子看起来不错,而且我确实喜欢map的建议,但是,对于只有一些对象具有异步方法的对象集合该怎么办?(上面我的观点3)我有一个想法,就是将每个元素的解析逻辑抽象到一个函数中,然后让它在异步调用响应中解析,或者在没有异步调用的地方解析它。那有意义吗?
user3205931

我还需要让map函数同时返回我正在构建的json对象和我需要做出的异步调用的结果,所以也不知道该怎么做-最后,由于我要遍历目录,因此整个过程都需要递归结构-我仍在咀嚼,但付费工作正在妨碍您:(
user3205931 2015年

2
@ user3205931 Promise很简单,而不是简单,那就是-它们不像其他东西那么熟悉,但是一旦您使用了它们,它们就会变得更好用。紧紧抓住,你会得到的:)
本杰明·格伦鲍姆

42

这是一个使用reduce的简单示例。它可以串行运行,保持插入顺序,并且不需要Bluebird。

/**
 * 
 * @param items An array of items.
 * @param fn A function that accepts an item from the array and returns a promise.
 * @returns {Promise}
 */
function forEachPromise(items, fn) {
    return items.reduce(function (promise, item) {
        return promise.then(function () {
            return fn(item);
        });
    }, Promise.resolve());
}

并像这样使用它:

var items = ['a', 'b', 'c'];

function logItem(item) {
    return new Promise((resolve, reject) => {
        process.nextTick(() => {
            console.log(item);
            resolve();
        })
    });
}

forEachPromise(items, logItem).then(() => {
    console.log('done');
});

我们发现将可选上下文发送到循环中很有用。上下文是可选的,并由所有迭代共享。

function forEachPromise(items, fn, context) {
    return items.reduce(function (promise, item) {
        return promise.then(function () {
            return fn(item, context);
        });
    }, Promise.resolve());
}

您的promise函数将如下所示:

function logItem(item, context) {
    return new Promise((resolve, reject) => {
        process.nextTick(() => {
            console.log(item);
            context.itemCount++;
            resolve();
        })
    });
}

谢谢您-您的解决方案对我有用,而其他解决方案(包括各种npm库)却没有。您已经将此内容发布到npm了吗?
SamF

谢谢。函数假定所有承诺均已解决。我们如何处理被拒绝的承诺?此外,我们如何处理带有价值的成功承诺?
oyalhi

@oyalhi我建议使用“上下文”并添加一个映射到错误的拒绝输入参数数组。这实际上是针对每个用例的,因为有些人会忽略所有剩余的承诺,而有些人则不会。对于返回的值,您也可以使用类似的方法。
史蒂芬·斯潘金

1

我经历了同样的情况。我使用两个Promise.All()解决了。

我认为这确实是一个很好的解决方案,所以我在npm上发布了它:https ://www.npmjs.com/package/promise-foreach

我认为您的代码将是这样的

var promiseForeach = require('promise-foreach')
var jsonItems = [];
promiseForeach.each(jsonItems,
    [function (jsonItems){
        return new Promise(function(resolve, reject){
            if(jsonItems.type === 'file'){
                jsonItems.getFile().then(function(file){ //or promise.all?
                    resolve(file.getSize())
                })
            }
        })
    }],
    function (result, current) {
        return {
            type: current.type,
            size: jsonItems.result[0]
        }
    },
    function (err, newList) {
        if (err) {
            console.error(err)
            return;
        }
        console.log('new jsonItems : ', newList)
    })

0

只是为了添加到提出的解决方案中,在我的情况下,我想从Firebase获取多个数据以获取产品列表。这是我的做法:

useEffect(() => {
  const fn = p => firebase.firestore().doc(`products/${p.id}`).get();
  const actions = data.occasion.products.map(fn);
  const results = Promise.all(actions);
  results.then(data => {
    const newProducts = [];
    data.forEach(p => {
      newProducts.push({ id: p.id, ...p.data() });
    });
    setProducts(newProducts);
  });
}, [data]);
By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.