Je vůbec nějaký důvod, proč se tahle verze nepoužije v distribuci? Podle mě je výrazně lepší.

Je tahleta továrníčka už v Nette/Extras v doku? Pokud ne, dáte ji tam pls?

Jenom chci nahlásit, že supertovárnička je boží.

Společně s novým macrem {widget} (které odstraní přebytečné render… metody, ve kterých se jen plnila template komponentami) se některé prezentery smrsknou jen na createMyControl() metody, coz je fakt jako cool.

Protože není problém udělat CTRL + C a CTRL + V do BasePresenteru nebo BaseControlu, když většinou stejně nějaký základní presenter je? :)

Továrnička createComponent je jen obecný mechanismus a neurčuje konkrétní implementaci. Jestli si to člověk naimplementuje přes switche, podmínky nebo přes toto je přece jen čistě na něm. Začátečníky by mohlo volání callbacků dost zmást, proto si myslím že je i dobře, že to není v distribuci.

romansklenar napsal(a): Začátečníky by mohlo volání callbacků dost zmást, proto si myslím že je i dobře, že to není v distribuci.

Mě se zdá, že by to začátečníky určitě nemátlo, protože pak by „Návod na použití component v presenteru“ scvrkl na toto:

1. Pro každou komponentu, kterou chceš použít v presenteru, vytvoř metodu createMyControl (nebo pro formuláře createMyForm), která vytvoří a vrátí nakonfigurovanou komponentu/formulář
2. V template vykresli komponentu pomocí {control MyForm} nebo {wiget MyControl:specialRender}
3. Pokud je zapotřebí dále manipulovat s komponentami například v renderView(), komponentu získáte pomocí $this->getControl('myForm')

Presenter se tedy za pomoci {widget} a této továrničky smrskne (v ideálním případě) do metod createMyControl()

Honza M. napsal(a): Ale přišel jsem na jiné proti… Muselo by se každému vysvětlit, aby se nepokoušel zavolat třeba $this->getComponent("template"), což pravda normálně nikoho ani nenapadne.

Dobrá připomínka, jméno komponenty ‚template‘ by v ‚createMyControl()` mělo být asi zakázané stejně jako 'component‘.

Editoval jasir (6. 7. 2009 19:31)

Asi nejvýstižnější by bylo createComponent<Name>, ale je to dost dlouhé.

jasir napsal(a):

Nebo zavést podtržítko…

To je proti Nette konvencím:

Coding standard

Functions and Methods

• camelCase

Honza M. napsal(a):

Asi nejvýstižnější by bylo createComponent<Name>, ale je to dost dlouhé.

To je popravdě nejvýstižnější, otázka je: je to create skutečně nutné?

Máte ji tam, mršku http://api.nette.org/…rol.php.html#a48

Vždyť to volání metod není citlivé na velikost písmen.

Hlásim nespokojenost s implementací chytřejší továrničky.

Potřeboval bych metodám createComponent<Name> předávat parametr $name, protože některé moje komponenty potřebují již v továrničce být připojené k presenteru.

Př.:

// v presenteru
protected function createComponentEditForm($name) {
        new PageForm($this, $name);
}

// PageForm.php
class PageForm extends AppForm {
        public function __construct($parent, $name) {
                parent::__construct($parent, $name);

                // definice
                ...

                $this["id"]->setValue = $this->presenter->getParam("id");
        }
}

Pokud by tenhle ne úplně šťastný příklad jako argument nestačil, zamyslím se a vymyslím lepší.

Proč to $this["id"]->setValue() nezavolat až v attached?

// class PageForm
protected function attached($presenter)
{
        parent::attached($presenter);

        if ($presenter instanceof Presenter) {
                $this["id"]->setValue = $presenter->getParam("id");
        }
}

Editoval LM (10. 7. 2009 18:07)

Jsem pro přidání, protože:

• při případném přejmenování ho změním na méně místech ;]
• méně psaní ;]
• můžu používat továrny na některé componenty (nebo jejich častí nastaveni, vytvoření)
• tahat si componenty z modelu (porad bojuju s myšlenkou ze tam např Form patří)
• dostane název case sensitive tak jak jsem si ho napsal v getComponent
• snadněji se mi migruje ze starého způsobu přes createComponent($name) { switch ($name) ....
• něco si můžu například kešovat (nebo mít v sesions) v namespace podle nazvu componenty.
• parametr bude nepovinej, takže ho použiju jen když ho potřebuju.

Editoval PetrP (11. 7. 2009 22:58)