AbcLinuxu:/ Poradna / Programovací poradna / Getter a Setter v PHP

Štítky: citace, data, PHP, programování, vývojář

Dotaz: Getter a Setter v PHP

25.2.2015 13:41 Marky
Getter a Setter v PHP

Přečteno: 1369×

Odpovědět | Admin

Ahoj, už tři dny mi nedá spát Kit-ova citace

"Když vývojář napíše getter/setter jako public, poruší tím zapouzdření objektu se všemi důsledky."

Copak to jde udělat private ? K čemu by pak takový getter měl sloužil a hlavně jak bych pak dostal data z těch class, když potřebuji z jedné metody získat více hodnot?

Řešení dotazu:

Nástroje: Začni sledovat (0) ?

Odpovědi

25.2.2015 16:18 Kit | skóre: 45 | Brno
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

Pokud má třída 3 privátní atributy, 3 gettery, 3 settery a žádnou další metodu, jedná se o 3 nezávislé věci, které třída dělá. Proto by měla být rozdělena do 3 nezávislých tříd. Setter je pak nahrazen konstruktorem, getter prezentační metodou.

Pokud je atribut privat a je k němu napsán public getter a public setter, v konečném důsledku se ten atribut chová jako public. A to je až na výjimky (messengery) zpravidla nežádoucí.

V PHP téměř nikdy nepotřebuji manipulovat s jednotlivými atributy objektu. Proto ani gettery/settery nedělám. Jednoduše je nepotřebuji. Vždy pracuji s objektem jako celkem, jinak by to bylo porušením SRP.

Pokud potřebuji z jedné metody dostat více hodnot, uložím je do objektu další třídy nebo v jednodušších případech použiji zmíněný messenger.

Použití nezávislých setterů s sebou rovněž nese riziko nekonzistence objektu. Pokud například chceš změnit jméno i příjmení, změníš jméno a než změníš příjmení, tak se někdo zeptá na stav objektu. V tu chvíli je tam nové jméno, ale staré příjmení. A to je problém, který se může projevit i u dvou getterů, které jdou po sobě. Mezitím se stav objektu mohl změnit.

Komentáře označují místa, kde programátor udělal chybu nebo něco nedodělal.

25.2.2015 16:48 rADOn | skóre: 44 | blog: bloK | Praha
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

Pokud je atribut privat a je k němu napsán public getter a public setter, v konečném důsledku se ten atribut chová jako public.

Je tu jeden rozdil – pokud se neco zmeni a ty chces ten atribut ziskat jinym zpusobem, treba ho spocitat z jinych, vytahnout z cache nebo cokoliv, musis opravit potencialne hodne kodu. Pokud jazyk nema zpusob jak zahakovat cteni/nastavovani atributu, tak je psani get/set metod rozumny zpusob jak byt predem pripraven. Kdyz jsou to "nezavisle veci" tak je to samozrejme zbytecne, ale treba v prikladu ktery uvadis jsou jmeno a prijimeni na sobe zavisle. Takze si bud udelas setter ktery nastavi oboji, nebo udelas dva settery ale opatris je vzajemnym zamykanim… v kazdem pripade budes chtit aby samotny atribut byl private aby ti na nej nikdo nesahal.

"2^24 comments ought to be enough for anyone" -- CmdrTaco

25.2.2015 17:07 Kit | skóre: 45 | Brno
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

Jenže já na ty privátní atributy nesahám, takže nic nemusím opravovat ani zamykat.

Komentáře označují místa, kde programátor udělal chybu nebo něco nedodělal.

25.2.2015 16:40 Mercuriuz | skóre: 16 | blog: Mercuriuz | V hospodě =)
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

Je to jen vtípek... myslím
Čistě teoreticky můžeš gettery/settery udělat protected a přístup k nim řešit přes child classu, např. pokud child classa poskytuje metody pro zpracování hodnot, může je takto předávat parentovi který zapouzdřuje výsledek.
Každopádně dělat private všechno, na co nemá chytat nic jiného než classa samotná je velmi užitečný "zlozvyk"

class Car
{
    private $VIN,
            $color,
            $owner;

    private function convertColorToRgb($hexColor)
    {
        /* funkce která zajistí convert */
        return $rgb;
    }

    protected function getData()
    {
        return array("VIN" => $VIN, "color" => $color, "owner" => $owner);
    }

    protected function setVIN($vin)
    {
        $this->VIN = $vin;
    }

    protected function setColor($color)
    {
        $this->color = $this->convertColorToRgb($color);
    }

    protected function setOwner($owner)
    {
        $this->owner = $owner;
    }
}

