我试图将一系列AJAX请求写入字典.
我试图使用promises,但是我要么正确地编写promise语法,要么我认为可能发生的是函数实际完成(for循环已完成,AJAX请求已发送)但AJAX请求仍然存在没有回来.因此,这仍然是一个空字典.
let dict = {};
let activeMachines = ["41", "42", "43"];
let dataPromise = new Promise (function (resolve, reject){
for(let i = 0; i < activeMachines.length; i++){
let machineID = activeMachines[i]
let getAPIData = new XMLHttpRequest();
let url = 'http://127.0.0.1:8000/processes/apidata/' +machineID + '/';
getAPIData.open('GET', url);
getAPIData.send();
getAPIData.onload = function(){
let APIData = JSON.parse(getAPIData.responseText);
dict['machine_' + machineID] = APIData[0].author_id;
dict['temp' + machineID] = APIData[0].tempData; //get value
dict['humid' + machineID] = APIData[0].humidData;
timeValue = String((APIData[0].dateTime));
dict['time' + machineID] = new Date(timeValue);
console.log("done");
}
}
resolve();
});
dataPromise.then(function() {
console.log(dict);
});
当所有XMLHTTPRequests都返回时,有没有办法“感知”?
解决方法:
@Rafael的答案会起作用,但它并没有说明出错的原因,因为你试图理解Promises的概念并自己写一个.
从根本上说,我认为你的方法有两个失误:1.创建一个Promise来处理对你所有任意“activeMachines”列表的调用,以及2.将你的resolve()调用放在错误的地方.
通常Promise看起来像这样:
const myPromise = new Promise(function(resolve, reject) {
doSomeAsyncWork(function(result) {
// Some kind of async call with a callback function or somesuch...
resolve(result);
});
}).then(data => {
// Do something with the final result
console.log(data);
});
您可以使用setTimeout()模拟某种任意异步工作:
const myPromise = new Promise(function(resolve, reject) {
// Resolve with "Done!" after 5 seconds
setTimeout(() => {
resolve("Done!");
}, 5000);
}).then(data => {
console.log(data); // "Done!"
});
但是,您的原始代码将resolve()调用放在一个奇怪的地方,甚至不传递任何数据.它看起来有点像这样:
const myPromise = new Promise(function(resolve, reject) {
// Resolve with "Done!" after 5 seconds
setTimeout(() => {
// Doing some work here instead of resolving...
}, 5000);
resolve();
}).then(data => {
console.log(data); // This would be "undefined"
});
你正在做一个console.log(“完成”);在您的原始代码中实际上您应该在哪里解决(someData);!
你也试图在你的Promise的异步函数中做副作用,这真的很奇怪,与Promise的工作方式相反.承诺应该是关闭并执行其异步工作,然后使用结果数据解决 – 字面上使用.then()链.
此外,您不应在Promise内部进行多次异步调用,而应将其概括为可重用且仅封装单个网络请求.这样你可以触发多个异步Promises,等待它们全部解决,然后做一些事情.
const activeMachines = ["41", "42", "43"];
// Make a reusable function that returns a single Promise
function fetchAPI(num) {
return new Promise(function(resolve, reject) {
const getAPIData = new XMLHttpRequest();
const url = "http://127.0.0.1:8000/processes/apidata/" + num + "/";
getAPIData.open("GET", url);
getAPIData.send();
getAPIData.onload = function() {
const APIData = JSON.parse(getAPIData.responseText);
const resolveData = {};
resolveData["machine_" + num] = APIData[0].author_id;
resolveData["temp" + num] = APIData[0].tempData; //get value
resolveData["humid" + num] = APIData[0].humidData;
timeValue = String(APIData[0].dateTime);
resolveData["time" + num] = new Date(timeValue);
resolve(resolveData);
};
});
}
// Promise.all() will resolve once all Promises in its array have also resolved
Promise.all(
activeMachines.map(ea => {
return fetchAPI(ea);
})
).then(data => {
// All of your network Promises have completed!
// The value of "data" here will be an array of all your network results
});
fetch()API很棒,你也应该学会使用它 – 但只有在你理解了Promise实际操作背后的理论和实践之后.