[www]Prosta aplikacja do notatek - Symfony 4. Opcje

[smk]

Cześć.
Prosiłbym o code review mojej prostej aplikacji do notatek (IMG:style_emoticons/default/smile.gif) 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

[markuz]

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.

[nospor]

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

[smk]

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 (IMG:style_emoticons/default/wink.gif)

[nospor]

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

[smk]

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 (IMG:style_emoticons/default/wink.gif) UserId również dopiero po fakcie zauważyłem - dzięki za informację, bo nie wiedziałem, że Doctrine utworzy właściwe kolumny.