Komponenta v modelu, nebo service?

Super, to mě vůbec nenapadlo, že to bude tak jednoduchý :).

Ještě bych měl jeden problém. V komponentě mám jeden submit, kterej potřebuju používat v HomePresenteru, ale při přidání článku (ArticlePresenter) bych potřeboval zobrazit jen ten selectBox bez tlačítka (tam mi stačí když si pak vytáhnu ID kategorie a uložím do DB).

Je možný to nějak udělat a nebo musím mít dvě prakticky identický komponenty, ale jednu s tlačítkem a druhou bez.

Díky.

EDIT: jediný co mě napadlo, tak to tlačítko schovat pomocí CSS, ale to asi nebude moc správně…

Tomáš Votruba napsal(a):

Ahoj, jestli to dobře chápu, potřebuješ zobrazit stejnout komponentu na více presenterech.

Pokud ano, stačí ji přesunout do BasePresenteru.

Dalším krokem by mohlo být vyčlenění komponenty do samostatné třídy (názorný příklad), abys měl BasePresenter čistý.

Editoval Freestyler (6. 7. 2014 13:10)

Díky, vypadá to moc pěkně, jen mi to nefunguje :(. Pročetl jsem jak best practise:formuláře jako komponenty, tak návod, který jsi postoval ty a dostávám:

Method AdminModule\HomepagePresenter::createComponentArticleMenu() did not return or create the desired component

Mám to takhle:

ComponentFactory

<?php

namespace App\Model;

use Nette\Application\UI\Form;

class ComponentFactory  {

protected function createComponentArticleMenu() {

     $form = new Form();
     $form->addSubmit('send', 'Odeslat');
	return $form;
}

}
interface IArticleMenuFactory {

   /** @return ComponentFactory */
    function create();
}

HomepagePresenter
<?php

namespace AdminModule;



/**
 * Homepage presenter.
 */
class HomepagePresenter extends BasePresenter
{

    /** @var \App\Model\IArticleMenuFactory
     *  @inject
     */

    public $componentFactory;

    protected function createComponentArticleMenu() {

	 return $this->componentFactory->create();
    }
}

V configu mám zaregistrován interface:

services:
	- App\Model\IArticleMenuFactory

V šabloně pak klasicky používám {control articleMenu}

Tomáš Votruba napsal(a):

Na to krásně hodí mít komponentu v samostatné třídě.

Pak bys mohl udělat něco jako:

HomepagePresenter.php

protected function createComponentArticleMenu()
{
	return $this->articleMenuFactory->create();
}

ArticlePresenter.php

protected function createComponentArticleMenu()
{
	$control = $this->articleMenuFactory->create();
	$control->useSubmit(FALSE);
	return $control;
}

Jak na to se dočteš v pěkném návodu

Jéje, tak to mě nenapadlo. A tam musím tu šablonu ručně vykreslit, jak je popsáno tady https://doc.nette.org/…n/components ? Nebo to jde nějak obejít :).

Paráda, díky. Škoda, že to v tom návodu není napsáno, začátečníky to může asi dost zmást.

matej21 napsal(a):

Pokud ma komponenta pouze vykreslovat svoji podkompoentu, ktera je formularem, tak bude stacit:

public function render()
{

$this['articleMenu']->render();
}

Šaman napsal(a):

Spíš doporučuji mít plnohodnotnou šablonu (já ji pojmenovávám stejně jako komponentu)

Jaká je přidaná hodnota? Je to víc psaní, nemluvě o exekuci šablony a Latte navíc…

Ahoj,
přepsal jsem kód s použitím generovaných továrniček a interfacu následovně:

IArticleMenuFactory

<?php

namespace App\Components;


interface IArticleMenuFactory {
    /** @return ArticleMenu */
    function create();
}

Továrnička ArticleMenu

<?php

namespace App\Components;

use Nette\Application\UI\Form;

class ArticleMenu extends \Nette\Application\UI\Control  {

/**
 * @var App\Model\Articles
 * inject
 */
    public $articles;

    protected function createComponentArticleMenu() {

     $form = new Form();
     $form->addSelect('category','Kategorie', $this->articles->getArticleCategories()->fetchPairs('id', 'category_name'));
     return $form;
}
public function render()
{

$this['articleMenu']->render();
}


}

Model Articles, metoda pro vytažení kategorií z DB.

*
   * Get all categories
   */
  public function getArticleCategories() {

	return $this->database->table('categories');

  }

Použití v ArticlesPresenteru

/**
     * @var App\Components\IArticleMenuFactory
     * @inject
     */

    public $articleMenuFactory;

    protected function createComponentArticleMenu()
    {
	return $this->articleMenuFactory->create();

    }

První problém je, že kód nefunguje, konkrétně presenter ArticleMenu, tam dostávám Call to a member function getArticleCategories() on a non-object, mám použít konstruktor? Vždyť @inject anotace by měla být identická ne?

Druhý problém je, že tohle všechno dělám proto abych neopakoval kód a měl znovupoužitelnou komponentu. Tzn. třeba v HomepagePresenteru budu muset zase tupě injectovat IArticleMenuFactory, definovat proměnnou $articleMenuFactory a vytvářet komponentu (createComponentArticleMenu()? Je to tak správně, nebo něco vyloženě nechápu na generovaných továrničkách? Nikde jsem nenašel dostatečně vysvětlující příklad pro začátečníka :(.

Díky.

Komponentu chci používat jen ve dvou presenterech, takže se asi to copy&paste přežije no. Můžu tedy používat jen @inject anotaci a nebo je lepší konstruktor (ten je asi čistější).

Můžeš mi ještě prosím mrknout co by tam tak mohlo být špatně a nette na mě řve s Call to a member function on a non object?

Díky.

matej21 napsal(a):

První problém je, že kód nefunguje, konkrétně presenter ArticleMenu, tam dostávám Call to a member function getArticleCategories() on a non-object, mám použít konstruktor? Vždyť @inject anotace by měla být identická ne?

defaultne je inject anotace povolena pouze pro presentery. Muzes to explicitne zapnout pro konkretni sluzbu, ze v neonu k jeji definici uvedes inject: true, konkretne:

services:
    -
        implement: App\Model\IArticleMenuFactory
        inject: true

ale je lepsi do komponent injectovat pres konstruktor

Druhý problém je, že tohle všechno dělám proto abych neopakoval kód a měl znovupoužitelnou komponentu. Tzn. třeba v HomepagePresenteru budu muset zase tupě injectovat IArticleMenuFactory, definovat proměnnou $articleMenuFactory a vytvářet komponentu (createComponentArticleMenu()? Je to tak správně, nebo něco vyloženě nechápu na generovaných továrničkách? Nikde jsem nenašel dostatečně vysvětlující příklad pro začátečníka :(.

pokud jednu komponentu pouzivas ve vsech presenterech (nebo v naproste vetsine), presun onu createComponent* metodu do base presenteru. par radku kodu muze take usetrit autowired component factories. pokud komponentu pouzivas pouze v nekterych presenterech a chces se vyhnout copy-pastovani tech createComponent*, muzes (za predpokladu, ze pouzivas php 5.4+) vytvorit traity

Ok, díky.

Ještě teda nevím, kde mám chybu v tom kódu, koukám do toho už dobu a pořád to nevidím, popostrčí mě prosím někdo?

Pavel Macháň napsal(a):

Freestyler napsal(a):

Komponentu chci používat jen ve dvou presenterech, takže se asi to copy&paste přežije no. Můžu tedy používat jen @inject anotaci a nebo je lepší konstruktor (ten je asi čistější).

Můžeš mi ještě prosím mrknout co by tam tak mohlo být špatně a nette na mě řve s Call to a member function on a non object?

Díky.

Konstruktor je určitě čistější a neporušuje zapouzdření (@inject musí mít public property). Osobně se snažím používat konstruktor všude (až na BasePresenter).

@Freestyler viz https://forum.nette.org/…nebo-service#…, pokud chces v komponentach pouzit @inject, musis to zapnout, jinak musis pouzit konstruktor

public function __construct (\App\Model\Articles $articles, \App\Model\Articles $categoryID) {
WTF? Takhle se předávají služby, takže teď to do $categoryID předá stejnou instanci jako do articles. Parametr si musíš predávat ručně, ideálně v tovární metodě v presenteru. Buď si najdi, jak předávat parametr metodě create() (budeš muset upravit rozhraní a config), nebo si na to $id napiš setter a v tovární metodě createComponent… si po vytvoření komponenty pomocí služby ještě zavolej ten setter.

A ta továrnička v konstruktoru? Předáváš ji parametr? Jo a to $categoryID dej jako první parametr konstruktoru, až pak ty autowirovaný.

Sem musíš ještě předat to $id. Kde jinde by ho aplikace vzala? Celé to přepisování bylo kvůli tomu, abys naučil metodu create akceptovat parametr. A presenter už by měl vědět, pro jakou kategorii tu komponentu vytváří.

<?php
    protected function createComponentArticleMenu()
    {
        return $this->articleMenuFactory->create($categoryId);
     }
?>

Tak se pro začátek vyprdni na generované továrničky a napiš si ji po svém (ještě mě napadlo jakou máš verzi Nette, protože ještě nedávno se tyhle složitější s parametrem musely psát ručně).

Tzn. Místo interface IArticle…Factory budeš mít třídu Article…Factory, která bude obsahovat metodu create($categoryId), uvnitř bude obyčejné vytvoření pomocí operátoru new, nastavení toho $id a vrátíš nakonfigurovanou instanci.
Tuhle tovární třídu si zaregistruješ v configu jako službu (takže do konstruktoru můžeš normálně předávat závislosti) a pak ji předáš do presenteru a použiješ jako doteď. Jinak řečeno v configu budeš mít

services:
	- App\Model\ArticleMenuFactory

a v presenteru ji budeš injectovat bez toho I.

Pak se ti to bude lépe ladit a taky pochopíš, co vlastně pak dělá ta generovaná továrna. Jen ti ušetři trochu psaní.

Jinak v tom pastbinu máš špatně odsazené

-
    implement: IArticleMenuFactory
    parameters: [categoryID]
    arguments: [%categoryID%]

ale předpokládám, že to by házelo jinou chybu a že to máš asi jen špatně vykopírované.

Edit: Resp. úplně nejjednodušší bude pro začátek nemít žádnou tovární třídu a prostě si tu komponentu vytvořit v presenteru v createComponent… pomocí new. (Repozitář si injectneš do presenteru a předáš jako parametr do konstruktoru).
Tohle odladíš a pak si vytvoříš tu tovární třídu (ta se dělá proto, abys nevytvářel tu komponentu na více místech pokaždé jinak).
A pokud budeš chtít (z hlediska čistoty návrhu už ti to nic nepřinese), tak pak můžeš přepsat továtní třídu na generovanou.

Editoval Šaman (21. 7. 2014 14:41)