Autorski skrypt "Boards" pod forum Opcje

[PanKrok]

Cześć,


Chciałem zapytać o opinię mojego hobbistycznego projektu forum internetowego opartego na SLIM4, potrzebuję feedbacku co poprawić, co zmienić, co jest ok (IMG:style_emoticons/default/smile.gif)

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

[nospor]

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'] (IMG:style_emoticons/default/questionmark.gif) 1;
a:
$page = (int) ($arg['page'] (IMG:style_emoticons/default/questionmark.gif) 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