我正在使用nodejs的优秀caolan“async”模块：

我有这个代码：

exports.manageComments = function(req, res) {
    var toDeleteIds = [];
    var deleteFunctions = [];
    if (req.body.doDelete) {
        toDeleteIds = req.body.doDelete;
    }
    var i;
    for ( i = 0; i < toDeleteIds.length; i++ ) {
        var deleteFunction = function(callback) {
            var id = toDeleteIds[i];
            console.log(id);
            Comment.findOne({_id:id}, function(err, found) {            
                if (!err) found.remove(callback);
            });
        }
        deleteFunctions.push(deleteFunction);
    }
    async.parallel(
        deleteFunctions,
        function(err,results) {
            exports.comments(req, res); //render a view
        }
    );
};

我的数组包含两个元素,但console.log()一直告诉我“未定义”.

我错过了什么？

解决方法:

你的问题是：

    var deleteFunction = function(callback) {
        var id = toDeleteIds[i];

因为在执行每个回调函数时,我将具有与toDeleteIds.length相同的值.闭包不会“捕获”外部变量在创建时所具有的值;它“捕获”对外部变量在执行时所具有的任何值的引用(在这种情况下,直到你的for循环完成之后才会这样).

为了在创建回调函数时“捕获”i的值,您需要使i成为您调用以创建回调函数的函数的参数.你需要类似的东西

    var deleteFunction = makeDeleteFunction(i, callback);

然后在回调之外创建一个单独的函数：

function makeDeleteFunction(i, callback) {
    return function(callback) {
        var id = toDeleteIds[i];
        console.log(id);
        Comment.findOne({_id:id}, function(err, found){            
            if (!err) found.remove(callback);
        });
     };
}