Problema nel concatenare stringhe

[maurocoviello]

Ciao a tutti
Ho un problema nel concatenare due stringhe
questo è il codice

for (i=1; i<=conta; i++) {
	if(document.getElementById(i).checked) {
	testo=testo+document.getElementById(i).value;
	}
}

Io devo ottenere una stringa che ha concatenati tutti i valori delle checkbox selezionate.
Il problema è nella concatenazione.

Se scrivo

for (i=1; i<=conta; i++) {
	if(document.getElementById(i).checked) {
	testo=document.getElementById(i).value;
	}
}

giustamente testo vale l'ultimo valore della checkbox selezionata, però deve valere tutti i valori concatenati.
Dove sbaglio?

[fcaldera]

ma il primo codice non funziona?

[maurocoviello]

no, non funziona, praticamente non succede nulla...

[fcaldera]

l'errore è nel ciclo for perchè stai facendo un getElementById su un numero (e non si può assegnare un numero come id.

quindi dai un id corretto alle tue checkbox e usa questo codice

           inputs = document.getElementsByTagName('input');
            
            for (i=0; i<inputs.length; i++) {
                if (inputs[i].type == 'checkbox') {
                    if(inputs[i].checked) {
                        testo += inputs[i].value;
                    }
                }
            }

[maurocoviello]

Grazie!
Ho fatto come mi hai indicato e funziona, però ho ancora un problemino:
la stringa che contiene il valore delle checkbox inizia con undefined.
Non capisco perché
grazie

[fcaldera]

prima del ciclo for inizializza la variabile


testo = "";

[maurocoviello]

è vero! grazie