[www] Laravel - Prośba o przejrzenie oraz uwagi Opcje

[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