[RESOLVED] newbie needs help with include

[rebz01]

ok when yall finish laughing at me can you help with what may be a simple solution

i have made a lil website for my gaming clan (its weak and in so many ways probably coded wrong)

but anyway here goes

link >> http://bf.gravediggaz.org

my question >>
i have index.php home.php links.php etc...
how do i get home.php to load in index.php by default as it stands now to view the home page you need to go to http://bf.gravediggaz.org/index.php?site=home

<?php
ob_start();
if($_GET['site'] == 'home')
{
include ("home.php"); 
}
?> <?php
ob_start();
if($_GET['site'] == 'videos')
{
include ("videos.php"); 
}
?> <?php
ob_start();
if($_GET['site'] == 'images')
{
include ("images.php"); 
}
?> <?php
ob_start();
if($_GET['site'] == 'links')
{
include ("links.php"); 
}
?>

any help will be much appreciated 🙂

ty in advance

rebz

[rincewind456]

Like this is one way

<?php
ob_start();

switch ($_GET['site']) {
	case 'videos':
		include ("videos.php");
		break;
	case 'images' :
		include ("images.php");
		break;
	case 'links':
		include ("links.php");
		break;

default:
	include ("home.php");
	break;
}
?>

ans yes I know I haven't included a case for home, beacuse there seems little point checking for it if it will default to it anyway. However if you were to change your default you would then need to add a case for home.

[rebz01]

thats done what i needed it to do
thank you so much 🙂

told you i was a noob 😉

[rebz01]

a quick question tho

i have Wamp2 and if i go to local host it gives me the following
Notice: Undefined index: home in C:\wamp\www\index.php on line 80
then loads the page underneath

but

on http://bf.gravediggaz.org/index.php it works fine

[Kudose]

The two servers probably have different [man]error_reporting[/man] levels.

[bradgrafelman]

And to elaborate...

You should never reference an array item which might not exist - especially with incoming data. Here's how I often handle such situations in a script like yours:

$site = (isset($_GET['site']) ? $_GET['site'] : '');
// Now replace every reference to $_GET['site'] with $site

This uses the ternary operator, which is basically a shorthand way of doing:

if(isset($_GET['site'])
{
    $site = $_GET['site'];
}
else
{
    $site = '';
}

[man]isset/man or [man]empty/man are two common ways to check whether a variable exists before using it.