Include files with $_GET switch statement? - php

Question

guys! I am a newbie. I think about it all day but still not much..
I have one index.php file, one switch statement:
$current_page = $_GET['page'];
switch ($current_page) {
case ('homepage'):
include 'contents/homepage.php';
break;
case ('about'):
include 'contents/about.php';
break;
case ('contacts'):
include 'contents/contacts.php';
break;
default:
include 'contents/homepage.php';
}
I include a file if the get attribute is some of the names.
And if the URL is for example: myproject/index.php?page=about
The proper content file is included.
But when the URL is just: myproject/index.php
No file is included.
I was thinkig to add something like:
if (!isset($_GET['page'])) {
header('Location: index.php?page=homepage');
}
But it is kind of ugly I think and the URI cannot be nver just index.php.. And what if the parameter is not "page"... It is a problem again.
Do you have in mind some simple understandable solution for this. How to include the proper contents according to the different pages?
Thank you very much!

Answer 1

First, when checking if an array's key is set, use array_key_exists instead. It is much faster for the parser.
Second, you shouldn't redirect the page. Instead, use a default value, like so:
// Set default value
$current_page = 'homepage';
// Change value if `page` is specified
if(array_key_exists('page',$_GET) {
$current_page = $_GET['page'];
}
// Check page
switch ($current_page) {
case 'about':
include 'contents/about.php';
break;
case 'contacts':
include 'contents/contacts.php';
break;
case 'homepage':
default:
include 'contents/homepage.php';
}
Additionally, as I have done above, you don't need to specify the include for homepage twice. Without a break statement, the parser will go into the next case statement. If you want to treat multiple values the same, you can simply specify them subsequently.
I hope this helps!

Answer 2

$current_page = isset($_GET['page'])?$_GET['page']:'homepage';

Answer 3

This should do it
$current_page = isset($_GET['page']) ? $_GET['page'] : null;
switch ($current_page) {
case ('homepage'):
include 'contents/homepage.php';
break;
case ('about'):
include 'contents/about.php';
break;
case ('contacts'):
include 'contents/contacts.php';
break;
default:
include 'contents/homepage.php';
}