从多个mongodb调用填充数组的范围问题

时间:2013-05-23 16:03:13

标签: javascript node.js mongodb

我有一个游戏网格,我试图将其放入一个行数组中,每个行都有许多块。这些块存储在同一个集合中,并具有行和列字段。

我目前的尝试如下。

exports.findById = function(req,res) {
    var id = req.params.id;
    console.log('Retrieving game #' + id);
    db.games.findOne( { "_id" : ObjectId(id) }, function(err, game_obj) {
        if(err || !game_obj) {
            console.log("no game");
            res.send({'error':'An error has occurred'});
        } else {
            var rows = [];
            function getRowBlocks(err,blocks) {
                if(err || !blocks) {
                    console.log("no blocks");
                    res.send({'error':'An error has occurred'});
                } else {                    
                    console.log(blocks[0].row + " has " + blocks.length + " blocks");                   
                    rows[blocks[0].row] = blocks;
                }
            }
            for(i=0;i<game_obj.farthest_row;i++) {  
                db.blocks.find( { "game_id" : ObjectId(id), "row" : i     }).sort( { "column" : 1 }).toArray(getRowBlocks);
            }
            res.send('game', { row_list : rows, game : game_obj});
        }
    });
}

但是由于mongodb find操作的范围要求每一行,我实际上无法获得存储在rows变量中的块。并且由于运行了多个操作,我无法简单地将res.send...置于toArray回调中。

我只能使用一个查询并在回调中构建数组然后传递它,但我认为考虑到多个查询是异步发生并且可能存在大量行,这将效率低得多

如果我可以将行数组传递给查询回调,我想我可以做到,但是我无法找出是否可能。

1 个答案:

答案 0 :(得分:1)

这就是async.parallel的用途:

// build a list of query-functions
var queries = [];
for (var i = 0; i < game_obj.farthest_row ; i++) {
  queries.push(function(i, cb) {
    db.blocks
      .find( { "game_id" : ObjectId(id), "row" : i })
      .sort( { "column" : 1 })
      .toArray(cb);
  }.bind(this, i)); // create a partial function to create a newly scoped 'i'
}

// pass the query functions to async, which will run them in parallel,
// gather all the results, and finally call the callback function with
// the results-array
async.parallel(queries, function(err, results) {
  if (err)
    // handle error

  // results now contains the query results (array-in-array)
  results.forEach(function(blocks) {
    getRowBlocks(null, blocks); // this probably requires a rewrite...
  });
});