【问题标题】:node-mysql timingnode-mysql 时序
【发布时间】:2012-06-13 16:08:52
【问题描述】:

我有一个这样的递归查询(注意:这只是一个例子):

var user = function(data)
{
  this.minions = [];
  this.loadMinions = function()
  {
    _user = this;
    database.query('select * from users where owner='+data.id,function(err,result,fields)
    {
      for(var m in result)
      {
        _user.minions[result[m].id] = new user(result[m]); 
        _user.minions[result[m].id].loadMinions();
      }
    }  
    console.log("loaded all minions");
  }
}
 currentUser = new user(ID);
 for (var m in currentUser.minions)
 {
   console.log("minion found!");
 }

这不起作用,因为时间都错了,代码不等待查询。

我试过这样做:

var MyQuery = function(QueryString){
    var Data;
    var Done = false;
    database.query(QueryString, function(err, result, fields) { 
        Data = result;
        Done = true;
    });
    while(Done != true){};
    return Data;
}

var user = function(data)
{
  this.minions = [];
  this.loadMinions = function()
  {
    _user = this;
    result= MyQuery('select * from users where owner='+data.id);
    for(var m in result)
    {
      _user.minions[result[m].id] = new user(result[m]); 
      _user.minions[result[m].id].loadMinions();
    }
    console.log("loaded all minions");
  }
}
currentUser = new user(ID);
for (var m in currentUser.minions)
{
  console.log("minion found!");
}

但他只是暂时冻结,我错过了什么吗?

【问题讨论】:

  • 您基本上错过了 Node.js 与您可能习惯于编程的环境之间的最完整且最重要的变化。您是如何做到的?

标签: javascript mysql node.js timing node-mysql


【解决方案1】:

解决问题的第一个障碍是理解 Node.js 中的 I/O 是异步的。一旦您知道这如何适用于您的问题,递归部分就会变得容易得多(尤其是如果您使用 Async 或 Step 等流控制库)。

这是一个示例,它可以完成您尝试做的一些事情(减去递归)。就个人而言,我会避免递归加载可能未知数量/深度的记录;而是按需加载它们,如下例所示:

var User = function(data) {
    this.data = data
    this.minions;
};

User.prototype.getMinions = function(primaryCallback) {
    var that = this; // scope handle
    if(this.minions) { // bypass the db query if results cached
        return primaryCallback(null, this.minions);
    }

    // Callback invoked by database.query when it has the records
    var aCallback = function(error, results, fields) {
        if(error) {
            return primaryCallback(error);
        }

        // This is where you would put your recursive minion initialization
        // The problem you are going to have is callback counting, using a library
        // like async or step would make this party much much easier

        that.minions = results; // bypass the db query after this
        primaryCallback(null, results);
    }

    database.query('SELECT * FROM users WHERE owner = ' + data.id, aCallback);
};

var user = new User(someData);    
user.getMinions(function(error, minions) {
    if(error) {
        throw error;
    }

    // Inside the function invoked by primaryCallback(...)
    minions.forEach(function(minion) {
        console.log('found this minion:', minion);
    });
});

在这个例子中最需要注意的是回调。 database.query(...) 是异步的,您不想占用事件循环等待它完成。这是通过向查询提供回调aCallback 来解决的,该回调在结果准备好时执行。一旦该回调触发并且在您对记录执行任何您想要执行的处理之后,您可以触发 primaryCallback 并获得最终结果。

【讨论】:

    【解决方案2】:

    每个 Node.js 进程都是单线程的,所以行

    while(Done != true){};
    

    接管线程,将Done 设置为true 的回调永远不会运行,因为thead 在无限循环中被阻塞。

    您需要重构您的程序,以便将依赖于查询结果的代码包含在回调本身中。例如,让MyQuery 接受回调参数:

    MyQuery = function(QueryString, callback){
    

    然后在您的 database.query 回调结束时调用回调 - 或者甚至将其提供为 database.query 回调。

    【讨论】:

      【解决方案3】:

      不幸的是,冻结是正确的行为,因为 Node 是单线程的。

      您需要一个调度程序包来解决这个问题。就个人而言,我一直在使用 Fibers-promise 来解决这类问题。你可能想看看这个或另一个承诺库或异步

      【讨论】:

        猜你喜欢
        • 2017-05-04
        • 1970-01-01
        • 2014-05-06
        • 1970-01-01
        • 1970-01-01
        • 1970-01-01
        • 2016-08-04
        • 2012-12-13
        • 2020-05-09
        相关资源
        最近更新 更多