Question

我收到了来自我的get请求的params，如下所示：

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

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

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

问题是回调被多次调用，因为它们的键值对被循环。

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

所以我已经使用app.get中的代码修复了它（我等到所有键都在outputCaption中定义了值之后才做任何事情）

它有效，但我无法想象它是最好的方法，所以我希望有一种不那么强硬的方式？感谢

server.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);
    };
  });
})

...

logic.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);

这样，您的代码仍然更清晰，更易于理解。

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

1 个答案: