Hi,

> Good coder will understand this at a glimpse:

[snip horrible code]

> if($pos_params!=false)

Assuming it's a boolean, $pos_params is *already* true or false, so testing
it like this is pretty much pointless. It makes more sense (and is much more
readable IMHO) to do something like this:

if ($pos_params) {
  // something
}

I'm not sure how the PHP interpreter works internally but that might even
save your script the overhead of an unnecessary comparison ;-)

Cheers
Jon

-- 
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to