Limitare selezione checkbox e aggiungere/rimuovere classe

[kikoweb]

Salve,
ho scritto un codice per limitare le selezioni di input checkbox a 2 disabilitando in automatico tutte le altre + il bottone x l'invio del form.
Inoltre quando si seleziona un checkbox viene aggiunta una classe per cambiare il background della riga all'interno della tabella.
Ora...funziona tutto però.... dato che secondo me è scritto molto male vorrei sapere se potete aiutarmi ad ottimizzare il codice riscrivendolo d'accapo in più mi servirebbe inoltre che vengano passato i due valori dei checkbox selezionati in 2 campi hidden.

---> FOGLI DI STILE (inserisco solo la classe che modifica il background della riga)

.selected { background-color: #E0EFF7; }

---> CODICE JAVASCRIPT

<script type="text/javascript">

/* Highlight Row */
function highlight(id, currentClass, checkboxId)
{
   var checkbox_element = document.getElementById(checkboxId);
   var row_element = document.getElementById(id);
   var lista_check = document.getElementsByTagName("input");
   var conta = 0;
        row_element.className = checkbox_element.checked ? 'selected' : currentClass;

for(i = 0; i < lista_check.length; i++){
if(lista_check.item(i).getAttribute("type") == "checkbox" && lista_check.item(i).checked == true){
conta++;
}
}
if (conta == 2) {
      document.getElementById('submit').disabled=false;

for(i = 0; i < lista_check.length; i++){
if(lista_check.item(i).getAttribute("type") == "checkbox" && lista_check.item(i).checked == false){
lista_check.item(i).disabled = true;
}
}


}
if (conta < 2) {
      document.getElementById('submit').disabled=true;

for(i = 0; i < lista_check.length; i++){
if(lista_check.item(i).getAttribute("type") == "checkbox" && lista_check.item(i).checked == false){
lista_check.item(i).disabled = false;
}
}


}
if(conta > 2){
      document.getElementById(checkboxId).checked = false;
}

}

</script>

--> FORM il campo $item->id viene generato con un ciclo foreach dove estrapola il valore da un database (inserisco solo alcune righe)

<form action="/confronta/?" method="get" name="mioform" accept-charset="utf-8">
<input type="hidden" name="valore1" value="">
<input type="hidden" name="valore2" value="">

<tr id="tr<?php echo "$item->id"; ?>" class="tr_class">
 <td class='num'><input id="<?php echo "$item->id"; ?>" type="checkbox" class="" value="<?php echo "$item->id"; ?>" onClick="JavaScript: highlight('tr<?php echo "$item->id"; ?>', '<?php echo $item->class ?>', this.id);"></td>
 <td class='num'><input id="<?php echo "$item->id"; ?>" type="checkbox" class="" value="<?php echo "$item->id"; ?>" onClick="JavaScript: highlight('tr<?php echo "$item->id"; ?>', '<?php echo $item->class ?>', this.id);"></td>
 <td class='num'><input id="<?php echo "$item->id"; ?>" type="checkbox" class="" value="<?php echo "$item->id"; ?>" onClick="JavaScript: highlight('tr<?php echo "$item->id"; ?>', '<?php echo $item->class ?>', this.id);"></td>
 <td class='num'><input id="<?php echo "$item->id"; ?>" type="checkbox" class="" value="<?php echo "$item->id"; ?>" onClick="JavaScript: highlight('tr<?php echo "$item->id"; ?>', '<?php echo $item->class ?>', this.id);"></td>
</tr>

 <input type="submit" id="submit" value="Confronta" disabled>
</form>

Funziona tutto ma come ho detto prima vorrei sapere se è possibile scrivere un codice migliore e in piu passare i due valori nei 2 campi hidden.
Spero che qualcuno sappia aiutarmi.

Grazie!!

[cavicchiandrea]

Forse si riesce ad eseguire un solo ciclo for da studiare e provare, se già usi jquery potrebbe migliorare

[kikoweb]

Ciao... si ma il problema è ... come?

[RoTeam]

Vedo tante graffe inutili e 2 condizioni di conta abbreviabili con un operatore ternario nel codice js

[kikoweb]

i consigli sono ben accetti ma anche qualche soluzione possibilmente scritta

[Vindav]

secondo me si può ridurre a questo:

cont = 0;

/* Highlight Row */
function highlight(id, currentClass, checkboxId)
{
   var row_element = document.getElementById(id);
   var lista_check = document.getElementsByTagName("input");
   row_element.className = checkboxId.checked ? 'selected' : currentClass;

   checkboxId.checked ? cont++ : cont--

   document.getElementById('submit').disabled= cont != 2;
   for(i = 0; i < lista_check.length; i++){
     if(lista_check.item(i).getAttribute("type") == "checkbox" &&     lista_check.item(i).checked == false){
       lista_check.item(i).disabled = cont == 2;
     }
   }
}​

Devi anche modificare la chiamata al metodo cosi: highlight('tr<?php echo "$item->id"; ?>', '<?php echo $item->class ?>', this);

Fai un po' di prove

[kikoweb]

ciao... beh ho provato la tua soluzione ma non funziona :/
non cambia il backgroung della riga, non limita la selezione a 2, non disabilita tutte le altre e non attiva il bottone ... eppure il tuo codice sembra scritto bene. Boh

[Vindav]

hai cambiato la chiamata al metodo?

[kikoweb]

si si

[kikoweb]

ricevo un errore

Uncaught ReferenceError: highlight is not defined