Using "or" correctly in PHP with an if statement

Question

I want to check if either of these checkboxes are checked and if either of them are i want the new assigned variable $open_monday to = "yes else "no".

   if (isset($_POST['open_monday_lunch'] or $_POST['opening_monday1'])) {

        $open_monday = "yes";

   }

   else { $open_monday = "no"; }

Is that the right way to do it? I have never used or before. I just get a blank pag when trying to run it as if the syntax is incorrect.

This is not how you use the or operator! do: isset($_POST['open_monday_lunch']) || isset($_POST['opening_monday1'])) You don't use it in the function call itself — Rizier123
– Rizier123, Commented Apr 30, 2015 at 9:34
possible duplicate of using AND/OR in if else PHP statement. Also, is it really that difficult to google? Or even turn on error reporting? — Huey
– Huey, Commented Apr 30, 2015 at 9:35
Your most obvious problem is that you have missed out some brackets. It should be if ( isset($_POST['open_monday_lunch']) or isset($_POST['opening_monday1']) ) { regardless of whether you use || or OR But you should also read this answer before moving from using '&&' and '||' to AND and OR stackoverflow.com/questions/2803321/and-vs-as-operator — RiggsFolly
– RiggsFolly, Commented Apr 30, 2015 at 9:43

Ayyanar G · Accepted Answer · 2015-04-30 12:00:51Z

2

Due to Operator precedence its always preferable to use && instead of and ,similarly || instead of or.please refer this link for additional info,

so try like this,

 if (isset($_POST['open_monday_lunch'] ) || isset( $_POST['opening_monday1'])) {

        $open_monday = "yes";

   }

   else { $open_monday = "no"; }

edited Apr 30, 2015 at 12:00

answered Apr 30, 2015 at 9:34

Ayyanar G

1,5451 gold badge11 silver badges24 bronze badges

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

7 Comments

Rizier123 Over a year ago

Why should the OP "try this"? Please add an explanation of what you did and why you did it that way not only for the OP but for future visitors to SO.

Springheeled Jack Over a year ago

Still showing as blank on this. what does || mean? Or?

Rizier123 Over a year ago

@SpringheeledJack || is the same operator as or just with a higher precedence

Rizier123 Over a year ago

@sgtBOSE I thought that they would recognize it, but apparently not :)

Rizier123 Over a year ago

This code has still a syntax error in it! And it is accepted ?!

|

Sougata Bose · Accepted Answer · 2015-04-30 09:39:03Z

0

You can check it with -

if (!empty($_POST['open_monday_lunch']) || !empty($_POST['opening_monday1'])) {

answered Apr 30, 2015 at 9:39

Sougata Bose

31.8k8 gold badges52 silver badges89 bronze badges

2 Comments

Rizier123 Over a year ago

empty() might not be necessary here or maybe even break the code if OP has a value attr. for the checkboxes with an empty value or false would it return false even if they are set

Sougata Bose Over a year ago

@Rizier123 thats the main point. If OP want to check for the same then can use this. :)

Collectives™ on Stack Overflow

Using "or" correctly in PHP with an if statement

2 Answers 2

7 Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

7 Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related