Need email validation without required

Question

I have used following code for email validation. It is running fine but i don't want to validate it when field will be empty.

    var filter = /^([\w-\.]+)@((\[[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.)|(([\w-]+\.)+))([a-zA-Z]{2,4}|[0-9]{1,3})(\]?)$/;
    var sEmail = $(this).val();

    if (!filter.test(sEmail)) {
        value = false;
        $(this).css('border', '1px solid red');
        $(this).attr('error','true');
    }else{
        $(this).css('border', '');
        $(this).removeAttr('error');
    }

trim `$(this).val()` and check its not empty prior to validating then? — SubjectCurio, Sep 29 '14 at 13:17
@MLeFevre Just by testing the boolean against `.val()` will give you the result. http://jsfiddle.net/MelanciaUK/jumkynzd/ — emerson.marini, Sep 29 '14 at 13:33
@MLeFevre Yes, I just got caught on this as mentioned in the answer given by T.J. Crowder. — emerson.marini, Sep 29 '14 at 13:48
@MelanciaUK Yep no worries, I replied to you before I saw below :) — SubjectCurio, Sep 29 '14 at 13:49

score 1 · Accepted Answer · answered Sep 29 '14 at 13:18

1

See comments:

var sEmail = $.trim($(this).val());
//           ^^^^^^^-------------^---- Trim whitespace

if (sEmail && !filter.test(sEmail))
//  ^^^^^^^^^^--- only validate if there's something in it

answered Sep 29 '14 at 13:18

T.J. Crowder

1,031,962
187
1,923
1,875

I don't think you need to trim the value to check whether it's empty or not. `.val()` when tested as boolean will return false if it's empty. http://jsfiddle.net/MelanciaUK/jumkynzd/ – emerson.marini Sep 29 '14 at 13:37
@MelanciaUK: If it's really empty, yes. But if the user types in `" "`, no. – T.J. Crowder Sep 29 '14 at 13:39
I see your point. :) – emerson.marini Sep 29 '14 at 13:39
Thanks T.J. It is working fine. – Terry Sep 29 '14 at 13:48

Need email validation without required

1 Answers1