falscher menü aufruf bringt leere Seite

[Loddar1]

Hi Leute
ich habe in meiner menue.php 5 menues (funktioniert)

in der index.php rufe ich innerhalb von <article></article>
die menüs mit href.. index.php?page=(seite integer) ab (funktioniert)

wenn ich jedoch in der browserzeile index.php?page=(nicht vorhandene seite) aufrufe kommt eine
Seite ohne <article></article> also leer.

der Aufruf der Seite geht mit:

if(empty($page))
   {
     header("Location:index.php?page=1");
   }

die anderen Seiten jeweils per DB select

mir gefällt die leere Seite nicht, da will ich eine Meldung oder sowas

Ich habe es hiermit

$stmt=$pdo->prepare("
SELECT ziel
FROM menu
WHERE id = :id
");
$stmt->bindParam(':id', $page, PDO::PARAM_INT);
$stmt->execute();
$stmt->fetch();
if ($stmt->rowCount() == 0)
   {
     if(file_exists('error.php') == true){require('error.php');}
   }

probiert, funktioniert aber nur solange ich nur menu abfrage
bei den anderen 4 menüs wird auch die error.php angezeigt.

gibt es eine einfachere Methode?

Danke im vorraus auch für Denkanstöße...

 

[bdt600]

Das wird dann wohl an

if(empty($page))

liegen.

Richtig debuggen

1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(-1);
3. Man verwendet ini_set('display_errors', true); damit die Fehler auch angezeigt werden.
4. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
5. An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde. Wichtig bei MySQL Fehlern (...not a valid MySQL result resource...): mysqli_error() verwenden oder Abfrage ausgeben und zb mit phpmyadmin testen.
6. Schritt 5 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
7. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
8. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
9. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

Die Ausgaben per echo bzw. var_dump kann man sich ersparen, in dem man einen Debugger benutzt.

 

[Loddar1]

Das wird dann wohl an

if(empty($page))

liegen.

Richtig debuggen

1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man schreibt an den Anfang des Scriptes die Zeile: error_reporting(-1);
3. Man verwendet ini_set('display_errors', true); damit die Fehler auch angezeigt werden.
4. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
5. An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde. Wichtig bei MySQL Fehlern (...not a valid MySQL result resource...): mysqli_error() verwenden oder Abfrage ausgeben und zb mit phpmyadmin testen.
6. Schritt 5 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
7. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
8. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
9. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

Die Ausgaben per echo bzw. var_dump kann man sich ersparen, in dem man einen Debugger benutzt.

Ich wusste nicht, das ich das error reporting erwähnen muß.
Aber vielleicht hast du ja nicht richtig gelesen oder ich habe mich falsch ausgedrückt aber an empty liegt es nicht.

 

[bdt600]

Schreibe doch mal vor

if(empty($page))

diese Zeilen:

if (isset($page))
    echo "page = $page";
else
    echo "page not set";

Ich wusste nicht, das ich das error reporting erwähnen muß.

Da man das nirgendwo sehen kann, wäre das nicht die dümmste Idee gewesen.

 

[Divo1984]

Zunächst denke ich solltest du deinen Code vereinfachen:

if ($stmt->rowCount() == 0)
   {
     if(file_exists('error.php') == true){require('error.php');}
   }

funktioniert zwar aber ist doch so:

if ($stmt->rowCount() == 0 && file_exists('error.php')) {
    require('error.php');
}

Viel kürzer und besser lesbar Ich vermute den Fehler in der SQL Abfrage. Hast du da mal den Rückgabewert per var_dump() geprüft? Hast du den SQL-Query mal direkt per PHPMyAdmin geprüft?

 

[Loddar1]

Hallo Divo1984
Ich glaube das auch du nicht richtig gelesen hast oder drücke ich mich wirklich falsch aus?
Deine schreibweise mag zwar kürzer sein und besser lesbar aber löst nicht meine frage!

ich habe es jetzt so gelöst:

if(($stmta->rowCount() == 0)
&&($stmtb->rowCount() == 0)
&&($stmtc->rowCount() == 0)
&&($stmtd->rowCount() == 0)
&&($stmte->rowCount() == 0))
{
if(file_exists('error.php') == true){require('error.php');}
}

macht genau das was es soll!

 

[bdt600]

Ich glaube das auch du nicht richtig gelesen hast oder drücke ich mich wirklich falsch aus?

Ja, du drückst dich falsch aus, aber was noch viel schlimmer ist, du postest nur Auszüge aus deinem PHP-Code. So muss man sich was zusammen reimen und muss raten, wo das Problem liegt.

PS: deine Lösung sieht nicht gerade gut aus, aber wie gesagt, mit den Bruchstücken an Code kann man auch nicht helfen.

 

[Loddar1]

Hi bdt600 was hätte ich denn noch posten sollen für mehr Verständnis?

 

[bdt600]

Hi bdt600 was hätte ich denn noch posten sollen für mehr Verständnis?

was noch viel schlimmer ist, du postest nur Auszüge aus deinem PHP-Code

Damit ist doch alles gesagt.

 

[Divo1984]

Um das mal aufzuklären - du fragst rowCount() nach fetch() ab, wodurch das Ergebnis verfälscht sein kann, da der Cursor so bereits bewegt ist. Mein Lösungsansatz:

<?php
// Prüfen, ob $page gesetzt ist, sonst umleiten
if (empty($page)) {
    header("Location: index.php?page=1");
    exit;
}

// Datenbankabfrage vorbereiten
$stmt = $pdo->prepare("
    SELECT ziel
    FROM menu
    WHERE id = :id
");
$stmt->bindParam(':id', $page, PDO::PARAM_INT);
$stmt->execute();

// Prüfen, ob die Seite existiert
if ($stmt->rowCount() > 0) {
    // Seite existiert, die Inhalte aus der Datenbank holen
    $seite = $stmt->fetch();
    // Inhalt rendern (z.B. innerhalb von <article>)
    echo "<article>";
    echo "Inhalt der Seite: " . htmlspecialchars($seite['ziel']);
    echo "</article>";
} else {
    // Seite existiert nicht, zeige Fehlermeldung an
    if (file_exists('error.php')) {
        require('error.php');
    } else {
        echo "<article>";
        echo "<h2>Seite nicht gefunden</h2>";
        echo "<p>Die angeforderte Seite existiert nicht.</p>";
        echo "</article>";
    }
}
?>

Das war auch der Grund, warum ich dich darauf verwiesen hatte deine SQL-Abfrage zu var_dumpen oder über PHPMyAdmin zu testen.