Dependency Injection Factories: konečně použitelné továrničky

Viděl bych to jako maximálně jako nepovinné rozšíření, ne jako výchozí způsob vytváření komponent ve frameworku. Klasické factory metody v presenterech s voláním new jsou podle mě většinou dostačující a potřeba použití DIC je dost omezená.

Dále lze kontainer získat v komponentě přes metodu

$this->getPresenter()->getContext();

Sice to funguje jen v případech, kdy už je komponenta k němu připojena, ale to bude téměř vždy, zejména pokud jsou komponenty vytvářeny jako lazy.

Metodu injectNeco je mozne pouzit aj v

HosipLan napsal(a):

…komponentu v komponentě v komponentě …

?

Editoval vvoody (23. 7. 2012 11:43)

@vvoody: A teď mi řekni, jak injektneš nesdílenou službu? Nijak, protože chceš injektovat její továrnu. A když dáš do typehintu Nette\Callback (protože takto jsou v současné době implementované továrničky), tak autowire nebude vědět, co ti tam má předat.

Už si rozumíme, jaký problém se snažím vyřešit?

Editoval HosipLan (23. 7. 2012 11:54)

Já to pochopil tak, že to chceš aby framework při požadavku na komponentu zavolal DIC a ten by veděl, jakou factory třídu zavolat, takže by se už nepsaly factory metody v presenterech ale jen injectXXX. Pokud to tak není a dál zůstává to, že framework jen zavolá factory metodu na presenteru při požadavku na novou komponentu pak v pořádku a špatně jsem to pochopil.

Co tedy přesně představuje Tvůj návrh, jen ta compiler extension?

… Presenter nemá co záviset na DI Containeru.

Ale závisí, až skutečně zmizí __construct( string $baseDir, Nette\DI\Container $context ) z Presenteru, tak teprve pak je možné se o nějakém porušení DI při volání getContext() bavit.

blindAlley napsal(a):

Já to pochopil tak, že to chceš aby framework při požadavku na komponentu zavolal DIC a ten by veděl, jakou factory třídu zavolat, takže by se už nepsaly factory metody v presenterech ale jen injectXXX. Pokud to tak není a dál zůstává to, že framework jen zavolá factory metodu na presenteru při požadavku na novou komponentu pak v pořádku a špatně jsem to pochopil.

Ano, špatně jsi to pochopil. Chci aby libovolná třída registrovaná/vytvářená přes DIC (tedy i presenter) mohla vyžadovat továrničku na (třeba) komponentu

public function __construct(IPaginatorFactory $factory)
{
	$this->paginatorFactory = $factory;
}

A potom tuto továrničku na službu budu volat v továrničce na komponentu

protected function createComponentPaginator()
{
	return $this->paginatorFactory->create();
}

Tohle je navíc, oproti současnému chování, kdy komponentu vytváříš „ručně“ v továrničce na komponentu. Díky tomu mohu mít komponentu jako takovou v DIC a injektovat do ní další služby (a další továrničky na komponenty, …).

… Presenter nemá co záviset na DI Containeru.

Ale závisí, až skutečně zmizí __construct( string $baseDir, Nette\DI\Container $context ) z Presenteru, tak teprve pak je možné se o nějakém porušení DI při volání getContext() bavit.

Evidentně nesleduješ vývoj, takže ti to přiblížím.

• Presenter už není přímo závislý na DIC
• Na předávání závislostí je možné použít metodu inject
• DIC se do presenteru předává pouze kvůli tomu, aby byla dodržena zpětná kompatibilita.

Editoval HosipLan (23. 7. 2012 12:59)

blindAlley napsal(a):

To je pak tedy v pořádku, asi se Ti tedy jedná opravdu jen o to dostat tu extension do DIC do frameworku?

Přesně tak.

Zatím to může zůstat klidně jako rozšíření mimo framework ne?

Zatím to není ani zdaleka tak super, jak by to mohlo být, kdyby se vymyslelo, jak to udělat pěkně a kdyby to podporovalo IDE. Takže se nemusíš bát, že bych já tlačil do FW něco co považuju za nedokonalé, nebo David něco takového přijal :)

Interface se generuje vždy. Jde o to, abych toho musel napsat co nejméně.

