javascript - nodejs and mongodb (mongojs): Trying to query and update database within a for loop - Stack Overflow

admin管理员组

文章数量:1384197

I'm writing a multiplayer game(mongojs, nodejs) and trying to figure out how to update user stats based on the oute of the game. I already have the code written to pute all the post game stats. The problem es when I try to update the users' stats in a for loop. Here's what i got:

//Game Stats
var tempgame = {
    gameid: 1234,
    stats: [
        {
            score: 25,
            user: 'user1'
        },
        {
            score: 25,
            user: 'user2'
        }
    ]
}


for(i = 0; i < tempgame.stats.length; i++){
    db.users.find({ username: tempgame.stats[i].user }, function(err, res){
        if( err != null){
            //handle errors here.
        } else {
            var userstats = res[0].stats;
            if( tempgame.stats[i].score > userstats.bestscore ){ //this is where it chokes 
                userstats.bestscore = tempgame.stats[i].score;
            }

            //code here to pass back new manipulated stats
        }
    });
}

Everything works fine until i try to use the tempgame object within the callback function. It says "cannot read property 'score' of undefined". Is this just a scoping issue?

Also i was thinking it could be an issue with the callback function itself. Maybe the loop would increment before the callback is even run. But even in that case, the score should be be there it would just be pulling from the wrong array index... that's what lead me to believe it may just be a scope issue.

Any help would be greatly appreciated.

I'm writing a multiplayer game(mongojs, nodejs) and trying to figure out how to update user stats based on the oute of the game. I already have the code written to pute all the post game stats. The problem es when I try to update the users' stats in a for loop. Here's what i got:

//Game Stats
var tempgame = {
    gameid: 1234,
    stats: [
        {
            score: 25,
            user: 'user1'
        },
        {
            score: 25,
            user: 'user2'
        }
    ]
}


for(i = 0; i < tempgame.stats.length; i++){
    db.users.find({ username: tempgame.stats[i].user }, function(err, res){
        if( err != null){
            //handle errors here.
        } else {
            var userstats = res[0].stats;
            if( tempgame.stats[i].score > userstats.bestscore ){ //this is where it chokes 
                userstats.bestscore = tempgame.stats[i].score;
            }

            //code here to pass back new manipulated stats
        }
    });
}

Everything works fine until i try to use the tempgame object within the callback function. It says "cannot read property 'score' of undefined". Is this just a scoping issue?

Also i was thinking it could be an issue with the callback function itself. Maybe the loop would increment before the callback is even run. But even in that case, the score should be be there it would just be pulling from the wrong array index... that's what lead me to believe it may just be a scope issue.

Any help would be greatly appreciated.

• javascript
• node.js
• mongodb
• mongojs

Share Improve this question Follow edited May 15, 2013 at 13:33 Alberto De Caro 5,20399 gold badges5050 silver badges7676 bronze badges asked Jan 22, 2013 at 18:14 v3xx3dv3xx3d 5766 bronze badges

Add a ment  | 

3 Answers 3

Sorted by: Reset to default 7

You've been tripped up by the notorious "defining functions inside a loop" problem.

Use "forEach" instead:

tempgame.stats.forEach(function (stat) {
    db.users.find({ username: stat.user }, function(err, res){
        if( err != null){
            //handle errors here.
        } else {
            var userstats = res[0].stats;
            if( stat.score > userstats.bestscore ){ //this is where it chokes 
                userstats.bestscore = stat.score;
            }

            //code here to pass back new manipulated stats
        }
    });
});

Part of your problem is as mjhm stated in his answer to your question, and is as you have suspected. The i variable is changing before the callback is invoked.

The other half of your problem is because your database calls have not returned yet. Due to the asynchronous nature of NodeJS, your loop will finish before your database calls plete. Additionally, your database calls are not necessarily ing back in the same order you called them. What you need is some sort of flow control like async.js. Using async.map will allow you to make all calls to the DB in parallel and return them as an array of values you can use, after all db calls have been pleted.

async.map(tempgame.stats, function(stat, callback){
  db.users.find({ username: stat.user }, function(err, res){
      if( err != null){
          callback(err);
      } else {
          callback(null, res[0].stats);
      }
  });      
}, function(err, stats){
  if(err){
    //handle errors
  } else{
    stats.forEach(function(stat){
      //do something with your array of stats
      //this wont be called until all database calls have been pleted
    });
  }
});

In addition to the above, if you want to return results back to the application, http://nodeblog.tumblr./post/60922749945/nodejs-async-db-query-inside-for-loop

本文标签： javascriptnodejs and mongodb (mongojs) Trying to query and update database within a for loopStack Overflow