switch-case inhalt changen

flo96 · 23. Mai 2016

Hallo Leute,

ich bin mal wieder unterwegs...

und zwar will ich den inhalt meiner website über die switch-case funktion ändern.

das läuft bis zu dem Punkt, andem es keine datei mit dem entsprechenden namen gibt.
Hier sollte er dann die error404.php aufrufen, was er aber nicht tut...

Vllt hat ja jemand ne idee:

PHP

//index.php
<? error_reporting(E_ALL); ?>
<!DOCTYPE HTML>
<html lang="DE">
<?
include("head.php");
?>
<?
include("body.php");
?>
</html>

Alles anzeigen

Im body.php soll der content ausgelesen werden:

PHP

<?
include pageFilenames(); // Binde die Dateien ein
function pageFilenames() { // Funktion für die switch - case funktion
$p = !empty($_GET["page"]) ? trim($_GET["page"]) : 'home'; // hol dir den gesuchten link
switch($p) {
case "home": // Fall home ->
$r = 'home.php'; // rufe inhalt von home.php ab
break; // breche funktion ab
case "galerie":
$r = 'galerie.php';
break;
case "land":
$r = 'land.php';
break;
case "natur":
$r = 'natur.php';
break;
case "portrait":
$r = 'portrait.php';
break;
case "technik":
$r = 'technik.php';
break;
case "guest":
$r = 'guest.php';
break;
case "imp":
$r = 'imp.php';
break;
case "kontakt":
$r = 'kontakt.php';
break;
case "daten":
$r = 'daten.php';
break;
default:
$r = 'error404.php';
break;
}
return $r;
}
?>

Alles anzeigen

Danke schonmal im vorraus

Basi · 23. Mai 2016

Dein Code ist so korrekt. Das Problem liegt also woanders.

An genau welcher Stelle des Codes erwartest du genau welches Ergebnis und was passiert stattdessen? Schau mal zwischendurch mittels var_dump was in $p und $r passiert.

Du brauchst übrigens nach default kein break mehr.

flo96 · 23. Mai 2016

also ganz einfach:
ich erwarte an letzter stelle, dass wenn keiner dieser cases vorkommt, der inhalt der error404.php ausgelesen wird, weil entweder der link zu nichts führt, oder weil höchstwahrscheinlich die Seite nicht vorhanden ist, oder der benutzer eine falsche Internetadresse eingegeben hat, welche auf ein Ziel führt, welches es gar nicht gibt.

Danke für die information, ich glaub das hatte ich übersehen. Klar brauche ich kein break mehr weil der default selbst der Abbruch der schleife ist...

Ich hab mal den script im localhost gerunt und der localhost spuckt mir folgendes aus:

Warning: include(land.php): failed to open stream: No such file or directory in C:\xampp\htdocs\dashboard\content.php on line 2

Warning: include(): Failed opening 'land.php' for inclusion (include_path='C:\xampp\php\PEAR') in C:\xampp\htdocs\dashboard\content.php on line 2

Notice: Undefined variable: r in C:\xampp\htdocs\dashboard\content.php on line 43
NULL
Notice: Undefined variable: p in C:\xampp\htdocs\dashboard\content.php on line 44
NULL

Klar dass er land.php nicht includen kann, da es sie schlichtweg (noch) nicht gibt.

Was bedeuten jetzt die anderen beiden genau?

Basi · 23. Mai 2016

Ohne den Code dazu kann man da jetzt wieder nichts sagen.

Wo werden die Fehler ausgespuckt? Du bist doch nicht zum ersten mal im Forum

flo96 · 23. Mai 2016

Basi schrieb:

Ohne den Code dazu kann man da jetzt wieder nichts sagen.

Wo werden die Fehler ausgespuckt? Du bist doch nicht zum ersten mal im Forum

Mal schauen ob ich die überhaupt richtig eingebaut hab die var_dumps...

PHP

