Čo je to model a načo slúži?

Model ti má zapouzdřovat konkrétní chování. Model by ses neměl snažit dělat univerzální jako v tvém případě. Zjednodušeně řešeno v modelu bys měl mít volání té tvé getTable.

@mystik
No ale mi to príde ako zbytočná vrstva navyše. Keď potrebujem len nejaké základné dáta, ktoré nemusím ošetrovať, len zistím, či nie sú false napr. volanie $db->table(‚authors‘). Pre toto mi to nedáva zmysel. Či je to len dobrá konvencia?

Editoval Čamo (6. 8. 2014 8:53)

@mystik
Hej to čo píšeš dáva zmysel.
Ten môj registerPresenter vyzerá takto:

<?php

namespace App\Presenters;

use	Nette,
	Nette\Security\Passwords,
	App\Model\UserModel;

/**
 * Registration presenter.
 */
class RegistPresenter extends \App\Presenters\BasePresenter
{
	/** @var App\Model\UserModel */
	private $userModel;

	public function __construct(UserModel $userModel)
	{
		$this->userModel = $userModel;
	}

	/**
	 * Registration form factory
	 * @return Nette\Application\UI\Form
	 */
	protected function createComponentRegistForm()
	{
		$form = new Nette\Application\UI\Form;
		$form->addText('username', 'User name:')
			->setRequired('Vyplňte prosím meno.')
			->setAttribute('class', 'formEl');

		$form->addPassword('password', 'Password:')
			->setRequired('Zadajte prosím heslo.')
			->addRule($form::MIN_LENGTH, 'Zadajte prosím heslo s minimálne %d znakmi', 3)
			->setAttribute('class', 'formEl');

		$form->addPassword('password2', 'Password check:')
			->setRequired('Zadajte prosím heslo.')
			->addRule($form::EQUAL, 'Heslá sa nezhodujú. Zopakujte prosím kontrolu.', $form['password'])
			->setAttribute('class', 'formEl');

		$form->addText('email', 'Email:')
			->setRequired('Zadajte prosím emailovú adresu.')
			//->addCondition()
			->addRule($form::EMAIL, 'Nezadali ste platnú mailovú adresu. Opravte si prosím chybu.', $form['password'])
			->setAttribute('class', 'formEl');

		$form->addSubmit('send', 'Registrovať');

		$form->onSuccess[] = $this->registFormSucceeded;
		return $form;
	}

	public function registFormSucceeded($form)
	{
		$values = $form->getValues();
		$hash = Passwords::hash($values['password']);
		$arr = array('password' => $hash,
					'username' => $values['username'],
					'email' => $values['email'],
					'status' => 10,
					'created' => time()
					);

		try
		{
			$user = $this->userModel->RegisterUser($arr, 'users', $values);
		}
		catch(\App\Model\Exceptions\DuplicateEntryException $e)
		{
			$form->addError($e->msg);
			return;
		}

		$this->userSess->username = $user['username'];
		$this->userSess->id = $user['id'];
		$this->userSess->created = $user['created'];
		$this->userSess->status = $user['status'];

		$this->flashMessage('Vitajte '.$values['username'].'. Vaša registrácia bola úspešná a loli ste automaticky prihlásený(á).');
		$this->redirect(':Default:');
	}
}

Neviem, či to hashovanie patrí do presentera, či do modelu…
A model vyzerá:

namespace App\Model;

use 	Nette,
	Nette\Security\Passwords,
	Nette\Diagnostics\Debugger;

/**
 * @method RegisterUser
 */
class UserModel
{
	/** @var Nette\Database\Context @inject */
	public $database;

	public function __construct()
	{

	}

	/**
	 * @param  array $params
	 * @param  string  $table
	 *
	 * @return activeRow
	 *
	 * @throw  Nette\InvalidArgumentException
	 * @throw  Model\Exceptions\DuplicateEntry
	 */
	public function RegisterUser($params, $table, $values)
	{
		if(!is_array($params))
		{
			throw new Nette\InvalidArgumentException('$params must be an array');
		}
		try
		{
			$row = $this->database->table((string)$table)->insert($params);
		}
		catch(\PDOException $e)
		{
			// This catch ONLY checks duplicate entry to fields with UNIQUE KEY
			$info = $e->errorInfo;

			// mysql==1062  sqlite==19  postgresql==23505
			if ($info[0] == 23000 && $info[1] == 1062)
			{
				$db = $this->database;
				// if/elseif returns the name of problematic field and value
				if( $db->table('users')->where('username = ?', $values['username'])->fetch() )
				{
					$msg = 'Meno '.$values['username'].' je už obsadené. Vyberte si prosím iné.';
				}
				elseif( $db->table('users')->where('email = ?', $values['email'])->fetch() )
				{
					$msg = 'Email '.$values['email'].' je už zaregistrovaný. Musíte uviesť unikátny email.';
				}

				throw new Exceptions\DuplicateEntryException($msg);

			}
			else throw $e;
		}

		return $row;
	}

