ho 4 campi da controllare che non siano vuoti, e lo faccio cosi

codice:
function controlla_campi_vuoti()
{
    var Text = document.getElementsByTagName("input");   
    var Textarea = document.getElementsByTagName("textarea");
    var ContatoreText = 0;
    var ContatoreTextArea = 0;
		    	
    for(i=0; i<Text.length; i++)
    {    
        if (Text[i].value == "")
        {
        	ContatoreText = ContatoreText+1;
        }
    }
      
    for(j=0; j<Textarea.length; j++)
    {
        if (Textarea[j].value == "")
        {
            ContatoreTextArea = ContatoreTextArea+1;
        }
    }
    
    if ((ContatoreText != 0) || (ContatoreTextArea != 0))
    {
   		alert("\n Hai lasciato qualche campo vuoto.");
   		return false;
    } 
}
e questo funziona egregiamente. Ora, volevo anche controllare la correttezza della email

codice:
function controlla_email()
{
   email = document.getElementById('email').value;
   if (!(email.match(/^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,4})+$/)))
   {
	alert("Inserisci una email valida");
	return false;
   }
}
entrambe le funzioni vengono richiamate sul submit del pulsante "INVIO".

Si può fare una cosa più elegante/funzionale? Magari tutto in una funzione...? Ora, non ricordo precisamente, ma qualcosa non funzionava in questi due controlli...