Ich bin nicht in der Lage, mehrere Ereignisse zu loopen und zu erstellen

Question

Ich weiß nicht, wie ich es nennen soll, aber was ich will, ist diese Änderungsfunktion für jeden Tag der Woche zu haben.

Ist etwas falsch mit meiner Verkettung? Die Schleife funktioniert definitiv und ich bekomme die Werte aus dem Array.

Answer 1

Wenn Sie eine Funktion innerhalb einer Schleife in Javascript verwenden und die Funktion die Schleifenvariable verwendet, erfasst sie die Variable selbst, nicht den aktuellen Wert. Am Ende der Schleife ist die Variable der letzte Wert, den i erreicht. Sie müssen alles in eine andere Funktion einfügen und den aktuellen Wert der Variablen an diesen übergeben.

for(i=0;i<daysofweek.length;i++){ 
    (function(i) { 
    $("input[name=" + daysofweek[i] + "]").change(function() { 
     if($(this).is(':checked')) { 
     $("#" + daysofweek[i] + "_content").show(); 
     } else { 
     $("#" + daysofweek[i] + "_content").hide(); 
     }  
    }); 
    })(i); //loop through all days of the week creating the event that show/hides days 
} 

Answer 2

Sie verlieren den richtigen Wert von i, weil es in der for Schleife nicht scoped ist.

Sie könnten use $.each() instead, die einen Verschluss schafft:

$.each(daysofweek, function(i, val) { 
    $("input[name=" + daysofweek[i] + "]").change(function() { 
     if (this.checked) { 
      $("#" + daysofweek[i] + "_content").show(); 
     } else { 
      $("#" + daysofweek[i] + "_content").hide(); 
     } 
    }); //loop through all days of the week creating the event that show/hides days 
}); 

Auch ich $(this).is(':checked')-this.checked geändert. Es ist viel schneller.

Sie können auch ein Bit by using .toggle() anstelle von show/hide vereinfachen.

$.each(daysofweek, function(i, val) { 
    $("input[name=" + daysofweek[i] + "]").change(function() { 
     $("#" + daysofweek[i] + "_content").toggle(this.checked); 
    }); //loop through all days of the week creating the event that show/hides days 
}); 

Answer 3

Variablen werden nicht in JavaScript Block-scoped, so dass die Funktionen Sie auf die Änderung() Methode übergeben werden den gleichen Wert von i alle haben.