Question

I am trying to validate email using javascript but for some reason it is not working.

PROBLEM:When I enter VALID email address it still alerts that that my address is faulty…
What I am missing here?
Here’s the code:

    $(document).ready(function(){

    $('.info').hide();
    $('#button').click(function(){

    var name = encodeURIComponent($('#name').val());
    var email = encodeURIComponent($('#email').val());
    var message = encodeURIComponent($('#message').val());
    var reg = /^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/;



    //if name is empty
    if($('#name').val() == 0){

    $('.info').html('<p style="font-weight:bold;text-align:center;font-size:20px;"> Molimo unesite Vaše ime!</p>').fadeIn(1000);
    }
    //if name is less than three characters
    else if(name.length < 3){
    $('.info').html('<p style="font-weight:bold;text-align:center;font-size:20px;"> Molimo unesite ime duže od tri slova!</p>').fadeIn(1000);

    }
     // if email is empty
     if ($('#email').val() == 0){
    $('.info').html('<p style="font-weight:bold;text-align:center;font-size:20px;"> Molimo unesite Email!</p>').fadeIn(1000);

    }
    // if email AND name are empty
    if($('#email').val() == 0 && $('#name').val() == 0 ){
    $('.info').html('<p style="font-weight:bold;text-align:center;font-size:20px;"> Molimo unesite Email!<br/>Molimo unesite ime!</p>').fadeIn(1000);

    }
    //if mail adress is NOT valid
    if(reg.test(email)==false){
    //alert('please enter valid mail!');
    $('.info').html('<p style="font-weight:bold;text-align:center;font-size:20px;">Molimo unesite ispravan Mail!</p>').fadeIn(1000);
    }

            //if mail adress IS VALID (this is not working...)
    if($('#name')!= '' && reg.test(email) !=false ){

            //alert(data);
    var data = 'name=' + name + '&email=' + email + '&message=' + message;

           $.ajax({
           type:"POST",
           url:'kontakt2.php',
           cache:'false',//IE FIX
           data: data,

           success: function(){
           //alert('THX for your mail!');
           } //end sucess 
           });
           } //end if statment for VALID mail
           //return false prevent Redirection
           return false;
           });


           });

Answer 1

the line

if($('#name')!= '' && reg.test(email) !=false )

should probably be

if($('#name').val()!= '' && reg.test(email) !=false )

(or just if (name != '' && reg.test(email) != false ) )