[deartify]

Mirki jakie są najgorsze rzeczy jakie można uczynić w projekcie symfony lub laravel i php samym w sobie. Szukam po internecie najgorszych praktyk, żeby sprawdzić czy sam tak nie robię ( ͡° ͜ʖ ͡°). Znalazłem taką stronę http://www.phpthewrongway.com Zgadzacie się z tym co tam jest napisane?
#php #laravel #symfony

[zakopiak]

@deartify: Z pewnością jedną z takich praktyk jest używanie larwy. (✌ ﾟ ∀ ﾟ)☞

[A.....W]

@deartify: SF
- zapytania w kontrolerach
- wywoływanie kontrolerów w komendach :D
- nie używanie stałych dla statusów encji (np w widokach twig)
- sporo można najanuszować w Doctrinie (sprawdzić poprawność mapowania), wiele wpisów na blogach jest o wydajności doctrin i jak łatwo można to zjanuszować
- złe rozłożenie zależności w bundlach i serwisach

[FREEQ]

@AVAW:

- wywoływanie kontrolerów w komendach :D

XD

[A.....W]

@FREEQ: #truestory nie byłem autorem tego pomysłu, jak zobaczyłem kod to tylko kazałem poprawić XD

[FREEQ]

@AVAW: Najgorsze co widziałem to w kontrolerze metoxę x() która walidowała, słała zapytania do bazy, budowała jakieś zależności. Ba, nawet HTML-e ktoś tam budował. Generalnie to około 450 linijek kodu w jednej metodzie o tak gigantycznej złożoności obliczeniowej, że aż trudno to pojąć. Więc wierzę Ci, że tak ktoś robił. :D

[Jare_K]

@deartify: IMO najgorsza z wszystkich praktyk to nie izolowanie domeny aplikacji od samego frameworka i DAL, czytaj: budowanie silnych zależności z konkretnymi narzędziami.

[Utiopa]

@zakopiak: nie obrażaj( ͡° ͜ʖ ͡°)

[Jurigag]

@deartify w takim laravel czy Symfony - korzystanie z eager loading, auto wiringu, pobieranie pełnych encji z bazy czy nawet często gęsto w widokach pętla po related encjach powodująca dodatkowe zapytania bo brak joina

[stash]

auto wiringu

@Jurigag: Dlaczego korzystanie z tego uważasz za złą praktykę? Jak przeglądam sobie zmiany w symfony to coraz bardziej rozbudowują tą funkcjonalność żeby jeszcze łatwiej budowało się serwisy.

[Jurigag]

@stash: bo wykorzystuje refleksje które są wolne

[stash]

@Jurigag: Symfony buduje cache dla serwisów, więc tylko minimalnie dłużej trwa wygrzewanie. Coś jeszcze jest w tym, co uważasz za wadę?

[Jurigag]

@stash: ale cache nie ma nic tutaj do rzeczy, chodzi mi o to że jeśli masz jakąś metodę w jakiejś klasie i chcesz aby automatycznie do takiej metody były dowiązane zależności musi być tu wykorzystana reflection - i tyle w temacie, i jest to wada i tyle, jak się da - nie powinno się korzystać i tyle, na poziomie developingu - spoko, ale na produkcji powinny być po prostu skonfigurowane pliki

[stash]

@Jurigag: Na produkcji w czasie tworzenia cache wykona się cała praca związana z refleksją i później już nie będzie żadnego narzutu. W samej dokumentacji jest to wytłuszczone https://symfony.com/doc/current/service_container/autowiring.html

Thanks to Symfony's compiled container, there is no runtime overhead for using autowiring.

Nadal nie rozumiem, co jest problemem?

[Jurigag]

@stash: to że w praktyce jest overhead - wystarczy sobie porównać wyniki wydajnościowe z kontrolerem jako serwis i napisanymi plikami konfiguracyjnymi i bez, zresztą wystarczy sprawdzić jakikolwiek inne DI, ten z symfony jest dosyć wolny

[stash]

@Jurigag: Co to ma do rzeczy z autowire, że jest to coś złego? DI w symfony jest jaki jest i korzystając z tego fw bierze się go zwykle z dobrodziejstwem inwentarza żeby wszystko było u standaryzowane.

[Jurigag]

@stash: po prostu lepiej ustawiać kontroler jako serwis i odpowiednie pliki konfiguracyjne, dużo czasu to nie zajmuje a szbyciej będzie działało podczas developingu i na produkcji też jest różnica