Diskuze ke článku: Inject a DIC továrničky

Sis naložil, co? :-) Za sebe bych úplně vyhodil kapitolu „jak to nedělat“.

Osobně bych přidal k příkladu pro formuláře více vysvětlení práce s parametry pro factories. Když na to prvně kouknu, tak vidím, že přidávám %userId% pro klíč create a pak to musím znova specifikovat samostatným klíčem parameters. Chybí mi tam, proč to tak je a jak tyhle parametry fungují.

Plus bych k tomu přidal i nějaké varování (podobné jako pro rozhraní výše v článku), že tyhle parametry neslouží k předávání jiných závislostí. Vidím celkem reálně přístup toho, že udělám chybně něco takového:

factories:
    userForm:
        create: App\CreateOrEditUserForm(%database%, %userId%)
        implement: App\ICreateOrEditUserFormFactory
        parameters: [database, userId]

a pak to chybně použiju:

class UserPresenter extends BasePresenter
{

    /**
     * @persistent
     */
    public $userId = 0;

    /**
     * @var ICreateOrEditUserFormFactory
     */
    private $userFormFactory;

    /**
     * @var Nette\Database\Connection
     */
    private $database;

    /**
     * @param ICreateOrEditUserFormFactory $userFormFactory
     */
    public function injectUserFormFactory(ICreateOrEditUserFormFactory $userFormFactory)
    {
        $this->userFormFactory = $userFormFactory;
    }

    /**
     * @param Nette\Database\Connection $database
     */
    public function injectDatabase(Nette\Database\Connection $database)
    {
        $this->database = $database;
    }

    /**
     * @return CreateOrEditUserForm
     */
    protected function createComponentUserForm()
    {
        return $this->userFormFactory->create($this->database, $this->userId);
    }
}

Předpokládá to sice nepochopení předchozí části článku, ale IMHO bych se na snahu snížit počet vlastních vazeb na další služby znovu odkázal.

Hezké, jen se mi nelíbí pořád ukazovat příklady na třídě „article“

<?php
factories:
    articles:
        create: ArticlesControl
        implement: IArticlesControlFactory
?>

tenhle příklad obsahuje 4 slova s dvojtečkou(tedy na první pohled důležitá), z nichž tři ale jsou klíčová a jedno ne – kdyby se to menovalo myOwnClass nebo nějaká taková blbost, možná to bude pro začátečníky jednodušší. Jelikož to articles by u někoho mohlo evokovat, že jde o klíčové slovo Nette. :/

Editoval Hafran (22. 11. 2012 21:19)

@HosipLan myslím si že to ani tam nepatrí, ešte si pamätám ako som sám z toho bol zmätený, nevedel som či pre mňa ten kód niečo znamená kedže je generovaný, ale na druhú stranu je napísaný v dokumentácií, takže jej autor si zrejme myslí že ho programátor potrebuje vidieť… s čím (imho) momentálne, keď už som v obraze, nesúhlasím. Aspoň u začiatočníkov si myslím, že to narobí menej „škody“, respektíve rýchlejšie pochopia danú problematiku, keď bude predstavená čo najmenším množstvom kódu. Snažím sa na to pozerať ako by to čítal človek, čo Nette pozná tak z 5–10%, nie ako taký čo ho pozná z 50–90%.

Já osobně mám trochu problém s tím, proč se pro takovou funkčnost mám psát interface, v jehož definici je přímo uveden návratový typ. Připadá mi to jako zneužívání rozhraní pro jiné účely, než pro jaké má sloužit, nebo mi něco zásadního uniká?

Naopak se zastanu toho, aby vygenerovaný kód v článku zůstal.

Jsem to tedy vzal od podlahy … :) Líp už to ale asi nedám, takže pokud to nestačí, bude to muset napsat někdo jiný ;)

Koho zajímá „history of factories, how it was and how it is“, tak v historii to samozřejmě zůstalo ;)

Editoval HosipLan (23. 11. 2012 0:38)

HosipLan napsal(a):

Jsem to tedy vzal od podlahy … :) Líp už to ale asi nedám, takže pokud to nestačí, bude to muset napsat někdo jiný ;)

Tak teraz je to ooooveľa lepšie. Nice :)

mne sa viac páčila tá prvá verzia, dole by si mohol pridať link „nebo jinak DIF továrničky pro nás pět“ ;-)

Asi patřím mezi těch pět nebo co, ale pro mne byla lepší původní verze. :-D Zejména protože jsem chtěl vědět jak to použít s komponentami. :-)

Poznatky k nové verzi:

1. Nejdříve mi přišla trochu wtf ta vlastní služba mailer, když Nette přece vytváří vlastní službu nette.mailer. Až když jsem se podíval do debugBaru tak jsem zjistil, že to je jiný mailer (Nette vytváří SendMailMailer, v článku je SmtpMailer). Ale pořád s tím mám problém, že na to nepotřebuji vlastní službu. Článek ve mne zkrátka vyvolal dojem „toto je jediný správný způsob jak odesílat e-maily“, což dle mého názoru není.
2. Očekával bych, že factory pro vytvoření věcí jako \Nette\Mail\Message nebo \Nette\Latte\Engine bude také poskytovat přímo framework jako služby nette.latteFactory a nette.messageFactory a že třeba \Nette\Mail\Message bude už přímo v sobě mít metodu injectMailer. Chystá se něco takového?

Editoval enumag (23. 11. 2012 6:59)

Jak Nette pozná, že má implementovat metodu create()? Nikde v configu není použito slovo create. Co kdyby interface obsahoval více funkcí? Nebo je create nějaké magické klíčové slovo?

