Want to protect your cyber security and still get fast solutions? Ask a secure question today.Go Premium

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 175
  • Last Modified:

problem with for loop

The code below works, but only if I include an alert.  When i remove the alert, it does not pass the array values through as it should

			var theoption = ["first","second"];
			for (var i=0;i<theoption.length;i++)
				{			
				$.getJSON("testsample?menu="+theoption[i]+"",function(results){
								var options="";
								
								for(resd in results.data)					
								{
									options+="<option value='"+results.data[resd].id+"' >"+results.data[resd].Title+"</option>";
								} 
								$("#"+theoption[i]+"test").html(options);
				});		
				alert(theoption[i])	
		}

Open in new window

0
debbieau1
Asked:
debbieau1
  • 2
1 Solution
 
Alexandre SimõesManager / Technology SpecialistCommented:
Try it like this:
var theoption = ["first", "second"];
for (var i = 0; i < theoption.length; i++) {
    getValues(theoption[i]);
}

function getValues(option){
    $.getJSON("testsample?menu=" + option + "", function (results) {
        var options = "";

        for (resd in results.data) {
            options += "<option value='" + results.data[resd].id + "' >" + results.data[resd].Title + "</option>";
        }
        $("#" + option + "test").html(options);
    });
}

Open in new window

0
 
Alexandre SimõesManager / Technology SpecialistCommented:
You're having this problem because you're using the loop index inside the ajax callback.
As the callback is asyncronous, when it actually arrives the value of i is no longer the one at request time giving you unexpected results.

The solution above sends the current 'theoption' to the processing function, decoupling the callback process from the loop itself.
0
 
debbieau1Author Commented:
Thanks a lot.  Great clear explanation
0

Featured Post

Keep up with what's happening at Experts Exchange!

Sign up to receive Decoded, a new monthly digest with product updates, feature release info, continuing education opportunities, and more.

  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now