piccolo problema validazione form

[eloderma]

ho questo controllo per un form ,e ho problemi con il controllo della mail, non è obbligatoria ma se inserita vorrei controllarla, se inserisco una mail errata mi avverte ma se dopo inserisco una giusta mi continua dire che è sbagliata, dov'è l'errore?

grazie

function controllo(azione) {
     // Variabili associate ai campi del modulo
     var nome = document.form1.nome.value;
     var cognome = document.form1.cognome.value;
     var telefono = document.form1.telefono.value;
     var email = document.form1.email.value;
     var via = document.form1.via.value;
     var cap = document.form1.cap.value;
     var citta = document.form1.citta.value;
     var checkEmail = /^([\w-]+(?:\.[\w-]+)*)@((?:[\w-]+\.)*\w[\w-]{0,66})\.([a-z]{2,6}(?:\.[a-z]{2})?)$/i;

        if ((nome == "") || (nome == "undefined")) {
           alert("Il campo Nome è obbligatorio.");
           return false;
        }
        else if ((cognome == "") || (cognome == "undefined")) {
           alert("Il campo cognome è obbligatorio.");
           return false;
        }
        else if ((telefono == "") || (telefono == "undefined")) {
           alert("Il campo telefono è obbligatorio.");
           return false;
        }
        else if ((email != "")) {
           if(!checkEmail.test( email )){
             alert("Non hai inserito un indirizzo email corretto.");
             return false;
           }
        }
        else if ((via == "") || (via == "undefined")) {
           alert("Il campo via è obbligatorio.");
           return false;
        }
        else if ((cap == "") || (cap == "undefined")) {
           alert("Il campo codice postale è obbligatorio.");
           return false;
        }
        else if ((citta == "") || (citta == "undefined")) {
           alert("Il campo citta è obbligatorio.");
           return false;
        }
        else {
           document.form1.action = azione;
           document.form1.submit();
        }
  } 

[Braco]

Originariamente inviato da eloderma
ho questo controllo per un form ,e ho problemi con il controllo della mail, non è obbligatoria ma se inserita vorrei controllarla, se inserisco una mail errata mi avverte ma se dopo inserisco una giusta mi continua dire che è sbagliata, dov'è l'errore?

grazie

function controllo(azione) {
     // Variabili associate ai campi del modulo
     var nome = document.form1.nome.value;
     var cognome = document.form1.cognome.value;
     var telefono = document.form1.telefono.value;
     var email = document.form1.email.value;
     var via = document.form1.via.value;
     var cap = document.form1.cap.value;
     var citta = document.form1.citta.value;
     var checkEmail = /^([\w-]+(?:\.[\w-]+)*)@((?:[\w-]+\.)*\w[\w-]{0,66})\.([a-z]{2,6}(?:\.[a-z]{2})?)$/i;

        if ((nome == "") || (nome == "undefined")) {
           alert("Il campo Nome è obbligatorio.");
           return false;
        }
        else if ((cognome == "") || (cognome == "undefined")) {
           alert("Il campo cognome è obbligatorio.");
           return false;
        }
        else if ((telefono == "") || (telefono == "undefined")) {
           alert("Il campo telefono è obbligatorio.");
           return false;
        }
        else if ((email != "")) {
           if(!checkEmail.test( email )){
             alert("Non hai inserito un indirizzo email corretto.");
             return false;
           }
        }
        else if ((via == "") || (via == "undefined")) {
           alert("Il campo via è obbligatorio.");
           return false;
        }
        else if ((cap == "") || (cap == "undefined")) {
           alert("Il campo codice postale è obbligatorio.");
           return false;
        }
        else if ((citta == "") || (citta == "undefined")) {
           alert("Il campo citta è obbligatorio.");
           return false;
        }
        else {
           document.form1.action = azione;
           document.form1.submit();
        }
  } 

Prova ad allargare le virgolette di uno spazio così:

else if ((email != " ")) { ...

se proprio non va prova il mio codice:

//Valida l'Email   (Campo non obbligatorio)                                
if (f.email.value.length != 0 ){
var reg = new RegExp("^([-_.#ìùòàè$&%+a-z0-9])+[@][a-z0-9]+(\\.[-_#ìùòàè$&a-z0-9]{2,})*(\\.[a-z]{2,4})$");
if(!f.email.value.match(reg)){
f.email.style.backgroundColor='yellow'
alert("Inserire un' Email valida") 
f.email.focus();
return false;                                                                             
}
 }