Ocena systemu Logowanie/Rejestracja PHP

Question

Cześć, mógłbym prosić o ocenę mojego systemu rejestracji/logowania?

Link: Github

Z góry dziękuję.

Comments

Warunki post skracaj .. 

$array1=['imie','nazwisko','email'];
$array2=['imie','nazwisko','email'];
 echo ($array1===$array2)?? false;

pierwsza to pola wymagane, druga to post.

 

---

Jaki sens ma zamknięcie tematu z oceną? Ocena to subiektywna opinia poszczególnych ludzi – to raz. Dwa – nie zawsze jedna osoba jest w stanie wszystko opisać i przejrzeć. Zamykając tego typu temat po uzyskaniu jednej opinii ograniczasz sobie tak naprawdę horyzont poznawczy.

Answer 1

System to zdecydowanie za dużo powiedziane.

1. Dlaczego nie używasz autoloadingu i namespaceów?
2. W register robisz require config, a w configu require register. Dobrze ze jest once dodane.
3. Nazwa Login to dla mnie po prostu login. Na pewno nie kojarzy się z procesem logowania. LoginAction byłoby o wiele lepsze. To samo register.
4. Prezentujesz bardzo charakterystyczny styl kodowania. Tzw egipski. Polega on na utworzeniu najwyższej piramidy z ifów. Można te ify kulturalnie napisać pod sobą.
5. Co ma login do bazy danych?
6. Pisz dokumentację.
7. csrf protection brak.

Podsumowując. Fajnie że próbujesz, korzystasz z git (huba przynajmniej). Uwierz mi jednak, że takie mechanizmy już powstały i są bardziej zaawansowane.

To co Ci mogę polecić to złap się za jakiś framework php. Zamiast skupiać się i marnować czas na kolejnym systemie walidacji (który nie będzie bezbłędny) nauczysz się jak naginać gotowe komponenty do swoich potrzeb i tworzyć stabilne oraz fajne aplikacje  

Comments

Dzięki za pomoc ;)

---

Niech napisze to jeszcze raz, brak mu podstaw do frameworka.

---

8. Zainteresuj się systemem szablonów, np. Twig.

Podziękujesz potem.