Loop with nested promise

Question

What is the best way to get result from a promise inside a For loop. In this example code resultArray is not complete when the loops ends.

  var resultArr = [];
  var itemArray1 = [1, 2, 3, 4, 5];
  var itemArray2 = ['a','b','c','d','e'];
  for (var a = 0; a < itemArray1.length; a++) {
      for (var b = 0; b < itemArray2.length; b++) {
          myPromise(a,b)
          .then(function(result) {
                if (result != null) {
                    resultArr.push(result);
                }
          });

      }
  }  

  // resultArray is still not complete


 function myPromise(a,b) {
     return new Promise(function(resolve, reject) {
         // request to mongodb
         myTable.findOne({ _id:a, name:b }, function(err,result) {
             if (err) throw err;
             resolve(result);
         });
     });
 }

You can use for-each loop..

Subburaj
– Subburaj

2016-04-20 06:22:03 +00:00
Commented Apr 20, 2016 at 6:22 — Subburaj
– Subburaj, Commented Apr 20, 2016 at 6:22
Use Promise.each or Promise.map

user5383152
– user5383152

2016-04-20 06:34:32 +00:00
Commented Apr 20, 2016 at 6:34 — user5383152
– user5383152, Commented Apr 20, 2016 at 6:34

JLRishe · Accepted Answer · 2016-04-20 10:07:37Z

1

In my opinion, the cleanest way to do things like this is to use Promise.all() with Array#map. Also, make sure to give keep your functions clean and concise, and give them meaningful names!

var itemArray1 = [1, 2, 3, 4, 5];
var itemArray2 = ['a','b','c','d','e'];

function flatten(arrays) {
    return [].concat(arrays);
}

function queryAs() {
    return Promise.all(itemArray1.map(queryBs))
        // the result is an array of arrays, so we'll flatten them here
        .then(flatten);
}

function queryBs(a) {
    return Promise.all(itemArray2.map(function (b) {
        return performQuery(a, b);
    }));
}

// resultArray is still not complete
function performQuery(a, b) {
    return new Promise(function(resolve, reject) {
        // request to mongodb
        myTable.findOne({ _id:a, name:b }, function(err,result) {
            if (err) throw err;
            resolve(result);
        });
    });
}

queryAs().then(function (results) {
    console.log(results);
});

edited Apr 20, 2016 at 10:07

answered Apr 20, 2016 at 7:05

JLRishe

102k19 gold badges138 silver badges171 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Danish Woman Over a year ago

Thank you very much. This was really a very clean way to do it!

user5383152 · Accepted Answer · 2016-04-20 06:41:45Z

1

You could use a combination of forEach and Promise.all

var resultArr = [];
var itemArray1 = [1, 2, 3, 4, 5];
var itemArray2 = ['a', 'b', 'c', 'd', 'e'];

var myPromise = (item1, item2) => new Promise((resolve, reject) => {
    myTable.findOne({
        _id: item1,
        name: item2
    }, function(err, result) {
        if (err) reject(err);
        else resolve(result);
    });
});

var promises = [];
itemArray1.forEach(item1 => {
    itemArray2.forEach(item2 => {
        promises.push(myPromise(item1, item2));
    });
});

Promise.all(promises).then(result => {
    console.log(result);
}).catch(err => {
    console.error(err.message);
});

answered Apr 20, 2016 at 6:41

user5383152

1 Comment

Danish Woman Over a year ago

Thank you for your answer. This was helpful too, also if I did not use this answer.

Collectives™ on Stack Overflow

Loop with nested promise

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related