How to handle for loop in node.js?

Question

I am having following code in node.js.

var months =  ['jan','feb','march','april','may','june','july','august','sep','oct','nov','dec']
for(var i=0; j=months.length,i<j; i++){
  var start = scope.getCurrentUTS(new Date(2013, i, 1));
  var end = scope.getCurrentUTS(new Date(2013, i, 31));
  var query = {};
  query["stamps.currentVisit"] = {
        "$gte" : start.toString(),
        "$lt" : end.toString()
  };

     //connect to mongo and gets count coll.find(query).count(); working fine
  mongoDB.getCount(query,function(result) {
        console.log(result,i);  
  });
}

Problem : Being code is running async, last line of code is not running as expected.

output expected is

10 0

11 1

12 2

.......

........

40 11

but it is giving output as

undefined 11

Krasimir · Accepted Answer · 2013-09-18 06:21:02Z

8

Probably some of your queries doesn't match anything. That's why it returns undefined as result. But there is another problem. The i in the async callback may be not what you expected. And will be probably equal to months.length. To keep the same i you should use something like:

var months =  ['jan','feb','march','april','may','june','july','august','sep','oct','nov','dec']
for(var i=0; j=months.length,i<j; i++){
    (function(i) {
        var start = scope.getCurrentUTS(new Date(2013, i, 1));
        var end = scope.getCurrentUTS(new Date(2013, i, 31));
        var query = {};
        query["stamps.currentVisit"] = {
            "$gte" : start.toString(),
            "$lt" : end.toString()
        };
        //connect to mongo and gets count coll.find(query).count(); working fine
        mongoDB.getCount(query,function(result) {
            console.log(result,i);  
        });
    })(i);
}

Also this

for(var i=0; j=months.length,i<j; i++){

Could be just:

for(var i=0; i<months.length; i++){

answered Sep 18, 2013 at 6:21

Krasimir

13.6k4 gold badges45 silver badges58 bronze badges

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

1 Comment

lakshmankashyap Over a year ago

use should be used let in place of var in for loop for better approach

Collectives™ on Stack Overflow

How to handle for loop in node.js?

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