[PHP] Includecode werkt niet, wat doe ik fout?

[DJFliX]

Deze code heb ik in een tabel geplaatst, maar er gebeurt niets (de pagina blijft gewoon zwart, nl. de achtergrondkleur van de site). Het gaat om www.flixite.com (nog verre van functionerende wlm-ripoff-theme) en in het nu nog zwarte vlak moet home.php komen, tenzij ?pagina=paginanaam.php wordt opgevraagd (bv. http://www.flixite.com/index.php?pagina=campzone.php) zodat campzone.php in die tabel terecht komt.

Ik heb dit script ergens van het forum gecopy'd en gepaste, en een beetje dingen geprobeerd met vervangen van paginanamen enzo, maar ik snap er eigenlijk helemaal niks van. Ik hoop dat iemand mij kan wijzen op mijn fout

Edit: Het script werkt, maar als er niets wordt gespecificeerd (bv ?pagina=phone.php) dan blijft de pagina leeg... waarom komt home.php hier niet gewoon te staan?

Edit2: Negeer dit maar ff, volgens mij ligt het probleem aan mijn webserver... home.php staat op mijn ftp, maar is niet te bereiken via http://www.flixite.com/home.php

code:

<?php

// Get home-page (home.php)

if($_GET['pagina']=="") {
include($_GET['home'].".php");
}

// Get variable page
else {
if(file_exists($_GET['pagina'].".php")) {
include($_GET['pagina'].".php");
}

// Go to error.php
else {
include("error.php");
}
}
?>

[ Voor 15% gewijzigd door DJFliX op 02-09-2006 18:14 ]

[Depress]

$_GET['home'] is niet defined.
Je zult daar gewoon

PHP:

include("home.php");

Moeten gebruiken.

[ Voor 59% gewijzigd door Depress op 02-09-2006 18:14 ]

[GlowMouse]

Ik denk dat onderstaande code beter zal werken. Niet alleen is $_GET['home'] wellicht niet gedefinieerd ($_GET verwijst naar variabelen die via de URL worden meegegeven), maar ook is jouw script erg onveilig (verander ?pagina=veilig maar eens voor de lol in ?pagina=index). Daarnaast bestaat elseif om een onoverzichtelijke else { if { } } constructie te vervangen.

PHP:

$valid_pages = array('a.php',
            'b.php',
            'c.php'
        );

// Get home-page (home.php)
if($_GET['pagina']=='') {
    include('home.php');
}
// Get variable page
elseif(in_array($_GET['pagina'].'.php', $valid_pages)) {
    include($_GET['pagina'].'.php');
}
else {
    include('error.php');
}

[Helmet]

wat jij doet is
als ($_GET['pagina']) leeg is, include $_GET['pagina'] (null) + .php je probeert in principe het bestand .php in te voeren.

Verder zou ik adviseren om het als volgt aan te pakken:

PHP:

<?
if (!isSet($_GET['action']) || empty($_GET['action'])) { $action = 'home'; }
else { $action = $_GET['action']; }
switch ($action)
{
case 'home':
echo 'homepage!';
break;
case 'intro':
include('intro.php');
break;
}
?>

[DJFliX]

Thanx het werkt hij mag van mij een slotje? Thanx all