jQuery multi part if statement not working

Question

I have a jQuery Mobile form I am working on where when the user tries to submit the form I would like to check the settings of 2 inputs. The first input is a jquery mobile flipswitch and the second is a checkbox.

If my flipswitch value is 1 and my checkbox is checked I would like to throw and error. The jQuery code below is not working for me. Pretty sure I'm on the right track but I don't know what I am doing wrong here. Anyone have any ideas?

my HTML

<select name="flip-min" id="flip-min" data-role="slider">
    <option id="filled" value="1">Filled</option>
    <option id="notFilled" value="0">Un-Filled</option>
</select>

<input type="checkbox" name="unable" id="unable"/>
<label for="unable">label text</label>

<input type="button" name="submit" value="submit">

my jQuery

$("input[name=submit]").on("click", function(){
    if($('#unable').is(':checked') && $('#flip-min').val() == 1){
        alert "error message";
        exit;
    }
});

Your [JS Console](http://webmasters.stackexchange.com/questions/8525/how-to-open-the-javascript-console-in-different-browsers) should tell you about a syntax error on that `alert` line. — Paul Roub, Oct 22 '14 at 20:34

score 1 · Accepted Answer · answered Oct 22 '14 at 20:38

alright, there appears to be a few things wrong with your code. First, there is no parenthesis around your alert function. This will definitely cause an error. Secondly, you have "exit;" at the end of the if statement when in reality, you want to use "return;". Here is what your code should look like:

$("input[name=submit]").on("click", function(){
    if($('#unable').is(':checked') && $('#flip-min').val() == 1){
        alert("error message");
        return;
    }
});

score 0 · Answer 2 · answered Oct 22 '14 at 20:38

0

'alert' is a function, so you have to call it setting the arguments within parentheses as follows.

alert("error message");

and instead exit you have to put 'return;'

answered Oct 22 '14 at 20:38

Diego Montoya

119
3

score 0 · Answer 3 · answered Oct 22 '14 at 21:06

@Austin, your code is correct But the alert syntax is incorrect!!!

Correct syntax

alert ("I have an error...!");

Also you can use console.log + development tools of your browser (F12) to monitor your messages and objects values.

Correct code + console.log statement

$("input[name=submit]").on("click", function(){
    if($('#unable').is(':checked') && $('#flip-min').val() == 1){
        console.log('I have an error...!');
        alert ("I have an error...!");
    }
});

Result enter image description here

I used Firefox + Firebug in my answer ;)

jQuery multi part if statement not working

3 Answers3