Automatické vložení závislosti, návrh aplikace

OK nyní mám

services:
basket: Basket

a třídu Basket

<?php
class Basket extends Nette\Object {

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

    /** @var Nette\Security\User */
    public $user;

    function __construct(Nette\Security\User $user, Nette\Database\Connection $database) {

?>

a pokud vytvořím objekt tak tam stejn musím jako parametry předat user a db nebo se pletu?

<?php
$x = new \Basket($this->user, $this->database);
?>

Aha :-) Takže pokud to vytáhnu z context tak se mi ty závislosti předají sami? Jdu to zkusit. Toto je dost zásadní informace.

Nastavuješ ty properties v tom konstruktoru? Mělo by to být takto:

function __construct(Nette\Security\User $user, Nette\Database\Connection $database) {
	$this->user = $user;
	$this->database = $database;
}

DI kontejner ti „jen“ předá ty služby do konstruktoru, nastavit do těch properties si to musíš v konstruktoru už sám.

batko napsal(a):
a pokud vytvořím objekt tak tam stejn musím jako parametry předat user a db nebo se pletu?

<?php
$x = new \Basket($this->user, $this->database);
?>

Obecně ano, jak by se tam taky jinak ty závyslosti dostali, než že je tam vy sám předáte.
V Nette ovšem využijte config a založte si službu (jak tu už je ostatně napsáno) a Nette se postará samo o předání závyslostí přes tzv. autowiring: https://doc.nette.org/cs/configuring#…
V Presenteru si pak službu injectujte: https://pla.nette.org/…ect-autowire

batko napsal(a):

<?php
 public function createComponentShowBasket() {
        return new \Todo\ShowBasket();
    }
?>

Buď to nechápu a nebo mi to přijde jako naprosto stejný problém, který si měl s třídou „Basket“. Componentu si nastav v configu a pak jí tahej z conteineru a o závislosti se postará auto-wiring.

zavislosti pri tvorbe komponenty predavat rucne nemusis, k tomu ucelu slouzi tovarnicka. nejdriv ukazu zpusob bez nette magie (bez dic tovarnicek):
mas tu tridu ShowBasket, ktera vyzaduje uzivatele a databazi. bez tovarnicky by sis opravdu musel do presenteru injectnout User a Connection a tyto sluzby predat pri vytvareni ShowBasket v createComponent medode (jak to pise @Šaman)

situaci si muzem vylepsit, ze si vytvorim tridu ShowBasketFactory se zhruba nasledujicim kodem:

class ShowBasketFactory
{
	/** @var Nette\Security\User */
	protected $user;
	/** @var Nette\Database\Connection */
	protected $connection;

	public function __construct(Nette\Security\User $user, Nette\Database\Connection $connection)
	{
		$this->user = $user;
		$this->connection = $connection;
	}

	public function create()
	{
		return new ShowBasket($this->user, $this->connection);
	}
}

v neonu tohle registruju jako beznou sluzbu

services:
	showBasketFactory: ShowBasketFactory

a nette uz se postara o zavislosti na Connection a User

v presenteru se potom uz nemusis starat o to, co trida ShowBasket pozaduje za zavislosti. tebe jen zajima, ze potrebujes instanci tridy a vis, ze k tomu mas tovarnicku. tak si tovarnicku injectnes do presenteru

class FooPresenter extends BasePresenter
{
	protected $showBasketFactory;
	public function injectShowBasketFactory(ShowBasketFactory $showBasketFactory)
	{
		$this->showBasketFactory = $showBasketFactory;
	}

