【问题标题】:Anonymous callback function in for loop in javascriptjavascript中for循环中的匿名回调函数
【发布时间】:2013-11-01 20:53:52
【问题描述】:

我正在尝试在 async.waterfall 中创建一个函数,该函数检查数组中的每个 id 是否在 mongodb 中有任何具有此 id 的条目(使用 Mongoose)。如果 id 已经存在,则将从数组中删除。我写了以下函数:

    function(eventIds, callback) {
        // check for duplicates
        for (var i = 0; i < eventIds.length; i++) {
            var query = Party.find({
                fbEventId: eventIds[i]
            });
            query.exec(function(err, doc) {
                if (err) return console.log(err);
                if (doc) {
                    // remove i from array
                    console.log(doc);
                }
            });
        }
        callback(null, eventIds);
    }

然而,这会发出警告,因为在 for 循环中构造了一个新函数。

如果我像下面这样在 for 循环之外创建函数,则会出现错误:ReferenceError: err is not defined。

    function(eventIds, callback) {
        // check for duplicates
        function checkDuplicate(err, doc) {
            if (err) return console.log(err);
            if (doc) {
                    // remove i from array
                console.log(doc);
            }
        }
        for (var i = 0; i < eventIds.length; i++) {
            var query = Party.find({
                fbEventId: eventIds[i]
            });
            query.exec(checkDuplicate(err, doc));
        }
        callback(null, eventIds);
    }

这样做的正确方法是什么?

【问题讨论】:

  • 请注意,callback 将在所有 checkDuplicate 调用执行/完成之前执行。你确定这是你想要的吗?
  • 不,这确实不是我想要的,谢谢你的提示 :)

标签: javascript node.js mongodb mongoose


【解决方案1】:

您正在调用该函数,而不是为其分配引用。

你的代码

query.exec(query.exec(checkDuplicate(err, doc));

应该是

query.exec(query.exec(checkDuplicate));

【讨论】:

    猜你喜欢
    • 2012-12-08
    • 2011-08-05
    • 2013-05-23
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2013-02-28
    • 2016-12-17
    相关资源
    最近更新 更多