Diskuze: Velikost třídy - výkon, paměť
V předchozím kvízu, Online test znalostí PHP, jsme si ověřili nabyté zkušenosti z kurzu.
Martin Konečný:6.11.2017 16:53
Mám entitu, která má několik relací na jiné entity, a jelikož projekt
není úplně malý, tak se to postupně trochu nakupilo. Možná by se dalo
něco delegovat jinam, ale velikost třídy hned nemusí znamenat porušení
SRP.
Nicméně to není odpověď na mou otázku 
Petr Homola, Marian Benčat Právě vůbec netuším, jak s tím PHP pracuje a zda je výkonnostní problém mít nějaký větší objekt. Nerad bych pak řešil problém, když by se instancí velké třídy udělalo třeba 100 během jednoho requestu. Děkuji za odpovědi!
David Hartinger Taktéž děkuji za odpověď. Výkon mě v tuto chvíli
netrápí, na PHP 7 to běží, jen bych nerad, aby později začal výkon
upadat kvůli takové kravině, jako je velký objekt 
Tak mě jen zajímalo, zda to
má nějaký patrný vliv.
Ohledně porušení SRP je to diskutabilní, třída je sice velká, ale stále
je to entita pracující jen se svými atributy. Navíc řada metod je spíše
pro intuitivnější použití objektu, abych např. místo
if ($item->type === "weapon" || $item->type === "armor" || $item->type === "shield")mohl napsat
if ($item->isWeapon() || $item->isArmor() || $tem->isShield())Tudíž si nemyslím, že by nutně SRP porušovalo, ale možné to
samozřejmě být může. Bez ukázky kódu to ale řešit moc nejde a nechce se
mi produkční kód celé třídy dávat veřejně.
Každopádně ještě jednou děkuji všem za odpověď.
David Hartinger:7.11.2017 13:27
Na tohle jsou návrhové vzory, ty metody isWeapon(),
isArmor() a podobně přeci nemusíš mít v té třídě, to je
úplně samostatná úloha něco rozlišovat 
Jestli jich je tam hodně,
rozdělil bych odpovědnost do další třídy a psal něco jako:
if ($item->getClassifier()->isWeapon()) // ...nebo třeba:
if ($itemClassifier->isWeapon($item)) // ...Záleží jak to máš napsané, co všechno se tam dělá, to musíš
vědět ty
Martin Konečný:7.11.2017 13:36
Hmm, je fakt, že hodně podobných metod je napláclých přímo v té
třídě. Nicméně bych teď nerad přepisoval kus kódu, tak pokud bude
později třeba, třeba se na to podívám. Díky
Martin Konečný:7.11.2017 14:18
Jsem si vědom toho, že odkládání je spíše negativní, jen se mi z
jistých důvodů nechce v tuto chvíli přepisovat fungující kód. Na
databázi to vázané je, tudíž nedokážu hned z hlavy odhadnout, jakou
práci by dalo tu třídu upravit. Osobně si nemyslím, že je to napsané
"špatně", spíš jen by to možná šlo napsat "lépe"
Jelikož to tedy nemá dopad na výkon, nechám to prozatím tak. Děkuji za
vysvětlení.
David Hartinger:7.11.2017 14:46
Myslel jsem to tak, že k tomu nemusíš změnit databázi. Dokud se
refaktoruje jen kód, tak se to docela dá. Databáze, soubory a pod. mají
obvykle více sideeffects
Martin Konečný:7.11.2017 15:03
Jasné, už rozumím. Zde je možná trochu problém, že je to entita,
která své atributy má přímo namapované na sloupce v databázi (používám
ORM). Tudíž pokud bych např. "typ" předmětu delegoval na jinou třídu,
musel bych obejít výchozí chování frameworku. Možná by to šlo řešit
jednoduše, ale nenapadá mě v tuto chvíli jak a nerad bych teď dával čas
do úprav, které by to nakonec kvůli omezení frameworku ještě mohly
zhoršit
Marian Benčat:7.11.2017 17:01
if ($item->getClassifier()->isWeapon()) // ...
šéfe kážeš o návrhových vzorech a pravidlech a hned v prvním
příkladu porušíš demetera... ty ty ty 
Zobrazeno 14 zpráv z 14.