Neporušuje NDBT princípy MVC?

osobne NDTB pouzivam cisto v modeloch. Akonahle totiz nemam 100% istotu, ze budem mat konzistentne data v db, osetrovat to cele v sablonach je prasacina a nocna mora (uz som to zazil).
Preto pouzivam NTDB na skladanie dotazov

<?php
if(...) $result->where(...);
?>

a vytahovanie dat z ref tabuliek

<?php
$data = array_map(function($row){
	return $row->toArray() + array(
		'category' => $row->categoryId ? $row->ref('category')->name : NULL,
	);
}, $result->toArray());

return $data;
?>

MVC rozhodně neporušuje, protože s ním nijak nesouvisí. Nejspíš máš na mysli vícevrstvou architekturu. A tady použití NDBT v šabloně může velmi snadno vést k obcházení vrstev.

DG
Netrúfam si s tebou polemizovať o tom čo je to MVC a už vôbec nie o multitier archytektúre. To čitam prvý krát. Ale teda mi povedz ak môžeš v čom je moja predstava MVC mylná:
Model-spravuje data oboma smermi
Controler-určuje čo sa bude diať podľa požiadavky uživateľa
View-zodpovedá za formátovanie dát, ktoré mu dodal contoler.
Prečo by mal view komunikovať s modelom. Jedným z hlavných argumetov, je presne to čo napísala plavaren. Keď potrebujem zmeniť model siahnem presne k nemu. Podľa mňa sa to takto triešti.

Jan Tvridk
Prečo odporúčaš miesto NDBT Dibi? Máš to v podpise.

Editoval Čamo (5. 8. 2014 11:39)

Čamo napsal(a):
Japlavaren
Presne tú nočnú moru mám na mysli.
Nemohol by si ten tvoj model trochu rozpísať resp previesť na ten príklad z dokumentácie ako book->ref(autor)->name?

Najprv troska teorie, ako to robim ja:

1. zasadne vymenonavam stlpce, ktore chcem v z modelu previest. s cachomvanim stlpcov, ktore sa maju vypisat som mal problem a chcem to mat plne pod kontrolou
2. na tabulky mam interface. ITableName::TABLE obsahuje nazov tabulky + mam tam konstanty zoradovania
   • ak je nazov modelu zhodny s tabulkou, implementujem interface ITableName (v modeli pouzivam self::TABLE)
   • v ostatnych modeloch pouzivam ITableName::TABLE
   • ak potrebujem zmenit hocico suvisiace s tabulkou, dam len vyhladat interface a viem, kde ju mam pouzitu

IBookTable.php (podobnym sposobom su napisane i ostatne I*Table)

<?php

namespace App\Model;

interface IBookTable
{
	const TABLE = 'book';

	const SORT_NAME = 'name';
	const SORT_NAME_DESC = '-name';
	const SORT_PRICE = 'price';
	const SORT_PRICE_DESC = '-price';
}

?>

BookModel.php

<?php

namespace App\Model;

class BookModel extends BaseModel implements IBookTable
{
	/** @var array */
	protected $sortOptions = array(
		self::SORT_NAME,
		self::SORT_NAME_DESC,
		self::SORT_PRICE,
		self::SORT_PRICE_DESC,
	);


	/**
	 * @param array $filter
	 * @param string $sort
	 * @param int $limit
	 * @param int|null $offset
	 * @return array
	 */
	public function getUsers(array $filter, $orderBy, $limit, $offset = NULL){
		if(!in_array($sort, $this->sortOptions)) {
			$this->log(new \InvalidArgumentException("Invalid sort value '$sort'"));
			return array();
		}

		// $this->db je z BaseModel typu Database\Context predana cez BaseModel::__construct()
		$result = $this->db->table(self::TABLE)
			->select('id, authorId, name, price, description')
			->order($this->processSort($sort))
			->limit($limit, $offset);

		if(isset($filter['name']) && $filter['name']) $result->where('name LIKE ?', "%$filter[name]%"));

		return array_map(function($row) {
			return $row->toArray() + array(
				'author' => $row->authorId ? $row->ref(IAuthorTable::TABLE)->name : NULL, // osetrenie pre nepovinne cudzie kluce
			);
		}, $result->fetchAll());
	}


	// funkcia sa nachadza v BaseModel
	/**
	 * convert initial minus to DESC attribute
	 * @param string $sort
	 * @return string
	 */
	protected function processSort($sort) {
		if (strpos($sort, '-') === 0) {
			return substr($sort, 1) . ' DESC';
		} else {
			return $sort;
		}
	}
}

?>

pisane z hlavy, moze obsahovat chyby

Editoval japlavaren (4. 8. 2014 21:14)

Jan Tvridk
Prečo odporúčaš miesto NDBT Dibi? Máš to v podpise.

Jednak ten podpis je z doby, kdy NDBT obsahovalo spoustu chyb (v Nette 2.0, teď už je to snad v klidu). Jednak má dibi mnohem mocnější jazyk na skládání složitých dotazů. Není prostě možné nahradit hromadu modifikátorů jedním otazníkem a očekávat, že to bude umět to samé. Neumí. A to i přesto, že se NDB pomocí temné (a nespolehlivé) magie snaží.