Dynamické vyžiadanie služby

Tak sa mi to celkom rýchlo podarilo vyriešiť.

Bude v pohode, ak to budem používať takto?

<?php
// container je injectnuty Nette\DI\Container
$this->container->getService("mailer.mailerService")->getAllByQuery($query);
?>

Ta služba bude registrovaná jen jednou DIC? Pokud jo tak ji můžeš normálně přes autowire předat do konstruktoru (nebo kamkoliv jinam).

@GEpic to viem, ale to by moj problem neriesilo :)

jak píše @CZechBoY, viděl bych to na vytvoření nějaké třídy/služby. něco jako:

<?php
class SearchService {
    /**
     * @var ModuleOne
     */
    private $moduleOne;
    /**
     * @var ModuleTwo
     */
    private $moduleTwo;

    /**
     * SearchService constructor.
     * @param ModuleOne $moduleOne
     * @param ModuleTwo $moduleTwo
     */
    public function __construct(ModuleOne $moduleOne, ModuleTwo $moduleTwo)
    {
        // autowiring komponent
        $this->moduleOne = $moduleOne;
        $this->moduleTwo = $moduleTwo;
    }

    /**
     * @param string $searchPhrase
     * @param \Nette\Security\User $user
     * @return array
     */
    public function search($searchPhrase, \Nette\Security\User $user)
    {
        // klidně to může být nějaká služba, která ti vrátí dostupné moduly
        $modules = $user->getIdentity()->getAvailableModules();

        $results = [];
        foreach ($modules as $moduleName) {
            $results += $this->getModule($moduleName)->search($searchPhrase);
        }

        return $results;
    }

    /**
     * @param string $name
     * @return IModule
     */
    private function getModule($name)
    {
        $map = [
            'moduleOne' => $this->moduleOne,
            'moduleTwo' => $this->moduleTwo
        ];

        if (!array_key_exists($name, $map)) {
            throw new \InvalidArgumentException;
        }

        return $map[$name];
    }
}
?>

Dá sa aj tak, zatiaľ som to vyriešil tak, že akonáhle mám modul v konštante SEARCH_MODULES tak to znamená, že som jej vytvoril službu aj na vyhľadávanie a priamo DI container mi ju vráti a môžem ju požiadať o vrátenie už nastavenej, ready šablony. V skratke takto:

<?php
	/**
     * Performs search in certain module
     * and renders out given result
     * @param string $resourceName
     */
    private function performSearch($resourceName) {

        $template = $this->getTemplate();

        if (in_array($resourceName, self::SEARCH_MODULES)) {
            $moduleSearch = $this->container->getService($resourceName.".".$resourceName."ModuleSearch");
            $moduleSearch->setup($this->getQuery());
            $template = $moduleSearch->getTemplate();
            $template->render();
        }
    }
?>

@Oli Samozrejme, nie je to finálne a netvrdím, že najlepšie a vlastne celkovo cítim code smell v tom, že niečo musí mať presný názov :-\

Každý modul registrujem ako extension: ArticleExtension, MailerExtension, InvoiceExtension atd atd atd…
Takisto v extensionoch definujem aj cesty pre slovníky pre Kdyby\Translation, takže viem o čom hovoríš.

To čo vlastne popisuješ v treťom odstavci sa mi páči, celkom sa s tým návrhom stotožňujem, čiže:

1. Vytvorím si napr.:

<?php
	- class ContactModuleSearch implements IModuleSearch ()
	- class InvoiceModuleSearch implements IModuleSearch ()
	- class MailerModuleSearch implements IModuleSearch ()
?>

2. V BaseModule, ktorý je rodičom všetkých modulov, mám vytvorenú triedu, ktorá jednoducho povedané získa z týchto searchov template z výsledkami vyhľadávania.

<?php
class GeneralSearch extends Control {

    /** @var ResourceService */
    private $resourceService;

    /** @var User */
    private $user;

    /** @var Container */
    private $container;

    private $query;

    /** List of modules that have search */
    const SEARCH_MODULES = ["mailer", "contact", "invoice"];

    public function __construct($query, ResourceService $resourceService, User $user, Container $container) {
        $this->query = $query;
        $this->resourceService = $resourceService;
        $this->user = $user;
        $this->container = $container;
    }

    /**
     * Checks allowed modules for current user
     * and performs search in them
     * For each module it renders its search result
     */
    public function render() {

        $activeResources = $this->resourceService->getAllActive();

        foreach ($activeResources as $activeResource) {
            if ($this->user->isAllowed($activeResource->getName(), "view")) {
                $this->performSearch($activeResource->getName());
            }
        }
    }

    /**
     * Performs search in certain module
     * and renders out given result
     * @param string $resourceName
     */
    private function performSearch($resourceName) {

        $template = $this->getTemplate();

        if (in_array($resourceName, self::SEARCH_MODULES)) {
            $moduleSearch = $this->container->getService($resourceName.".".$resourceName."ModuleSearch");
            $moduleSearch->setup($this->getQuery());
            $template = $moduleSearch->getTemplate();
            $template->render();
        }
    }

    private function getQuery() {
        return $this->query;
    }

}
?>

Aby sa zachovala štruktúra a zmenil som obsah funckie performSearch, tak potrebujem v nej pracovať s triedami, ktoré majú implementované rozhranie IModuleSearch a nad nimi zavolať metodu, ktorá vráti template na základe nejakého $query stringu.

Otázkou teda je ako do tohto controlu dostať(injectnúť triedy, ktoré implementujú dané rozhranie) a ako sa s ním dá následne pracovať… Skúsim pohľadať, ale idem vybehnúť na obed :)

Zatiaľ ďakujem za nakopnutie a nápady

Editoval iNviNho (29. 9. 2016 12:07)

Za mě, pokud musíš udělat něco takového:

const SEARCH_MODULES = ["mailer", "contact", "invoice"];

Pak moduly a veškerá abstrakce postrádá smysl. Navíc když už to registruješ jako extension, pak už to nějaký zápis v configu má, proč se toho nedržet?

Editoval GEpic (29. 9. 2016 12:36)

@iNviNho tak si vytáhni službu s tím, že ji nepotřebuješ getByType($type, $need=false) a pak zkontroluj jestli ti teda DIC něco vrátil nebo ne ($service===null).

Editoval CZechBoY (29. 9. 2016 14:00)

@CZechBoY ďakujem za nakopnutie
@Oli Super super super
Ďakujem vážim si to aj váš čas.

Celkovo môže byť problém aj v tom, že nepoznám až tak dobre silu DI a čo všetko pomocou nej dokážeme. Zajtra to v robote skúsim a sám som zvedavý ako to celé funguje a čo mi vráti napr. new Statement()

btw nemôže nastať ešte nejaky problém v tom, že triedu GeneralSearch vytváram cez IGeneralSearch, kedže query jej musím vždy dodať ručne?

<?php
/**
 * ....
 */
interface IGeneralSearch {

    /**
     * @return GeneralSearch
     */
    function create($query);

}
?>