Mongoose find() 回调问题

Question

我找不到让我的查找功能正常工作的方法。 我想从 bills 集合中找到所有 billId，然后检查事务数据库中的每个 billId。

问题是，由于 aSynch - 我认为，id 不会在我的循环中更新。 代码如下：

Bills.find({type: bill_type, endDate: {"$gte" : new Date(year + "-" + month + "-1")}}).find(function(err, bills){
        if(err)
            res.send(err);

        details.bills = bills;

        for(key in bills){
            var billId = bills[key]._id;
            console.log("BillId: " + billId); // Here the ids are unique (which is what I want)

             Transactions.find({billId : billId}, function(err, transactions){
                if (err) {
                    console.log('error: '+ err)
                } else {
                    console.log("Transaction BillId: " + billId); // Here I get always the same ID - which is not quiet what I need.
                }
            });


            //console.log(transactions);
        }

       res.send(details);  
    });

控制台的结果是：

BillId: 549bf0597886c3763e000001
BillId: 54a014bfac01ca3526000001
BillId: 54a015753547a6c026000001

^ 好结果 - 来自 for() 中的第一个 console.log。

然后：

Transaction BillId: 54a015753547a6c026000001
Transaction BillId: 54a015753547a6c026000001
Transaction BillId: 54a015753547a6c026000001

^ 错误的结果 - 它重复了 for() 的最后一个结果，我需要所有结果。

在第一个 console.log() 中，当我从第一个 find() (Bills.find()) 获得结果时，我可以看到日志上的所有 id，但是当我尝试将它们放入第二个 Find ( Transactions.find())，他们重复最后一个 id。 因此，在当前状态下，我无法为每个 id 查询数据库。对此的任何帮助表示赞赏。

如果您需要任何说明，请告诉我。

提前致谢！

Answer 1

循环内闭包函数的经典错误。

您的变量值会在每个循环步骤中发生变化。另一方面，当变量已经多次更改时，您的函数将在稍后（一段时间后）执行。

查看链接以进行澄清：

Creating closures in loops: A common mistake

JavaScript closure inside loops – simple practical example

Javascript infamous Loop issue?

Answer 2

如果你的Transactions.find 函数是异步的，它的回调会在稍后的时间点运行，在for ... in 循环结束很久之后。那时，billId 具有在最后一次运行 for ... in 循环时分配的值。因此，回调的所有运行都会打印出该值。

如果您想保留该值，则必须在某处创建closure，如下所示：

(function (id) {
    Transactions.find(..., function (...) {
        // ... use `id` here
    });
})(billId);

---

如果bills 是Array 或者如果您使用像Lo-Dash 这样的实用程序库，您可以使用像Array.prototype.map 或lodash.each 这样使用闭包的迭代函数，这样您就不会必须单独创建一个。

Answer 3

完美！

感谢您的提示。我使用Underscore each function 解决了它。

方法如下： 我没有使用for 循环，而是简单地将for 替换为_.each，如下例所示：

Bills.find({type: bill_type, endDate: {"$gte" : new Date(year + "-" + month + "-1")}}).find(function(err, bills){
    if(err)
        res.send(err);

    details.bills = bills;

    _.each(bills, function(item){        
        var billId = item._id; 
        console.log("BillId: " + billId); 

         Transactions.find({billId : billId}, function(err, transactions){
            if (err) {
                console.log('error: '+ err)
            } else {
                console.log("Transaction BillId: " + billId);
            }
        });


        //console.log(transactions);
    });

   res.send(details);  
});