Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> [www] Laravel - Prośba o przejrzenie oraz uwagi
maurepass
post 16.07.2019, 18:55:06
Post #1





Grupa: Zarejestrowani
Postów: 1
Pomógł: 0
Dołączył: 15.07.2019

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


Cześć

Od pewnego czasu w ramach uczenia się programowania rozwijam aplikację którą używamy u mnie w pracy.
Co jakiś czas jak zdobędę nową wiedzę to ją poprawiam.
Pod względem funkcjonalnym spełnia pokładane w niej oczekiwania.

Zastanawiam się jak się ma napisany przeze mnie kod do praktyk stosowanych przez osoby z dużo większym doświadczeniem oraz w których miejscach można ten kod napisać lepiej.

Będę wdzięczny za wszelkie uwagi

https://github.com/maurepass/metsoft_laravel

Pozdrawiam
Go to the top of the page
+Quote Post
nospor
post 16.07.2019, 19:16:38
Post #2





Grupa: Moderatorzy
Postów: 36 440
Pomógł: 6289
Dołączył: 27.12.2004




- Brak typowania paramatrow oraz zwrotek metod

- porownania bez typowania == rowniez nie napawaja optymizmem

- mieszanie polskiego i angielskego nazewnictwa. zdecyduj sie na jeden i sie go trzymaj. najlepiej zdecyduj sie na angielski

- nie pracowalem z laravel ale z tego co widze to masz podatnosci na ataki SQLInjection w kodach ala
$offer_no = '%'.$offer_no.'%';
- 'date_stats_from', "date_stats_to" - czemu raz apostrof a raz cudzyslow i to zaraz obok? Uzywaj najlepiej apostrofu ciagle dla takich przypadkow jak tutaj

- brak testow

- brak okreslania wielkosci kolumn tekstowych


Zainteresuj sie tez programami jak php-cs-fixer oraz psalm i phpstan. Pomoga ci poprawic jakosc kodu


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

"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
Pyton_000
post 16.07.2019, 19:42:10
Post #3





Grupa: Zarejestrowani
Postów: 8 068
Pomógł: 1414
Dołączył: 26.10.2005

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


Jak na kod Laravel to nie jest źle. Widać że uczyłeś się z tutoriali o stricte Laravelu. Jak na początek to jest fajnie.

Możesz teraz poczytać o Dependency Injection i zacząć stosować (Głównie chodzi o zamianę statycznych metod wstrzykiwaniem zależności do konstruktora/metody)
Go to the top of the page
+Quote Post
nospor
post 17.07.2019, 10:13:23
Post #4





Grupa: Moderatorzy
Postów: 36 440
Pomógł: 6289
Dołączył: 27.12.2004




Cytat
Jak na kod Laravel to nie jest źle.
Hm, czyli jednak przyznajesz sam ze w laravel nie da sie nic porzadnie napisac? wink.gif


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

"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
Pyton_000
post 17.07.2019, 10:15:48
Post #5





Grupa: Zarejestrowani
Postów: 8 068
Pomógł: 1414
Dołączył: 26.10.2005

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


Nie, raczej miałem na myśli że praktyczne większość kursów Laravela promuje taki styl pisania, co nie jest złe dla małych projektów ale dla większych już tak. Więc wg. konwencji Laravela to jest spoko smile.gif Wg. konwencji Symfo9ny nie smile.gif
Go to the top of the page
+Quote Post
nospor
post 17.07.2019, 10:33:28
Post #6





Grupa: Moderatorzy
Postów: 36 440
Pomógł: 6289
Dołączył: 27.12.2004




A juz sie ludzilem ze poszedles po rozum do glowy* wink.gif



*jakby ktos nie wiedzial to my z Pytonem tak sobie dogadujemy od czasu do czasu i nie ze ja go tutaj niby obrazam wink.gif


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

"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
dublinka
post 23.07.2019, 13:36:05
Post #7





Grupa: Zarejestrowani
Postów: 594
Pomógł: 66
Dołączył: 22.02.2008
Skąd: Dublin

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


Cytat(nospor @ 17.07.2019, 10:33:28 ) *
A juz sie ludzilem ze poszedles po rozum do glowy* wink.gif



*jakby ktos nie wiedzial to my z Pytonem tak sobie dogadujemy od czasu do czasu i nie ze ja go tutaj niby obrazam wink.gif


taaa....jedzccie razem na wakacje z Waszymi kobietami to zobaczymy smile.gif))

Ten post edytował dublinka 23.07.2019, 13:36:27


--------------------
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: 28.03.2024 - 09:13