problema con ciclo for

[techno]

Salve ho questo script:

for(var menuopen=0;menuopen<=5;menuopen++){
			if($("ul.m"+menuopen).is(":visible")){
				$("ul.m"+menuopen).slideUp("fast", function(){
					alert(menuopen);
					$("li#m" + menuopen + ">a").removeClass("menucaton").addClass("menucat");
					$("ul.m" + menuopen + ">li").remove()
					$("ul.m" + menuopen).css("display","none");
				});
			}
		}

cioè verifico se un menu è aperto così lo chiudo ma il fatto e che io conto fino a 5 nel ciclo for ma in alert(menuopen) mi da 6.

comè possibile ciò????

[br1]

Se usi menuopen++ l'incremento viene fatto al termine del ciclo...

[techno]

be certo che deve essere fatto al termine del ciclo ma se la condizione è situato al terzo ciclo mi da sempre 6 in alert.

[techno]

e alert è dentro la condizione come fa a dare 6 e questo che non capisco

[fcaldera]

dà 6 (correttamente) perchè hai creato una closure
quando l'evento si verifica, il for ha portato la variabile menuopen a 6 , pertanto l'alert stampa l'ultimo valore di menuopen all'interno del ciclo (mentre tu ti aspetti che venga stampato il valore che era attuale nella singola iterazione).

[br1]

Originariamente inviato da fcaldera
dà 6 (correttamente) perchè hai creato una closure
quando l'evento si verifica, il for ha portato la variabile menuopen a 6 , pertanto l'alert stampa l'ultimo valore di menuopen all'interno del ciclo (mentre tu ti aspetti che venga stampato il valore che era attuale nella singola iterazione).

Non saprei (anzi non so) dirlo meglio

[fcaldera]

a occhio devi modificare in

for(var menuopen=0;menuopen<=5;menuopen++){
	if($("ul.m"+menuopen).is(":visible")) {
               
               (function(i) {
   
 	           $("ul.m"+i).slideUp("fast", function(){
			alert(i);
			$("li#m" + i + ">a").removeClass("menucaton").addClass("menucat");
			$("ul.m" + i + ">li").remove()
			$("ul.m" + i).css("display","none");
		    });

                })(menuopen);
	}
}

[techno]

e si oppure imposto il valore invece che 5 a 4 perchè appunto mi ritorna 5.