Re: Someone please help me with this PHP script.
| From: | M. Sokolewicz | Date: | Sun, 18 Dec 2005 09:34:37 +0000 |
| Subject: | Re: Someone please help me with this PHP script. | ||
| References: | 1 2 | Groups: | php.general |
| Request: | Send a blank email to php-general+get-227592@lists.php.net to get a copy of this message | ||
Aside from the 20 suggestions you've already got, let me add this one:
elseif( ($page != "one") or ($page != "two") or ($page != NULL) ) {
Echo "<html><head><title>Undefined!</title></head><body>page isn'tdefined correctly!</body></html>"; } is useless code. Why? $page = null; never happens (you made sure of that further up the page). $page = "one"; never happens (you already pulled that one out earlier). $page = "two"; never happens (you already pulled that one out earlier). So... if *any* of those returns true you show an error... may I suggest just changing it to an else instead? a lot more logical(!). Besides that, were you to move this to its own if() you'd notice it would always display because if the page is "one" it *can not* be "two" or null, and will thus be displayed. Which means that you'd have an error which would always display... oh well, ignore my rant and first make sure to follow the suggestions provided by others - tul Wolf wrote:
What is it doing, or not doing? Try changing the <? to <?php and see if that works. Robert Erik Johnson wrote:I do not know why this isn't working, but it would be very helpful if someone looked over it. <? $defaultpage = "http://lom.game-host.org/uploads/erik/"; If($page == NULL) { $page = "one"; } if($page == "one") { Echo "<html><head><title>Page One</title></head><body>It works!!<br><a href=\"" . $defaultpage . "index.php?page=two\">Nice...</a></body></html>"; } elseif($page == "two") { Echo "<html><head><title>Page Two</title></head><body>This is page " . $page . " -- <a href=\"" . $defaultpage . "index.php ?page=\">;D</a></body></html>"; } elseif( ($page != "one") or ($page != "two") or ($page != NULL) ) { Echo "<html><head><title>Undefined!</title></head><body>page isn't defined correctly!</body></html>"; } if($page == "") { Echo "<br><br><br><br><br><br><br><br><a href=" . $defaultpage . ">Main Page..</a>"; } ?> Thank you, Erik Johnson