Drukowana wersja tematu

Kliknij tu, aby zobaczyć temat w orginalnym formacie

Forum PHP.pl _ Oceny _ Autorski skrypt "Boards" pod forum

Napisany przez: PanKrok 15.01.2021, 14:47:56

Cześć,

Chciałem zapytać o opinię mojego hobbistycznego projektu forum internetowego opartego na SLIM4, potrzebuję feedbacku co poprawić, co zmienić, co jest ok

Tematyka: Forum internetowe
Demo: https://boards-dev.ct8.pl
GitHub: https://github.com/pankrok/boards

Napisany przez: nospor 15.01.2021, 15:15:18

Czesc akcji w ogole nie dziala, jak np stworzenie watku oraz inne.

Stosuj sie do PSR
Uzywaj stalych wciec. Niektory pliki wygladaja jak jakas choinka
Uzywaj php-cs-fixer to pomoze ci zapanowac nad tym balaganem

Tak sie nie pisze
$plotRead->timeline >= strtotime($plot['created_at'])) ? $unread = true : $unread = false;

Tak sie pisze
$unread = $plotRead->timeline >= strtotime($plot['created_at']);
Prawda ze czytelniej?

Zakladam ze uzywac php7. Uzywaj wiec pelnego typowania parametrow w funkcjach

Jak cos ma byc INT to ma to byc zawsze INT
NIe
$page = $arg['page'] 1;
a:
$page = (int) ($arg['page'] 1);

Do porownania uzywaj === a nie ==

Strona jak ta
https://boards-dev.ct8.pl/user/PanKrok/1000000

powinna zwrocic 404 a nie strone z pustym uzytkownikiem

Strona jak
https://boards-dev.ct8.pl/user/PanKrok/1000000 or 1=1
na pewno nie powinna zwrocic mi bledu o braku jezyka

Napisany przez: LowiczakPL 16.01.2021, 09:52:31

Do poprawy tryb dzień/noc nie działa poprawnie.

Nie może być tak że user zmienia sobie na noc a podczas przełączania stron najpierw apka domyślnie ustawia dzień po czym sprawdza że jednak user wybrał noc to przełącza na noc, dzięki temu jest stroboskop.

Napisany przez: gino 21.01.2021, 15:06:30

1. albo user albo e-mail a wygląda na to że chcesz walidować tylko e-mail, popraw chciaż opis pola
2. zakładanie konta - dorobiłbym osobiście potwierdzenie założenia konta na e-mial (dokończenie rejestracji)
3. trochę ciężko nawiguje się po zalogowaniu, chwile musiałem szukać pola gdzie wpisać wiadomość, submit na ENTER mnie nie przekonuje, odruchowo można chcieć przejść do następnej linii a tutaj koniec. Po to jest button.
4. ogólnie trochę nieład na stronie i ciężko się połapać gdzie się jest