validazione form con controllo di tipo checkbox

[micdas]

Ciao,
Avevo un form con normali controlli di tipo input governati da un JS per la sua convalida.
Il JS è semplice:

function check_valid(theForm) {
	for (i=0; i < theForm.elements.length; i++) {
		if (theForm.elements[i].value == "" && theForm.elements[i].className == "si") {
			alert (theForm.elements[i].name + ": campo vuoto non accettabile");
			theForm.elements[i].value = "";
			theForm.elements[i].focus();
			return false;
		}
	}
}

Finora erano semplici controlli di tipo "text", "textarea" o "select".
Di questi alcuni sono obbligatori ed altri no.
A quelli obbligatori, per permetterne la validazione dal JS ho aggiunto una classe in questo modo:

<form name="prenotazione" action="form_processa.php" method="post" onsubmit="return check_valid(this);">
<input type="text" name="nome" size="20" value="" class="si" />
<input type="text" name="azienda" size="20" value="" />
<input type="text" name="telefono" size="20" value="" class="si" />
<select name="giorno" class="si"> 
       ...................
</select>
       ...........................
</form>

A questo form ho dovuto aggiungere un controllo di tipo checkbox

<input type="checkbox" name="privacy" class="si" />

Ma non riesco a farlo validare dal JS, nonostante la classe.
Dove sbaglio? Come dovrei modificare il JS per controllare che il controllo "checkbox" sia stato flaggato?

Grazie per l'aiuto che mi vorreste dare

[cavicchiandrea]

Prova cosi:

function check_valid(theForm) {
	for (i=0; i < theForm.elements.length; i++) {
		if (theForm.elements[i].value == "" && theForm.elements[i].className == "si") {
			alert (theForm.elements[i].name + ": campo vuoto non accettabile");
			theForm.elements[i].value = "";
			theForm.elements[i].focus();
			return false;
		}else if (theForm.elements[i].type=="checkbox" && theForm.elements[i].checked && theForm.elements[i].className == "si") {
			alert (theForm.elements[i].name + ": campo non spuntato");
			theForm.elements[i].focus();
			return false;
		}
	}
}

[micdas]

cavicchiandrea
ti ringrazio per la tua risposta
Stamane ho sperimentato la nuova funzione aggiornata come mi hai suggerito.
Tuttavia rimaneva un problema nella "if".

if (theForm.elements[i].value == "" && theForm.elements[i].className == "si") {
.................
}else if (theForm.elements[i].type=="checkbox" && theForm.elements[i].checked && theForm.elements[i].className == "si") {
................
}

In questo modo viene testato prima la proprietà value del controllo, poi viene valutato se il controllo in oggetto è una checkbox e, di conseguenza il form non viene mai accettato.

Ho pensato di invertire le condizioni nella "if" e di cambiare leggermente il codice ed ho parzialmente ovviato al problema.

if (theForm.elements[i].name == "privacy") {      // questo è il nome della checkbox
...............
} else if (theForm.elements[i].value == "" && theForm.elements[i].className == "si") {
...................
}

A sto punto c'è un altro problema: non posso bloccare il FORM se theForm.elements[i].checked ma se non è checked .
per cui la mia, temporanea e provvisoria soluzione sarebbe la seguente:

function check_valid(theForm) {
	for (i=0; i < theForm.elements.length; i++) {
		if (theForm.elements[i].name == "privacy") {
			if (theForm.elements[i].checked) {
				var errato="";
			} else {
				errato = theForm.elements[i].name + ": campo non spuntato";
			}
			if (errato != ""){
				alert(errato);
				theForm.elements[i].value = "";
				theForm.elements[i].focus();
				return false;
			}
		} else if (theForm.elements[i].value == "" && theForm.elements[i].className == "si") {
			alert (theForm.elements[i].name + ": campo vuoto non accettabile");
			theForm.elements[i].value = "";
			theForm.elements[i].focus();
			return false;
		}
        }
}

Quel che in questa routine mi disturba un poco è quel if (theForm.elements[i].checked) {
Non potrei dire da subito di controllare se il controllo non è checked?
come si potrebbe dirlo?
In questo modo accorcerei la routine ed eviterei del codice inutile in questo modo:
invece di dire:

if (theForm.elements[i].checked) {
	var errato="";
} else {
	errato = theForm.elements[i].name + ": campo non spuntato";
}
if (errato != ""){
	alert(errato);
	theForm.elements[i].value = "";
	theForm.elements[i].focus();
	return false;
}

potrei scrivere semplicemente:

if (theForm.elements[i].non-checked) {    // non so come indicare lo stato non-checked 
	alert("devi flaggare il controllo");
	theForm.elements[i].focus();
	return false;
}

Grazie ancora

[cavicchiandrea]

Dovrebbe funzionare, comunque la logica del controlli per l'elemento "globale" form
if type=='text' && value=="" x input text
if type=='select' && selectedIndex x le select
if type == 'checkbox' o 'radio' && checked == true/false x radio e spunta
se devi validare tutti gli elementi del form altrimenti fai i controlli specifici
if document.nomeform.nomecampo.value/checked/selectedIndex per dettaglio clicca qui

[micdas]

Grazie cavicchiandrea e ottima la sintesi dei controlli che mi hai fornito.

Grazie ai tuoi suggerimento ho ridotto il codice in questo modo e funziona egregiamente

function check_valid(theForm) {
	for (i=0; i < theForm.elements.length; i++) {
		if (theForm.elements[i].name == "privacy") {
			if (theForm.elements[i].checked == false) {
				alert(theForm.elements[i].name + ": campo non spuntato");
				theForm.elements[i].value = "";
				theForm.elements[i].focus();
				return false;
			}
		} else if (theForm.elements[i].value == "" && theForm.elements[i].className == "si") {
			alert (theForm.elements[i].name + ": campo vuoto non accettabile");
			theForm.elements[i].value = "";
			theForm.elements[i].focus();
			return false;
		}
}

Per il checkbox uso il name anzichè il type perchè, almeno in questo momento, non ho intenzione di controllare altri eventuali checkbox.