First Implementation #12

Closed
caroline.slt wants to merge 0 commits from Registrierung into dev
Collaborator
No description provided.
caroline.slt added 4 commits 2026-05-26 23:19:36 +02:00
Owner

Hi, magst du bei zukünftigen PRs den Titel so wie die Aufgabe nennen?

Hi, magst du bei zukünftigen PRs den Titel so wie die Aufgabe nennen?
caroline.slt added 1 commit 2026-05-27 19:26:02 +02:00
niklas.ortmann requested changes 2026-05-28 11:12:14 +02:00
niklas.ortmann left a comment
Owner

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.

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();
Owner

Nicht das DAO instanziieren. Das DAO ist nur ein Interface. Bitte eine Klasse z.B LocalUser aus dem UserDAO-Interface erstellen und dieses instamziieren.

Nicht das DAO instanziieren. Das DAO ist nur ein Interface. Bitte eine Klasse z.B LocalUser aus dem UserDAO-Interface erstellen und dieses instamziieren.
caroline.slt marked this conversation as resolved
@@ -1,32 +1,92 @@
<?php
session_start();
Owner

Statt jedes mal session_start() aufzurufen, kann in einer index-controller.php einmalig die session gestartet werden.

Statt jedes mal session_start() aufzurufen, kann in einer index-controller.php einmalig die session gestartet werden.
caroline.slt marked this conversation as resolved
@@ -1,0 +33,4 @@
$error = "Login fehlgeschlagen. Bitte überprüfe deine Eingaben.";
}
}
?>
Owner

Bitte php und das Formular trennen z.B. in eine register-controller.php im ordner php auslagern (s. dibos Musterbeispiel03).

Bitte php und das Formular trennen z.B. in eine register-controller.php im ordner php auslagern (s. dibos Musterbeispiel03).
caroline.slt marked this conversation as resolved
@@ -0,0 +1,6 @@
<?php
session_start();
Owner

ist die session_start() notwendig? sie läuft doch schon bereits oder?

ist die session_start() notwendig? sie läuft doch schon bereits oder?
caroline.slt marked this conversation as resolved
@@ -13,3 +24,1 @@
<input type="text" name="Vorname" class="login-input" required
placeholder="Vorname"
value="<?php //DB-Daten ?>">
<form method="post" action="index.php?pfad=profile">
Owner

Vielleicht auch hier php und Formular trennen. Z.B. eine profile-controller.php erstellen (s. o.).

Vielleicht auch hier php und Formular trennen. Z.B. eine profile-controller.php erstellen (s. o.).
caroline.slt marked this conversation as resolved
@@ -1,0 +37,4 @@
exit();
}
}
?>
Owner

auch hier die php in eine eigene register-controller.php schieben...

auch hier die php in eine eigene register-controller.php schieben...
caroline.slt marked this conversation as resolved
data/users.json Outdated
@@ -0,0 +1,7 @@
[
Owner

Der Ansatz mit der .json ist richtig gut! Das klaue ich mir auch für meine Aufgabe

Der Ansatz mit der .json ist richtig gut! Das klaue ich mir auch für meine Aufgabe
caroline.slt marked this conversation as resolved
@@ -0,0 +1,64 @@
<?php
require_once "UserDAOInterface.php";
class UserDAO implements UserDAOInterface {
Owner

Ich würde die Klasse eher LocalUser oder UserFix nennen und das Interface UserDAO

Ich würde die Klasse eher LocalUser oder UserFix nennen und das Interface UserDAO
caroline.slt marked this conversation as resolved
@@ -0,0 +1,7 @@
<?php
interface UserDAOInterface {
Owner

s.o. -> Lieber UserDAO nennen...

s.o. -> Lieber UserDAO nennen...
caroline.slt marked this conversation as resolved
@@ -0,0 +1,7 @@
<?php
interface UserDAOInterface {
public function findUser($email);
Owner

Bitte jede Methode kommentieren!
Was macht die Methode? -> Funktion
Welche Eingabe wird erwartet?
Was wird ausgegeben?

Schau sonst mal in meinem Branch "BeitragErstellen" vorbei.

Bitte jede Methode kommentieren! Was macht die Methode? -> Funktion Welche Eingabe wird erwartet? Was wird ausgegeben? Schau sonst mal in meinem Branch "BeitragErstellen" vorbei.
caroline.slt marked this conversation as resolved
@@ -1,23 +1,37 @@
<?php
session_start();
Owner

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?

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?
caroline.slt marked this conversation as resolved
caroline.slt added 1 commit 2026-05-31 21:35:58 +02:00
caroline.slt added 1 commit 2026-05-31 21:42:49 +02:00
caroline.slt added 1 commit 2026-05-31 21:53:35 +02:00
caroline.slt added 1 commit 2026-05-31 22:21:55 +02:00
caroline.slt added 1 commit 2026-05-31 22:26:44 +02:00
caroline.slt added 1 commit 2026-05-31 22:34:19 +02:00
caroline.slt added 1 commit 2026-05-31 22:38:32 +02:00
caroline.slt added 1 commit 2026-05-31 22:42:20 +02:00
caroline.slt added 1 commit 2026-05-31 22:49:17 +02:00
caroline.slt added 1 commit 2026-05-31 22:54:02 +02:00
caroline.slt closed this pull request 2026-05-31 22:55:11 +02:00
caroline.slt reopened this pull request 2026-05-31 22:55:15 +02:00
niklas.ortmann added 1 commit 2026-06-01 22:48:42 +02:00
niklas.ortmann added 1 commit 2026-06-01 22:51:50 +02:00
niklas.ortmann added 1 commit 2026-06-01 22:52:45 +02:00
niklas.ortmann added 1 commit 2026-06-01 22:53:30 +02:00
niklas.ortmann closed this pull request 2026-06-01 23:04:54 +02:00

Pull request closed

Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: niklas.ortmann/webprogrammierung#12