Ako správne získavať dáta z databáze a zobrazovať ich

Ano, tento zapis je v souladu s filosofii NDBT, ale: https://forum.nette.org/…principy-mvc

Myslím, že to je přesně to, na co narážel matej21. Ten výše uvedený kód funguje, ale správně by veškeré připojení k databázi měl obstarat model a presentery by měly komunikovat pouze s modelem a nikoli s databází.

Tiez by ma zaujimalo, ako taky model v Nette vyzera, priklad na model som nenasiel, len ako pouzit model v presenteri.

Nejake zakladne pravidla a tak, napr. insertovanie, edits atd.

Takže mi zrejme nezostáva nič iné len experimentovať a študovať kódy iných :D

Pokud je nějaký příklad na správný model, tak se také hlásím.
@Lkopo Ale mohu lehce popsat architekturu svého modelu, který používám.
Je to vlastně spousta tříd, které injektuji do presenteru jako závislosti.
A ty třídy dělají aplikační logiku, mají napojení na databázi a tak podobně.
Mám například v modelu třídu UserHandler, která má veškeré metody potřebné pro práci s uživateli a každý presenter, který ji chce, ji dostane přes DI.
malá ukázka:

<?php

    private $customOtherDependecyFromModel;

    function __construct(dependencies) {
		//injecting all dependencies
    }

 	public function authenticate(array $credentials) {
        //some magic
        return $user;
    }

    public function getUserFromLoginName($loginName) {
	    //some magic
        return $user;
    }

    public function registerNewClient($firstName, $lastName, $email, $linkToAccountActivation) {
        //some magic, registering to database, sending registration e-mail
	}
    public function registerNewAdmin($firstName, $lastName, $email, $link) {
        //some magic, registering to database, sending registration e-mail
    }

	public function promoteToLector(Client $client, $commission) {
		//some magic, promoting and sending e-mail
    }

    public function activateAccount($activationParameter) {
        //some magic
    }

    public function setNewPassword($password, $activationParameter) {
        //some magic
    }
?>

a potom mám podobné třídy na další věci, mám messageHandler, který obstarává samotné odesílání zpráv, několik database handlerů, které mají samotné připojení k databázi (tato třída má pouze database handler jako závislost a sama přímo napojení na databázi nemá), notificationHandler, který generuje novinky o tom, co se děje, fileHandler, který obsluhuje úložiště uživatelů a tak podobně…

Editoval Azathoth (18. 9. 2014 20:35)

@PeterB doporučuji toto: http://www.zdrojak.cz/…neho-navrhu/ dalo mi to hodně.
(pokud melu blbosti, opravujte mne)
U malých projektů je to jedno, ale u velkých už na tom záleží.
Snaž se, aby každá třída měla zodpovědnost pouze za jednu věc. Tedy presenter má mít na starosti pouze zobrazování a nějaké úkolování modelu, takže v presenteru by mělo být akorát
$this->template->firmy = $this->mujDatabazovyModel->dejMiVsechnyFirmy

A třída MujDatabazovyModel by měla poskytnou všechny firmy.
Pokud jsem správně pochopil mvc a single responsibility principle, tak presenter, tedy to, co poskytuje data uživateli, by vůbec neměl vědět o tom, co se používá pro připojení k databázi.

@PeterB je a není to špatně. To co jsi udělal ty je VC (template + presenter). Ono to takhle je popsáno i v dokumentaci. Byly tu i nějaké diskuse jestli to je správně nebo špatně. Jde o to aby se začátečník „nezahltil“. Já jsem třeba začínal stejně, po napsání 2 presenterů jsem hodil práci s databází do modelu, ale komponenty vytvářel v presenteru. Pak jsem experimentoval, že jsem třeba formuláře vytvářel jako potomky třídy \Nette\Application\UI\Form, pak jako \Nette\Application\UI\Control až nakonec teď téměř všechny služby generuju pomocí továrničky a pokud je to trochu komplexní tak z toho dělám extension.

Tím chci říct, že to co jsi udělal ty je první krok k tomu ten kód napsat ve správném MVC. K tomu ale musíš dospět postupně. Něco uděláš a řekneš si, sakra tohle bych mohl udělat obecněji. Pokud si to neřekneš, tak to asi nepotřebuješ. Například firemní presentace o 10 stránkách a blogu by klidně šla napsat způsobem jak jsi naznačil. ;-)

Azathoth napsal(a):

$this->template->firmy = $this->mujDatabazovyModel->dejMiVsechnyFirmy

…tomuto chápem, mám teda iba jedinú otázku, ako by sa mal volať model ktorý vytvorím (myslím, podľa obecných zásad Nette, alebo modely sú načítané všetky – usudzujem podľa use Nette, App\Model; ), kde potom definujem triedu a tieto metódy?

