[js/html] Zaznacz/odznacz Opcje

[kaszanka]

Hi,

Mam kodzik:

[HTML] pobierz, plaintext 
<script type="text/javascript">
function checkAll(c){
for(var i=0, t=document.forms['form'].elements['id_send'];i<t.length;i++){
      if(t[i].type.toLowerCase()=='checkbox'){
            t[i].checked=c;
      }
   }
}
</script>
[HTML] pobierz, plaintext 

i kod html:

[HTML] pobierz, plaintext 
<form method="post" action="" name=form>
<input type="checkbox" onclick="checkAll(this.checked)">&nbsp; zaznacz / odznacz wszystko
<input type="checkbox" class="checkbox" name="id_send[]" value="'.['pozycja:id'].'"> // tu jest pętla php gdzie 'id_send' idzie to tablicy
</form>
[HTML] pobierz, plaintext 

I teraz w przypadku zapisu:
<input type="checkbox" class="checkbox" name="id_send" value="'.['pozycja:id'].'">
działa zaznaczanie wszystkich checkbox, natomast jak w pierwszym przykładzie gdy jest name="id_send[]" to nie działa.
Jak to zmodyfikować?

[kamil4u]

Jeszcze dodam kilka uwag do tej prostej pętli:
- nie zadeklarowana zmienne t
- przy każdym 'obrocie' funkcji sprawdzasz wielkość tablicy "i<t.length;" co tylko nie potrzebnie przedłuża czas działania (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
- i<t.length; warunek jest i tak niepotrzebny

Moja propozycja:

Kod

<script type="text/javascript">
function checkAll(c,t,i){
for(i=0; t=document.forms['form'].elements['id_send'][i++];){
if(t.type.toLowerCase()=='checkbox'){
t.checked=c;
}
}
}
</script>

Wiem, że to w tym skrypcie to nieistotne, ale gdy będziesz pisał(może już piszesz) większe skrypty warto zwracać uwagę na takie 'pierdoły' (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)