Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [www] Laravel - Prośba o przejrzenie oraz uwagi
Forum PHP.pl > Inne > Oceny
maurepass
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
nospor
- 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
Pyton_000
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)
nospor
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
Pyton_000
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
nospor
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
dublinka
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))
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-2019 Invision Power Services, Inc.