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, Apr 30 '15 at 09:34
possible duplicate of [using AND/OR in if else PHP statement](http://stackoverflow.com/questions/4405795/using-and-or-in-if-else-php-statement). Also, is it really that difficult to google? Or even turn on error reporting? — Huey, Apr 30 '15 at 09: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` http://stackoverflow.com/questions/2803321/and-vs-as-operator — RiggsFolly, Apr 30 '15 at 09:43

Ayyanar G · Accepted Answer · 2015-04-30T12:00:51.160

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 '15 at 12:00

answered Apr 30 '15 at 09:34

Ayyanar G

1,545
1
11
24

1

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. – Rizier123 Apr 30 '15 at 09:36
Still showing as blank on this. what does || mean? Or? – Springheeled Jack Apr 30 '15 at 09:40
@SpringheeledJack `||` is the same operator as `or` just with a higher precedence – Rizier123 Apr 30 '15 at 09:41
1

@sgtBOSE I thought that they would recognize it, but apparently not :) – Rizier123 Apr 30 '15 at 09:42
@Rizier123 they are still not getting that. – Sougata Bose Apr 30 '15 at 09:43
I get this message if I dont check that box using this code. Notice: Undefined index: open_monday1 in /Applications/MAMP/htdocs/welcometowarwick/scripts/php/insert_restaurant.php on line 108 – Springheeled Jack Apr 30 '15 at 09:59
1

This code has still a syntax error in it! And it is accepted ?! – Rizier123 Apr 30 '15 at 10:07

score 0 · Answer 2 · answered Apr 30 '15 at 09:39

0

You can check it with -

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

answered Apr 30 '15 at 09:39

Sougata Bose

31,517
8
49
87

`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 – Rizier123 Apr 30 '15 at 09:45
@Rizier123 thats the main point. If OP want to check for the same then can use this. :) – Sougata Bose Apr 30 '15 at 09:50

Using "or" correctly in PHP with an if statement

2 Answers2