bedingtes Include

[cool_water_at]

Was PHP angeht bin ich leider noch ein Newbie ...
Ich habe jetzt durch googeln bzw. mein Vorwissen von anderen Programmiersprachen den einen PHP code zu schreiben.

Leider wird der Code nicht so abgearbeitet wie ich mir das vorstelle:

<?php
    ## INHALT DYNMAISCH ERZEUGEN ## 
      $content = $_GET["lifeonair"] . ".php";
    
    
    
     include('header.php');
    
    
    //wenn noch kein Link geklickt wurde, öffne die Datei 'start.php' 
      if(!$content) { 
           include("start.php"); 
         exit; 
      } 
    // Sicherheitsproblem unterbinden ************ 
      if (strtolower(substr($content,0,7)) == "http://" || 
          strtolower(substr($content,0,8)) == "https://" || 
          strtolower(substr($content,0,6)) == "ftp://"){ 
           include("fehlerseite.php"); 
           exit; 
      } 
      else if (!file_exists($content)){ 
           include("fehlerseite.php"); 
           exit; 
      } 

    //wenn $content einen Inhalt hat, öffne diesen 
      include($content); 

    
    
    include('sidebar.php');
    include('footer.php');
?>

was der Code eigentlich machen soll ist folgendes:

wenn keine Referenz also bsp. (index.php&lifeonair=links) - sprich wenn ich nur: index.php eintippe dann möchte ich, dass die "start.php" datei importiert wird.
das eben was nicht geht.
Wo liegt der Fehler?
wenn ich index.php eintippe - dann wird sofort die Fehlerseite angezeigt.

Danke für eure Hilfe schon im Voraus.

[pekka]

die() statt exit() verwenden.

[ministry]

Das ist Murks. Du schaffst eine Sicherheitslücke.

Mach es so:

Ein Array mit allen erlaubten Seitennamen
Abfrage, ob einer davon übergeben wurde, und nur dann includieren, andernfalls start.php einbinden

if (isset($_GET['lifeonair']) && in_array($_GET['lifeonair'], $allowed_pages)) include($_GET['lifeonair'] . ".php");
else include("start.php"); 

die() statt exit() verwenden.

Wieso?

[cool_water_at]

$allowed_pages ??

ist dan die Variable für ein Array?
könntest du bitten ein Beispiel posten - danke!

[penizillin]

halte dich an http://tut.php-q.net/frames.html und den rest der seite.

[ministry]

Das Beispiel steht doch schon da. Du musst nur noch ein Array mit deinen Seiten anlegen. (In dem Bespiel heißt das $allowed_pages, aber das kannst du ja ändern).

[cool_water_at]

vielen dank das wird mir weiter helfen!

[w-shadow]

mal ne andere frage: wie kann nach

      $content = $_GET["lifeonair"] . ".php"; 

folgendes if noch wie gewünscht funktionieren?

      if(!$content) { 
           include("start.php"); 
         exit; 
      } 

nach der zuweisung steht doch mindestens ".php" in $content, oder nicht???

[onemorenerd]

Mach mal ein var_dump() des if-Ausdrucks und von $content. Ich glaub das nämlich nicht ...

[cool_water_at]

Original geschrieben von onemorenerd
Mach mal ein var_dump() des if-Ausdrucks und von $content. Ich glaub das nämlich nicht ...

und wie geht das ?

var_dump($content) ???

und dann??

[cool_water_at]

Array.

hallo noch mal

ich habe nun das mit dem Array probiert - es wird leider noch immer nur die start.php angezeigt:

auch wenn im meinem Webbrowser "index.php?lifeonair=empfang" eintippe - die datei ist vorhanden.

    ## LIFE ON AIR ARRAY für erlaubte Links ## 

    $allowed_pages = array(); // ein Leeres Array erzeugen    
    $allowed_pages['start'];
    $allowed_pages['empfang'];
    $allowed_pages['links'];
    
    
 
    include('header.php');
    
    
    if (isset($_GET['lifeonair']) && in_array($_GET['lifeonair'], $allowed_pages)) { 
        
        include($_GET['lifeonair'] . ".php");
    
    }
    
    
    else {
        include("start.php");
    }

    
    
    include('sidebar.php');
    include('footer.php'); 

[pekka]

Das Array ist leer. Im Manual unter "Arrays" anschauen, wie man Elemente zuweist.

[cool_water_at]

<?php
    ## LIFE ON AIR ARRAY für erlaubte Links ## 

    $allowed_pages = array(); // ein Leeres Array erzeugen    
    $allowed_pages['start']="start.php";
    $allowed_pages['empfang']="empfang.php";
    $allowed_pages['links']="links.php";
    
    
 
    include('header.php');
    
    
    if (isset($_GET['lifeonair']) && in_array($_GET['lifeonair'], $allowed_pages)) { 
        
        include($_GET['lifeonair'] . ".php");
    
    }
    
    
    else {
        include("start.php");
    }

    
    
    include('sidebar.php');
    include('footer.php');
?>

Am Array ist es leider nicht gelegen - es wird leider noch immer nur die start.php seite angezeigt

    include $allowed_pages[$_GET['lifeonair']]; 

statt

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

nützt leider auch nichts

[onemorenerd]

$_GET['lifeonair'] ist ja auch nicht "empfang.php" sondern "empfang".

array_key_exists

[cool_water_at]

Hallo
ich möchte zu dem Array noch eine wechselnde Header grafik machen.

sprich ich brauche ein Variable und in die Variable soll jedes Mal ein Bild reingeladen werden sofern man eine Seite aufruft.


empfang: soll ein eignes Bild haben
start soll ein eigenes Bild haben.

wie kann ich dass dazu programmieren?