JavaScript for in loop issue

Question

i need to get 4 request from the server, i have stored all this 4 urls in this way. for information this is raw url (http://www.earthtools.org/timezone/40.714352/-74.0059731);

var nationZone = {
    getNewYorkLocalTime :  'getTime.php?lat=40.7143528&lan=-74.0059731',
    getLondonLocalTime : 'getTime.php?lat=51.5001524&lan=-0.1262362',
    getChennaiLocalTime :  'getTime.php?lat=13.060422&lan=80.249583',
    getBangaloreLocalTime:'getTime.php?lat=12.9715987&lan=77.5945627'

}

this is calling my getTime.php and retrieving the result.

for that, i created this for in loop, but this prints only one time intead of 4 times? how can i make this to call 4 request ?

for(zone in nationZone ){
    if (window.XMLHttpRequest){
        zone=new XMLHttpRequest();
    }else{
        zone=new ActiveXObject("Microsoft.XMLHTTP");
    }
    zone.onreadystatechange=function() {
        if(zone.readyState==4 && zone.status==200){
            alert(zone.responseText);
        }
    }
    zone.open("GET",nationZone[zone],true);
    zone.send();
}

I do not mind using a solution that involves an additional third party library

possible duplicate of Access outside variable in loop from Javascript closure — Felix Kling
– Felix Kling, Commented Nov 17, 2011 at 10:12

Niels · Accepted Answer · 2011-11-17 10:13:56Z

1

You can try this if your using jQuery, but I doubt about it to work, because cross domain ajax is not allowed.

var nationZone = {
    getNewYorkLocalTime :  'getTime.php?lat=40.7143528&lan=-74.0059731',
    getLondonLocalTime : 'getTime.php?lat=51.5001524&lan=-0.1262362',
    getChennaiLocalTime :  'getTime.php?lat=13.060422&lan=80.249583',
    getBangaloreLocalTime:'getTime.php?lat=12.9715987&lan=77.5945627'

}

$.each(nationZone , function(key, value){
    $.get(value, function(response){
        alert(response);
    });
});

answered Nov 17, 2011 at 10:13

Niels

50.1k4 gold badges68 silver badges86 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Anders Tornblad · Accepted Answer · 2011-11-17 10:10:52Z

1

You are using the same name for the loop variable and the XHR. Don't do that.

EDIT: Also, this question has the jQuery tag, but you are not using jQuery. Instead of doing your own XHR stuff, use jQuery's ajax implementation through $.ajax or $.get. Also consider using JSONP, if the remote server supports it.

answered Nov 17, 2011 at 10:10

Anders Tornblad

19.4k9 gold badges53 silver badges67 bronze badges

1 Comment

Felix Kling Over a year ago

While this is true, this is not the only problem and changing the name won't solve it.

Arindam · Accepted Answer · 2011-11-17 11:34:47Z

0

zone variable inside the if/else block is overwriting the zone from the for loop. Hence your loop only has one zone object to handle by the time it finishes the first iteration.

Use some other var name inside the function. The line :

`zone.open("GET",nationZone[zone],true);`

should say

`newZoneVariable.open("GET", nationZone[zone], true);`

where newZoneVariable is the var used inside the loop body.

answered Nov 17, 2011 at 11:34

Arindam

9981 gold badge8 silver badges22 bronze badges

Collectives™ on Stack Overflow

JavaScript for in loop issue

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related