problema nel convalidare un form

[markella92]

è corretto secondo voi questo script per verificare se i campi (mittente, oggetto e testo della e-mail) sono corretti all'interno di un form?

function controllaEmail() {
var email = document.ciao.email.value;
var oggetto=document.ciao.oggetto.value;
var testo=document.ciao.testo.value;
var filtro = /^([a-zA-Z0-9_.-])+@(([a-zA-Z0-9-])+.)+([a-zA-Z0-9]{2,4})+$/;
if (!filtro.test(email)) {
alert("Attenzione, indirizzo email non valido");
document.ciao.email.select();
return false;
}else if ((oggetto.value==="") || (oggetto.value==="undefined")) {
alert("Il campo Oggetto è obbligatorio.");
document.ciao.oggetto.focus();
return false;
}else if ((testo.value==="") || (testo.value==="undefined")) {
alert("Il campo Testo è obbligatorio.");
document.ciao.testo.focus();
return false;
}else{
return true;
}

quando esce l'alert che non ho inserito la mail, premo ok e mi sottomette la pagina...mi sapreste aiutare?? GRAZIE MILLE in anticipo

[cavicchiandrea]

Prova questo

function controllaEmail() {
var email = document.ciao.email.value;
var oggetto=document.ciao.oggetto.value;
var testo=document.ciao.testo.value;
var filtro = /^([a-zA-Z0-9_\.\-])+@(([a-zA-Z0-9])+\.)+([a-zA-Z0-9]{2,4})+$/;
if (!filtro.test(email)) {
alert("Attenzione, indirizzo email non valido");
document.ciao.email.select();
return false;
}else if ((oggetto.value=="") || (oggetto.value=="undefined")) {
alert("Il campo Oggetto è obbligatorio.");
document.ciao.oggetto.focus();
return false;
}else if ((testo.value=="") || (testo.value=="undefined")) {
alert("Il campo Testo è obbligatorio.");
document.ciao.testo.focus();
return false;
}else{
return true;
}

[markella92]

sia con il return controllaEmail() che senza il return (nell'input) mi sottomette la pagina anche con i campi e gli allerta non vengono visualizzati proprio...

[cavicchiandrea]

Metti la pagina online e riporta qui il link

[markella92]

function controllaEmail() {
var email = document.ciao.email.value;
var oggetto = document.ciao.oggetto.value;
var testo = document.ciao.testo.value;
var filtro = /^([a-zA-Z0-9_.-])+@(([a-zA-Z0-9-])+.)+([a-zA-Z0-9]{2,4})+$/;


if (!filtro.test(email)) {
alert("Attenzione, indirizzo email non valido");
document.ciao.email.select();
return false;
}else if ((oggetto.value ==="") || (oggetto.value==="undefined")) {
alert("Il campo Oggetto è obbligatorio.");
document.ciao.oggetto.focus();
return false;
}else if ((testo.value ==="") || (testo.value==="undefined")) {
alert("Il campo Testo è obbligatorio.");
document.ciao.testo.focus();
return false;
}else{
return true;
}
}

E L'INPUT:
<input type="image" name="btn-send" src="images/btn-send.png" align="right" onClick="return controllaEmail()"/>

[markella92]

in questo modo mi esegue solo il controllo sulla email mentre l'oggetto e il testo della email il controllo non viene eseguito....

[cavicchiandrea]

togli onclick="return controllaEmail()" a metti il contro nel form onsubmit="return controllaEmail()"

[markella92]

niente.....

[markella92]

continua a verificare solo l'email.....

[cavicchiandrea]

continua a verificare solo l'email.....

Metti la pagina online e riporta qui il link