[www]Prosta aplikacja do notatek - Symfony 4. |
[www]Prosta aplikacja do notatek - Symfony 4. |
2.08.2018, 16:31:02
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 Kodu w sumie jest mało, trochę było zabawy z jQuery i ajaxem, ale nawet warte są uwagi czy struktura plików jest w miarę dobra itp. Jeśli macie jakieś uwagi, piszcie! https://github.com/selfmadeking/symfony4-notes-app |
|
|
2.08.2018, 16:37:51
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. -------------------- |
|
|
2.08.2018, 16:49:22
Post
#3
|
|
Grupa: Moderatorzy Postów: 36 519 Pomógł: 6308 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 |
|
|
2.08.2018, 16:51:35
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 |
|
|
2.08.2018, 16:55:25
Post
#5
|
|
Grupa: Moderatorzy Postów: 36 519 Pomógł: 6308 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 |
|
|
2.08.2018, 16:57:35
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 UserId również dopiero po fakcie zauważyłem - dzięki za informację, bo nie wiedziałem, że Doctrine utworzy właściwe kolumny. |
|
|
Wersja Lo-Fi | Aktualny czas: 26.09.2024 - 21:19 |