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

- 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

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)

Hm, czyli jednak przyznajesz sam ze w laravel nie da sie nic porzadnie napisac?

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 Wg. konwencji Symfo9ny nie

A juz sie ludzilem ze poszedles po rozum do glowy*



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

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