Promise.all（[..]）、私は次のようになり、個々の約束の数持って

Question

を解決することはありません：。彼らはすべての要求を実行するために）getFromApi（と呼ばれるこの方法を使用

var performancePerMonth = new Promise(function(resolve, reject) { 
    var path = some_path; 
    getFromApi(path); 
}); 

を：

function getFromApi(path, callback) { 

    request({ 
    method: 'GET', 
    url: base_uri + path, 
    headers: { 'TOKEN': 'MY_ACCESS_TOKEN' } 
    }, 

    function (error, response, body) { 
    console.log('Status:', response.statusCode); 
    console.log('Headers:', JSON.stringify(response.headers)); 
    console.log('Response:', body); 
    return body; 
}); 

私は一緒にリクエストを実行し、すべての結果を含む配列を取得する必要があるため、私はPromise.allにすべての個々の約束を追加しました：

exports.perform = function(req, response) { 
    Promise.all([performancePerMonth, performancePerPlayer,..]).then(function(results) { 
    console.log('Then: ', results); 
    }).catch(function(err) { 
    console.log('ERROR Promise.all'); 
    console.log('Catch: ', err); 
    }) 
}; 

問題は、getFromApi（）メソッドのログからわかるようにリクエストが正常に実行されても、結果の配列は返されません。どうやらメソッドが実行されますが.then内の関数になることはありませんし、この：

console.log('Then: ', results); 

が（その下のキャッチ機能もない）に達することはありません。

プログラムがすべての要求が実行され、結果を返すまで実行されないかのように見えます。

Answer 1

あなたは約束を決して決してしません。

var performancePerMonth = new Promise(function(resolve, reject) { 
    var path = some_path; 
    getFromApi(path, resolve); 
}); 

あなたgetFromApi機能もコールバックを持っていますが、それが実行されることは決してありません。おそらく、代わりにreturn body;

Answer 2

function getFromApi(path, callback)のcallback(body);は、コールバック引数は今少し呼び出すように変更され、その無名関数を使用することができます代わりに、匿名関数のrequestのためのコールバックとしてあなたが

function getFromApi(path, callback) { 
    request({ 
    method: 'GET', 
    url: base_uri + path, 
    headers: { 'TOKEN': 'MY_ACCESS_TOKEN' } 
    }, callback); 
} 

を持っていることを使用することができます受け入れたいです解決必要に応じて/拒否、performancePerMonthなど

var performancePerMonth = new Promise(function(resolve, reject) { 
    var path = some_path; 
    getFromApi(path, function (error, response, body) { 
     if (error) { 
      reject(error); 
     } else { 
      console.log('Status:', response.statusCode); 
      console.log('Headers:', JSON.stringify(response.headers)); 
      console.log('Response:', body); 
      resolve(body); 
     } 
    }); 
}); 

にあなたはresponse.statusCodeを確認したいことがあり、1つのそのインディカですTES成功した要求の前に盲目的にあまりにも体を解決する - 多くのリクエストAPIのあなたは

var performancePerMonth = new Promise(function(resolve, reject) { 
    var path = some_path; 
    getFromApi(path, function (error, response, body) { 
     if (error) { 
      reject(error); 
     } else { 
      console.log('Status:', response.statusCode); 
      console.log('Headers:', JSON.stringify(response.headers)); 
      console.log('Response:', body); 
      if (response.statusCode >= 200 && response.statusCode < 300) { 
       resolve(body); 
      } else { 
       reject(response.statusCode); 
      } 
     } 
    }); 
});