Question

我是这个node.js的新手..我对这个回调有点困惑..在我的应用程序内部for循环我调用异步函数调用，我想我的问题是在我得到响应之前异步调用my for循环获取循环。

我的代码：

async.forEach(Object.keys(config), function(key, next) {
        search(config[key].query, function(err, result) { // 
        console.log("fffffffffff="+ util.inspect(result))-------- >>>Getting undefined..
            if (err) return next(err) // 
            var json = JSON.stringify({
                "result": result
            });
            results[key] = {
                "result": result
            }
            console.log("rrrrrrrr="+util.inspect(results[key]))
            next() // <---- critical piece.  This is how the forEach knows to continue to the next loop.  Must be called inside search's callback so that it doesn't loop prematurely.                   
        })
    },
    function(err) {
        console.log('iterating done');

         res.writeHead(200, {
        'content-type': 'application/json'
    });
    res.end(JSON.stringify(results));  
    });


}

搜索功能代码：

var matches = [];
    var qrySubString = query.substring(0, 4);
    client.query("select * from xxxxxxxxx where level4 ILIKE '%" + query + "%'", function(err, row1, fields) {
        for (var id in row1.rows) {                
            var match, name;                
            if (query == row1.rows[id].level4) {
                match = true;
                name = row1.rows[id].level4;
            }
            else {
                match = false;
                name = query;
            }
            matches.push({
                "id": id,
                "name": row1.rows[id].level4,
                "score": 100,
                "match": match,
                "type": [{
                    "id": "/people/presidents",
                    "name": "US President"
                }]
            })
        }           
        callback(matches);
    })

我想在成功执行1个搜索功能后执行for循环，我想我必须使用async for loop。请指导我解决这个问题。谢谢提前..

Answer 1

我已将您的代码示例缩减为以下几行，以便更容易理解该概念的解释。

var results = [];
var config = JSON.parse(queries);
for (var key in config) {
    var query = config[key].query;
    search(query, function(result) {
        results.push(result);
    });
}
res.writeHead( ... );
res.end(results);

前面代码的问题是search函数是异步的，所以当循环结束时，没有调用任何回调函数。因此，results列表为空。

要解决此问题，您必须将循环后的代码放在回调函数中。

    search(query, function(result) {
        results.push(result);
        // Put res.writeHead( ... ) and res.end(results) here
    });

但是，由于回调函数被多次调用（每次迭代一次），您需要知道所有回调都已被调用。为此，您需要计算回调数，并检查该数字是否等于异步函数调用的数量。

要获取所有密钥的列表，请使用Object.keys。然后，为了遍历此列表，我使用.forEach（您也可以使用for (var i = 0, key = keys[i]; i < keys.length; ++i) { .. }，但这可能会产生问题，请参阅JavaScript closure inside loops – simple practical example）。

这是一个完整的例子：

var results = [];
var config = JSON.parse(queries);
var onComplete = function() {
    res.writeHead( ... );
    res.end(results);
};
var keys = Object.keys(config);
var tasksToGo = keys.length;
if (tasksToGo === 0) {
   onComplete();
} else {
    // There is at least one element, so the callback will be called.
    keys.forEach(function(key) {
        var query = config[key].query;
        search(query, function(result) {
            results.push(result);
            if (--tasksToGo === 0) {
                // No tasks left, good to go
                onComplete();
            }
        });
    });
}

注意：前一个示例中的异步代码是并行执行的。如果需要以特定顺序调用函数，则可以使用递归来获得所需的效果：

var results = [];
var config = JSON.parse(queries);
var keys = Object.keys(config);
(function next(index) {
    if (index === keys.length) { // No items left
        res.writeHead( ... );
        res.end(results);
        return;
    }
    var key = keys[index];
    var query = config[key].query;
    search(query, function(result) {
        results.push(result);
        next(index + 1);
    });
})(0);

我所展示的是概念，您可以在实施中使用众多（第三方）NodeJS模块中的一个，例如async。

Answer 2

你已经正确地诊断出了你的问题，所以工作很好。一旦你调用了你的搜索代码，for循环就会继续正确。

我是https://github.com/caolan/async的忠实粉丝，这对我很有帮助。基本上你会得到类似的东西：

var async = require('async')
async.eachSeries(Object.keys(config), function (key, next){ 
  search(config[key].query, function(err, result) { // <----- I added an err here
    if (err) return next(err)  // <---- don't keep going if there was an error

    var json = JSON.stringify({
      "result": result
    });
    results[key] = {
      "result": result
    }
    next()    /* <---- critical piece.  This is how the forEach knows to continue to
                       the next loop.  Must be called inside search's callback so that
                       it doesn't loop prematurely.*/
  })
}, function(err) {
  console.log('iterating done');
});

我希望有所帮助！

Answer 3

我喜欢在这个场景中使用递归模式。例如，像这样：

// If config is an array of queries
var config = JSON.parse(queries.querrryArray);   

// Array of results
var results;

processQueries(config);

function processQueries(queries) {
    var searchQuery;

    if (queries.length == 0) {
        // All queries complete
        res.writeHead(200, {'content-type': 'application/json'});
        res.end(JSON.stringify({results: results}));
        return;
    }

    searchQuery = queries.pop();

    search(searchQuery, function(result) {
        results.push(JSON.stringify({result: result}); 
        processQueries();            
    });

}

processQueries是一个递归函数，它将查询元素从要处理的查询数组中提取出来。然后，当查询完成时，回调函数再次调用processQueries。当没有任何疑问时，processQueries知道结束。

最简单的方法是使用数组，但可以修改它以使用我想象的对象键/值。

Answer 4

Node.js在async await中引入了7.6，这使得Javascript更加美丽。

var results = [];
var config = JSON.parse(queries);
for (var key in config) {
  var query = config[key].query;
  results.push(await search(query));
}
res.writeHead( ... );
res.end(results);

为了实现这一目标，search功能必须返回promise或必须async功能

如果它没有返回Promise，您可以帮助它返回Promise

function asyncSearch(query) {
  return new Promise((resolve, reject) => {
   search(query,(result)=>{
    resolve(result);
   })
  })
}

然后将此行await search(query);替换为await asyncSearch(query);

node.js中的async for loop

4 个答案: