I seem to be having some issues with making HEAD
requests, and preserving the integrity of data in an array.
Given this snippet:
var ima
You have a single i
variable which is shared by all of the callbacks.
Since AJAX is asynchronous, all of the callbacks run after your loop is finished, and they all get the same i
.
To fix this, you need to move the AJAX call into a separate function that takes i
as a parameter.
Thus, each callback will get a separate i
parameter.
You can scope I like so:
success: function(i){
return function(message){
imageData.push([imageTemp[i], ajaxSizeRequest.getResponseHeader('Content-Length')]);
}
}(i)
The problem is that the single variables i
and ajaxSizeRequest
being captured by the callback function are the same variables for all instances of the callback function. I think if you call a function and pass the index variable to it and, at the same time, scope the request variable locally to the function itself use the response parameter of the done handler, you should end up with independent variables captured by the callback. It should then reference each array element and each response variable correctly.
var imageData = Array();
for(var i = 0; i < imageTemp.length; i++){
updateImageData( i );
}
function updateImageData( i )
$.ajax({
type: "HEAD",
async: true,
url: imageTemp[i],
}).done(function(message,text,jqXHR){
imageData.push([imageTemp[i], jqXHR.getResponseHeader('Content-Length')]);
});
}
looks like your i
isnt properly closed-in
in addition, you can't use ajaxSizeRequest
because it too is pointing to just one request (probably the last, because the loop will execute very fast)
just wrap your success
callback function as follows, changing the reference to ajaxSizeRequest
:
success: (function(i){
return function(data,status,xhr){
imageData.push([imageTemp[i], xhr.getResponseHeader('Content-Length')]);
};
})(i)
If anyone still having trouble with this, and since this post is, like, 5 years-old already, here's a more 'modern' version of the answer: just use let
instead of var
in the original post's for
loop.
Info: Is there any reason to use the “var” keyword in ES6? and: MDN - Let syntax