controllo form con javascript

[jessica1234]

controllo form con javascript

ecco il codice html:
<form name="contform">
<table>
<tr>
<td><em>Nome</em></td>
<td><input type="text" size"30" maxleght="28" name="nome"></td>
</tr>
<td><em>Email</em></td>
<td><input type="text" size="30" maxleght="28" name="email"</td>
</tr>
<tr>
<td><em>Argomento</em></td>
<td>
<select name="argomento">
<option value="-">- Scegli un argomento -</option>
<option value="prob">Problemi tecnici</option>
<option value="sugg">Suggerimenti</option>
<option value="altro">Altro</option>
</select>
</td>
</tr>
<tr>
<td><em>Testo</em></td>
<td>
<textarea rows="5" cols="30" name="testo"></textarea>
</td>
</tr>
</table>
<br>
<hr size="2" width="450" align="left">
<input type="reset" value="Annulla">     
<input type="button" value="Invia" onClick="ControllaForm();" />
</form>

ecco la funzione che ho creato ma che non funziona per controllare la validità del form:
<script language="javascript">
<!-- function ControllaForm() {
var nome = document.contform.nome.value;
var email_reg_exp = /^([a-zA-Z0-9_.-])+@(([a-zA-Z0-9-]{2,})+.)+([a-zA-Z0-9]{2,})+$/;
var argomento = document.contform.argomento.options
[document.contform.argomento.selectedIndex].value;
var testo = document.contform.testo.value;
//Controllo nome
if ((nome == "") || (nome == "undefined")) {
alert('Il campo Nome è obbligatorio.');
document.contform.nome.focus();
return false;
}
//Controllo indirizzo email valido
else if (!email_reg_exp.test(email) || (email == "") || (email == "undefined")) {
alert("Inserire un indirizzo email corretto.");
document.contform.email.select();
return false;
}
//Controllo argomento
else if ((argomento == "") || (argomento == "undefined")) {
alert("Il campo Argomento è obbligatorio.");
document.contform.argomento.focus();
return false;
}
//Controllo testo
if ((testo == "") || (testo == "undefined")) {
alert("Il campo Testo è obbligatorio.");
document.contform.testo.focus();
return false;
}
//Invia form
else {
document.contform.action = "elabora_dati.asp";
document.contform.submit();
}
}
//-->
</script>

MA DOVE SBAGLIO?????
chi mi può aiutare???
grazie mille!!!

[vnotarfrancesco]

Ciao jessica1234, chiudo questo thread perché è identico a questo che hai inserito qui.
Porre la stessa domanda più volte in un forum si chiama crossposting e non è ammesso dal nostro regolamento.

11. Crossposting

11.0. Non è consentito postare lo stesso messaggio o parti di esso in più aree del forum, salvo casi di particolare interesse che dovranno comunque essere autorizzati da un Admin.

11.1. Messaggi crossposting verranno chiusi e quando possibile riuniti.
Se ripetuti potranno portare all'allontanamento dell'utente.

Valerio Notarfrancesco