What is wrong with my PHP?

[kau]

if ($bbuserinfo[usergroupid != 5 OR $bbuserinfo[usergroupid != 6)

This IF statement is not working. What is wrong?

[EvilLS1]

You forgot the last bracket on $bbuserinfo[usergroupid].

[Jolten]

You might also want to encompass each side of the expression in a container.

if (($bbuserinfo[usergroupid] != 5) OR ($bbuserinfo[usergroupid] != 6))

[Dean C]

Moved to the correct forum

[Xenon]

also it's easier to use the in_array for such things:

PHP Code:

if (!in_array($bbuserinfo['usergroupid'], array(5,6))) 


btw the statement above is wrong on several places, because it would always be true
i assume you need an and

[clamcrusher]

and also, i beleive you should be using || instead of OR

AND and OR are bitwise operators. If you dont know what those are, you should probably avoid them altogether

use && instead of AND
use || instead of OR

[Zachery]

Originally Posted by clamcrusher

and also, i beleive you should be using || instead of OR

AND and OR are bitwise operators. If you dont know what those are, you should probably avoid them altogether

use && instead of AND
use || instead of OR

|| and OR are the same thing.

Somthing about the boolean logic is bad.

[Brad]

Originally Posted by clamcrusher

and also, i beleive you should be using || instead of OR

AND and OR are bitwise operators. If you dont know what those are, you should probably avoid them altogether

use && instead of AND
use || instead of OR

http://www.vbulletin.com/docs/html/m...andards_and_or

from: http://www.vbulletin.com/docs/html/codestandards

[Tekton]

Originally Posted by Brad.loo

http://www.vbulletin.com/docs/html/m...andards_and_or

from: http://www.vbulletin.com/docs/html/codestandards

Meh, I think I'll continue using && and || ^__^;;

[Xenon]

bit wise operators are & and |
&& and || as well as AND and OR are boolean operators in PHP

using AND/OR in conditions and &/| for bitwise ops makes the code more readable, that's why i defined AND/OR as macros in my C++ as well