PHP's starting to piss me off!

[jacrewq]

This is part of my code:

if ($group == 0) $cfg->errmsg ("Please select a group.$group");

Notice that there should be an error message if $group is zero. And then in the error message the value of $group is displayed.

So the script outputs this: Please select a group.Ontario

That means that $group is Ontarion and ZERO in the SAME TIME!!!!

WHT??

[Mordecai]

If you're checking if a string is a number, most of the time, it returns -1, it might just be zero in your case. Don't try to check a string against a number, unless you convert it some how.

[epimeth]

try === instead of ==

[planetsim]

What he had was suffient. He Doesnt need the === as that will make sure its identical which is not required.

$group should actually be the Goup names id instead of the Group itself.

[jacrewq]

what do you mean by

$group should actually be the Goup names id instead of the Group itself.

[Mordecai]

He means you shouldn't use the name for the ID. I.e. this example:
Person id #1 = Mordecai
Person id #2 = George Bush
Person id #3 = Pacman
if ((person id #3) == "Pacman") { }
this equals: if (3 == "Pacman"). Obviously, 3 does not equal Pacman. Therefore, you must use either:
if ("Pacman" == "Pacman")
or
if ((person id #3) == (person id #3))

Hope this helps, despite being a very confusing example that means almost nothing.

In simple terms: stick with either an ID or a name, don't mix and match, since they're obviously not the same when measuring equality.

[Bunkermaster]

Originally posted by Mordecai
Hope this helps, despite being a very confusing example that means almost nothing.

O_o


w00t??

[Sxooter]

Note that if you're check to see if a $var got set, then use isset:

if (!isset($group)) die("Group var ain't set pa!");

If it's set to nothing, then test for that:

if ($group=="") die("witty error message goes here");