controllo dati form

[passasooz]

buongiorno...

ho un problema alquanto "stupido" ma non capisco come mai mi salta la lettura di una funzione che ho creato...

compilo i dati di un form premo il submit per inviarli, e invece di controllare se i dati sono compilati o meno, lo script salta questo controllo e invia ugualmente i dati... come mai?

il codice è il seguente

funzione controlla

<script>
Modernizr.load({
    test: Modernizr.mq('only all'),
    nope: 'js/vendor/respond.min.js'
});
$(document).ready(function(){
$('a[href*=#]').click(function() {
    if (location.pathname.replace(/^\//,'') == this.pathname.replace(/^\//,'') && location.hostname == this.hostname) {
        var $target = $(this.hash);
        $target = $target.length && $target || $('[name=' + this.hash.slice(1) +']');
        if ($target.length) {
            var targetOffset = $target.offset().top;  
            $('html,body').animate({scrollTop: targetOffset}, 1000); 
            return false;
        }
    }
});
$('.contattami a img').css('border','0px');
});
function controlla(){
    if((document.getElementById(mail)=="") || (document.getElementById(subj)=="") || (document.getElementById(messaggio)=="")){
        alert("prova");
return false;
    }
}
</script>

ho incollato tutto lo <script> magari sbaglio qualcosa nell'inserimento che non mi accorto...

il form invece è questo

<form method="post" onsubmit="return controlla()" action="sendmail.php">
<input type="text" name="email" id="mail" placeholder="Inserisci inidirizzo e-mail" />
<input type="text" name="subj" id="subj" placeholder="Inserisci un oggetto" />
<textarea name="messaggio" id="messaggio" placeholder="Inserisci testo del messaggio"></textarea>

diciamo che il comando onsubmit viene saltato proprio.. passa subito all'action

[cavicchiandrea]

document.getElementById(mail)=="" mancao i value document.getElementById(mail).value==""

[passasooz]

giusto non ci avevo fatto caso nella fretta, comunque non è solo quello il problema visto che ancora quell'if sembra che non venga visto...

[cavicchiandrea]

 function controlla(){ 
    if((document.getElementById('mail').value=="") || (document.getElementById('subj').value=="") || (document.getElementById('messaggio').value=="")){          alert("prova");  return false; 
    }  }

cosi deve andare

[passasooz]

gli apici vero!!! assurdo .... vabhe... grazie, senti ma... il getElementByName si può fare avevo letto... no???

[cavicchiandrea]

gli apici vero!!! assurdo .... vabhe... grazie, senti ma... il getElementByName si può fare avevo letto... no???

Non ho capito la domanda riproponi con meno puntini...

[passasooz]

esiste un metodo che si chiama getElementByName ? anzichè utilizzare il ById?

[cavicchiandrea]

http://www.w3schools.com/jsref/met_d...entsbyname.asp