Question

我有一个关于node.js / javascript语法和异步调用的初学者问题。我正在使用node-dbi从MySQL服务器中提取一些信息。

我有2张桌子。客户和区。

我需要编写一个执行此操作的函数：

for (i=0;<zone.count;i++){
  for (j=0;j<client.count;j++){
     //loop through the entire client table for each zone record & run a condition
  }
}

这是我在node-dbi中的语法：

db.fetchAll('SELECT * from Zone', null, function(err, result){
    if (result) {
        db.fetchAll('SELECT * from Client', null, function(err, result){
            if (result) {
                //do something to all client records for each zone record
            }
        });
    }
});

因为它很明显，我的结果和错误的变量都是冲突的。有人可以解释一下这个异步函数的语法吗？

谢谢！

Answer 1

为每个变量指定一个特定于该函数的名称：

db.fetchAll('SELECT * from Zone', null, function(zoneErr, zoneResult){
    if (zoneResult) {
        db.fetchAll('SELECT * from Client', null, function(clientErr, clientResult){
            if (clientResult) {
                //do something to all client records for each zone record
            }
        });
    }
});

此外，您应该重构一下以使其更具可读性并删除深度嵌套。

这是考虑它的一种方式：

var getZones = function() {
  var result;
  db.fetchAll('SELECT * from Zone', null, function(zoneErr, zoneResult){
     if (zoneResult) {
       callback();
       result = true; // assumes you need to keep track of success of failure of result
     } else {
       result = false;
     }
  }
  return result;
};


var getClients { ... same code as above for clients };

getZones(getClients);

但重构不在袖口 - 它可能不适合你的情况。

问题应该通过第一段代码来解决

Answer 2

将err和结果重命名为errZone，resultZone / errClient，resultClient作为Squadrons建议，或使用async https://npmjs.org/package/async

// WARNING UNTESTED CODE
// npm install async
// or add async into your package.json dependencies
var async = require('async');
async.parallel({ 
        zone: function (callback) {
            db.fetchAll('SELECT * from Zone', null, function (err, result) {
                if (err || !result) {
                    callback(err || 'No Zone results found', result)
                }
            });
        },
        client: function (callback) {
            db.fetchAll('SELECT * from Client', null, function (err, result) {
                if (err || !result) {
                    callback(err || 'No Client results found', result)
                }    
            });
        }
    },
    function (err, results) {
        if (err) throw err;
        for (var i = 0; i < results.zone.count; i++) {
            for (var j = 0; j < results.client.count; j++) {
                //loop through the entire client table for each zone record & run a     condition
            }
        }
    });

node js：使用相同的参数名在另一个函数内执行一个函数

2 个答案: