PHP-General - How the Heck is this not working?

Firstly, I'm not a new coder nor am I new to PhP, but for some reason my latest section of code seems to have some weird error that I can't work out.

I have value read in from a database of 3 values separated by colons - for example 5:3:0. The format is always the same, and it always goes from high to low, with the highest possible value being 8 and the lowest possible at 0.

From a db table, I pull a numeric value out of the 'slots' field and assign it to a variable name called $oa1 : . The field is a TINYINT type in the database

I then use the three values as thresholds to trigger a message on the screen. The code is below.



What this does, is to produce 1 of 4 results.
greater than first threshold - Available
Less than or equal to the first & greater than the second - Limited
Less than or equal to the second & greater than the third - xx slots remaining
equal to or less than the third - fully booked.

The problem is that no matter which order the case statements are in, setting 0 slots always marks it as available, that is : greater than the first threshold. In this case, it reads 0 slots free as being greater than 5.
Every other number falls neatly into the sections that it needs to, and if I set the last threshold and the free slots to 1, it manages to display the fully booked status.

I've added some error trapping to this (not shown here) and the values are being exploded correctly and it isn't using a default value - it really is assuming that 0 is greater than 5. Any ideas on what I've done wrong?

Hi,

in PHP, neither is 0 > 5 nor '0' > '5'. So obviously there's something else going on.

Make a var_dump() of each value and show the result here.

You should also consider normalizing your database and getting rid of this weird "a:b:c" stuff.

Here's the error-checked output and var dumps

================= Using PRINT command ================
Slots value: 0
First Threshold: 5
Second Threshold: 3
Third Threshold: 0
================== VAR_DUMP Values ===================
dump of slots int(0)
dump of thresholds array(3) { [0]=> string(1) "5" [1]=> string(1) "3" [2]=> string(1) "0" }
Result: Should say WAPW_OA_BOOKED_TEXT string(75) "WAPW_OA_AVAILABLE_TEXT"

First thing I noticed was that the thresholds are strings and the slots value is an integer. Wonder if it's worthwhile converting the thresholds to int values too?

As for the database data - I can't change that format as it has to interface with a different system in that format.

*lol*

You have put conditions instead of values into your switch statement. That's not how it works. The first condition expression evaluates to false, so you compare 0 with false. And since this is true, you end up with the first case being executed.

So you simply didn't use the switch correctly. To be honest, I didn't notice this at first either.

Just replace that stuff with an if statement, and you're done. You should generaly avoid using switch, because it's cumbersome, difficult to read and error-prone (thanks to the fall through mechanism).