Drukowana wersja tematu

Kliknij tu, aby zobaczyć temat w orginalnym formacie

Forum PHP.pl _ Oceny _ [www] Laravel - Prośba o przejrzenie oraz uwagi

Napisany przez: maurepass 16.07.2019, 18:55:06

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

Napisany przez: nospor 16.07.2019, 19:16:38

- 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

Napisany przez: Pyton_000 16.07.2019, 19:42:10

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)

Napisany przez: nospor 17.07.2019, 10:13:23

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

Napisany przez: Pyton_000 17.07.2019, 10:15:48

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

Napisany przez: 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

Napisany przez: dublinka 23.07.2019, 13:36:05

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))

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)