Login / Registrierung von Usern

Da ich in den Thema auch noch an lernen bin gebe ich meine Antwort als Frage raus weil ich es erstmal so versuchen würde bevor ich da eine Extra SELECT abfrage mache.

Würde es nicht reichen die Spalte mit den securityid auf Unique/Primary Key zu setzen ????

Hallo zusammen,

zu dem Code: wenn ich es richtig sehe nutzt fetch() dies liefert laut der Docu nur im Fehlerfall ein False zurück, ansonsten (je nach Einstellung) das Resultset

http://php.net/manual/en/pdostatement.fetch.php

D.h. du solltest hier prüfen wie viele Einträge mit der sid zurückkommen.

Dafür kannst du auch rowCount nutzen: http://php.net/manual/en/pdostatement.rowcount.php

basti1012

Auch dein vorgehen ist gängig, wenn wir prinzipiell davon ausgehen das es besser ist solches verhalten über Exceptions zu handeln (was es im bereich von OOP ist), dann würde hier nur ein Insert gestartet werden und dann eine PDOException gefangen.

D.h. in MySql das Feld auf UNIQUE stellen und nach dem Insert per Try Catch prüfen

<?php

try {
   $prep->execute($values);
 
} catch (PDOException $e) {
   if ($e->errorInfo[1] == 1062) {
     // doppelter eintrag
   } else {
      // anderer FEhler
   }
}

Der Fehlercode 1062 wäre laut der List: https://www.fromdual.com/mysql…es-and-messages-1050-1099 Duplicate entry.

ACHTUNG: Das ist Datenbank spezifisch, ich vermute man könnte den Fehler auch generisch finden, weiß gerade nur nicht wie.

Grüße

Timo

Damit die try/catch Variante sicher funktioniert, sollte sichergestellt sein, dass die Objekt-Instanziierung von PDO mit der entsprechenden Eigenschaft PDO::ATTR_ERRMODE auf PDO::ERRMODE_EXCEPTION gesetzt ist. Sollte PDO nicht auf Exceptions eingestellt sein, bietet das PDOStatement-Objekt auch von Haus aus eine Fehler-Behandlungsmöglichkeit: http://php.net/manual/de/pdostatement.errorcode.php

Ich habe mir das Fetch auch nochmal angeschaut, tatsächlich liefert das bei keinem Ergebnis False zurück, d.h. im Standard, liefert Fetch bei einem SQL Fehler UND bei leerem Result ein False.

Jetzt wäre meine Frage, was funktioniert denn nicht? Ist es bei deinem sid test immer false also kann sich jeder user auch mit existierender sid registrieren?

Falls ja: dann solltest du mal prüfen ob dort sql error zurückgegeben werden. Falls nein, musst du glaube ich nochmal genauer erklären, was nicht funktioniert.

Aber dennoch würde ich fetch nicht so benutzen (da ich es nicht so benutze und so wie Arne Drews geschrieben hat, nur mit dem ERRMODE_EXCEPTION) ist mir das bisher noch nicht aufgefallen. Ein RowCount zu machen um zu prüfen ob es existiert wäre die bessere alternative.

Grüße

Timo

Wenn Du das für die bessere Alternative hältst, dann erwähne aber auch, dass PDOStatement::rowCount() für SELECT Queries nur abhängig vom DBMS vertrauenswürdige Ergebnisse liefert. Ich halte das nicht für die bessere Wahl, anstatt Fehler abzufragen, wie es gedacht ist.

Arne Drews das ist auch richtig. Das mit Fetch ist eigentlich auch der Richtige weg. PHP und PDO sind furchtbar. Hab mir nebenher nochmal die Doku und einige eingräte auf StackOverflow angeschaut, scheint aber auch nicht das Stabilste Construct zu sein.

MariusB Dann vermute ich eher, das du irgendwo in der Abarbeitung einen Logik Fehler hast. Ich kann frühestens heute Abend das ganze nochmal genau anschauen. Vielleicht kannst du den gesamten Code Posten und die Tabellen. Ich weiß was du versuchst und ich weiß auch das es heir einfachere oder Übersichtlichere Wege gibt das zu lösen.

