Autorski skrypt "Boards" pod forum |
Autorski skrypt "Boards" pod forum |
15.01.2021, 14:47:56
Post
#1
|
|
Grupa: Zarejestrowani Postów: 1 Pomógł: 0 Dołączył: 15.01.2021 Ostrzeżenie: (0%) |
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 |
|
|
15.01.2021, 15:15:18
Post
#2
|
|
Grupa: Moderatorzy Postów: 36 482 Pomógł: 6303 Dołączył: 27.12.2004 |
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 -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
Wersja Lo-Fi | Aktualny czas: 19.06.2024 - 14:43 |