node.js async for loop

Question

My Scenario:

Inside a forEach loop i executed a function with callback as like below:

var rows = [];
rows.forEach(function(elem, i){
///done some stuff

     client.search(query).then(function(facResult){
     //done some stuff         
     });    
});

My problem is for first iteration the "client.search(query)" is executed before finishing this stuff the second iteration is started because client.search(query) is executed with callback. I have to push first all result into an array.

So before i am getting first result the second is appended, sometimes i am not getting even the first result.

What i have to do to execute the for loop in order. I searched a lot , some suggestions is to use timedelay, but i don't need that. Help me to solve this. Thanks in advance.

You can use async library. Look at this post: stackoverflow.com/questions/9644197/… — Aviv Shaked
– Aviv Shaked, Commented Feb 11, 2015 at 5:44

Community · Accepted Answer · 2017-05-23 12:12:38Z

0

You are running into race-condition issues because of using 'forEach'. Instead of getting all the rows into 'rows', use a 'cursor' instead.

You can see the following answer for implementation which is similar to your use-case : https://stackoverflow.com/a/18119789/802651

edited May 23, 2017 at 12:12

CommunityBot

11 silver badge

answered Feb 11, 2015 at 5:34

Adarsh Konchady

2,7375 gold badges32 silver badges52 bronze badges

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

2 Comments

Subburaj Over a year ago

its not a mongodb connection rows. Actually its just an array.. see my editd post..

Adarsh Konchady Over a year ago

Could you create a fiddle for this then?

Subburaj · Accepted Answer · 2015-02-11 06:40:30Z

0

I used async library for this as like below:

async.forEach(myVar.rows, function (elem, next){ 

    client.search(query).then(function(facResult){
     //done some stuff      
      if(--myVar.rows.length == 0){
          loop finishes //done some stuff
      }

     });  

});

answered Feb 11, 2015 at 6:40

Subburaj

5,21211 gold badges48 silver badges95 bronze badges

Collectives™ on Stack Overflow

node.js async for loop

2 Answers 2

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related