Potlačení kruhovych závislostí při vytváří služeb v DI

@hAssassin: Vidím dvě cesty, jak z toho ven:

1. Akademická – uprav si model, aby neměl kruhové závislosti
2. Pragmatická – vytvoř si service locator a injektni ho do všech repozitářů (při injektnutí se repozitář může automaticky do service locatoru zaregistrovat)

Jinak ješte abych to uvedl na konkrétním případě. Mám produkty a kategorie. Přičemž vazba mezi nimi je m:n. Mám tedy ProductRepository a CategoryRepository a chtěl bych volat nad ProductRepository metodu getCategories($productId) a současně nad CategoryRepository volat getProducts($catId).

Současně si ale myslím, že by nebylo správný hrabat v ProductMapperu do tabulky category ani v CategoryMapperu do tabulky product. Mapper at si pracuje pouze nad svojí tabulkou, resp. několika tabulkami, které tvoří logický celek (což zde už neplatí).

Pak ale potřebuju mít kruhovou závislost…

heh :) shit… A co si myslíš o tom tagování a injektování v initialize() přes afterCompile() ?

@enumag já bych to zobecnil na skupinu tabulek pod jedním repozitářem, v případě, že ty tabulky jsou úzce svázány a navenek je není potřeba rozlišovat (viz třeba Closure Tables). Ale jinak to dělám úplně stejně tj. přes fasádu.

@enumag > díky za odpověď. Ano, nad tím sem taky uvažoval, ale nejsem si jistý jestli by to vlastně vubec něco řešilo, protoze by se ta kruhová závislost přenesla jen z repozitářů do fasád. Nebo si teď nejsem jistý jak to přesně myslíš:

1. fasáda přímo pracuje z několika repositáři, které se pouze injektnou do ní při jejím vytvoření, nebo
2. fasáda pracuje s ostatními fasádami a přes ně přistupuje až k repozitáři (čili injektují se zase fasády).

Předpokládám, že myslíš asi a), protože b) vlastně nic neřeší?

@enumag > ano chápu a asi to nejčistčí řešení jak z toho ven. Já mám problém ten, že chci mít možnost přes entitu přitupovat k jiným entitám přes cizí klíče. Čili když se entita (např produkt) inicializuje, nastaví si categoryId na id daný kategorie (za předpokladu, že je tam vazba 1:n). A pokud někde (třeba až v šabloně) zavolám $entity->getCategory() natáhne a vrátí mi to právě danou kategorii (pokud existuje). K tomu sice musí být entita závislá na repozitáři, což mi zase tak nevadí, ale repozitář (produkt) je pak závislí na jiném (category).

Možnost, že by entita byla závislá místo toho nebo současne přímo na facádě, mi nepříjde jako zrovna nejlepší.

Aha, YetORM presně neznám, každopádně to přímo v entitě nemám, ta metoda getCategories() je pouze zkratka, uvnitř se volá repozitář pro danou entitu a v něm ses volá repozitář cizí, proto ho tam potřebuju. Představa, že bych z repozitáře nebo z entity volal fasádu a ta by volala cizí repo se mi nelíbí a nelíbí se mi ani fakt, že bych o tuto funkcionalitu přisel :-)

Jo, taky mi to nedělá úplně dobře… :-) Jak jsem psal, YetORM moc neznám, prakticky vůbec, mohl bys nastínit jak tu metodu implementovat?

Napadá mě určitá struktura modelu (zatím opravdu jen nápad): Mějme nějaký modelContainer (něco jako service locator), který bude zastřešovat všechny repository a k nim příslušné entity a mappery. Zároveň všechny entity budou striktně propojené s repozitářem např. nějak takto:

$entity = $repository->createEntity();
//nebo
$entity = new Entity();
$repository->attach($entity);

pak požadavek volaní metod ekvivalentních metodám ref resp. related z NDB by probublal z dané entity/kolekce přes repozitář do modelContainer, kde by se zavolal patřičný repozitář, který by to zpracoval a vrátil. A teď se do mě pusťte :D