<?
include pageFilenames(); // Binde die Dateien ein
function pageFilenames() { // Funktion für die switch - case funktion
$p = !empty($_GET["page"]) ? trim($_GET["page"]) : 'home'; // hol dir den gesuchten link
switch($p) {
case "home": // Fall home ->
$r = 'home.php'; // rufe inhalt von home.php ab
break; // breche funktion ab
case "galerie":
$r = 'galerie.php';
break;
case "land":
$r = 'land.php';
break;
case "natur":
$r = 'natur.php';
break;
case "portrait":
$r = 'portrait.php';
break;
case "technik":
$r = 'technik.php';
break;
case "guest":
$r = 'guest.php';
break;
case "imp":
$r = 'imp.php';
break;
case "kontakt":
$r = 'kontakt.php';
break;
case "daten":
$r = 'daten.php';
break;
default:
$r = 'error404.php';
}
return $r;
}
var_dump($r);
var_dump($p);
?>

Alles anzeigen

Ich habe sie nun an das ende der content.php gesetzt...

im grunde würde die zusammengesetzte datei so aussehen:

PHP

<? error_reporting(E_ALL); ?>
<!DOCTYPE HTML>
<html lang="DE">
<?
include("head.php");
?>
<body>
<? include("header.php"); ?>
<hr />
<? include("nav.php"); ?>
<hr />
<section>
<?
include pageFilenames(); // Binde die Dateien ein
function pageFilenames() { // Funktion für die switch - case funktion
$p = !empty($_GET["page"]) ? trim($_GET["page"]) : 'home'; // hol dir den gesuchten link
switch($p) {
case "home": // Fall home ->
$r = 'home.php'; // rufe inhalt von home.php ab
break; // breche funktion ab
case "galerie":
$r = 'galerie.php';
break;
case "land":
$r = 'land.php';
break;
case "natur":
$r = 'natur.php';
break;
case "portrait":
$r = 'portrait.php';
break;
case "technik":
$r = 'technik.php';
break;
case "guest":
$r = 'guest.php';
break;
case "imp":
$r = 'imp.php';
break;
case "kontakt":
$r = 'kontakt.php';
break;
case "daten":
$r = 'daten.php';
break;
default:
$r = 'error404.php';
}
return $r;
}
var_dump($r);
var_dump($p);
?>
</section>
<hr />
<footer>Impressum | Kontakt | Datenschutz</footer>
</body>
</html>

Alles anzeigen

wenn jemand (m)ein Fehler entdeckt wäre das super...

Gruß
Flo

cottton · 23. Mai 2016

Du gibst $p nicht aus der Funktion zurück.
Und den zurückgegebenen Wert $r setzt Du ausserhalb der Func nicht, sondern nutzt den Wert direkt um die Seite einzubinden:

PHP

include pageFilenames();

Eine Möglickeit wäre:

PHP

function pageFilenames() { // Funktion für die switch - case funktion
$p = !empty($_GET["page"]) ? trim($_GET["page"]) : 'home'; // hol dir den gesuchten link
switch($p) {
case "home": // Fall home ->
$r = 'home.php'; // rufe inhalt von home.php ab
break; // breche funktion ab
// ...
default:
$r = 'error404.php';
}
return array($p, $r);
}
list($p, $r) = pageFilenames();
include $r;

Alles anzeigen

Oder per Ref:

PHP

function pageFilenames(&r, &$p) { // Funktion für die switch - case funktion
$p = !empty($_GET["page"]) ? trim($_GET["page"]) : 'home'; // hol dir den gesuchten link
switch($p) {
case "home": // Fall home ->
$r = 'home.php'; // rufe inhalt von home.php ab
break; // breche funktion ab
// ...
default:
$r = 'error404.php';
}
return;
}
$r = $p = '';
pageFilenames($r, $p);
include $r;

Alles anzeigen

Oder aber auch:

PHP

function getPage() {
return !empty($_GET["page"]) ? trim($_GET["page"]) : 'home';
}
function getFilename() {
$p = getPage();
switch($p) {
case "home":
$r = 'home.php';
break;
// ...
default:
$r = 'error404.php';
}
return $r;
}
$p = getPage();
$r = getFilename();
include $r;

Alles anzeigen

flo96 · 23. Mai 2016

bei der ersten methode sagt er mir das:

Notice: Array to string conversion in C:\xampp\htdocs\dashboard\content.php on line 2

Warning: include(Array): failed to open stream: No such file or directory in C:\xampp\htdocs\dashboard\content.php on line 2

Warning: include(): Failed opening 'Array' for inclusion (include_path='C:\xampp\php\PEAR') in C:\xampp\htdocs\dashboard\content.php on line 2

