Loop to get asynchronous data in nodejs sequelize

Question

I'm working with nodejs and sequelize framework but I'm having issues trying to retrieve some data

getAllMedicamentsWithTotal: function () {
    return medicamentService.getAll().then(function success(medicaments) {
        for(var i = 0 ; i < medicaments.length; i++){
            getTotalMedicament(medicaments[i]).then(function (total) {
                medicaments[i].dataValues.total = total;
            })
        }
    }, function fail() {

    })
}

I have this function who gets all the medicament with its total in the stock. But the outer promise end before the callback its executed. I know there is an error over looping promises but is there a better way to do this than fixing this piece of code?

To give a little more context I have a table inventory who has a foreign key to the table medicaments, and in the inventory table there is the quantity of the medicaments. I want to get something like [{name1, sum(stockQuantity1)}, {name2, sum(stockQuantity2)}, ...] but I cant manage to do so.

I would appreciate any help

use Promise.all instead of a for loop bluebirdjs.com/docs/api/promise.all.html — Daniel Lizik
– Daniel Lizik, Commented May 24, 2016 at 15:53
So inside the outer promise I would put something like return Promise.all(...)?. I'm not sure the the callback function will be executed before the response is sent — ocespedes
– ocespedes, Commented May 24, 2016 at 16:14

Community · Accepted Answer · 2017-05-23 11:44:06Z

1

You have to wrap everything in a promise like this

getAllMedicamentsWithTotal: function () {
  return new Promise((resolve, reject) => {

    // call your service first
    medicamentService
      .getAll()

      // with your array of results we need to map out an array of promises
      // then feed that array into Promise.all() 
      .then(medicaments => Promise.all(medicaments.map(med => {
        return getTotalMedicament(med);
      }))

      // the result of Promise.all() is an array of results
      // reduce the totals to get your accumulated total and resolve it to the caller
      .then(arrayWithTotals => {
        let total = arrayWithTotals.reduce((acc, obj) => acc += obj.dataValues.total, 0);
        resolve(total);
      });
  });

// elsewhere...

getAllMedicamentsWithTotal().then(total => { /** your total */ });

As an aside it looks like you are doing a lot of logic for something that can most likely be done with a join query. IIRC sequelize calls this the include property on your query object.

edited May 23, 2017 at 11:44

CommunityBot

11 silver badge

answered May 24, 2016 at 16:27

Daniel Lizik

3,1422 gold badges24 silver badges44 bronze badges

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

1 Comment

ocespedes Over a year ago

I found that doing a simple group by would do the trick ... still I have an error with the group by and the join but that is a separate issue thanks for this answer

Collectives™ on Stack Overflow

Loop to get asynchronous data in nodejs sequelize

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related