Osobně se mě taky moc nezdá, ze interface určuje přímo třídu, kterou vrací. To by měla podle mě být záležitost konkrétní implementace interface. Mělo by tam být spíš Nette\Mail\IMessage.

HosipLan napsal(a):
@enumag přidal jsem poznámku, že to není best-practise.

OK, to už vypůadá lépe. :-)

Úkol pro všechny: vymyslete lepší téma, kolem kterého půjde článek napsat – emaily se zdají být dost kontroverzní. Ideálně něco, co vůbec není ve frameworku.

Přesně to mne taky napadlo. Že bych to přepsal na něco jiného než maily a komponenty. A stejně jako ty jsem se zasekl, protože mne nenapadá vhodné téma. :-D

Co ta druhá část ohledně injectMailer() v \Nette\Mail\Meassage?

EDIT: Trochu jsem nad tím přemýšlel.

Jaký example vlastně hledáme?

1. Něco co se použije vícekrát (jinak je to služba),
2. ale současně něco co není vykreslitelné (jinak je to komponenta),
3. a současně to musí mít nějaké závislosti (jinak stačí new).

To už myslím docela výrazně omezuje možnosti. Kromě toho, jaký byl důvod proč @HosipLan tuhle feature vůbec vymýšlel? Pokud vím tak to bylo kvůli komponentám (@Hosiplan: oprav mne pokud se mýlím).

Z téhle úvahy mi vychází, že komponenty budou jako příklad přeci jen nejlepší, co myslíte?

Editoval enumag (23. 11. 2012 10:30)

@HosipLan: Upravil jsem svůj příspěvek výše. ;-)

Off topic: Stejně mám pocit, že ta třída Nette\Mail\Message by ten mailer vůbec neměla potřebovat a tedy by neměla mít ani metodu send. Myslím, že by bylo lepší odesílat přes $mailer->send($message);.

Je to neporovnateľne pochopiteľnejšie ako prvá verzia;)
Jediná poznámka.. Pre nováčika by mohlo byť trošku mätúce kde sa vzalo ‚$this->database‘.

Možno by to ešte mohlo pokračovať tým, že formulár so spracovaním by šiel zvlášť do továrničky a tá by bola tiež nastavená tým istým spôsobom aj s nastavením ‚database‘ a ‚mailingFactory‘ cez inject.

Editoval romiix.org (23. 11. 2012 12:01)

Podle mě je to dost těžko pochopitelná a dost magická feature a jako takovou bych ji vůbec do frameworku nezařazoval.

• Je to takové programování v konfiguraci. Pokud přesně vím mechanizmus jakým to funguje, můžu některé třířádkové třídy napsat v neonu místo PHP.
• Není zřejmé na čem funguje konfigurace setup. Jestli na továrničce nebo na vytvářeném objektu.
• Nějaký kód stejně psát musím (ten interface).

<offtopic> Ostatně soudím, že sekce factories by měla být zničena. Neexistuje žádný rozdíl mezi továrničkou a službou, která má metodu na továrničkování. </offtopic>

Hele super, ale klidně bych tam vrátil jak vlastně ta vygenerovaná třída vypadá.

Honza Marek napsal(a):

• Je to takové programování v konfiguraci.

Ano Nette se stává o něco více deklarativní a zbavujeme se dalšího boilerplate (a toho se vždy rád zbavím). Imho nikdo a nic tě nenutí to používat (jenom pokud to používat budeš usnadníš si práci).

<offtopic> Ostatně soudím, že sekce factories by měla být zničena. Neexistuje žádný rozdíl mezi továrničkou a službou, která má metodu na továrničkování. </offtopic>

Ten rozdíl je právě v tom generování třídy (resp. tak to vidím já).

A už je to tady. :-(

@Michalek: Ten příklad je špatný. Snažíme se vymyslet něco lepšího, ale to bude chvíli trvat. Faktem je, že pro mailer nemáš vytvářet vlastní službu jelikož takovou už poskytuje samo Nette.

EDIT:

A ta druhá část se ti nezdá také oprávněně. Tohle je v podstatě ukázka toho jak bys to musel udělat dříve. Tj. továrnu si naprogramovat sám (to je ta třída MailingMessageFactory) a pak si ji zaregistrovat jako službu. Ten nový (preferovaný) způsob je potom níže, kde už to dáš přímo do sekce factories – to znamená že tu factory ty sám nepíšeš jako nějakou třídu, ale necháš Nette aby ji vyrobilo. To šlo používat i dříve, akorát takové továrny doteď nebyly injectovatelné, protože je Nette nevyrábělo jako třídu, ale pouze jako metodu DI containeru.

Myslím, že tohle stačí jako názorný příklad, že jakékoli „jakto bylo dříve“ nebo cokoli podobného začátečníky jen zbytečně mate a bude pro ně nejlepší ten nový správný způsob uvést hned na začátku a cokoli dalšího neuvádět buď vůbec nebo pod velkým červeným nápisem „takhle ne, tohle je jen ukázka pro detailnější pochopení“.

EDIT2:

@HosipLan: Lepší téma zřejmě nikoho nenapadá, takže asi čekáme než @dg povolí komponenty jako príklad, je to tak? Ty e-maily bych tam nechal jen opravdu nerad, připadá mi to hodně matoucí.

Editoval enumag (23. 11. 2012 19:00)

IMHO je to názvosloví dost matoucí (obecně v nette, ne v tom článku). Vzhledem k tomu, že továrničky ve smyslu $container->createFoo() jsou neperspektivní by stálo za to se zamyslet jak to znormálnět (pokud možno bez BC breaku).

Editoval mkoubik (23. 11. 2012 22:08)