Co sądzicie o tym skrypcie rejestracji i logowania, Temat z ankietą Opcje

[Zodiak]

Oto on na innym forum dostałem radę żeby go uszczuplić, niestety nie powiedziano mi "z czego go uszczuplić", jak na moje, należy wywalić kod css, no bo po co on? Ale czy kod php jest w porządku? Czy da się i jak stworzyć tam treści dostępne tylko dla zarejestrowanych?

Miło by mi było gdyby ktoś przejrzał i napisał co można tam polepszyć, a może lepiej napisać własny od nowa?

[blooregard]

Miło by mi było gdyby ktoś przejrzał i napisał co można tam polepszyć, a może lepiej napisać własny od nowa?

Znaczy się to nie Twój skrypt?

Co mi się nasuwa po pobieżnym przejrzeniu (że działa w miarę poprawnie - świadczą komentarze pod)

1. Nie jest obiektowy (porównaj sobie np. z modułem sfGuard z Symfony)
2. Całkowity brak separacji warstwy logiki od prezentacji, fragmenty HTML "echowane" - masakra przy jakichkolwiek modyfikacjach
3. Wszystkie wartości (np. długość loginu) ustawione na sztywno - lepsze byłoby rozwiązanie np. ze stałymi
4. Narzuca odgórnie pewne rozwiązania techniczne (m.in. strukturę tabel w bazie), przez co integracja z istniejącymi serwisami może być problematyczna

Generalnie - dla amatora do prostych serwisów może być, ale jak dla mnie - bezużyteczny

[Zodiak]

2. Całkowity brak separacji warstwy logiki od prezentacji, fragmenty HTML "echowane" - masakra przy jakichkolwiek modyfikacjach

OK, powiedzmy że wezmę smarty i na tym oprę rejestrację, ale nie widzę tam żadnych zmiennych od rejestracji, hmm, jak byś polecał zrobić to najprościej w oparciu o smarty(lub inny system szablonów)?