1

async モジュールを使用して、Node.js に固​​有の「コールバック地獄」を削減しようとしています。基本的にasync.series、データベースから情報を取得してビューに表示するために使用しようとしています。それでも、私の見解では結果が得られません。

これは私がこれまでに持っているコードです:

// Search
exports.search = function(req, res) {

    var x = [];



        async.series([
            function(cb) {
                Lang.find({ lang: req.query.keyword }).sort({ verbal: -1 }).exec(function(err, langs) {              

                    cb(null, langs);

                });
            },

            function(cb) {
                Human.find({}, function(err, humans) {            

                    cb(null, humans);
                });
            }], 

            function(err, results) {
                if (err) {
                    res.send(500);
                }

                for(var i = 0; i < results[0].length; i++) {
                    for(var j = 0; j < results[1].length; j++) {
                        if(results[1][j]._id == results[0][i].human) {
                             x.push(results[1][j]);
                        }
                    }
                } 

                res.render('myView', { title: 'Search Results', humans: x });
            }
        );
    }

まず、Lang モデル (MongoDB) にクエリを実行し、一致するレコードを見つけたいと考えていますreq.query.keyword。その後、Human モデルにクエリを実行し、その特定の言語スキルを持つすべての Human を見つけたいと考えています。また、FOR ループは、人間が複数の言語を持っている可能性があるため、配列から重複を排除することを目的としています。

4

1 に答える 1

0

私があなたのコードを正しく理解していれば、async.waterfall代わりに使用することをお勧めします。ある関数の結果を引数として次の関数に渡します。他にもいくつかの最適化を行うことができます。

async.waterfall([
  function(done) {
    Lang
      .find({ lang: req.query.keyword })
      .sort({ verbal: -1 }) // (not really useful)
      .select('human')      // not strictly necessary, but saves a bit of space
      .exec(done);          // short for : 
                            //   .exec(function(err, results) {
                            //     done(err, results);
                            //   });
  },
  function(langs, done) {
    // extract the 'human' property from each result
    var _ids = langs.map(function(lang) {
      return lang.human;
    });

    // perform a query finding all humans in the list of ids
    Human
      .find({ _id : { $in : _ids } })
      .exec(done);
  }
], function(err, humans) {
  if (err)
    return res.send(500);
  res.render('myView', {
    title : 'Search Results',
    humans: humans
  });
});

編集:$in順序を保持しないため、この回答は willを使用することを示唆している$orため、2番目のクエリの代わりにこれを試してください:

  ...
  function(langs, done) {
    var query = langs.map(function(lang) {
      return { _id : lang.human };
    });

    Human
      .find({ $or : query })
      .exec(done);
  }
  ...
于 2013-10-30T10:05:12.667 に答える