Controllo doppio su un form

[freizer]

ho creato un form di raccolta dati.

nei controlli ho sia un controllo che verifica la compilazione di alcuni campi e sto aggiungendo un altro controllo che verifichi la corretta compilazione del campo email.

il codice java che ho inserito è questo:

<script language="JavaScript">
var controllo = true
function valida() {
controllo = true
if (document.form1.nome.value == "") {controllo = false};
if (document.form1.cognome.value == "") {controllo = false};
if (document.form1.indirizzo.value == "") {controllo = false};
if (document.form1.citta.value == "") {controllo = false};
if (document.form1.provincia.value == "") {controllo = false};
if (document.form1.email.value == "") {controllo = false};
if (controllo == false) {
alert ('Attenzione. Non sono stati compilati dei campi obbligatori!');
}

return controllo

}

function emailCheck(emailStr) {

var emailPat = /^(\".*\"|[A-Za-z]\w*)@(\[\d{1,3}(\.\d{1,3}){3}]|[A-Za-z]\w*(\.[A-Za-z]\w*)+)$/;
var matchArray = emailStr.match(emailPat);
if (matchArray == null) {
alert("Email non corretto");
return false;
}

return true;
}
</script>

e nel campo di inizio form ho inserito:

<form name="form1" action="inviadati.asp" method="post" onSubmit="return valida(); return emailCheck(this.email.value);">

ma il controllo non lo fa!

dove sbaglio?

[balun]

qui:
onSubmit="return (emailCheck(this) && valida(this));">

[balun]

tu puoi dare un occhiata al mio post form check?

[freizer]

@ balun

mi spiace ma non saprei come aiutarti



il mio codice l'ho aggiornato in questo modo

<script language="JavaScript">
var controllo = true
var controlloemail = true
function valida() {
controllo = true
if (document.form1.nome.value == "") {controllo = false};
if (document.form1.cognome.value == "") {controllo = false};
if (document.form1.indirizzo.value == "") {controllo = false};
if (document.form1.citta.value == "") {controllo = false};
if (document.form1.provincia.value == "") {controllo = false};
if (document.form1.email.value == "") {
controllo = false
} else {
emailCheck(document.form1.email.value);
};
if (controllo == false) {
alert ('Attenzione. Non sono stati compilati dei campi obbligatori!');
} else {
if (controlloemail == false) {
controllo = false
alert ('Email non corretta!');
}
}

return controllo

}

function emailCheck(emailStr) {
var emailPat =
/^(\".*\"|[A-Za-z]\w*)@(\[\d{1,3}(\.\d{1,3}){3}]|[A-Za-z]\w*(\.[A-Za-z]\w*)+)$/;
var matchArray = emailStr.match (emailPat);
if (matchArray == null) {controlloemail = false}
}
</script>

e nel form

<form name="form1" action="inviadati.asp" method="post" onSubmit="return
valida();">


ma anche se inserisco una mail valida continua a dirmi 'Email non corretta!'

come mai?