Multiple conditions using an AND if Statement, javascript

Question

Can anyone tell me what's wrong with this if statement?

If I use either of the two main conditions on their own the statement works fine but when I add that middle && statement, it stops working. I've searched online and can't see what's wrong.

P.S. I can even change that middle && to a || statement and it works as well. I'm so confused.

if ((containerId.id == "LineOne" && dropLoc == "dropLocation1.1") && (containerId.id == "LineTwo" && dropLoc == "dropLocation2.2"))
        {
            alert("finished");
            cdpause();      
        }

AND means AND. containerId.id cannot be "LineOne" and "LineTwo" at the same time. — Joel
– Joel, Commented Oct 31, 2017 at 10:14
I fifgured it out... My main issue is i'm an idiot... but besides that I have the same variables containerId, and dropLoc being used twice... I'm trying to use an array now... thanks — TheWee Seal
– TheWee Seal, Commented Oct 31, 2017 at 10:49

Community · Accepted Answer · 2020-06-20 09:12:55Z

4

I've searched online and can't see what's wrong.

I can even change that middle && to a || statement and it works as well

Because containerId.id can't be LineOne and LineTwo at the same time.

Similarly, dropLoc can't have two values at the same time.

But it can have one of the two values, so replace && with ||.

   if ((containerId.id == "LineOne" && dropLoc == "dropLocation1.1") || 
       (containerId.id == "LineTwo" && dropLoc == "dropLocation2.2"))
   {
        alert("finished");
        cdpause();      
   }

edited Jun 20, 2020 at 9:12

CommunityBot

11 silver badge

answered Oct 31, 2017 at 10:13

gurvinder372

68.6k11 gold badges78 silver badges98 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Nina Scholz · Accepted Answer · 2017-10-31 10:12:25Z

0

You could combinte the two checks with an OR, because, you can not have two different values at the same time.

Beside that, you need no brackets, because of the operator precedence of logical AND && over logical OR ||.

if (
    containerId.id == "LineOne" && dropLoc == "dropLocation1.1" ||
    containerId.id == "LineTwo" && dropLoc == "dropLocation2.2"
) {
    alert("finished");
    cdpause();      
}

answered Oct 31, 2017 at 10:12

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

Comments

user7838288 · Accepted Answer · 2017-10-31 10:13:01Z

0

Like Satpal said

 if(containerId.id == "LineOne" && dropLoc == "dropLocation1.1") || (containerId.id == "LineTwo" && dropLoc == "dropLocation2.2")

answered Oct 31, 2017 at 10:13

user7838288

Comments

Patrick Hund · Accepted Answer · 2017-10-31 10:16:51Z

0

You already had the correct solution, you should have a || (or) instead of a && (and) in the middle.

It's basic boolean logic: you have two expressions with "and", and you want to execute your code if either of those expressions are true, so you join those expressions with "or".

"If the name is John and it's Monday OR if the name is Jane and it's Tuesday , then remind them to shop for groceries." => on Monday, it's John's turn to go shopping, on Tuesday, it is Jane's.

answered Oct 31, 2017 at 10:16

Patrick Hund

20.4k12 gold badges71 silver badges97 bronze badges

Collectives™ on Stack Overflow

Multiple conditions using an AND if Statement, javascript

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related