Warning: include(guest.php): failed to open stream: No such file or directory in C:\xampp\htdocs\dashboard\content.php on line 44

Warning: include(): Failed opening 'guest.php' for inclusion (include_path='C:\xampp\php\PEAR') in C:\xampp\htdocs\dashboard\content.php on line 44
string(9) "guest.php" string(5) "guest"

bei der zweiten methode das:

Warning: include(): Filename cannot be empty in C:\xampp\htdocs\dashboard\content.php on line 43

Warning: include(): Failed opening '' for inclusion (include_path='C:\xampp\php\PEAR') in C:\xampp\htdocs\dashboard\content.php on line 43
string(0) "" string(0) ""

bei der dritten sagt er mir folgendes:

Fatal error: Call to undefined function getFilenames() in C:\xampp\htdocs\dashboard\content.php on line 44

Also irgendwie haben die Vorschläge zwar gut ausgesehen, aber gebracht hatten sie leider nichts

cottton · 24. Mai 2016

Du musst es Dir ja auch genauer ansehen, und nicht einfach reinkopieren.

Basi · 24. Mai 2016

Zitat

aber gebracht hatten sie leider nichts

Genau so wenig wie deine Fehlermeldungen ohne deinen Code um überhaupt zu wissen was du da machst..

Und zwar so, dass die Zeilennummern mit den Fehlermeldungen übereinstimmen.

Warum nicht einfach:

PHP

<?php
function getFileName($baseName, $default)
{
if(empty($baseName))
return $default;
$baseName = htmlspecialchars(trim($baseName));
if(!is_file($baseName .".php"))
return "error404.php";
else
return $baseName .".php";
}
include getFileName($_GET["page"], "home.php");
?>

Alles anzeigen

Nach wie vor bin ich der Meinung, der Fehler liegt nicht in deinem Code.

Code

Warning: include(land.php): failed to open stream: No such file or directory

Sagt doch eigentlich alles und ist selbsterklärend.

Das widerspricht dann wiederrum deiner Aussage, alles funktioniere, bis "error404.php" aufgerufen werden soll.

flo96 · 24. Mai 2016

Basi :

wenn du der Meinung bist, dass es nicht an dem Code liegt, woran könnte es dann liegen?

könnten sich die .htaccess und diese Code gegenseitig abschießen?

in meiner .htaccess steht nämlich das drin:

Code

ErrorDocument 404 /error404.php

Ist das vllt der Fehler?

Basi · 24. Mai 2016

Ich bin an der Stelle raus.

Kein Code. Keine genaue Beschreibung was passiert. Ich rate doch nicht.

flo96 · 30. Mai 2016

So hier nocheinmal ein Versuch mich zu erklären:

Ich habe basiert auf eine index.php eine Seite geschrieben, welche sich über eine switch-case-Funktion den angefragten Inhalt holt.
Ist dieser nicht da, soll er die Error404 ausspucken welche ich gemacht habe...

Die Funktionen dazu ist diese hier:

PHP

<?
include pageFilename(); // Binde die Dateien ein
function pageFilename() { // Funktion für die switch - case funktion
$p = !empty($_GET["page"]) ? trim($_GET["page"]) : 'home'; // hol dir den gesuchten link
switch($p) {
case "home": // Fall home ->
$r = 'home.php'; // rufe inhalt von home.php ab
break; // breche funktion ab
case "galerie":
$r = 'galerie.php';
break;
case "land":
$r = 'land.php';
break;
case "natur":
$r = 'natur.php';
break;
case "portrait":
$r = 'portrait.php';
break;
case "technik":
$r = 'technik.php';
break;
case "guest":
$r = 'guest.php';
break;
case "imp":
$r = 'imp.php';
break;
case "kontakt":
$r = 'kontakt.php';
break;
case "daten":
$r = 'daten.php';
break;
default:
header('HTTP/1.0 404 Not Found');
$r = 'error404.php';
}
return $r;
}
?>

Alles anzeigen

Die zugehörige Navigation beläuft sich hierauf:

HTML

