Lesson 7's Try It

shahz · June 4th, 2015, 04:28 AM

Hey guys,

In the Lesson 7's Try It activity, towards the end of it, there is a switch statement:

Code:

switch (isset($_GET['content'])) :
	case 'gents' :
	case 'sporting' :
	case 'women' :
		include 'content/catnav.php';
	endswitch;

The code looks perfectly normal, but at this stage, the sidebar is showing on the "about" page too, even when we did not add it to the switch statement. Am I missing something?

I did resort to using if-else statements that work flawlessly though.

nbnyc · August 23rd, 2015, 09:20 AM

hi, shahz -

just came here to ask about this code as well.

i think what's happening is that this line:

Code:

switch (isset($_GET['content']))

is evaluating to "true" (because isset() returns a boolean)... and then perhaps after that _any_ non-null value of $_GET['content'] will be cast to "true" and trigger the case.

seems to me that the switch condition should instead be

Code:

switch ($_GET['content'])  // because we want to switch based on the content!

however this results in errors where "content" is not set, so it all needs to be wrapped in an "if" statement that evaluates isset().

(SPOILER ALERT):

Code:

<?php
		if (isset($_GET['content'])):
				switch ($_GET['content']):
					case 'gents':
					case 'sporting':
					case 'women':
						include 'content/catnav.php';
				endswitch;
		endif;
?>

works as expected:

- subnav links present on category pages for Gents, Sporting, Women
- subnav links not present on home, about, categories pages