controllo campi form

[evetta]

ho messo questo controllo javascript per controllare i campi di un form, ma non funziona..

[CODE]function MM_validateForm() {
with(document.form1){
if (document.form1.nome.value==""){
alert ("Errore: compilare il campo NOME");
document.form1.nome.focus();
return false;
}
if (document.form1.cognome.value==""){
alert ("Errore: compilare il campo COGNOME");
document.form1.cognome.focus();
return false;
}
if (document.form1.eta.value==""){
alert ("Errore: compilare il campo ETA'");
document.form1.eta.focus();
return false;
}
if (document.form1.cellulare.value==""){
alert ("Errore: compilare il campo CELLULARE");
document.form1.cellulare.focus();
return false;
}
if (document.form1.professione.value==""){
alert ("Errore: compilare il campo PROFESSIONE");
document.form1.professione.focus();
return false;
}
if (document.form1.email.indexOf("@")==(-1) || email.indexOf(".")==(-1)) {
alert("Inserisci un indirizzo e-mail valido");
document.form1.email.focus();
return.false;
}
if (document.form1.citta.value==""){
alert ("Errore: compilare il campo CITTA' DI PROVENIENZA");
document.form1.citta.focus();
return false;
}
}

[fcaldera]

a occhio mi sembra che manchi una parentesi graffa di chiusura
al blocco che inizia con

 with (document.form1) {

ad ogni modo prima di aggiungerla, ti suggerisco di eliminare direttamente

with (document.form1) {

visto che per come stai validando il form è una istruzione che non ti serve.