IT rekvalifikácia. Seniorní programátori zarábajú až 6 000 €/mesiac a rekvalifikácia je prvým krokom. Zisti, ako na to!

Diskusia – 4. diel - Zapuzdrenie v PHP

Späť

Upozorňujeme, že diskusie pod našimi online kurzami sú nemoderované a primárne slúžia na získavanie spätnej väzby pre budúce vylepšenie kurzov. Pre študentov našich rekvalifikačných kurzov ponúkame možnosť priameho kontaktu s lektormi a študijným referentom pre osobné konzultácie a podporu v rámci ich štúdia. Toto je exkluzívna služba, ktorá zaisťuje kvalitnú a cielenú pomoc v prípade akýchkoľvek otázok alebo projektov.

Komentáre
Avatar
Kit
Tvůrce
Avatar
Kit:3.8.2013 13:32

Spíš se používá pravidlo, že metoda by neměla mít víc než 20 řádek a třída 60 řádek, řádky se zalamují po max 78 znacích. Slovo "neměla" však neznamená nepřekročitelný limit.

Odpovedať
3.8.2013 13:32
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
David Hartinger
Vlastník
Avatar
Odpovedá na Kit
David Hartinger:3.8.2013 13:56

To máš s atributy a phpdoc 2 metody na třídu, to je poněkud neefektivní.

Odpovedať
3.8.2013 13:56
New kid back on the block with a R.I.P
Avatar
Petr Nymsa
Tvůrce
Avatar
Odpovedá na Kit
Petr Nymsa:3.8.2013 13:58

60 řádků na třídu ? To mi přijde hodně nizký limit :) Vždyť ani taková třída která je jako CRUD wrapper nad DB poruší podle mě tenhle limit

Odpovedať
3.8.2013 13:58
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
Kit
Tvůrce
Avatar
Odpovedá na David Hartinger
Kit:3.8.2013 14:03

Metody bývají obvykle výrazně kratší, takže se jich tam 8-10 vejde. Kromě toho to není nepřekročitelné maximum.

Petr Nymsa: CRUD wrapper nad DB se mi do 60 řádek v pohodě vejde.

Odpovedať
3.8.2013 14:03
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Petr Nymsa
Tvůrce
Avatar
Odpovedá na Kit
Petr Nymsa:3.8.2013 14:06

Jo teď si na to tak koukám a vejde se :) Ale i tak, občas je třída, kde rozložení na víc komponent by spíš ublížilo a sem tam se ten limit překročí. Třeba mám třídu pro různé efekty nad textem, obrázky apod. Jsou to různé metody a je zbytečné to rozdělovat do více tříd

Odpovedať
3.8.2013 14:06
Pokrok nezastavíš, neusni a jdi s ním vpřed
Avatar
Kit
Tvůrce
Avatar
Odpovedá na Petr Nymsa
Kit:3.8.2013 14:12

Pokud se sem-tam ten limit překročí, nic se neděje, ale třeba 200řádková třída už je nepřehledná a je lepší ji v návrhu logicky rozdělit na více komponent. Obvykle s tím není problém a řešení je najednou čistější a hezčí.

Odpovedať
3.8.2013 14:12
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Michal Žůrek - misaz:3.8.2013 14:35

já tyto limity moc neřeším...

 
Odpovedať
3.8.2013 14:35
Avatar
Kit
Tvůrce
Avatar
Odpovedá na Michal Žůrek - misaz
Kit:3.8.2013 14:44

Také mám v provozu jednu funkční aplikaci, která má 700 řádek bez tříd a funkcí, prostě nudli. Když jsem v tom měl něco modifikovat, tak se mi moc nechtělo a celé se mi to také nechce přepisovat. Předělávej něco, co funguje.

Proto je dobré tu dekompozici udělat hned na začátku.

Odpovedať
3.8.2013 14:44
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Odpovedá na Kit
Michal Žůrek - misaz:3.8.2013 14:52

Já mám i 1400 řádků a taky funguje.

Spíš de o to že příliš neřeším max délku řádku, max počet řádků v třídě atd.

 
Odpovedať
3.8.2013 14:52
Avatar
Kit
Tvůrce
Avatar
Odpovedá na Michal Žůrek - misaz
Kit:3.8.2013 15:02

Nedávno jsem ve Wordpressu našel skript, který má 35k řádek a také funguje. Rozhodně to však není dobrý příklad, jak se má programovat. Ten kód vypadá přímo hnusně.

Odpovedať
3.8.2013 15:02
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Robíme čo je v našich silách, aby bola tunajšia diskusia čo najkvalitnejšia. Preto do nej tiež môžu prispievať len registrovaní členovia. Pre zapojenie sa do diskusie sa zaloguj. Ak ešte nemáš účet, zaregistruj sa, je to zadarmo.

Zatiaľ nikto nevložil komentár - buď prvý!