Was habe ich für einen Denkfehler?

[nils320]

Hallo,
ich habe ein Ticket-System programmiert.
Um dieses abzusichern habe ich folgenden Code eingebaut:

$query_view_02 = "SELECT * FROM tickets WHERE ID=$ticketid";
$result_view_02 = mysqli_query($verbindung, $query_view_02);
$row_view_02 = mysqli_fetch_array($query_view_02);
$userid_jetzt = $_SESSION['userid'];
if($_SESSION['userid'] != $row_view_02['USER_ID']){

    $error_no_perm = ' <div class="container"><div class="alert alert-danger">Dachtest du echt, dass ich so doof bin? :D <br> Du hast keine Berechtigungen dieses Ticket zu Sehen!</div></div>';
    echo $_SESSION['userid'];
    echo "<br>";

    echo $userid_jetzt;
    echo $error_no_perm;
    header('Location: viewticket.php');
    exit;
}

Es wird folgendes ausgegeben:

So wie der Code geschrieben ist, sollte das aber eigentlich nicht passieren...
Was habe ich falsch gemacht?
Mehrere Augen sehen schließlich mehr Fehler, als nur 2 Augen

Viele Grüße
Nils

 

[Gelöschtes Mitglied 36401]

Da hilft nur

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.

Beim Einsatz von SQL-Datenbanken sollte das Abrufen aller Spalten eines Datensatzes per SELECT * vermieden werden. Stattdessen sollten die gewünschten Spaltennamen explizit aufgelistet werden (SELECT col1, col2 FROM …). Einige Vorteile:

• Häufig werden in einer Abfrage nicht alle Spalten benötigt, die aktuell existieren oder in Zukunft existieren könnten, sodass unnötige Daten übertragen werden.
• Bei expliziter Angabe der gewünschten Spalten können Datenbanksysteme unter Umständen Indizes zur Beantwortung von Abfragen nutzen.
• Eine Auflistung der Spaltennamen macht den Code verständlicher, da die Tabellenstruktur nicht erst im Datenbankschema nachgelesen werden muss.
• Bei Änderungen am Datenbankschema (etwa bei Entfernung oder Umbenennung einer Spalte) schlägt eine Query, die kein SELECT * nutzt, sofort fehl. Das ist hilfreich, da so keine Stelle im Code übersehen werden kann, an der wahrscheinlich auch die PHP-Datenstrukturen an das neue Schema angepasst werden müssen.

 

[sysop]

...
if($_SESSION['userid'] != $row_view_02['USER_ID']){

Offenbar sind die Werte in der Session und der gelesenen Variablen nicht gleich.

Beachte:
9 ist nicht gleich '9' ist nicht gleich '9 ' ist nicht gleich ' 9'

 

[nils320]

Ich habe es jetzt gelöst.
Danke für die Hilfe

Viele Grüße
Nils

 

[nookie]

Da hilft nur

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.

Kommt mir so bekannt vor.(bdt600)

 

[Gelöschtes Mitglied 36401]

Kommt mir so bekannt vor.

Wenn du auch auf phpforum.de unterwegs bist, ist das kein Wunder, habe ich von dort.

(bdt600)

Sollte mir das etwas sagen?