nodejsが要求のあるforループ内に配列をプッシュ

Question

ノードを非同期的な知識なしに開始し、データをコールバックの一部として配列にプッシュする方法が不思議です。出力配列は宣言されていますが、要求コールバック内で参照されているときは未定義です。その変数をコールバック関数のスコープに渡すだけの簡単な方法はありますか？

理想的には、呼び出し元に返された各リクエストの結果の配列を返信したいと考えています。

const request = require('request'); 

module.exports = { 

apiRatingCall: function (input, callback) { 

var output = [] 

    for (var i = 0; i < input.length; i++) { 
    var options = { 
    url: 'someAPIURL' + '?longitude=' + input[i].longitude + '&latitude=' + input[i].latitude + '&name=' + input[i].name, 
    headers: { 
     'x-api-version': 2 
    } 
    }; 

    request(options, function (error, response, body) { 
    if (!error && response.statusCode == 200) { 

     var info = JSON.parse(body) 
     output.push(info) // this is not working as ouput is undefined at this point 
    } 
    }) 
    } 
    callback(output) 
    } 
} 

おかげ

Answer 1

私の答えは、私は

を約束し、この問題のための唯一の方法で実行することができます

module.exports = { 

apiRatingCall: function (input, callback) { 

var output = [] 

for (var i = 0; i < input.length; i++) { 
var options = { 
    url: 'someAPIURL' + '?longitude=' + input[i].longitude + '&latitude=' + input[i].latitude + '&name=' + input[i].name, 
headers: { 
    'x-api-version': 2 
} 
}; 

request(options, function (error, response, body) { 
    if (!error && response.statusCode == 200) { 

    var info = JSON.parse(body) 
    output.push(info) // this is not working as ouput is undefined at this point 
    } 
}) 
} 
setTimeout(function(){ 
    callback(output) 
},500) 

} 
} 

Answer 2

あなたはforループの終了を確認し、それによって、プログラミングに多分非常に規範的ではなく、あなたのcallback

var loopCallback = function(error, response, body) { 
    if (!error && response.statusCode == 200) { 
    var info = JSON.parse(body) 
    output.push(info) 
    } 
    if(i == input.length - 1){ //checking index for completion of loop 
    callback(output); 
    } 
} 

request(options, loopCallback}) 

Answer 3

asyncモジュールを見てください。 bluebirdと約束を使用したくない場合は、配列とコレクションを使って非同期操作を行うのが良いモジュールです。

npm install async -Sを実行します。

const request = require('request'); 
const async = require('async); 

module.exports = { 

    apiRatingCall: function (inputs, callback) { 

    var output = [] 


    async.each(inputs, function(input, cb) { 
     var options = { 
     url: 'someAPIURL' + '?longitude=' + input.longitude + '&latitude=' + input.latitude + '&name=' + input.name, 
     headers: { 
     'x-api-version': 2 
     } 
     request(options, function (error, response, body) { 
     if (!error && response.statusCode == 200) { 

      var info = JSON.parse(body) 
      output.push(info); 
      cb(); 
     }else{ 
      //if any of the request fails then return error 
      cb(error); 
     } 
    }) 

    }, function(err){ 
     if(!err){ 
     return callback(output); 
     }else{ 
     //if any of the request return error 
     return callback(err); 
     } 
    }); 
}