index.php?page=10

[nets]

You're better off having an array with the page names as keys, with their corresponding value as the location. Also in the interest of security, unless you need to use PHP within the "included" file, don't include the page. By including the page you're increasing the chance of having malicious PHP evaluated.

<?php

  $page[''] = 'pages/home.htm';
  $page['news'] = 'pages/news.htm';
  $page['info'] = 'pages/info.htm';

  if(file_exists($_tmp=$page[$_GET['page']]))
    print file_get_contents($_tmp);
  else
    print "<strong>The page you requested could not be found.</strong>\r\n";

?>

Use functions:

<?
function page($url)
{
include <<<EOF
{$url}.php
EOF;
}
?>

Then:

<?
include('func.php');
$url = $_GET['page'];
page($url);
?>

That is a severe risk to your site's security, since anyone could obtain secret information hidden on your server or (if "allow_url_fopen" is enabled in php.ini) even include a malicious PHP page located on their own server. Furthermore, it was pointless splitting it into a function and using heredoc syntax.

[ItchysPaws]

with the switch code, your including files, its best having it as functions with the processed html/php in that.

[Josh-H]

Some nice lil codes/tuts forming in this thread!

[ScottDiamond]

Would I put this in my iFrame or remove the iFrame and replace it with that? And renaim it to home.php ?

[nets]

Would I put this in my iFrame or remove the iFrame and replace it with that? And renaim it to home.php ?

You'd replace the Iframe with the PHP code, you shouldn't need to rename anything (as it will be using the files which were being viewed through your Iframe). Be careful about whose script you choose, as some of the scripts posted here aren't secure.