javascript per sommare i calcoli da un form

[gianvenezia]

Premesso che questo script funziona perfettamente ed è affidabile,
cè il modo di ridurlo un po?

function Somma() { // ok

var elementi=document.form1.modulo1;
var elementi2=document.form1.modulo2;


var valore=0;
 for (i=0; i<elementi.length; i++) {
  if (elementi[i].type=="checkbox" && elementi[i].checked) valore+=parseFloat(elementi[i].value);
 }
 
 if (i==0) { // funzione nel caso ci sia solo una fattura nell'elenco
 	if (elementi.type=="checkbox" && elementi.checked) valore+=parseFloat(elementi.value);
 } 
 
  for (i=0; i<elementi2.length; i++) {
  if (elementi2[i].type=="checkbox" && elementi2[i].checked) valore+=parseFloat(elementi2[i].value);
 }
 
 if (i==0) { // funzione nel caso ci sia solo una fattura nell'elenco
 	if (elementi2.type=="checkbox" && elementi2.checked) valore+=parseFloat(elementi2.value);
 } 
document.form1.totalone.value=Math.round(100*valore)/100
;
}

[gianvenezia]

in pratica ho un form con numerosi checkbox S/N che hanno valore numerico.
un altro campo modulo read only, deve mostrare la somma che il javascript deve calcolare.

[fcaldera]

non capisco cosa ti servono i blocchi

if (i==0)....

e cosa sono modulo1 e modulo2??

Puoi postare la pagina con le checkbox comprese??

[gianvenezia]

servivavo ad un altro script, che ho modificato per il mio scopo.

comunque li ho tolti come mi hai detto tu, e in effetti lo script funziona lo stesso:

function Somma() { // ok

var elementi=document.form2.modulo_1;
var elementi2=document.form2.modulo_2;


var valore=0;
 for (i=0; i<elementi.length; i++) {
  if (elementi[i].type=="checkbox" && elementi[i].checked) valore+=parseFloat(elementi[i].value);
 }
 
 	if (elementi.type=="checkbox" && elementi.checked) valore+=parseFloat(elementi.value);
 
  for (i=0; i<elementi2.length; i++) {
  if (elementi2[i].type=="checkbox" && elementi2[i].checked) valore+=parseFloat(elementi2[i].value);
 }
 
 	if (elementi2.type=="checkbox" && elementi2.checked) valore+=parseFloat(elementi2.value);

 
document.form2.totalone.value=Math.round(100*valore)/100
;
}

questi sono i 2 moduli della form:

<td class=""><input name="modulo_1" type="checkbox" onClick="Somma();" id="modulo_1" value="1" >1 </td>
<td class=""><input name="modulo_2" type="checkbox" onClick="Somma();" id="modulo_2" value="8" >1 </td>