Funzione JavaScript: dove sbaglio?

[leandro-2003]

Salve a tutti, ho cercato un pò su Internet e ho creato queste due funzioni:

function verifyNumbers(){
var input1 = document.funzione.inizio.value;
var input2 = document.funzione.fine.value;
if(isNaN(input1)){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
if(isNaN(input2)){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
}

function verifyLength(){
var input1 = document.funzione.inizio.length;
var input2 = document.funzione.fine.length;
if(input1 > 4){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
if(input2 > 4){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
}

Il form contiene questi attributi:

name="funzione" onsubmit="return verifyNumbers(); return verifyLength();"

E i due campi di input hanno rispettivamente i nomi "inizio" e "fine".
Se inserisco delle lettere verifyNumbers fa il suo dovere; se inserisco nel campo più di 4 cifre non succede niente.
Dove sbaglio?
Grazie a tutti.

[Alhazred]

onsubmit="return verifyNumbers(); return verifyLength();"

potrei sbagliarmi, ma il primo return penso che inibisca l'esecuzione della seconda funzione, prova a mettere solo il "return verifyLength()" e vedi se la funzione viene eseguita correttamente.

[leandro-2003]

Funziona!
Allora ho provato invertendo l'ordine dei return e...
Funziona!
Non avrei mai pensato di risolvere invertendo l'ordine... grazie comunque

[Alhazred]

Secondo me è comunque sbagliato, perché nelle 2 funzioni non ritorni né true, né false, mentre dato che nell'onclick richiedi un return, un valore dovrebbe essere ritornato.

In parole povere, se viene generato l'alert la funzione deve ritornare false, altrimenti deve ritornare true.

[leandro-2003]

Alla fine dei 4 if c'è "return false;". Non basta?

[Alhazred]

Non ci avevo fatto caso, sorry...

[Alhazred]

Ad ogni modo, se l'ordine delle funzioni nell'evento onclick è ora
verifyLength();
verifyNumbers();

fa questa prova: inserisci un input che non genera alert in verifyLength(), ma che li generi in verifyNumbers(), funziona regolarmente?

[cavicchiandrea]

Se invertendo l'ordine hai (o pensi) d'aver risolto c'è qualcosa che non va comunque dato che verifyLength() come l'hai scritto non potrà mai essere soddisfatto

function verifyNumbers(){
var input1 = document.funzione.inizio.value;
var input2 = document.funzione.fine.value;
if(isNaN(input1) && isNaN(input2)){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
}

function verifyLength(){
var input1 = document.funzione.inizio.value.length;
var input2 = document.funzione.fine.value.length;
if(input1 < 4 && input2 < 4){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
}

Inoltre se le due funzioni sono obbligatorie meglio eseguire onsubmit così

onsubmit="return(verifyNumbers() &&  verifyLength())"

Concludo dicendo che se non c'è qualche altro motivo le funzioni possono anche essere unite

[leandro-2003]

Ho modificato lo script (invece di > ho messo !=) e come consigliato da cavicchiandrea (che ringrazio ) ho unito le due funzioni e adesso il codice è questo

function verify(){
    var value1 = document.funzione.inizio.value;
    var value2 = document.funzione.fine.value;
    var length1 = document.funzione.inizio.length;
    var length2 = document.funzione.fine.length;
    if(isNaN(value1)){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
    if(isNaN(value2)){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
    if(length1 != 4){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
    if(length2 != 4){ alert("Prima di continuare devi inserire nei campi un numero di 4 cifre."); return false; }
}

<form ... onsubmit="return verify();">

E funziona a meraviglia!

@cavicchiandrea
Prima avevo risolto così, mistero comunque il codice da te fornito non era funzionante.

@Alhazred
Se non sbaglio verifyNumbers() funzionava correttamente.

[leandro-2003]

Orrore
Se metto 4 cifre da sempre errore... non capisco...