switch problem

[MrAdam]

hey everyone. iv got a problem with my switch statement...

switch ($f) {case "add":	addItem();	break;case "remove":	removeItem();	break;case "clear":	clearItems();	break;}

.. i've never tried doing something like that before... but it doesn't seem to work (but gives no errors) .. can anybody see a problem?-Thanks

[aspnetguy]

I don't see anything wrong with it but try this. I added a default case to see echo a string when $f doesn't match any of the other cases (which sounds like what is happening)

switch ($f) {case "add":	addItem();	break;case "remove":	removeItem();	break;case "clear":	clearItems();	break;default :	echo "value fell through";	break;}

[MrAdam]

giving out "value fell through" :S

[aspnetguy]

so try trimming the whitespace from $f and converting it to lower case just to be sure. It is usually some little thing like that.

[MrAdam]

al give it a shot.nope, it's nothing to do with whitespace, wrong case or owt.check this link out..http://www.brothers-united.newbiestyle.co....tions.php?f=addcan see there..

[aspnetguy]

how are you setting $f? $f = $_GET['f']; //?????

[MrAdam]

yeah ..

[aspnetguy]

just for kicks try this

switch ($_GET['f']) {case "add":	addItem();	break;case "remove":	removeItem();	break;case "clear":	clearItems();	break;default :	echo "value fell through";	break;}

it might be an issue with the variable and the querystring being the same name?!?

[MrAdam]

sorted! worked when i changed the variable to "function". i don't know why though lol. but am happy. thankya mate!

[aspnetguy]

sorted! worked when i changed the variable to "function". i don't know why though lol. but am happy. thankya mate!

I guess PHP doesn't like setting variables that have the same name as the querystring.

[aquatsr]

I guess PHP doesn't like setting variables that have the same name as the querystring.

Hmmm no... it's not that. I don't think the problem has been solved completely .

[MoZo1]

You should copy your entire code here, cause the switch is good, and with the $_GET array, it should run on every PHP versions.Oh, and you can try it with else if, I doubt it will work.

[aspnetguy]

It was obviously a problem with $f and the way it was being defined. changing f= to function= in the quesrtstring seemed to solve the problem...why would this be?

[justsomeguy]

Hard to say without the code. It might be that $f got overwritten somewhere.