controllo dinamico

[jet]

Ho la necessità di controllare un form con numero di elementi variabili; lo script che ho trovato controlla solo il primo elemento e quindi salta alla pagina prova.apsx;
dove sbaglio?

<form name="form" method="Post" action="prova.aspx" onSubmit="return CheckForm(this);">

 <script type="text/JavaScript"> 
<!--
function CheckForm() {
 
  var f = document.forms[0];
  var n = f.elements.length;
 
  for(var i = 0; i < n; i++){
    if(f.elements[i].value == ''){
      return alert('Error: value '+f.elements[i].name+ ' is empty');
      return false;
    }
  }
}
// -->
</script>

	<input type="text" name="0" />


	<input type="text" name="1" />


	<input type="submit" name="submit" value="vai" />
</form>

[cavicchiandrea]

Sbagli perché controlli solo il valore dell'elemento e il pulsante ha un valore definito "vai" fai cosi:
if(f.elements[i].type == text'' && f.elements[i].value == ''){

[jet]

Grazie; con il tuo consiglio l'errore è che manca una parentesi tonda alla fine del tuo post;
ma comunque dovrei fare il controllo nel pulsante?

[jet]

l'errore era text'' cambiato con 'text', ma comunque continua a controllare solo il primo elemento.

[cavicchiandrea]

togli il return nel alert

[jet]

Funge! Se invece delle text metto le textarea, cosa cambierebbe?

[jet]

Ho tolto il controllo sul type e funge. In pratica si doveva solo togliere il return sull'alert.

[cavicchiandrea]

Credo tutto le texarea non fanno parte della "collezione" input e non dispongono di type pertanto dovrai ciclare (for nuovo) solo quell'elemento specifico
var area = document.getElementsByTagName("texarea")
if(area.length>0){
qui metti il ciclo for for(.......)
}else{
qui metti l'avviso se le form è presente una sola textarea
}

[jet]

In realtà il controllo mi serviva proprio per le textarea, ma come ho detto, semplicemente togliendo return sull'alert è tutto ok. Ho già testato il controllo e va bene.
Grazie!