node.js:对于每个over行并异步更新?

时间:2013-06-08 23:08:40

标签: node.js asynchronous node-mysql

我需要查询数据库中的行,每行处理一些信息,然后用结果更新每一行。

这是我的示例代码,其目的是遍历每一行并更新标签:

var mysql = require('mysql');

var db = mysql.createConnection(config.database);

db.connect(function() {
    db.query('SELECT id FROM testTable', function (err, rows) {
        if (err) {
            console.log(err);
        } else {
            if (rows.length) {
                for (var i = 0, len = rows.length; i < len; i++) {
                    var row = rows[i];
                    console.log(row);

                    var label = "Label_"+row.id;

                    db.query('UPDATE testTable SET label = ? WHERE id = ?', [label, row.id], function(err, result) {
                        if (err) {
                            console.log(err);
                        } else {
                            console.log("Set label on row %s", row.id);
                        }
                    })
                }
            }
        }
    })
});

这个输出是:

{ id: 1 }
{ id: 2 }
{ id: 3 }
{ id: 4 }
Set label on row 4
Set label on row 4
Set label on row 4
Set label on row 4

所以,正如你所看到的,我已经将第4行更新了四次,而不是每次更新四行。虽然我新的查询将是非阻塞的,但我认为每个值的值都会发生变化。

我知道我可以将我的代码更改为使用rows.forEach(function(){...}),然后逐个执行每个UPDATE,这样就可以了。但为了帮助我理解,我想知道如何以异步方式正确执行更新。

1 个答案:

答案 0 :(得分:5)

您的row变量是回调函数中的闭包。在循环遍历所有结果列表之前,不会调用回调函数。 sql查询是正确的,但是在每个回调中打印出row.id的值只会给你每次for循环的最后一次迭代,因为这是每次回调的闭包状态。

您可以使用underscore模块来避免这种情况。它还可以帮助您简化逻辑。

npm install underscore

然后你的代码看起来像这样:

var mysql = require('mysql');
var _ = require('underscore');

var db = mysql.createConnection(config.database);

db.connect(function() {
  db.query('SELECT id FROM testTable', function (err, rows) {
    if (err) { console.log(err); return; }
    _.each(rows, function(one) {
      console.log(one);
      var label = "Label_"+one.id;
      var sql = 'UPDATE testTable SET label = ? WHERE id = ?';
      db.query(sql, [label, one.id], function(err, result) {
        if(err) { console.log(err); return; }
        console.log("Set label on row %s", one.id);
      });
    });
  });
});