Grüße

Timo

Zitat von timtim

PHP und PDO sind furchtbar.

WHAT?! Na wenn Du meinst...

Arne Drews

Ich bin mittlerweile im Enterprise Umfeld unterwegs, viel Azure, hauptsächlich .NET / Node.js. Wenn man sich an vieles dort gewöhnt hat und ich alle 2-3 Jahre mal wieder en PHP Projekt anfange, dann frage ich mich immer warum (mache es aber dennoch). Script sprachen vs. Application Sprachen.

Ich arbeite hauptberuflich auch im .NET Umfeld, PHP ist nur ein Hobby, aber die Behauptung PHP und PDO sei furchtbar ist imho unpassend.

Du befindest Dich hier im PHP-Bereich, da kannst Du erstmal nicht von .NET und Azure ausgehen. PDO ist für PHP bspw. aus meiner Sicht eine der besten Erweiterungen der letzten Jahre.

Aber ich teile Deine Meinung, dass man Script-Sprachen ( bspw. PHP ) und "Application"-Sprachen ( bspw. C# ) nicht vergleichen kann/sollte.

Genau deshalb finde ich aber die Beurteilung über PHP und PDO unpassend an der Stelle. Für die Umsetzung von Webprojekten ist PHP eine akzeptable Wahl und in Bezug auf Datenbanken dann PDO eine der besten.

MariusB

Ich werde versuchen mir das heute Abend genauer anzuschauen und durchzuspielen. Kann dir dann bestimmt helfen eine Lösung zu finden

Arne Drews Das ist richtig, ich nutze es deswegen ja auch für hobby Projekte. Scherzhaft sage ich auch nur, das alle Programmiersprachen furchtbar sind und ich die nutze, welche einem am wenigsten furchtbar erscheinen. Das oben, ging auch eher wieder an die Standard-Konfiguration und das vorgehen das Rückgaben "mixed" sind mit einer Inkonsequenten (und zum teil schlecht dokumentierten) Logik. Zudem, fällt es mir schwer, kein konsequentes OOP und Enterprise Architekturen zu verwenden und PHP zwingt einem manchmal da Brüche vorzunehmen . 60% meines heutigen codes sind in JS/TS und das mag ich weit weniger als PHP, dennoch mache ich damit fast alles.

Naja, PHP bleibt typschwach, von daher ist mixed so ziemlich die genaueste Angabe der Rückgabe. Wäre diese fest typisiert, müsste PHP Overloading, wie echte Programmiersprachen beherrschen. Manche sind zwar der Meinung, dass es das kann, aber meine Meinung dazu ist: Nein.

Dein weg geht schon, kann es sein, das du lediglich in dem teil hier:

<?php
if(!$error) {
        $statement = $pdo->prepare("SELECT * FROM securityid WHERE sid = :sid");
        $result = $statement->execute(array('sid' => $sid));
        $user = $statement->fetch();
        if($user !== false) {
            echo 'ID ung��g<br>';
            $error = false;
            echo 'ID g��g<br>';
        }    
    }

//Das machen musst
if(!$error) {
        $statement = $pdo->prepare("SELECT * FROM securityid WHERE sid = :sid");
        $result = $statement->execute(array('sid' => $sid));
        $user = $statement->fetch();
        if($user === false) {//<--- änderung hier
            echo 'ID ung��g<br>';
            $error = true;<-- änderung hier
        }    
    }

Weil, es gibt "FALSE" zurück, wenn die sid nicht existiert. Damit willst du es ja beenden wenn ->fetch() false zurück gibt (keine reihe gefunden).

Grüße

Timo

In meiner Korrektur waren keine zwei Echos und registrieren kann er nicht wenn ein error kommt... dann hast du die Anpassungen nicht übernommen. Habe sie extra markiert.

Und dann solltest du noch prüfen ob PDO ein Fehler in der Abfrage hat.

Grüße

Timo