我正在尝试基于数组中的信息进行多个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) {
})
答案 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)
})