Link v modelu

No, „obalování“ výsledků z databáze tagy bys měl nechat až na šabloně :)

Každopádně, metodu link ve svém modelu si zpřístupníš tak, že ho podědíš od Controlu. Ale opravdu takovéto řešení nedoporučuji.

PetrP napsal(a):

Ještě je možné zvrácené řešení Environment::getApplication()->getPresenter()->link(...);. Ale je to už za hranicí normality. ;]

Když už se rozhodnu, že budu vytvářet linky z modelu, což je samo o sobě „zvláštní“, tak mi toto řešení přijde nejčistší…

To ti ale nebrání přesunout to „obalování“ až do šablony, ne? Model data dodá (například celou tabulku v nějakém poli) a v šabloně to projdeš, obalíš, upravíš a vykreslíš.

Jaký smysl by potom měly šablony, kdyby v nich bylo jen {$output} :)

m0t3jl napsal(a):

Mě by docela zajímala ta otázka použití modelu jako komponenty. Když teď máme ty super továrničky, tak mě to už párkrát svádělo si udělat model jako extender Controlu, ale pořád se něčeho bojím, takže jsem to reálně nezkoušel. Nemuselo by to být úplně špatné, kdybych například v presenteru, který renderuje např. pohled na detail produktu v e-shopu, použil model toho produktu jako komponentu, co myslíte?

(Jestli jsem teď někoho neskutečně pohoršil nebo ho přivedl do nauseátního stavu, omlouvám se ;))

Já jsem nad podobnou věcí také přemýšlel, ale co by to přineslo, kromě toho, že by model byl dostupný přes getComponent()? Signály nepřijímá, nevykresluje se se, nepřesměrovává…?

romansklenar napsal(a):

Dávat model jako Control je blbost. Pokud to chcete udělat pro to, aby byl model součástí komponentového stromu, podědil bych ho od Component.

No, to jsem myslel svým příspěvkem. Asi to nebylo zřejmé.

Občas jsem taky v situaci, kdy bych si potřeboval šáhnout na presenter – například pro jazyk, podle kterého bude model vybírat data.

Ale to je dost málo na to udělat model potomkem Component.

Editoval jasir (22. 7. 2009 20:57)

LastHunter napsal(a):

jasir napsal(a):
Ale to je dost málo na to udělat model potomkem Component.

Výhodu vidím v lazy přístupu, tzn., vytvoří se, až když ho potřebuji.

To je pravda, ale to samé vyřeší třeba něco takového:

<?php
public function getModel() {
   static $model;
   if(!$model) {
	$model = new Model;
   }
   return $model;
}
?>

…nebo podobně jedoduché řešení ala továrnička
Ale máš pravdu, tady už nějaké + je.

romansklenar napsal(a):

jasir: Proč by to mělo být málo?

class Model extends Component implements IModel
{
	...

	/**
	 * @return Presenter
	 */
	public function getPresenter()
	{
		return $this->lookup('Nette\Application\Presenter', TRUE);
	}
}

Promiň, teď ti nerozumím. Myslel jsem tím, že přidávat model do stromu jen pro možnost získání prezenteru je málo. Můžu přeci použít třeba (zde na fóru tedy nedávno označené za nenormální) Environment::getApplication()->getPresenter(). Ale stejně – má z hlediska návrhu model vědět o prezenteru? Nebo mi něco uniká?

Editoval jasir (22. 7. 2009 22:17)

BTW: Vzpomínám si, že jsem umisťoval logiku odesílání e-mailu do modelu, takže jsem potřeboval metodu createMyTemplate z Controlu (Model jsem podědil od něj). Je to správné nebo jsem měl e-mail odesílat v souvisejícím Controlu?

Ještě další důvod proč sahat z modelu na presenter:
Když mám validaci v modelu, tak potřebuju vyhazovat případně chybové hlášky, tedy Presenter::flashMessage(). Otázka je jestli to neni špatněj návrh.

PetrP moje modely vracajú len true a false, podľa čoho presenter vyhodnotí úspešnosť operácie a model môže prípadnu chybu zalogovať.

PetrP napsal(a):

Jod napsal(a):

PetrP moje modely vracajú len true a false, podľa čoho presenter vyhodnotí úspešnosť operácie a model môže prípadnu chybu zalogovať.

$model->getNecoById($id) vrací taky true? ;] jen žertuju, rozumim ti.

Oba máte pravdu že je to lepší, na většině míst to tak používám. Jen mám pořád koncepční problem s tím jestli vytváření Forms (kvůly validaci, atd) a případné callbacky nepatří do modelu. Ale protože pak mám potřebu komunikovat z presenterem, tak to asi neni ideální. ;/

O formulářích v modelu se uvažuje a David má kvůli tomu přijít do Nette s něčím novým, aspoň takhle jsem to někde na fóru zahlédl.

Se mi tohle zrovna hodilo, sorry šoupnul jsem to do tohodle threadu, jsem narychlo hledal IModel :-) ale to je fuk.

<?php

class ModelLoader extends Object {

    protected $models = array();

    /**
     * Model loader
     * @param string $name Model name
     * @return IModel
     * @throws MemberAccessException
     */
    public function &__get($name)
	{
        if (isset($this->models[$name])) {
            return $this->models[$name];
        } elseif (class_exists($name . 'model')) {
            $r = new ReflectionClass( $name . 'model' );
            if ($r->implementsInterface('IModel')) {
                $this->models[$name] = $r->newInstance();
                unset($r);
                return $this->models[$name];
            }
            throw new MemberAccessException(ucfirst($name) .
                'Model does not implement IModel.');
        }
        throw new MemberAccessException(ucfirst($name) .
            'Model does not exist.');
	}
}

někde ve startup…

$this->model = new ModelLoader;
$this->model->articles->getArticle();

Editoval amsys (26. 10. 2009 7:12)