• Jetzt anmelden. Es dauert nur 2 Minuten und ist kostenlos!

Mein erstes PHP-Script bitte verbessern.

P

patrick227

Neues Mitglied
Hallo,
das Programm generiert mir ein simples Layout (ähnlich Blog, ohne Kommentare; txt01 ist ein Beitrag, txt01_short die Vorschau des Beitrages) habe es nach dem quake.net-Tutorial geschrieben und auf meine Bedürfnisse umgestalten.
Ich denke es ist selbsterklärend, möchte nur wissen was ich besser machen kann und ob es sicher ist:

PHP: 
<?php
$section = array();
$section['licence'] = 'inc/licence.html';
$section['contact'] = 'inc/contact.html';
$section['txt01_short'] = 'inc/txt01_short.html';
$section['txt02_short'] = 'inc/txt02_short.html';
$section['txt01'] = 'inc/txt01.html';
$section['txt02'] = 'inc/txt02.html';


include 'inc/header.html';
echo "    <body>\n";
include 'inc/logo.html';


if (isset($_GET['section'], $section[$_GET['section']])) {
    $ort = $_GET['section'];
    if ($ort == 'contact') {
        include $section['contact'];
    } elseif ($ort == 'txt01') {
        include $section['licence'];
        include $section['txt01'];
    } elseif ($ort == 'txt02') {
        include $section['licence'];
        include $section['txt01']; }
} else {
    include $section['licence'];
    include $section['txt02_short'];
    include $section['txt01_short'];
}
include 'inc/footer.html';
echo "    </body>\n";
echo "</html>\n";
?>
lg Patrick

EDIT: auf meinem Testserver funktioniert es
 
das ist schon soweit ok, etwas umständlich, aber das kommt noch.
wesentlich kürzer ist es so:
PHP: 
include 'inc/header.html';
echo "    <body>\n";
include 'inc/logo.html';


if(file_exists("inc/".$_GET['section'].".html"))
{
  include("inc/".$_GET['section'].".html");
}
else {
    include ('inc/licence.html');
    include ('inc/txt02_short.html');
    include ('inc/txt01_short.html');
}

include 'inc/footer.html';
echo "    </body>\n";
echo "</html>\n";
vorausgesetzt, du übergibst in der $_GET variablen nur den namen der html-datei, also z.b. einen link wie: http://blabla.com?section=license

tust du nun noch in die header.html deinen body-tag und in die footer.html das schliessen des body-tags. ist es so wie es sein sollte ....
 
Danke, ich wollte egentlich mit der $_GET variabel den Seitennamen übergeben der dann aus mehreren html-Dateien sich zusammensetzt.

eine normale seite setzt sich so zusammen:
logo
license
text1
footer

die contact-Seite:
logo
textcontact
footer

Ist von meinem Vorhaben nun abzuraten und soll ich mit der $_GET variablen nur den namen der html-datei übergeben?
 
dein script ist ok und auch sicher. wenn es funktioniert, dann lass es so.
da es dein erstes script ist ist das schon recht ordentlich.

eventuell solltest du dir (um weiter zu lernen) mal den switch befehl ansehen. damit fallen dann deine ganzen if und elseif anweisungen weg. damit wird dein script übersichtlicher und leichter erweiterbar.
 
Du musst dich einloggen oder registrieren, um hier zu antworten.

Neueste Beiträge

Zurück
Oben