根据数组的大小进行多个api调用

时间:2018-07-10 01:34:44

标签: node.js api asynchronous promise request

我正在尝试基于数组中的信息进行多个API调用。例如,我有一个[[伦敦],[纽约],[墨西哥],[中国],[东京]]的数组

我想使用开放式天气API获取所有天气信息。我正在尝试使用promise.all。在渲染到最前面之前,我需要所有数据,但是我的代码无法正常工作。

let cities = ['London', 'New York', 'Mexico', 'China', 'Tokyo'] 
let promisesArray = []
return new Promise(function(resolve, reject) {
    for(let i = 0; i < cities.length; i++) {

        let cities = ['London', 'New York', 'Mexico', 'China', 'Tokyo'] 
        let url = 'api.openweathermap.org/data/2.5/weather?q=' + cities[i]
        request(url, function(error, response, body){

            if (err) {
                reject(error);
            }

            var data = JSON.parse(body)
            var results = data.Search;
            promisesArray.push(resolve(results));
        });

    }
})

    Promise.all(promisesArray)
        .then(function(results) {

        })
        .catch(function(error) {

        })

1 个答案:

答案 0 :(得分:3)

您混淆了几件事,但这是正确的主意。您的promiseArray需要包含承诺,而不是数据。

因此,您应该遍历您的城市,在承诺调用request中为每个城市创建一个承诺,当请求返回调用resolve时。使用array.map比使用for循环要干净一些。

下面是一个带有模拟request函数的示例:

// fakes request
function request(url, cb) {
  setTimeout(() => cb(null, 200, `{"Search": "success for ${url}" }`), 200)
}


let cities = ['London', 'New York', 'Mexico', 'China', 'Tokyo']

// promisesArray will hold all the promises created in map()
let promisesArray = cities.map(city => {
  // make a new promise for each element of cities
  return new Promise((resolve, reject) => {
    let url = 'http://api.openweathermap.org/data/2.5/weather?q=' + city
    request(url, function(error, response, body) {
      if (error) {
        reject(error);
      }
      var data = JSON.parse(body)
      var results = data.Search;
      // resolve once we have some data
      resolve(results);
    });
  })
})

Promise.all(promisesArray)
  .then(function(results) {
    console.log(results)
  })
  .catch(function(error) {
    console.log(error)
  })