javascript - 在循环多个 db.Phrase.find 调用时，如何使用回调正确地从 mongodb 传递数据

Question

我从 get 请求中接收参数，如下所示:

{ location: 'Venice', weather: 'Dry', what: 'Yoga', who: 'Bob' }

然后，我查询一个 mongodb 数据库，该数据库循环访问每个键和值对，并在数据库中查询它们的并集。

然后，我将返回的值保存到outputCaption，然后使用回调将outputCaption 传回。

问题是回调被调用的次数与它们的键值对循环的次数一样多。

我被迫这样做，因为我需要 db.Phrase.find 调用中的回调，但我多次调用它......

所以我使用app.get中的代码修复了它(我等到所有键都在outputCaption中定义了值之后才进行任何操作)

它有效，但我无法想象这是最好的方法，所以我希望有一种不那么黑客的方法？ 谢谢

服务器.js

var express = require('express');
var db = require('./modules/db')
var logic = require('./modules/logic')

...

app.get('/phrase', function(req, res){
  logic(req.query, function(outputCaption){
    var flag = true
    for (key in outputCaption){
      if (outputCaption[key] === null){
        console.log('Incomplete')
        var flag = false;
      }
    }
    if (flag === true) {
      console.log(outputCaption);
    };
  });
})

...

逻辑.js

var db = require('./db')

var logic = function(params, callback){
  var outputCaption = {
    who: null, 
    what: null, 
    location: null, 
    weather: null
  };
  for (key in params){
    category = key.toLowerCase();
    option = params[key].toLowerCase();
    db.Phrase.find({$and: [
      {category: category}, 
      {option: option}
      ]}, function(err, phrases){
        if (err) return console.error(err);
        var options = Object.keys(phrases).length
        var idxOfOptionPicked = Math.floor(Math.random() * options)
        outputCaption[phrases[idxOfOptionPicked].category] = phrases[idxOfOptionPicked].phrase
        callback(outputCaption)
    })
  }
}

module.exports = logic;

Answer 1

不要触发多个查询并在客户端执行结果的并集，而是使用查询运算符来允许 MongoDB 为您执行并集。

这样你:

• 避免多次点击数据库。
• 避免使用多个回调处理程序。
• 可以在单个回调处理程序中对结果进行后处理。

您可以修改代码以从请求参数准备查询对象，

  var params = {location: 'Venice', weather: 'Dry', what: 'Yoga', who: 'Bob' };
  var query = {};
  var conditions = [];
  Object.keys(params).forEach(function(key){
   var $and = {};
   $and["category"] = key.toLowerCase();
   $and["option"] = params[key];
   conditions.push($and);
  });
  (conditions.length > 1)?(query["$or"] = conditions):(query = conditions[0]);

现在构造的查询如下所示:

{ '$or': 
   [ { category: 'location', option: 'Venice' },
     { category: 'weather', option: 'Dry' },
     { category: 'what', option: 'Yoga' },
     { category: 'who', option: 'Bob' } 
   ] 
}

您可以将此对象传递给 find() 方法，以在一次点击中获取结果:

db.Phrase.find(query,callback);

这样您的代码就会变得更干净、更容易理解。