Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [www]Prosta aplikacja do notatek - Symfony 4.
smk
post 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 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
Go to the top of the page
+Quote Post
markuz
post 2.08.2018, 16:37:51
Post #2





Grupa: Zarejestrowani
Postów: 1 236
Pomógł: 276
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.


--------------------
Go to the top of the page
+Quote Post
nospor
post 2.08.2018, 16:49:22
Post #3





Grupa: Moderatorzy
Postów: 34 263
Pomógł: 5678
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


--------------------

Najlepsze kawałki programistyczne || Dowcipy o informatykach || Forum PHP dla opornych
Klasy: Pager (stronicowanie) | Cache | ShoutBox (Chat) | Widok | Ładne url'e

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
smk
post 2.08.2018, 16:51:35
Post #4





Grupa: Zarejestrowani
Postów: 8
Pomógł: 0
Dołączył: 15.06.2018

Ostrzeżenie: (0%)
-----


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
Go to the top of the page
+Quote Post
nospor
post 2.08.2018, 16:55:25
Post #5





Grupa: Moderatorzy
Postów: 34 263
Pomógł: 5678
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


--------------------

Najlepsze kawałki programistyczne || Dowcipy o informatykach || Forum PHP dla opornych
Klasy: Pager (stronicowanie) | Cache | ShoutBox (Chat) | Widok | Ładne url'e

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
smk
post 2.08.2018, 16:57:35
Post #6





Grupa: Zarejestrowani
Postów: 8
Pomógł: 0
Dołączył: 15.06.2018

Ostrzeżenie: (0%)
-----


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.
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Wersja Lo-Fi Aktualny czas: 15.11.2018 - 16:17