Switch Conditionals

redfire

Okay I'm currently working on a new template, and instead of making a hole new page for every thing I'm setting it up just it just changes the one area.

<?php switch($_GET['id']) 
{
default:
   include('home.php');
  break;  

case 'news':
   include('news.php');
  break;  

case 'forums':
   include('forums.php');
  break;  

}
?>

and I have the links set-up like

<a href="index.php?id=news">News</a>[quote][/quote]

I have it set-up like that but it doesn't want to work 🙁

NogDog

Put the default case after all the others. (It will always execute, so you never get to any of the other cases.)

redfire

😃 thanks.

Weedpacket

No, the default case will only execute if none of the others match, irrelevant of where it appears.

$a = 2;

switch($a)
{
	default:
		echo "Whatever";
	break;
	case 1:
		echo "One";
	break;
	case 2:
		echo "Two";
	break;
}

Two

NogDog

Weedpacket wrote:
No, the default case will only execute if none of the others match, irrelevant of where it appears.
$a = 2;

switch($a)
{
	default:
		echo "Whatever";
	break;
	case 1:
		echo "One";
	break;
	case 2:
		echo "Two";
	break;
}
Two

Interesting...the switch manual page at php.net says

A special case is the default case. This case matches anything that wasn't matched by the other cases, and should be the last case statement.

But I guess it doesn't say it must be the last case statement. 🙂

ClarkF1

I only read that after I'd coded my website........nothing had gone wrong so I left it with the default first.

It's only first because it's the first thing I coded.

Weedpacket

Yeah..... I noticed that clause too, and don't really see much justification. Certainly there should be some logical ordering to the cases, and that would usually see the default case being handled last, but there are plenty of cases where the default could come sooner:

switch($foo)
{
case 1:
case 2:
case 3:
case 4:
default:
       // Here be code. Cases 1..4 handled explicitly because they are
       // by far the most common. It would be silly to just leave them
       // to be handled by the default.
case 5:
       // code
case 6:
       // code
//...
case 40:
       // code
       // Since otherwise there'd be ~40 tests needed in almost every case.
}