@PeterB
Ve složce, tedy i v namespace App\Model si vytvoříš PHP Class, nějak si ji pojmenuješ a začneš tam psát ty metody, dáš tam jako property $database a tak dále. A potom ji v neon.config přidáš mezi služby (jak je services: tak tam přidáš nový řádek, kde bude pomlčka a jméno té php class spolus s celým namespace, takže tam bude:
services:

• App\Model\MyDatabaseHandler (nebo jiný název, který zvolíš pro tu třídu)

) a injectuješ do presenteru, asi takhle:

/**
* @var \App\Model\MyDatabaseHandler
* @inject
*/
public $myDatabaseHandler;

Podívej se na nette sandbox, jakým způsobem je dělaný UserManager. Tak takhle bys to měl mít, obdobně.
Ale název té třídy je čistě na tobě, prostě to pojmenuj tak, abys tomu rozuměl a aby název přibližně odpovídal tomu, co to bude dělat.

@PeterB Na foru bude urcite nekolik ukazek kodu jak se to ma delat. Myslenka je ale zhruba takovahle:
Napises nejakou tridu, ktera se stara o praci s databazi, rekneme treba clanky:

namespace App\Model;

class ArticleRepository extends \Nette\Object
{

	private $context;

	function __construct(\Nette\Database\Context $context)
	{
		$this->context = $context;
	}

	public function findArticleById($id)
	{
		return $this->context->table('articles')->get($id);
	}

Potom tu třídu zaregistruješ jako službu do config.neon (musí být někde, kam „vidí“ Nette Autoloader, typicky řekněme někde v App + podsložky). Protože jsi to zaregistroval jako službu, tak ti tam ten Context Nette injectne automaticky.:

services:
	- App\Model\ArticleRepository
	# můžeš si tu službu i pojmenovat - mujClanek: App\Model\ArticleRepository
	# a potom k tomu přistoupis jako k @mujClanek

To že jsi to zaregistroval jako službu v configu ti dovoluje to načíst pomocí DI v presenteru.

class ArticlePresenter extends BasePresenter
{

	public function __construct(\App\Model\ArticleRepository $articleRepository)
	{
		$this->articleRepository = $articleRepository;
	}


	public function renderDetail($id)
	{
		$this->template->article = $this->articleRepository->findArticleById($id);
	}
}

@Bambulko komponenty jsou jednoduše řečeno znovupoužitelné kusy kodu. Například paginator (stránkovač) chceš vždycky, když vypisuješ nějakej přehled např. článků. Je blbost to psát pořád dokola, tak si uděláš komponentu, které předáš celkový počet článků a kolik jich chceš zobrazit a ona ti to „rozřeže“. Potom když chceš takhle vypsat novinky, tak už si zavoláš jen tu komponentu a nastavíš ji.

Továrnička není nic jiného než návrhový vzor factory. Prostě metoda, která vytvoří jinou třídu. Její výhoda je, že pokaždé vytvoří novou instanci.

Editoval Oli (18. 9. 2014 22:11)

Mám takú otázku ohľadom toho, či záznamy vkládať cez presenter alebo model, keďže takisto musím v presenteri povedať, že tieto $values chcem vložiť/upraviť.

Čiže v modeli mám to asi takto

public function findOneById($id)
{
    $post = $this->database->table(self::TABLE_NAME)->get($id);

    return $post;
}

A v presenteri mám:

$post = $this->postRepository->findOneById($postId);
$post->update($values);

Je to správne riešenie? Keďže na insert sa v modeli robí metóda (napr. add), ale na update alebo delete si nie som istý.

Editoval Lkopo (18. 9. 2014 23:27)

@Lkopo nevím, jak ostatní, ale já mám na všechno model, v presenterech se snažím nic neupravovat, ale vždy volat model. I když tohle má asi každý programátor jinak.

Editoval Azathoth (18. 9. 2014 23:31)

Ano, je to víc psaní než kdybys to dělal v presenteru. Ale já se snažím držet single responsibility principle a to toho mi úprava dat presenterem nějak nesedí.

No a ako poviem modelu len po odoslaní formulára updatne tabuľky? V návode bolo v presenteri:

$post = $this->database->table('posts')->get($postId);
$post->update($values);

No a ja mám

$post = $this->postRepository->findOneById($postId);
$post->update($values);

Naozaj je lepšie to spraviť týmto štýlom?

$post = $this->postRepository->findOneById($postId);
$this->postRepository->updateData($post, $values);

Editoval Lkopo (19. 9. 2014 0:20)

Naozaj je lepšie to spraviť týmto štýlom?

$post = $this->postRepository->findOneById($postId);
$this->postRepository->updateData($post, $values);

Ano ;)