Login - System

11/26/2016 20:50 Mad0ck#1
closed
11/26/2016 21:36 Syc#2
lass dir vor jeder if-abfrage einfach die entsprechenden werte ausgeben. dann siehst du ja wo das problem liegt
12/11/2016 19:12 #Givenchy*#3
Falls du noch Hilfe brauchst kannst du mich anschreiben.

Würde dir noch empfehlen trim zu verwenden.

mfg
12/11/2016 19:26 Sycrog#4
Quote:
Mit der Abfrage if ($_SERVER["REQUEST_METHOD"] == "POST") {
wird die $error irgendwie immer auf true gesetzt und so erstellt er mir nie einen DB Eintrag auch wenn alle Felder ausgefüllt sind.
( Wenn alle ausgefüllt sind, sollte er eigentlich nie $error auf true setzen )
Die If-Abfrage weist keinen Wert zu, $error wird dadurch nicht verändert.
Übrigens kannst du auch einfach das globale array $_POST prüfen: if(!empty($_POST))

Für Usereingaben würde ich dir [Only registered and activated users can see links. Click Here To Register...] empfehlen.

Quote:
$vornameErr = $nachnameErr = $emailErr = $passwortErr ="";
$vorname = $nachname = $email = $passwort = $passwort2="";
1. Zeile kannst du einfacher mit einem Array umsetzen, dass je nach Fehler entsprechend gefüllt wird.
2. Zeile setzt du ja sowieso, optimalerweise mit filter_input()

Anmerkungen:
- Die Ausgabe der Fehler fehlt.
- Du überprüfst nirgends, ob der User schon existiert. Kann zu Problemen führen, wenn die email bspw. unique ist.
12/12/2016 21:14 Mad0ck#5
Quote:
Originally Posted by Sycrog View Post
Die If-Abfrage weist keinen Wert zu, $error wird dadurch nicht verändert.
Übrigens kannst du auch einfach das gloable array $_POST prüfugen: if(!empty($_POST))

Für Usereingaben würde ich dir [Only registered and activated users can see links. Click Here To Register...] empfehlen.



1. Zeile kannst du einfacher mit einem Array umsetzen, dass je nach Fehler entsprechend gefüllt wird.
2. Zeile setzt du ja sowieso, optimalerweise mit filter_input()

Anmerkungen:
- Die Ausgabe der Fehler fehlt.
- Du überprüfst nirgends, ob der User schon existiert. Kann zu Problemen führen, wenn die email bspw. unique ist.

Danke für dein Feedback :)
Das weiß ich sehr zu schätzen !

Mittlerweile wurde jedoch von mir schon alles anders realisiert, da ich mittlerweile auch schon andere Wege und Mittel kenne ^-^


lg
Mad0ck

Quote:
Originally Posted by #Givenchy* View Post
Falls du noch Hilfe brauchst kannst du mich anschreiben.

Würde dir noch empfehlen trim zu verwenden.

mfg

Falls mal andere Probleme anfallen meld ich mich :*

Danke für dein Angebot =)

Derzeit bin ich jedoch in andere Sachen verwickelt wie zB Empfangen von Datenpacketen via Wlan auf dem Rasp Pi
lg
12/13/2016 13:30 Sycrog#6
Quote:
Originally Posted by Mad0ck View Post
Danke für dein Feedback :)
Das weiß ich sehr zu schätzen !

Mittlerweile wurde jedoch von mir schon alles anders realisiert, da ich mittlerweile auch schon andere Wege und Mittel kenne ^-^
Der Fortschritt ging aber schnell. Falls du Feedback zu deiner neuen Lösung möchtest, kannst du mir gerne eine PN schicken.
12/13/2016 17:39 Mad0ck#7
Quote:
Originally Posted by Sycrog View Post
Der Fortschritt ging aber schnell. Falls du Feedback zu deiner neuen Lösung möchtest, kannst du mir gerne eine PN schicken.
Komplett neu wurde es auch nicht gemacht =)
Das Rad wird ja auch nicht jedes mal neu erfunden ^^

Jedoch hat sich beim html code etwas geändert und wie du auch schon sagtest , wurde eine Variable nie verändert,da ich nen dummen Fehler gemacht habe :)

Kann dir heute Abend sonst spätestens morgen Vormittag den neuen Code schicken.
Wenn du mal Zeit hast würd ich dumm sein sowas abzulehen !

Gibt sicher noch einiges was man verbessern kann :§

lg