[www] Laravel - Prośba o przejrzenie oraz uwagi |
[www] Laravel - Prośba o przejrzenie oraz uwagi |
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 |
|
|
16.07.2019, 19:16:38
Post
#2
|
|
Grupa: Moderatorzy Postów: 36 519 Pomógł: 6308 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 |
|
|
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) |
|
|
17.07.2019, 10:13:23
Post
#4
|
|
Grupa: Moderatorzy Postów: 36 519 Pomógł: 6308 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?
-------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
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 Wg. konwencji Symfo9ny nie
|
|
|
17.07.2019, 10:33:28
Post
#6
|
|
Grupa: Moderatorzy Postów: 36 519 Pomógł: 6308 Dołączył: 27.12.2004 |
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 -------------------- "Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista "Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer |
|
|
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%) |
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 )) Ten post edytował dublinka 23.07.2019, 13:36:27 -------------------- |
|
|
Wersja Lo-Fi | Aktualny czas: 24.09.2024 - 00:41 |