Seitenstruktur

[alexf812]

hallo,

mal ne kurze frage.
ab und zu seh ich seiten wo man immer nur eine index datei hat und dann en parameter dran. wenn ich nun eine grosse seite hab, wird das dann immer mit ner riesen switch-anweisung gelöst oder gibts da ne andere lösung?

ist es denn besser immer inde.php?seite=... oder für jedne menüpunkt ne eigne datei? im switch werden ja m.E. auch nur die einzelnen datein eingebunden oder?

[rewboss]

Es muss kein Switch sein. Es könnte z.B. so aussehen:

PHP-Code:

<?php
 // Dokument-Head includen
 include("head.inc");

 // Navigation includen
 include("navigation.inc");

 // Inhalt includen
 if(!isset($_GET['page']) || !file_exists($_GET['page'].'.inc')){
  include("index.inc");
 }
 else{
  include($_GET['page'].'.inc');
 }

 // Rest des Dokuments includen
 include("foot.inc");

?>

Der Vorteil: Stell dir vor, du willst die Navigationsleiste neu gestalten, hast aber 100 Seiten auf deiner Website. Mit dieser Methode brauchst du nur eine Datei zu bearbeiten.

[-Andi-]

Zitat von rewboss

Es muss kein Switch sein. Es könnte z.B. so aussehen:

PHP-Code:

<?php
 // Dokument-Head includen
 include("head.inc");

 // Navigation includen
 include("navigation.inc");

 // Inhalt includen
 if(!isset($_GET['page']) || !file_exists($_GET['page'].'.inc')){
  include("index.inc");
 }
 else{
  include($_GET['page'].'.inc');
 }

 // Rest des Dokuments includen
 include("foot.inc");

?>

Der Vorteil: Stell dir vor, du willst die Navigationsleiste neu gestalten, hast aber 100 Seiten auf deiner Website. Mit dieser Methode brauchst du nur eine Datei zu bearbeiten.

Bei einem Switch kann man auch mehrere Seiten einbinden und muss nur noch eine Datei bearbeiten

[Fex]

Hi,

PHP-Code:

include($_GET['page'].'.inc'); 


solcher Art Konstrukte bergen immer ein gewisses Sicherheitsrisiko!
Besser mit Arrays arbeiten! Spart auch jegliche switch-Anweisungen:

PHP-Code:

<?php 
$arrSitePages = array(); 
$arrSitePages['home'] = 'index.php'; 
$arrSitePages['topic1'] = 'news/neuigkeiten.php'; 
$arrSitePages['topic2'] = 'help/hilfe.htm'; 

if (!isset($_GET['page']) || !file_exists($arrSitePages[$_GET['page']])) { 
    //wenn nicht vorhanden wird der Variablen '$page' der Wert 'home' zugewiesen; 
    $page = "home"; 
} else { 
    //sonst der Wert des URL-Parameters mit Namen = 'page' 
    $page = $_GET['page']; 
} 

include("{$arrSitePages[$page]}"); 
?>

Gruß Fex

[rewboss]

Zitat von Fex

PHP-Code:

include($_GET['page'].'.inc'); 


solcher Art Konstrukte bergen immer ein gewisses Sicherheitsrisiko!

Deswegen das hier:

PHP-Code:

 if(!isset($_GET['page']) || !file_exists($_GET['page'].'.inc')){
  include("index.inc");
 } 


...oder lauert auch da ein Sicherheitsrisiko, das ich noch nicht kenne?

[DSB]

Soweit ich weiß funktioniert file_exists($_GET['page']) auch über HTTP-Verbindungen.
Wenn in $_GET['page'] "http://www.boserBub.com/boesesScript" steht, könnte man so eigenen Code einschleusen.

[rewboss]

Zitat von DSB

Soweit ich weiß funktioniert file_exists($_GET['page']) auch über HTTP-Verbindungen.

Nur wenn allow_url_fopen aktiviert ist.

[DSB]

Es kommt halt immer darauf an für wen oder was man programmiert.
Wenn Du immer Zugrif auf die Servereinstellungen hast und Dir sicher sein kannst, dass das nie eingeschaltet wird, bei jedem neuen Script immer an den Zusammenhang denkst und Du nur für Dich programmierst, dann kann man das so machen.
Für z.B. unseren MySQLDumper, der auf vielen verschiedenen Servern läuft, wäre das undenkbar und stellt dann auf manchen Servern ein Sicherheitsrisiko dar. Darauf wollte Fex hinweisen und da hat er prinzipiell recht.

[steffenk]

Daher würde ich auch eine Methode mit einem statischen Array vorziehen, wo die gültigen Seiten eingetragen sind, und dann mit in_array() abfragen.

[rewboss]

Nun ja, allow_url_fopen kann nur durch Bearbeiten von php.ini selbst gesetzt werden, genau aus diesem Grund. Aber es stimmt schon, lieber ganz auf Nummer Sicher gehen. (Es war ja auch nur ein hypothetisches Beispiel...)

[alexf812]

und könnte man das Sicherheitsrisiko nicht ausschalten in dem man

PHP-Code:

file_exists("./".$_GET['page'].'.inc') 


schreibt?

[DSB]

Sobald in der Zeichenkette "http://" vorkommt, wir der Rest davor ignoriert, glaube ich.
Käme auf einen Test an.

[|23|]

Taugt so eine möglichkeit? Ist für eine kleine Seite.

PHP-Code:

            switch ($_GET['id']) 
            {
                case 1:
                    include('home.php');
                    break;
                   
                case 2:
                    include('company.php');
                    break;
                   
                case 3:
                    include('products.php');
                    break;
                   
                case 4:
                   // include('kon.php');
                    break;
                   
                case 5:
                    // include('imp.php');
                    break;
                    
                case 5:
                    // include('imp.php');
                    break;
                    
                case 7:
                    include('login.php');
                    break;
                   
                default:
                    include('home.php');
            } 


[DSB]

Funktionieren tut das.
So würde ich es aber eleganter, kürzer und leichter erweiterbar finden:

$seiten=ARRAY(1=>'home',2=>'company',3=>'products',...);
// existiert die übergebene Seitennummer im Array? Wenn nicht auf "home" setzen
$seitenindex=(in_array($_GET['id'],$seite)) ? $_GET['id']:1;
include($seiten[$seitenindex].'.php');

[321]

Ich prüfe jeweils, dass kein / in 'page' übergeben wird.
Also kein Ordnerwechsel durch den Anwender!

Weiter: bei 'nichtvorhandener Seite' eine kleine 'notfound-Seite' statt 'home'
Kein Muss aber ein Chic