	public function createComponentShowBasket()
	{
		return $this->showBasketFactory->create();
	}
}

nette vidi metodu inject* a podle typehintu zjisti, ze pozadujes sluzbu ShowBasketFactory, proto ji injectne do presenteru (pri vytvareni teto sluzby mimochodem zjisti, ze ta factory pozaduje User a Connection, proto tam tyto sluzby pri vytvareni vlozi)
v createComponent metode akorat zavolas metodu create() na tovarnicce, ktera ti vrati tu spravnou instanci

EDIT:
jeste k DIC tovarnickam. to je to, co jsem psal v tom mem odkazovanem prispevku. DIC tovarnicky tento proces zjednodusuji, aby jsi nemusel neustale psat tovarnicky (ShowBasketFactory) cele rucne. pri pouziti DIC tovarnicek ti staci napsat rozhrani:

interface ShowBasketFactory
{
	/**
	* @return ShowBasket
	*/
	public function create();
}

a toto rozhrani neregistrovat nyni jako sluzbu, ale do sekce factories takto:

factories:
	showBasketFactory:
		implement: ShowBasketFactory

nette se nyni postara o vytvoreni tridy, ktera implementuje toto rozhrani (trida je v podstate podobna tride ShowBasketFactory, kterou jsem vytvoril vyse). diky tomu, ze je implementovano to rozhrani, muzes si pote v presenteru pozadat o zavislost uplne stejne, jako v pripade vlastni tridy

EDIT2:
a v cem je toto delsi reseni lepsi nez to, co pise @Šaman? vem si, ze bys tuto komponentu chtel vytvaret treba ve dvou presenterech. nyni by jsi musel injectovat vsechny zavislosti one komponenty do vsech presenteru. a kdyby doslo ke zmene zavislosti teto komponenty, musel by jsi to resit ve vsech presenterech, kde komponentu vytvaris. timhle zpusobem, jak pisu, akorat upravis tridu ShowBasketFactory, v pripade, ze pouzijes DIC factory a to rozhrani, nebudes muset upravovat nic. proste pridas zavislost do konstruktoru ShowBasket a nette se postara o zbytek.

EDIT3:
o DIC tovarnickach najdes jeste dalsi info zde https://doc.nette.org/…tion/factory

Editoval matej21 (9. 6. 2013 19:03)

asi mas spatne odsazeni a factories mas jako sluzbu pod services, treba takhle ma vypadat config:

services:
	sluzba1: Foo
	sluzba2:
		class: Bar
factories:
	tovarnicka1:
		implement: FooBar

EDIT: @batko ok, byl jsi rychlejsi :)

Editoval matej21 (9. 6. 2013 18:35)

@Šaman: j melo by to tak byt, ale myslim, ze jeste to nejde, viz: https://github.com/…Compiler.php#L258 – klic implement je jen pokud je definice non-shared a podle tohohle https://github.com/…Compiler.php#L184 je shared v pripade, ze se definice nachazi v configu pod sekci services

kdyz pouzijes ty DIC tovarnicky, tak pouzi jen tu definici v sekci factories, tu definici v sekci services tedy smaz

Udělal jsem i příklad https://doc.nette.org/…tion/factory

a hází to chybu

Service ‚mailingMessage‘: Unknown key ‚implement‘ in definition of service

jo tak tim to asi bylo :-) nasadil jsem to tam, ale hazi mi to dasi chybu ale to je změnou metod v Nette\Database\Connection::table() is deprecated; use SelectionFactory::table() instead.

tohle jsem vyresil hravě, ale…

Nette\DI\ServiceCreationException

No service of type FrontModule\ShowBasketFactory found. Make sure the type hint in FrontModule\TestPresenter::injectShowBasketFactory() is written correctly and service of this type is registered.

ja proste uz nevim kde muze byt chyba

No tak to vypadá že to funguje. Chtěl bych ti moc poděkovat. není to se mnou lehký :-) a pradvěpodobně zítra přijde další dotaz :-) Ted to jdu otestovat jak se to chová a co to umí :-)

Jednoduše si o ty služby řekni v konstruktoru komponenty, Nette je automaticky předá. Příklad:

class ShowBasket extends Nette\Application\UI\Control
{

    protected $repository;

    public function __construct(FooRepository $repository) {
        parent::__construct();
        $this->repository = $repository;
    }

}

Šaman:

Ano vím, že je to nedoporučované, v podstatě jsem myslel přesně nějakou „supertovárničku“.

Já jsem se tam hlavně poněkud neobratně vyjádřil. Já ten výraz „tahat“ používám jak pro $this->context->... tak pro inject* atd. V podstatě jediné, co pro mě není „tahat“ je explicitní volání new ..., takže chápu, že použití tohoto výrazu může být z mé strany občas trochu zavádějící. Pokusím se příště radši více rozepsat. :D

Já jsem v tom předchozím postu chtěl pouze naznačit, že auto-wiring prostě nikdy nemůže fungovat při explicitním volání new \Todo\ShowBasket();. To jestli už pak použije DIC továrnu, $this->context->…, inject* už jsem neřešil. ;)