Edit: vyřešilo by to např. teoreticky neomezené odkazování přes cizí klíč.

Editoval castamir (20. 4. 2013 16:36)

@castamir > ano, o něco takového se pokouším, akorát že bez service lokatoru. Tady by se však celkem mohl hodit. A ano, chci mimojiné dosáhnout nekonečné odkazování přes cizí klíč :-)

Každopádně mě teda ještě napadlo povolit možnost, aby fasáda mohla přistupovat k více repozitářům, tak jak psal enumag, potom sama fasáda musí všechny tyto repozitáře vyžadovat a ty pak sama rozstrkat do repozitářů dál. Např. ProductFacade vyžaduje ProductRepos a CategoryRepos, ty se do ní injektnou a ona pak ví že CategoryRepos musí předat do ProductRepos a ProductRepos do CategoryRepos. Címž se sice kruhová závislost vytvoří, ale až po tom, co jsou inicilizovaný všechny potřebný repozitáře.

Má to dvě nevýhody:

1. porušuje to SRP, protože fasáda částečně přebírá činnost kontejneru/service locatoru,
2. porušuje to DRY, protože teoreticky bude potřeba totéž udělat i v CategoryFacade (stejný kód, šlo by to ale asi pořešit automaticky už v CompilerExtension).

Co myslíte?

Konfigurační soubor by neměl nahrazovat programování. Takže úvaha by se měla odvíjet od toho, jak bych stejný problém řešit čistě v PHP. No a vyrobil bych si na to továrničku.

Tedy místo původního

services:
	a: ClassA
	b: ClassB

kde potřebujeme objekty @a a @b nějak propojit, si vytvořím továrnu:

class ABFactory
{
	function __construct(A $a, B $b)
	{
		// zašmodrchám je
		$a->setB($this->b);
		$b->setA($this->a);

		$this->a = $a;
		$this->b = $b;
	}

	function getA() {
		return $this->a;
	}

	function getB() {
		return $this->b;
	}
}

A využiju jejích služeb:

services:
	abFactory: ABFactory(ClassA, ClassB)
	a: @abFactory::getA
	b: @abFactory::getB

Co bys na tom chtěl automatizovat?

Snažím se implementovat Davidovo řešení, ale dostávám se do fáze "Reference to missing service @abFactory::getA", v jaké verzi Nette tenhle postup funguje, prosím? Jsem s aktuálním 2.0 v pytli?

Jak bych pak prosím nahradil službu definovanou takto?

a:
		class: A
		setup:
			- setXY(%xy%)

Edit:
Možná je to tím že neumím továrničce předat ty původní modely?

testFactory: Models\TestFactory(Models\A, Models\B)

mi vygeneruje

/**
* @return Models\TestFactory
*/
protected function createServiceTestFactory()
{
	$service = new Models\TestFactory('Models\\A', 'Models\\B'');
	return $service;
}

Ale ostatní služby, které dřív vyžadovaly službu A mi po přepsání a na:

a:
	@testFactory::getA

hlásí že služba typu Models\A neexistuje a že mám zkontrolovat typehint.

TestFactory vypadá takto:

<?php
namespace Models;

class TestFactory {

    public function __construct(\Models\A $a, \Models\B $b) {
        $this->a = $a;
        $this->b = $b;
    }

    public function getA() {
        return $this->a;
    }

    public static function getB() {
        return $this->b;
    }

}

Editoval miler (22. 10. 2013 11:17)

Nedokázal by mi prosím někdo zkušenější poradit jak se z této šlamastiky vymotat? Už jsem prošel forum 4× tam a zpět, vyzkoušel spousty tipů a triků ale ne a ne s tím hnout. Díky moc.

Dočasně jsem to vyřešil tak že jsem ty metody co potřebují jiný model dal do jiného spolčného modelu.

Editoval miler (24. 10. 2013 12:18)