pbannist Posted October 1, 2007 Share Posted October 1, 2007 I have the code below in default.php, why oh why dont the 'blanks' swap about when you visit default.php?page=services and then default.php?page=about if you take out setting up the trg variables in the default: case then it all works fine, any help appreciated <?php switch ($_REQUEST["page"]) { case "about": $trg1 = "blank3"; $trg2 = "blank1"; $trg3 = "blank2"; $txt = "about.php"; break; case "services": $trg1 = "blank1"; $trg2 = "blank3"; $trg3 = "blank2"; $txt = "services.php"; break; default: $txt = "blank.php";} $trg1 = "blank1"; $trg2 = "blank2"; $trg3 = "blank3"; ?> <?php echo($trg1); ?><br> <?php echo($trg2); ?><br> <?php echo($trg3); ?><br> <?php echo($txt); ?> Quote Link to comment https://forums.phpfreaks.com/topic/71335-switch-misbehaving-driving-me-round-the-bend/ Share on other sites More sharing options...
pbannist Posted October 1, 2007 Author Share Posted October 1, 2007 forget it! just found it, stray bracket at end of - $txt = "blank.php";} theres an hour I'll never get back Quote Link to comment https://forums.phpfreaks.com/topic/71335-switch-misbehaving-driving-me-round-the-bend/#findComment-358931 Share on other sites More sharing options...
wildteen88 Posted October 1, 2007 Share Posted October 1, 2007 forget it! just found it, stray bracket at end of - $txt = "blank.php";} theres an hour I'll never get back This is why it is recommended to always format your code example: switch($_REQUEST["page"]) { case "about": $trg1 = "blank3"; $trg2 = "blank1"; $trg3 = "blank2"; $txt = "about.php"; break; // ... } Quote Link to comment https://forums.phpfreaks.com/topic/71335-switch-misbehaving-driving-me-round-the-bend/#findComment-359305 Share on other sites More sharing options...
cooldude832 Posted October 1, 2007 Share Posted October 1, 2007 also don't use $_REQUEST as that represents a larger data pool than what you are looking at ($_GET) Quote Link to comment https://forums.phpfreaks.com/topic/71335-switch-misbehaving-driving-me-round-the-bend/#findComment-359314 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.