class CarLoader extends Car
{
    public function loadCar($vin)
    {
        $this->setVIN($vin);
        $data = $this->getDatabaseData($vin)
        $this->setColor($data['color']);
        $this->setOwner($data['owner']);
    }

    private function getDatabaseData($vin)
    {
        /* funkce která načte auto z databáze a uloží hodnoty do lokálních proměnných $color a $owner */
        return array("color" => $color, "owner" => $owner);
    }
}

class CarToXml extends Car
{
    public function getXml()
    {
        $data = $this->getData();
        /* funkce která z dat v proměnné $data udělá xml */
        $return $xml;
    }
}

Je to samozřejmě jen příklad, jde o to, že child classy přesně ví, co mají do parenta posílat za data a jakým způsobem s ními pracovat, ani jeden setter/getter není z venku dostupný a stejně jde dělat vše co je potřeba.

Linux users everywhere, raise your keyboards into the air, we´re warriors, warriors of the world.

25.2.2015 16:47 Mercuriuz | skóre: 16 | blog: Mercuriuz | V hospodě =)
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

$ před return ve funkci getXml() je samozřejmě jen typo

Linux users everywhere, raise your keyboards into the air, we´re warriors, warriors of the world.

25.2.2015 16:51 Kit | skóre: 45 | Brno
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

No nazdar... Jako vtípek je to ale docela dobré. Jenom trochu ... nepoužitelné.

Komentáře označují místa, kde programátor udělal chybu nebo něco nedodělal.

25.2.2015 17:17 Mercuriuz | skóre: 16 | blog: Mercuriuz | V hospodě =)
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

no tak nejvhodnější příklad to zrovna není, ale nic lepšího sem zrovna nevymyslel

Linux users everywhere, raise your keyboards into the air, we´re warriors, warriors of the world.

25.2.2015 17:33 Kit | skóre: 45 | Brno
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

Co třeba tohle? Trochu jsem to předělal podle sebe.

class Car {
    private $vin;
    private $color;
    private $owner;
    private $db;

    function __construct($db, $vin) {
        $this->db = $db;
        $this->vin = $vin;
        $data = $this->findCar($vin);
        $this->color = self::convertColorToRgb($data['color']);
        $this->owner = $data['owner'];
    }

    private function findCar($vin) {
        /* funkce která načte auto z databáze a uloží hodnoty do lokálních proměnných $color a $owner */
        return $select->fetch(); //array("color" => $color, "owner" => $owner);
    }

    private static function convertColorToRgb($hexColor) {
        /* funkce která zajistí convert */
        return $rgb;
    }

    function __toString() {
        /* funkce která z dat v proměnných $vin, $color a $owner udělá $xml */
        return $xml;
    }
}

Komentáře označují místa, kde programátor udělal chybu nebo něco nedodělal.

25.2.2015 18:04 DarkKnight | skóre: 26
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

A co kdyz nechce pouzivat Active Record? Co kdyz se nehodi do navrhu architektury? Co kdyz potrebuju hloupy model, ale s kontrolou vstupu?

25.2.2015 18:24 Kit | skóre: 45 | Brno
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

Tak si udělej hloupý model s kontrolou vstupu. Tohle není Active Record, protože neumožňuje modifikaci v DB.

Je to hodně zjednodušená verze architektury, kterou používám. V reálu tam proměnnou $data a příslušné atributy vůbec nemám. View (který mám v jiné třídě) dostane záznam přímo z DB přes messengera. Konverze barev, data, času apod. do HTML či XML je jeho starost.

V daném případě jsem považoval za zbytečné to dělit do svých čtyř vrstev a napsal jsem to do jedné třídy. Vadí to moc?

Komentáře označují místa, kde programátor udělal chybu nebo něco nedodělal.

25.2.2015 22:18 kralyk z abclinuxu | skóre: 29 | blog:
Rozbalit Rozbalit vše Re: Getter a Setter v PHP

K tomhule 3 připomínky:

Celkem není důvod nepoužít getter/setter například pro tu barvu. Zapouzdření není svatá kráva. Pokud budou ty gettery/settery napsaný tak, že nemůžou dostat objekt do nekozistentního stavu, tak v tom opravdu nevidim problém. Pokud by existovala třída Color, která by sama o sobě zajišťovala konzistentní uložení barvy, tak by klidně Car mohlo mít barvu jako veřejný atribut, pokud nejsou na barvu nějaké další omezující podmínky.
Funkce convertColorToRgb nemá v třídě Car co pohledávat, vzhledem k tomu, že se stará o konverzi barev. Taky se mi na ní nelíbí, že je statická, ale přitom zamíchaná mezi nestatické.
Funkce findCar mi tam taky moc nesedí, čekal bych, že bude vracet instanci Car a že bude statická nebo úplně mimo třídu Car.