NodeJS Closure - オブジェクトの配列をループし、各オブジェクトのデータでサーバーにpingを実行

Question

JSONファイルから読み込まれるサーバーJSONオブジェクトのリストがあります。

私は各サーバーオブジェクトをループし、オブジェクトから値を取り出し、文字列と連結してURLを作成します。次に、URLを使ってリクエストを出して、サーバーが起動しているかどうかを確認します。取得リクエストの結果は、各サーバーオブジェクトのステータスを設定する必要があります。

問題は、各ループでサーバーオブジェクトの値が未定義として表示されていることです。私はどこで問題を解決するかわからない。

var servers = require('./data/servers.json'); 
var ping = require('http'); 

var newServers = statusChecker(servers); 

function statusChecker(servers) { 
var i; 
for(i=0;i<servers.length;i++) { 
    servers[i]['status'] = function(j) { 
     return function() { 
       var proto = 'http://' 
       var server = servers[i].servername; 
       var healthEndPoint = '/health-end-point'; 
       var url = proto+server+healthEndPoint; 
        //This logs the url with the server part undefined 
        console.log(url); 
        ping.get(url, function(res){ 
         console.log(res); 
         return true; 
        }).on('error', function(e){ 
         return false; 
        });    
     }() 
    }(i); 
} 
return servers; 
} 

Answer 1

あなたは、同期ループ内で非同期コードping.getを挿入しようとしているので、ステータスはundefinedです。あなたのしたいが、約Promise.all()、Promises、async/await

---

ここで見てみ

つもりがpingここ

function asynchronousPing(oneServer) { 
    return new Promise((resolve, reject) => { 
     const proto = 'http://' 
     const server = oneServer.servername; 
     const healthEndPoint = '/health-end-point'; 
     const url = proto + server + healthEndPoint; 

     ping.get(url, function (res) { 
     resolve(true); 
     }).on('error', function (e) { 
     resolve(false); 
     }); 
    }); 
    } 

---

は、あなたがそれを呼び出す方法ですになります関数の例です

Promise.all(servers.map(x => asynchronousPing(x))) 
    .then((allStatus) => { 
     console.log(allStatus); 
    }) 
    .catch((err) => { 
     console.log(err); 
    });