<nav role="navigation">
<ul>
<li class="cat1">
<a href="index.php?page=home" title="Home">Home</a>
</li>
<li class="cat2">
<a href="index.php?page=galerie" title="Galerie">Galerie</a>
<ul>
<li><a href="index.php?page=land" title="Landschaft">Landschaft</a></li>
<li><a href="index.php?page=natur" title="Natur">Natur</a></li>
<li><a href="index.php?page=portrait" title="Portrait">Portrait</a></li>
<li><a href="index.php?page=technik" title="Technik">Technik</a></li>
</ul>
</li>
<li class="cat3">
<a href="index.php?page=guest" title="Gästebuch">Gästebuch</a>
</li>
<!--<li>
<a href="">Shop</a>
</li>-->
</ul>
</nav>

Alles anzeigen

Wenn ich nun auf meinem Test-Webspace die Testdomaine aufrufe, kommt der gewünschte Inhalt der home.php.

Wenn ich nun einen Link öfnnen möchte der hier gelistet ist, bei dem ich weiß, dass es diese Datei nicht gibt, bringt er nichts heraus als das blanke Layout.

Wenn ich aber nun in die Adressleiste gehe und dort statt index.php?page=guest dort eingebe index.php?page=lol (diese Seite gibt es sicherlich nicht) spuckt er mir die error404 raus, welche ich auch wollte.

woran liegt es jetzt, dass er eine angefragte datei welche es nicht gibt auf dem Webspace einmal per Link nicht auf die error404 weiterleitet, aber die angefragte Datei aus der Adressleiste welche ich per hand eingegeben habe auf die error404 weiterleitet?

Ich habe eine kleine Vermutung aber bin mir dabei nicht sicher:

Es liegt um genau zu sein doch an meinem Script, er ist zwar an sich nicht falsch, aber er will eine Datei einbinden die es nicht gibt. Für diesen Fall dass er sie nicht einbinden kann, gibt es keine Exit-Bedingung, welche quasi den default aufruft.

Ich bedanke mich für jegliche Hilfe

EDIT:

Wenn ich das richtig sehe bzw. denke, müsste ich eine kleine Funktion schreiben, welche prüft, ob die z.B. im Fall Gästebuch angefragte Datei included werden kann, falls ja die datei auslesen, falls nein auf die error404 weiterleiten

Basi · 30. Mai 2016

Jetzt verstehe ich was du meinst. Natürlich bekommst du nichts angezeigt wenn die Datei zwar nicht vorhanden ist, die Switch Abfrage aber greift. Das ist so logisch (und die Fehlermeldungen weisen noch mal expliziter darauf hin), dass ich es nicht mal in Betracht gezogen habe

Was du brauchst ist eine Prüfung ob die Datei existiert:

PHP

switch($p) {
case "home":
$r = (is_file('home.php')) ? 'home.php' : 'error404.php';
break;
case "galerie":
$r = (is_file('galerie.php')) ? 'galerie.php' : 'error404.php';
break;
case "land":
$r = (is_file('land.php')) ? 'land.php' : 'error404.php';
break;
case "natur":
$r = (is_file('natur.php')) ? 'natur.php' : 'error404.php';
break;
case "portrait":
$r = (is_file('portrait.php')) ? 'portrait.php' : 'error404.php';
break;
case "technik":
$r = (is_file('technik.php')) ? 'technik.php' : 'error404.php';
break;
case "guest":
$r = (is_file('guest.php')) ? 'guest.php' : 'error404.php';
break;
case "imp":
$r = (is_file('imp.php')) ? 'imp.php' : 'error404.php';
break;
case "kontakt":
$r = (is_file('kontakt.php')) ? 'kontakt.php' : 'error404.php';
break;
case "daten":
$r = (is_file('daten.php')) ? 'daten.php' : 'error404.php';
break;
default:
$r = 'error404.php';
}

Alles anzeigen

Der Fehler lag also nicht im Code oder im Dateisystem, sondern in deiner Logik.

Nach wie vor ist jedoch das hier zwecks Dynamik und verringertem Änderungsaufwand die bessere Lösung.

flo96 · 30. Mai 2016

Genau

Die Funktion an sich hatte gestimmt

Dann bin ich auch erst beim verfassen des Beitrags auf die Idee gekommen dass es vllt daran liegen könnte

Danke für die Hilfe

Ich werde es direkt ausprobieren xD

EDIT:

Danke es funktioniert