Or operator in if statement

Question

Is there an 'or' operator for an if statement? I want to change the value of a text field depending on the content of another. If the value is "Your Company" OR (in german) "Ihr Unternehmen", it's supposed to leave the #cmp field blank. Otherwise it should take over the value. I can't get it working with an or operator (second code block), nor with an else if statement.

// Working for one string:
if ($("#listencmp").val() != "Ihr Unternehmen") {
     var comp = $("#listencmp").val();
     $("#cmp").val(comp);
 } else {
    $("#cmp").val("");
 }

// Not working for two strings:
if ($("#listencmp").val() != "Ihr Unternehmen" || "Your Company") {
     var comp = $("#listencmp").val();
     $("#cmp").val(comp);
 } else {
    $("#cmp").val("");
 }

score 2 · Answer 1 · answered Oct 20 '13 at 14:56

2

Think about the or operator. You are OR'ing a Boolean expression and a String. You need to build a second Boolean expression.

if ($("#listencmp").val() != "Ihr Unternehmen" || $("#listencmp").val() == "Your Company")

answered Oct 20 '13 at 14:56

struthersneil

2,700
10
11

Amine Hajyoussef · Accepted Answer · 2013-10-20T15:07:46.637

1

var listencmp = $("#listencmp").val();
if ( listencmp == "Ihr Unternehmen" || listencmp  == "Your Company") {
 $("#cmp").val("");
}
else{
 var comp = $("#listencmp").val();
 $("#cmp").val(comp);
}

side note: it's better to use strict equality (replace == with === )

edited Oct 20 '13 at 15:07

answered Oct 20 '13 at 14:54

Amine Hajyoussef

4,381
3
22
26

score 0 · Answer 3 · answered Oct 20 '13 at 14:55

Use this:

if ($("#listencmp").val() != "Ihr Unternehmen" || 
     $('#listencmp').val() != "Your Company") {
   // do the stuff here, this will work fine!
}

You need to complete the conditions then move on the next one. And the next ones needs to have two conditions to compare on.

score 0 · Answer 4 · answered Oct 20 '13 at 14:58

You have to use the compare statement to the second condition too:

if ($("#listencmp").val() !== "Ihr Unternehmen" || $("#listencmp").val() === "Your Company") {
  // your code here
}

Probably would be a more recommended to use strict equality operator (===).

score 0 · Answer 5 · answered Oct 20 '13 at 15:01

0

You should be using the OR operator to check if your val() === both values. If you stick to exclusion, go for the && operator instead.

answered Oct 20 '13 at 15:01

Sylvain

158
1
9

itsmikem · Answer 6 · 2013-10-20T15:37:09.037

0

I would like to add that this can be a good use of the ternary operator, achieved in one line of code.

HTML:

<input type="text" id="cmp"/>
<input type="text" id="listencmp" value="Your Company"/>

Javascript:

$("#cmp").val(($("#listencmp").val() === "Your Company" || $("#listencmp").val() === "Ihr Unternehmen")?"":$("#listencmp").val());

edited Oct 20 '13 at 15:37

answered Oct 20 '13 at 15:09

itsmikem

2,118
2
26
31

Or operator in if statement

6 Answers6