Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www]Prosta aplikacja do notatek - Symfony 4.
Forum PHP.pl > Inne > Oceny
smk
Cześć.
Prosiłbym o code review mojej prostej aplikacji do notatek 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
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 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
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 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.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.