[aswettla]

Hi,

I'm using the following code to produce a index.php?p=contact style thingy. Here is my code, then I try to run index.php?p=contact i get the following error.

Quote

Parse error: parse error, unexpected T_VARIABLE, expecting '(' in c:\program files\easyphp1-7\www\czeg\index.php on line 2

my code:

<?php
switch $_GET['p']
{
    case "login":
        include('login.php'); 
        break;
    case "news":
        include('news.php'); 
        break;
    case "pilots":
        include('pilots.php'); 
        break;
    default:
        include('main.php'); 
}
?>

Could somebody please help!

This post has been edited by aswettla: 14 April 2005 - 04:08 PM

[supersloth]

wouldnt you be better served just doing include('$_GET['p'].php'); ???

[aswettla]

Super, now I fixed it up using your suggestion and I get this error.

Parse error: parse error, unexpected T_CASE in c:\program files\easyphp1-7\www\czeg\index.php on line 4

[supersloth]

i dunno i probably didn't use the correst syntax for the get, i usually just use $p, maybe someone else can help with the correct syntax since i'm at work and don't have access to a php enabled box.

[Amadeus]

Try

include($_GET['p'].".php");

[snoj]

Just fixing your original code.

<?php
//You need teh parentheses '(' and ')'.
switch($_GET['p']) 
{
   case "login":
       include('login.php');
       break;
   case "news":
       include('news.php');
       break;
   case "pilots":
       include('pilots.php');
       break;
   default:
       include('main.php');
}
?>

Also note that while Sloth's and Amadeus' code is simpler (and far easier to do) they also open up scripts that you may not want people to see/use. But that's just what I think. And it depends on alot of things.


Oh and before I click 'Add Reply'.

<?php
//If you'd like to check if $_GET['p'] is valid.
$page = ($_GET['p'] == ('login'||'news'||'pilots')) : $_GET['p'] ? 'main';
include($page.'.php');
?>

This post has been edited by hotsnoj: 14 April 2005 - 06:22 PM