[Orolhawion]

Zitat von meici

also ist da jetz was falsch oder hat die seite mir das nur falsch angezeigt?

der validator zeigt dir an was nicht valide ist, deshalb heisst er wohl auch so. und der validator hat immer recht, denn sonst wärs ja kein validator. das heisst im klartext: wenn der validator sagt, er hat nen fehler gefunden, dann existiert der fehler auch. nämlich in dem skript das man durch den validator jagt, in diesem fall also in deinem script. wo der fehler liegt, sagt dir der validator auch, in diesem fall sogar ziemlich präzise, wie devar schon bemerkt hat.

[meici]

achso
lern lesen und schreiben
sry für die dumme frage

[meici]

soweit is der script jetzt fehlerfrei
muss mir jetzt nur noch was mit dem menü überlegen
Vielen Dank für eure Hilfe

[meici]

einmal brauch ich jetz ncoh eure Hilfe

Parse error: syntax error, unexpected T_CASE in /usr/export/www/vhosts/funnetwork/hosting/berta211/facharbeit php/inhalt.php on line 21

wird mir in dem "haupt" div ausgegeben.

PHP-Code:

<?php

    if(isset($_GET['section'])) {
        switch($_GET['section']) {
            case "home":
                include "home.php";
                break;

            case "info":
                include "info.php";
                break;

            case "infodef":
                include "infodef.php";
                break;

            case "minfo":
                include "minfo.php";
                break

            case "minfodef":                    <- hier ist Zeile 21
                include "minfodef.php";
                break

            case "mgestaltung":
                include "mgestaltung.php";
                break

            case "mdidaktik":
                include "mdidaktik.php";
                break

            case "mtheorie":
                include "mtheorie.php";
                break

            case "zukunft":
                include "zukunft.php";
                break

            case "studiengang":
                include "studiengang.php";
                break

            case "beruf":
                include "beruf.php";
                break

            case "zukunft2":
                include "zukunft2.php";
                break

            case "meinung":
                include "meinung.php";
                break

            case "bsp":
                include "bsp.php";
                break

            case "bsp1":
                include "bsp1.php";
                break

            case "bsp2":
                include "bsp2.php";
                break

            case "bsp3":
                include "bsp3.php";
                break

            case "imprint":
                include "imprint.php";
                break

            default:  // Wenn eine ungültige Section angegeben wurde
                    // sollen die News gezeigt werden
                include "home.php";
                break;
        }
    } else {
        // wenn section nicht angegeben wurde
        // sollen die News angezeigt werden.
        include "home.php";
    }
?>

das is die inhalt.php
aber irgendwie find ich den fehler nicht

[kampfgnom]

Du hast hinter den meisten break 's das ";" vergessen!

[meici]

danke...
stimmt, ist mir gar nicht aufgefallen...

[Orolhawion]

es gibt editoren, die sowas während dem tippen schon anzeigen. vielleicht solltest du sowas benutzen, denn damit reduziert man den suchaufwand, wenn nicht sogar schon das auftreten solch einfacher syntaxfehler. im anhang siehst du wie pdt (eclipse) das ganze darstellt. oft findet man den eigentlichen fehler, in der zeile davor.

[Basti]

und dir fällt wirklich nichts auf an deiner langen Liste da? Schau dir nochmal dn letzten Punkt von den Tipps aus meinem vorigen Posting an:

PHP-Code:

<?php

    if(isset($_GET['section'])) {
        switch($_GET['section']) {
            case "home":
                include "home.php";
                break;

            case "info":
                include "info.php";
                break;

            case "infodef":
                include "infodef.php";
                break;

            case "minfo":
                include "minfo.php";
                break

            case "minfodef":                    <- hier ist Zeile 21
                include "minfodef.php";
                break

            case "mgestaltung":
                include "mgestaltung.php";
                break

            case "mdidaktik":
                include "mdidaktik.php";
                break

            case "mtheorie":
                include "mtheorie.php";
                break

            case "zukunft":
                include "zukunft.php";
                break

            case "studiengang":
                include "studiengang.php";
                break

            case "beruf":
                include "beruf.php";
                break

            case "zukunft2":
                include "zukunft2.php";
                break

            case "meinung":
                include "meinung.php";
                break

            case "bsp":
                include "bsp.php";
                break

            case "bsp1":
                include "bsp1.php";
                break

            case "bsp2":
                include "bsp2.php";
                break

            case "bsp3":
                include "bsp3.php";
                break

            case "imprint":
                include "imprint.php";
                break

            default:  // Wenn eine ungültige Section angegeben wurde
                    // sollen die News gezeigt werden
                include "home.php";
                break;
        }
    } else {
        // wenn section nicht angegeben wurde
        // sollen die News angezeigt werden.
        include "home.php";
    }
?>

Unzählige Redundanzen, Wiederholungen. Das kann man doch ganz einfach auch so formulieren:

PHP-Code:

<?php

$aPages = array(
    'home',
    'info',
    'infodef',
    'minfo',
    ...
);

// erster Eintrag der Liste sei der Default-Wert
$sDefaultPage = $sPage = reset($aPages);

if (isset($_GET['section']))
    $sPage = $_GET['section'];

if (!in_array($sPage, $aPages))
    $sPage = 'error404';

include 'pages/' . $sPage;

Damit bist du schonmal ein ganzes Stück weiter. Noch einfacher geht es eben dann, wenn du erkennst, dass diese Liste sich dann exakt mit der Liste der verzeichniseinträge in deinem pages-Verzeichnis decken wird (bis auf die Endung, die hier fehlt). Dann brauchst du die Liste nämlich nicht mehr doppelt zu führen (einmal hier im Array und einmal in Form von Dateinamen in deinem Verzeichnis), sondern guckst eben direkt im Verzeichnis nach, ob eine entsprechende Seiten-Date existiert.

Also:

PHP-Code:

<?php

$sPage = 'home';

if (isset($_GET['page']))
    $sPage = $_GET['page'];

if (!preg_match('|^[a-z0-9]+$|', $sPage))
    die('invalid page id');

$sFilePath = sprintf('pages/%s.php', $sPage);

if (!is_writable($sFilePath))
    die('page not found');

include($sFilePath);

Kannst die Dingens natürlich auch weiterhin „Sections“ nennen – wusste nur nicht, was das hier bedeuten soll.

preg_match() filtert die Eingaben nach dem Muster: Nur Kleinbuchstaben (des englischen Alphabets) und Ziffern. Sowas ist eben wichtig, damit ein Angreifer nicht mit geschickten Parametern dateien einbinden kann, die außerhalb des angegeben Verzeichnisses liegen.

Und klar, um die Fehler darzustellen gibt es feinere Lösungen. In jedem Fall solltest du die korrekten Status-Header setzen (z.B. 404) und dann eben deine Fehlermeldung ausspucken. (Hier auch wieder die übergebenen Parameter niemals ungefiltert ausgeben, sonst lässt dich dein Prof womöglich durchrasseln!)

Basti