First Implementation #12
Reference in New Issue
Block a user
Delete Branch "Registrierung"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Hi, magst du bei zukünftigen PRs den Titel so wie die Aufgabe nennen?
Ich finde die Implementation bis dato richtig gut!
Ich habe meist nur formale sachen Kommentiert.
Schön, dass du an die Rückmeldungen an den Nutzer und an das Überprüfen doppelter Email-Adressen denkst.
@@ -0,0 +2,4 @@session_start();require_once "includes/UserDAO.php";$dao = new UserDAO();Nicht das DAO instanziieren. Das DAO ist nur ein Interface. Bitte eine Klasse z.B LocalUser aus dem UserDAO-Interface erstellen und dieses instamziieren.
@@ -1,32 +1,92 @@<?phpsession_start();Statt jedes mal session_start() aufzurufen, kann in einer index-controller.php einmalig die session gestartet werden.
@@ -1,0 +33,4 @@$error = "Login fehlgeschlagen. Bitte überprüfe deine Eingaben.";}}?>Bitte php und das Formular trennen z.B. in eine register-controller.php im ordner php auslagern (s. dibos Musterbeispiel03).
@@ -0,0 +1,6 @@<?phpsession_start();ist die session_start() notwendig? sie läuft doch schon bereits oder?
@@ -13,3 +24,1 @@<input type="text" name="Vorname" class="login-input" requiredplaceholder="Vorname"value="<?php //DB-Daten ?>"><form method="post" action="index.php?pfad=profile">Vielleicht auch hier php und Formular trennen. Z.B. eine profile-controller.php erstellen (s. o.).
@@ -1,0 +37,4 @@exit();}}?>auch hier die php in eine eigene register-controller.php schieben...
@@ -0,0 +1,7 @@[Der Ansatz mit der .json ist richtig gut! Das klaue ich mir auch für meine Aufgabe
@@ -0,0 +1,64 @@<?phprequire_once "UserDAOInterface.php";class UserDAO implements UserDAOInterface {Ich würde die Klasse eher LocalUser oder UserFix nennen und das Interface UserDAO
@@ -0,0 +1,7 @@<?phpinterface UserDAOInterface {s.o. -> Lieber UserDAO nennen...
@@ -0,0 +1,7 @@<?phpinterface UserDAOInterface {public function findUser($email);Bitte jede Methode kommentieren!
Was macht die Methode? -> Funktion
Welche Eingabe wird erwartet?
Was wird ausgegeben?
Schau sonst mal in meinem Branch "BeitragErstellen" vorbei.
@@ -1,23 +1,37 @@<?phpsession_start();muss wirklich überall die session nochmal gestartet werden? Reicht es nicht, in der index.php bzw. in einer index-controller.php diese einmal zu starten?
Pull request closed