![]() |
![]() ![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 15.06.2018 Ostrzeżenie: (0%) ![]() ![]() |
Cześć.
Prosiłbym o code review mojej prostej aplikacji do notatek ![]() https://github.com/selfmadeking/symfony4-notes-app |
|
|
![]()
Post
#2
|
|
Grupa: Zarejestrowani Postów: 1 240 Pomógł: 278 Dołączył: 11.03.2008 Ostrzeżenie: (0%) ![]() ![]() |
Usuń katalog .idea z repozytorium
Resztę ciężko ocenić, widać, że część kodu jest kopiowana albo lubisz pisać komentarze po angielsku. Możesz zainteresować się testami jednostkowymi a najlepiej TDD. -------------------- |
|
|
![]()
Post
#3
|
|
![]() Grupa: Moderatorzy Postów: 36 477 Pomógł: 6301 Dołączył: 27.12.2004 ![]() |
Usuwaj tez prosze takie komentarze
/** * Created by PhpStorm. * User: KRÓL ŻYCIA * Date: 21.07.2018 * Time: 15:50 */ Sa totalnie zbedne. PRacujac na git widac kto i kiedy co tworzyl jak uzywasz krotkiej notacji array to sie jej trzymaj a nie widze $forms = array(); ma byc $forms = []; pliki .gitignore w katalogach ktore chcesz by byly w git? Wywal je -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
![]()
Post
#4
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 15.06.2018 Ostrzeżenie: (0%) ![]() ![]() |
Usuwaj tez prosze takie komentarze /** * Created by PhpStorm. * User: KRÓL ŻYCIA * Date: 21.07.2018 * Time: 15:50 */ Sa totalnie zbedne. PRacujac na git widac kto i kiedy co tworzyl jak uzywasz krotkiej notacji array to sie jej trzymaj a nie widze $forms = array(); ma byc $forms = []; pliki .gitignore w katalogach ktore chcesz by byly w git? Wywal je Te komentarze created by to muszę zobaczyć w ustawieniach phpstorma, aby je wyłączyć. Tablice racja. Z .gitignore nie ruszałem nic, jak Symfony podobno samo dobrze generuje ten plik. Jak widać nie do końca ![]() |
|
|
![]()
Post
#5
|
|
![]() Grupa: Moderatorzy Postów: 36 477 Pomógł: 6301 Dołączył: 27.12.2004 ![]() |
nawet jak ci phpstorm doda komentarz to mozesz go recznie usunac. On naprawde na stale tam nie jest wstawiany.
Trzymaj sie PSR nie zadne user_id a userId. Doctrine utworzy ci wlasciwe kolumny user_id -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
![]()
Post
#6
|
|
Grupa: Zarejestrowani Postów: 8 Pomógł: 0 Dołączył: 15.06.2018 Ostrzeżenie: (0%) ![]() ![]() |
nawet jak ci phpstorm doda komentarz to mozesz go recznie usunac. On naprawde na stale tam nie jest wstawiany. Trzymaj sie PSR nie zadne user_id a userId. Doctrine utworzy ci wlasciwe kolumny user_id Ustawienia phpstorma już zmienione ![]() |
|
|
![]() ![]() |
![]() |
Wersja Lo-Fi | Aktualny czas: 10.06.2024 - 21:49 |