Vytvořil jsem nový pullrequest, který na základě předaného interface generuje injektovatelné továrničky: https://github.com/…tte/pull/839 (to samé co předtím, ale nyní je třeba napsat interface)

To že se ti generuje třída je pochopitelné, můžeš záviset na jejím interface a implementace je ti ukradená. To že se generuje interface, na kterým závisíš a který ti bude mimo jiné chybět v API dokumentaci, už ti tak ukradené není.

Další nemilý problém je tohle. Dokud aplikaci nespustíš, tak interfacy neexistují. A pokud aplikaci nemáš jak spustit a máš „pouze“ testy (protože třeba píšeš knihovnu) tak jsi úplně ztracen.

Zkoušel jsem oba přístupy a tohle zkrátka rock-solid a funguje to perfektně

class Article
{
    public $title;
    public function __construct($title) { $this->title = $title; }
}

interface IArticleFactory
{
    /** @return Article */
    function create($title);
}

A jednoduchá konfigurace

factories:
    article:
        class: Article
        factory: IArticleFactory
        arguments: [%title%]
        parameters: [title]

A dokonce díky tomu, že napíšeš interface, tak se ti konfigurace ještě zjednodužší, protože spousta věcí se z něj dá odvodit ;)

factories:
    article: IArticleFactory

Editoval HosipLan (3. 11. 2012 19:13)

Skvela prace :)

komponenta

namespace Components;
class NowPlaying extends Nette\Application\UI\Control
{
	public function __construct($repository1, $repository2)
	{
		parent::__construct();
		// ..
	}
}

Multiplier tam predavat nemusis a $channel taky ne. V momentě připojení je budeš mít přístupné jako $this->parent a $this->name.

interface

namespace Components;
interface INowPlaying
{
	/** @return NowPlaying */
	function create();
}

konfigurace

factories:
	nowPlaying:
		class: Components\NowPlaying
		implement: Components\INowPlaying
		arguments: [@repositoryOne, @repositoryTwo]

použití

private $nowPlayingFactory;

public function injectNowPlaying(Components\INowPlaying $factory)
{
	$this->nowPlayingFactory = $factory;
}

protected function createComponentNowPlaying()
{
	$factory = $this->nowPlayingFactory;
	return new Multiplier(function($channel) use ($factory) {
            return $factory->create();
        });
}

Editoval HosipLan (26. 11. 2012 14:43)

Samozřejmě je to chyba ;) Už je to správně.

ale predsalen je tu problem,

Component with name ‚jednotka‘ does not exist

sablona {control nowPlaying-jednotka}

cize predsalen treba niekde predavat aspon ten $name

edit: tak netreba, zabudol som pridat return do tovarnicky multipliera

Editoval bazo (26. 11. 2012 15:48)

Ano, jde

interface IFooFactory
{
    /** @return Foo */
    function create($something);
}

factories:
	interface: IFooFactory
	parameters: [something]
	setup:
		- setSomething(%something%)

Editoval HosipLan (3. 12. 2012 11:34)

no konecne som to skusil, ale zakrici to na mna

Nette\DI\ServiceCreationException

Service ‚object‘: Missing item ‚something‘.

to preda len string. no skusil som uz asi vsetky kombinacie, nakoniec uz ani nepotrebujem, co som potreboval. ale skombinovat autowire a predavanu hodnotu cez create sa asi neda.

namiesto parameters by malo byt arguments, vtedy mozes tu premennu podsunut rucne

třída Menu nemá implementovat IMenuFactory, protože její implementaci ti vygeneruje Nette a zároveň zařídí autowire.

Stačí takto

services:
    someService:
        class: SomeService

factories:
    MenuFactory:
        implement: IMenuFactory

Lze využít automatické injectování pomocí @inject anotace (aktuálně implemetováno v dev verzi) i na továrničky ?

U služby to není problém. Ale u továrny MenuFactory (viz můj kód výše) pomocí následujího to nelze.

/**
 *	@var IMenuFactory
 *	@inject
 */
public $menuFactory;

Funguje to totiž jenom na klasické třídy ne na interface. Jde to obejít nebo existuje jiný přístup?

Lehce jsem nahlídl do kódu, a je tam funkce class_exists(), na které to vlastně stojí a padá.
A která nekontroluje existenci interface.