http.get请求循环关闭问题(节点)

时间:2015-01-24 22:05:32

标签: node.js loops get request closures

我希望能够在命令行中输入一系列URL,并按照提供的顺序获取请求的结果。我已经研究了闭包来实现这个目标,因为回调函数不会按顺序执行(更有可能是一个响应最快的回调)

for (var i = 2; i < process.argv.length; i++) {
  (function(index) {
    http.get(process.argv[index], function (response) {
      response.pipe(bl(function (err, data) {
        console.log(data.toString());
      }));
    });
  })(i);
}

for参数的for循环。然后每次迭代调用IIFE匿名函数,后者又调用get请求等等。 但问题仍然存在,它是乱序执行的。不按用户要求的顺序。

我做错了什么?

1 个答案:

答案 0 :(得分:1)

如果您只是尝试按顺序记录结果,但是您希望并行运行请求,那么通常的解决方案是在数组中收集结果然后输出结果请求已经完成。

var count = process.argv.length - 2;
var results = new Array(count);
for (var i = 2; i < process.argv.length; i++) {
  (function(index) {
    http.get(process.argv[index], function (response) {
      response.pipe(bl(function (err, data) {
        results[index - 2] = data.toString();
        --count;
        if (count === 0) {
            for (var j = 0; j < results.length; j++) {
                console.log(results[j]);
            }
        }
      }));
    });
  })(i);
}

这种蛮力方法也可以稍微聪明一些,以便在完成所有请求之前更快地输出结果,而不是等待所有请求完成。

另一种方法是为每个请求使用promises,然后使用Promise.all()等待所有异步操作完成,Promise.all()也将收集所有结果。

而且,像Async这样的nodejs库也有很多功能来管理大量的异步操作。


如果您想要序列化请求(在前一个请求完成时仅启动下一个请求),它实际上会更简单一些。这可以这样做:

function runAll() {
    var index = 2;
    function next() {
        if (index < process.argv.length) {
            http.get(process.argv[index], function (response) {
                response.pipe(bl(function (err, data) {
                    console.log(data.toString());
                    ++index;
                    next();
                }));
            });
        }
    }
    next();
}

而且,这是一个使用promises并行运行请求的版本,但是按顺序返回结果:

var Promise = require('bluebird');

function runAll() {
    var promises = [];
    for (var i = 2; i < process.argv.length; i++) {
        promises.push(new Promise(function(resolve, reject) {
            http.get(process.argv[index], function (response) {
                response.pipe(bl(function (err, data) {
                    resolve(data.toString());
                }));
            });
        }));
    }
    return Promise.all(promises);
}

runAll().then(function(resultArray) {
    // resultArray contains an array of the results from all the operations, in order
    for (var i = 0; i < resultArray.length; i++) {
        console.log(resultArray[i]);
    }
});