Valid php syntax

[nate2099]

Hi, Is this valid syntax?? If I set $postcode to 1111 it still comes back false..

Code: Select all

if ((($postcode < 2000) && ($postcode > 2599)) || (($postcode < 2620) && ($postcode > 2898)))
                { $error =  "ERROR: Invalid postcode for NSW"; }

Thanks,

Nathan.

[grahame]

It's valid syntax, but incorrect logic. You can't have a number that's both less than 2000 and greater than 2599, so the first half will never be true. You can't have a number less than 2620 and greater than 2898, so the second half will never be true either.

I think you mean

if ($postcode < 2000 || ($postcode > 2599 && $postcode < 2620) || $postcode < 2898) { $error = "Out of state"; }

[nate2099]

oops... of course... I feel silly now!!!
Thanks!!