3

我有一个这样的递归查询（注意：这只是一个例子）：

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!");
}

但他只是暂时冻结了，我错过了什么吗？

4

3 回答 3

2

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

while(Done != true){};

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

您需要重构您的程序，以便将依赖于查询结果的代码包含在回调本身中。例如，MyQuery使用回调参数：

MyQuery = function(QueryString, callback){

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

于 2012-06-13T16:26:36.300 回答
2

解决问题的第一个障碍是理解 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最终结果。

于 2012-06-13T16:53:24.873 回答
1

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

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

于 2012-06-13T16:28:04.653 回答