Forum PHP.pl [Powered by Invision Power Board]

Napisany przez: smk 2.08.2018, 16:31:02

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

Napisany przez: markuz 2.08.2018, 16:37:51

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.

Napisany przez: nospor 2.08.2018, 16:49:22

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

Napisany przez: smk 2.08.2018, 16:51:35

Cytat(nospor @ 2.08.2018, 17:49:22 )

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

Napisany przez: nospor 2.08.2018, 16:55:25

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

Napisany przez: smk 2.08.2018, 16:57:35

Cytat(nospor @ 2.08.2018, 17:55:25 )

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.

Drukowana wersja tematu