syntax error: unexpected T_BOOLEAN_AND...

[jswebdev]

i'm receiving the error unexpected T_BOOLEAN_OR for the following line of code:

PHP: Select all

if (($_POST['title']) == '') || (($_POST['desc'])=='') || (($_POST['state'])=='') || (($_POST['date']) == '') { 

considering that so far i've made several fairly complex little apps using similar lines of code-- i simply can not see what is wrong here. does this mean that the error in this case must be conditional based on the context in which this line appears?

i thought it might have something to do w/ single vs double quoted strings, but this line produces the same error:

PHP: Select all

if (($_POST['title']) == "") || (($_POST['desc'])=="") || (($_POST['state'])=="") || (($_POST['date']) == "") { 

signed:
Stumped in the Valley

(PS: ya know, i avoided posting this question because i think it seems so "obvious"... ugh!)

[benbacardi]

try using this code instead:

PHP: Select all

if ((($_POST['title']) == '') || (($_POST['desc'])=='') || (($_POST['state'])=='') || (($_POST['date']) == '')) { 

you may need to encase the whole of the if statement conditions within one large ( ) which is what i've added above...

[rproctor]

Off the top of my head it looks to me like you missed a ) at the end
(($_POST['date']) == '') {
Should be
(($_POST['date']) == '')) {

[benbacardi]

he didnt miss one out at the end, because he didnt put one in at the beginning! you need to add one 2 the beginning AND the end, which is what i did in the code above...

[Pheonix]

Benbacardi's method would solve the issue, however to be much more easly read, I would write it like this

PHP: Select all

if ($_POST['title'] == '' || $_POST['desc']=='' || $_POST['state']=='' || $_POST['date'] == '') { 

This should have the exact same result, (I'm pretty sure it should, but once in a while I miss the obvious.

Further more, I would do this

PHP: Select all

if (empty($_POST['title']) || empty($_POST['desc']) || empty($_POST['state']) || empty($_POST['date']) ) {