Node JS Promise.all和forEach

一尘不染

Node JS Promise.all和forEach

javascript

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

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

对于n级嵌套，该模式应该是可重复的。
我需要使用promise.all或一些类似的技术来确定何时解析封闭例程。
并非每个元素都必然涉及进行异步调用。因此，在嵌套promise.all中，我不能仅仅基于索引对我的JSON数组元素进行分配。不过，我确实需要在嵌套的forEach中使用诸如promise.all之类的内容，以确保在解析封闭例程之前已进行了所有属性分配。
我正在使用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();

阅读 394

2020-05-01

共1个答案

一尘不染

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

每当您在中创建承诺时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]
});

2020-05-01