Multiple If conditions in Javascript

Question

I Have javascript function who check input value and if requiment inputs ar note lenght 0 sent ajax post.

    function sendEmail(){
            var nn = jQuery("#your-name").val().length;
            var ee = jQuery("#friend-email").val().length;

            // var n = document.getElementById("your-name").value;
            // var e = document.getElementById("friend-email").value;

            if ( nn == 0 && ee == 0 ) {

                jQuery("input#your-name").css("border", "1px red solid");
                jQuery("input#friend-email").css("border", "1px red solid");

            } 
            else if (ee == 0) {

                jQuery("input#friend-email").css("border", "1px red solid");

            }

            else if ( nn == 0 ) {

                jQuery("input#friend-name").css("border", "1px red solid");

            }
            else {

                ajax();
            }
}

Everything works well, exept: If email is set and name is is not..Nothing happens (jQuery("input#friend-name").css("border", "1px red solid");)

Whats wrong on this? See full example in jsfidle: http://jsfiddle.net/4X2J8/

Why `#friend-name` in the last `else if` when you're using `#your-name` at the top and in the first `if` condition? — Mat, May 18 '14 at 06:48
Your fiddle example is pointless, because you did not even embed jQuery there … — CBroe, May 18 '14 at 06:50
@Mat Because i need check if both: email and name input are set — Kārlis Millers, May 18 '14 at 06:59
@KārlisMillers: you're checking the length of one thing, and setting the color of some other thing. — Mat, May 18 '14 at 06:59

score 4 · Accepted Answer · answered May 18 '14 at 06:58

You should correct name field's id: "friend-name" to "your-name".

  function sendEmail(){
        var nn = jQuery("#your-name").val().length;
        var ee = jQuery("#friend-email").val().length;
        // var n = document.getElementById("your-name").value;
        // var e = document.getElementById("friend-email").value;
        if ( nn == 0 && ee == 0 ) {
            jQuery("input#your-name").css("border", "1px red solid");
            jQuery("input#friend-email").css("border", "1px red solid");
        } 
        else if (ee == 0) {
            jQuery("input#friend-email").css("border", "1px red solid");
        }
        else if ( nn == 0 ) {
            jQuery("input#your-name").css("border", "1px red solid");
        }
        else {
            ajax();
        }
    }

Multiple If conditions in Javascript

1 Answers1