convalidazione form non funzionante

[luchett]

Salve a tutti ,
ho la seguente funziona che effettua una convalida di alcuni form,
(a seconda di quello che le passo nel parametro), ma non mi effettua
tutti i controlli.

function CheckCampi(f) {	
	alert("ci sono 1")
	if (document.gallery && f == document.gallery){
		if (f.file.value == ""){
			alert("Attenzione! Inserire il file da caricare!");
			f.file.focus();
			return(false)
		}	
	}
	alert("ci sono 2")
	if (f.titolo.value == ""){
		alert("Attenzione! Inserire il titolo");
		f.titolo.focus();
		return(false)	
	}
	alert("ci sono 3")	
	if (f.testo.value == ""){
		alert("Attenzione! Inserire il testo");
		f.testo.focus();
		return(false)
	}
return(true)
}

Ho messo quelle alert è ho constatato che apre solo la prima ?? e non entra nella prima if.

Come è possibile???

[fcaldera]

	if (document.gallery && f == document.gallery){
		if (f.file.value == ""){

scommetto che quello in grassetto è un campo tipo input type="file"... nel qual caso hai usato un nome riservato "file" per chiamare quell'input.

cambia il nome di quell'input (con un nome non riservato)

[luchett]

errore da principiante

Grazie mille per l'aiuto

[luchett]

Scusate nuovamente.

ora l'ho modificata così

function CheckCampi(f) {	
	if (f.titolo.value == ""){
		alert("Attenzione! Inserire il titolo");
		f.titolo.focus();
		return(false)	
	}
	if (f.t.value == ""){
		alert("Attenzione! Inserire il testo");			              f.t.focus();
		return(false)
	}
	if (document.gallery && f == document.gallery){
		if (f.f.value == ""){
			alert("Attenzione! Inserire il file da caricare!");
			f.f.focus();
			return(false)
		}	
	}

return(true)
}

quando il form non è "gallery", quindi effettua i primi due controlli solamente, al secondo entra se il campo è vuoto (t è una textarea) ma effettua ugualmente il submit, invece dovrebbe uscire e basta.
La funziona la richiamo così:

<input type="submit" value="inserisci" onClick="return CheckCampi(this.form);">

[fcaldera]

non si scrive return(false) ma
return false;

analogamente anche return true;

mi domando comunque che senso ha chiamare in modo equivoco i vari campi "t" oppure "f" (f.f.focus()) quando

1) potresti usare nomi più descrittivi
2) assegnare degli id ai vari campi e accedervi con document.getElementById() (modo approvato dal w3c)


Ciao

[luchett]

Prendo i tuoi consigli e li faccio miei.

Grazie mille

[luchett]

però il problema del return sussiste

continua a fare il submit nonostante entra nella if

[fcaldera]

assicurati che il tag form sia qlcs del genere

<form... onsubmit="return checkCampi(this)">

in modo che la validazione sia controllata all'evento onsubmit del form

altrimenti prova a vedere se c'è qualche errore con la console Javascript di firefox