Question

我对Node.js中的递归异步请求的行为有疑问。

以下功能旨在从MongoDB返回搜索结果。如果初始搜索结果为空，我将文本拆分为单个单词，然后尝试递归fetchResult（...）为每个单词，并将 res 对象作为参数传递。

function fetchResult(text, res){
    var MongoClient = require('mongodb').MongoClient;

    MongoClient.connect(mongoURL + "/search", function (err, db) {

        if(err) throw err;

        db.collection('results', function(err, collection) {

            // search for match that "begins with" text
            collection.findOne({'text':new RegExp('^' + text, 'i')}, function(err, items){

               var result = (items == null || items.result == null) ? "" : items;

                if (result){
                    res.send(result);
                }
                else {
                    // no result, so fire off individual word queries - if any spaces found
                    if (text.indexOf(' ') > -1){
                        // split string into array
                        var textArray = text.split(" ");

                        // recursively process individual words
                        for (index = 0; index < textArray.length; index++) {
                            // ***** RACE CONDITION HERE? *****
                            fetchResult(textArray[index], res);
                        }
                    }
                    else {
                        // just return empty result
                        res.send(result);
                    }
                }
            });
        });
    });
}

我怀疑这可能会导致一些竞争条件，因为对 res 的引用是异步的，并且当我运行代码并观察到以下错误时确认了这一点：

Error: Can't set headers after they are sent.
    at ServerResponse.OutgoingMessage.setHeader (_http_outgoing.js:344:11)

所以我的问题是：如何在序列中实现执行单个字符串查询所需的递归行为，仅在我们找到第一个结果时返回（或者在搜索返回时在过程结束时返回）根本没有结果）？

Answer 1

您的代码需要进行一些重构。

首先，从MongoClient.connect功能中删除fetchResult来电。您可以连接一次并存储db对象以供以后使用。

其次，使用您的方法，您将返回查询中每个单词的响应。我不认为递归呼叫是去这里的方式。

如果您的第一个查询失败，您将有多个异步查询，并且您不知何故需要聚合结果......这就是它变得棘手的地方。我对Mongo并不熟悉，但我相信你可以通过find $in检索一组结果来避免这种情况。见this question，也许有帮助。

Node.js中异步递归的问题

1 个答案: