Stevie -- You've already been beaten up about posting tons of code, so I won't add to that (much, anyway; DON'T, and ABSOLUTELY DON'T 'TOFU' POST!). And you've heard that you needed the ; on the last line and to correct your occasional $includes problems, so that's done.
Yes, you should work on reducing the problem to its minimum cause. While
a default case is not required for a switch statement, you could have the
same problem by replacing all of those cases with a default -- and then
you not only might find the answer yourself but you also wouldn't take so
much heat for posting so much junk :-)
You should also read the manual to see how to use include(). I'll bet a
twinkie that /includes doesn't exist on your server (who would give a web
site access to files right off of the system root directory??) but am not
going to tell you any more because you should go and look it up yourself.
No, this reply is more about style and tricks. Your code is long and
difficult to debug because of its sheer bulk, and it needn't be.
I would write your entire 200 (or 151) lines of code as
<?php
$page = $_GET['id'] ;
if ( is_file("/includes/$page.html") )
{ include("/includes/$page.html") ; }
?>
if you didn't need to exclude any possibilities or, much more sanely,
<?php
$includes = array ( "home", "arts", ... "world") ;
$page = $_GET['id'] ;
if ( in_array($page,$includes) && is_file("/includes/$page.html") )
{ include("/includes/$page.html") ; }
?>
(just from the top of my head, though, without real thought for security)
and in reality I'd probably not bother with the $page temp variable.
Good luck, and enjoy :-)
HTH & HAND
:-D
--
David T-G * There is too much animal courage in
(play) [EMAIL PROTECTED] * society and not sufficient moral courage.
(work) [EMAIL PROTECTED] -- Mary Baker Eddy, "Science and Health"
http://justpickone.org/davidtg/ Shpx gur Pbzzhavpngvbaf Qrprapl Npg!
pgp00000.pgp
Description: PGP signature
