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); ?> 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 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; // ... } 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) 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
Archived
This topic is now archived and is closed to further replies.