	/**
	 * This method checks if user exists
	 * @param array
	 * @param string
	 *
	 * @return DB row or false
	 */
	public function SignInUser($values, $table)
	{
		$row = $this->database->table($table)->where('username = ?', $values['username'])->limit(1)->fetch();

		if($row)
		{
			if(Passwords::verify($values['password'], $row['password']))
			{
				return $row;
			}
		}
		// here MUST NOT be else!!!
		return false;

	}

Editoval Čamo (6. 8. 2014 21:32)

@Šaman

1. Ano ten inject nefunguje, kôli prehladnosti som zmazal konštruktor a bez neho je to blbosť
2. To hashovanie som premiestil do modelu takže sa volá registerUser($values, $table);
3. Nemôže to tak byť? Nemôžu existovať dve tabuľky, v ktorých by mohli byť usery registrovaní? Napr. modul Blog a modul Eshop budú mať zvlášť tabuľky. Či mal by mať každý modul vlastný model?
4. Neviem čo je CRUD(zistím) ja mám zatiaľ len BaseModel a *Model
5. Nerozumiem prečo nie model ale repository. Som asi hold zelenáč…

Díky

Editoval Čamo (6. 8. 2014 11:20)

3. To ma asi ešte čaká
4. Tú spojenie s databázou dám do baseModelu a ten CRUD asi zatiaľ v takomto jednoduchom modeli nemá zmysel vyčleniť.
5. Tak zatiaľ model keď aj vedúci to tak píše.

Díky za obšírne vysvetlenie.

PS: ako som to zmenil potrebujem otestovať is_array($values), čo samozrejme nefunguje.
Prepísal som to na $values instanceof Nette\Utils\ArrayHash , ale niesom si istý, či je to správne(či je dosť univerzálne pre akceptáciu polí).

Editoval Čamo (14. 10. 2014 21:24)

@Čamo jeste jedna vec jak registrujes, zda email co uzivatel zadal je opravdu funkcni email? Ja ty veci resim eventama vcetne flash message apod.

@japlavaren
Hej ten filter je určite lepší ako ten môj nápad s NULLmi v parametroch. Ja som sa chcel vyhnúť skladaniu toho filtra v presentery. Ale asi nejde mať všetko dokonalé.

Díky, že sa snažíte mi to vysvetliť.

Keby niekto videl nejakú problematickú časť v tom mojom UserModeli, tak to by som uvítal. Na príklade sa to ľahšie pochopí. Už som podľa Šamana presunul DB do BaseModelu a tiež hashovanie hesiel do modelu.

@jenicek
Pokiaľ viem tak email sa nedá takto overiť. Ale toto tu nemôžeme rozvíjať je to príliš OUT.
Radšej mi vysvetli, čo myslíš pod tým if-else hell. Lebo je si neviem predstaviť, že by som niečo vyhodil.

Editoval Čamo (6. 8. 2014 21:06)

Rád by som to urobil ale neviem s tým robiť. Ak existuje nejaký instantný návod, tak pošli odkaz. Už dlhšie sa na to chystám, ale ešte som sa nedokopal k tomu.

Editoval Čamo (6. 8. 2014 23:16)

@Čamo Myslim hned metodku RegisterUser testovani zda jde o array bych dal pryc a misto toho bych napsal array primo k parametru metody. Table je tam uplne zbytecne melo by byt jako parametr tridy, ktera vlastni tuto metodu atd.

Proste zkratil bych tem metodam zodpovednost na minimum, a rozdelil na metod.

Přesně abys věděl co co dělá a mohl si kousek použít i jinde.

A hlavně mrkni na ten YetOrm mě příjde dobrej, jednoduchej, přehlednej,
vím že existuje LeanMapper a je lepší o moc, ale moc jsem mu nepřišel zatím na zub.

Muj rychlej nástřel jak bych to přibližně s Nette\Database\Context psal.

function insert(array $values)
{
	return $this->getTable()->insert($values);
}

function findEmailOrUsername($email, $username)
{
	return $this->getTable()->where("email = ? or username = ?", $email, $username)->fetch();
}

function register(array $values)
{
	try {
		return $this->insert($values);
	} catch(\PDOException $e) {
		if($this->fineEmailOrUsername($values["email"], $values["username"]){
			throw new Exceptions\DuplicateEntryException;
			/*
				V presenteru bych ji zachytil a napsal bych flashmessage až tam
				navíc tam máš dostupné values tak si tam můžeš napsat co chceš.

				"Zadaný email $values["email"] nebo uživatelské jméno $values["username"]
				je již registrované."

				Protože za tyhle hlášky by neměla být zodpovědná vyjímka ani model.
				Model má vykonat jen úlohu co po něm chceš a ne aby ještě něco vypisoval,
				zjištoval a rozhodoval.

				Rozhodovat má presenter nebo komponenta od toho tu jsou.
			*/
		}
		throw $e;
	}
}

Editoval jenicek (7. 8. 2014 11:34)

Ještě bych spojil podmínky if($row && Passwords::verify($values['password'], $row['password'])), ten kaskádový dvojif vypadá divně.
A nemusíš si tam pak zdůrazňovat, že se nesmí elsovat.

Editoval Šaman (7. 8. 2014 12:39)

Šaman napsal(a):

Ještě bych spojil podmínky if($row && Passwords::verify($values['password'], $row['password'])), ten kaskádový dvojif vypadá divně.
A nemusíš si tam pak zdůrazňovat, že se nesmí elsovat.

Ve skeletonu tam je totiž else: uživatel nenalezen.