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