controllo form

[Sclerato]

cosa c'e' che non va!? trovate qualhc evidente errore?

function error(elem, text) {
	window.alert(text);
	elem.select();
	elem.focus();
}

function checkform(){
	// agenzia
	if (document.form1.id_agenzia.selectedIndex == "") { 
		window.alert("Scegli la tua filiale.");
	    document.form1.id_agenzia.focus()
		return false;
	} 
	// email
	var email = document.form1.email.value; 
	var email_reg_exp = /^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-]{2,})+\.)+([a-zA-Z0-9]{2,})+$/; 
	if (!email_reg_exp.test(email) || (email == "") || (email == "undefined")) { 
		error(document.form1.email,"Inserire un indirizzo email corretto.");
		return false;
	}
	// caso fattura alla ragione sociale
	if (document.form1.tipo_ordine.value == "ragione"){
		if (document.form1.nome.value == "") { 
		error(document.form1.nome,"Inserisci il nome.");
		return false;
		} 
		if (document.form1.via.value == "") { 
		error(document.form1.via,"Inserisci l'indirizzo.");
		return false;
		} 
		if (document.form1.cap.value == "") { 
		error(document.form1.cap,"Inserisci il c.a.p.");
		return false;
		} 
		if (document.form1.citta.value == "") { 
		error(document.form1.citta,"Inserisci la citta.");
		return false;
		} 
		if (document.form1.iva.value == "") { 
		error(document.form1.iva,"Inserisci la partita iva.");
		return false;
		} 
	}
return true;
}