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

Posledné komentáre sú na spodnej časti poslednej stránky.

Neaktivní uživatel

Tvůrce

Neaktivní uživatel:3.8.2013 15:03

Mno ono jde o to rozdělit si třídu tak, aby nebyla tak dlouhá. Asi děláš něco špatně, pokud máš tolik řádků.

Odpovedať

Neaktivní uživatelský účet

Neaktivní uživatel

Tvůrce

Neaktivní uživatel:3.8.2013 15:14

Obecně je nejlepší ponechat třídu pod 60 řádků, pokud to nenaruší funkčnost třídy. Jestli není nutné mít všechny komponenty na stejném místě, je vhodné třídu rozdělit. Také je dobré zbavit se zbytečných metod a datových členů, které se nevyužívají. Komentáře bych ani tak nepočítal, ty přece zvyšují přehlednost.

Odpovedať

Neaktivní uživatelský účet

Kit

Tvůrce

Kit:3.8.2013 15:24

Ono to zpočátku trochu bolí, protože v té třídě chci tohle a ještě tohle... Jenže když si uvědomíš, že téměř všechny neprimitivní proměnné objektu jsou vlastně samostatnými vnořenými objekty, jde to najednou mnohem jednodušeji.

Odpovedať

Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.

Neaktivní uživatel

Tvůrce

Neaktivní uživatel:3.8.2013 15:26

Tak tak. Například třída Form v Nette je rozdělená snad do tří tříd. Ovšem mě přijde že pan Grudl se snaží, aby to vypadalo co nejsložitěji, jelikož většina věcí co tam má nakompentované se dá udělat mnohem jednodušeji.

Odpovedať

Neaktivní uživatelský účet

David Hartinger

Vlastník

David Hartinger:3.8.2013 15:41

Mít Form ve třech 60ti řádkových třídách mi přijde jako nesmysl. Dekompozice v takovéhle granularitě je jen na škodu, sice se snižuje počet řádků, ale zvyšuje se počet závislostí, počet souborů ve složkách a zanoření ve jmenných prostorech. Třída má být dlouhá tak akorát, moc krátká je také špatně.

Odpovedať

New kid back on the block with a R.I.P

Kit

Tvůrce

Kit:3.8.2013 15:42

605 řádek třídy Form je skutečně moc. Je důsledkem potřeby něco udělat až příliš univerzálně. Přitom mi ta třída připadá zbytečná, resp. jsem zatím nepřišel na to, k čemu bych ji použil.

Odpovedať

Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.

Neaktivní uživatel

Tvůrce

Neaktivní uživatel:3.8.2013 15:44

Moje slova.

Odpovedať

Neaktivní uživatelský účet

Kit

Tvůrce

Kit:3.8.2013 15:48

Forms v Nette je rozdělen do osmi souborů PHP, které mají dohromady 1631 řádek. Přitom by pro běžné použití určitě stačila desetina.

Editované

Odpovedať

Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.

Neaktivní uživatel

Tvůrce

Neaktivní uživatel:3.8.2013 15:51

Nedávno jsem si Nette smazal, tak teď nevím. Ale určitě není nutné mít na mail čtyři třídy a jedno rozhrání, na databází nepočitatelně... Ta databáze v Nette je dobrý příklad bordelu.

Odpovedať

Neaktivní uživatelský účet

Kit

Tvůrce

Kit:3.8.2013 15:57

Pardon, zapomněl jsem na 2520 řádek v podadresářích. Takže dohromady mají Forms v Nette 4151 řádek PHP.

Database je samostatná kapitola. 4645 řádek ve 23 souborech.

Odpovedať

Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.

Posledné komentáre sú na spodnej časti poslednej stránky.

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.

Zobrazené 10 správy z 53.

Najčastejšie vyhľadávané

Diskusia – 4. diel - Zapuzdrenie v PHP