Načítání modelů a Dependency Injection (doplňte návod)

Já používám prostě jenom:

services:
	models: Nette\DI\NestedAccessor(@container, 'models')
	models_foo: Models\Foo
	models_bar: Models\Bar

(v models.neon)
Používá se to úplně stejně, člověk se obejde bez extension, akorát je to delší o ten prefix models_ u každé služby.

souhlas s hosiplan :) od vydani CMS jsem taky dosel k posunu ze to neni zrovna nejstastnejsi reseni :) slovo model je az moc vseobecne. Btw, zrovna v tom priklade z prvniho prispevku by se extension mela jmenovat repositories, popripade entityRepositories

Jestli mohu jednu drobnou připomínku. Úplně poslední příklad bych místo

$results = $this->models->balance->fetchAll();

raději použil

$results = $this->models->user->getByName('Jan Novák');

aby se použilo vše, co je v předchozím kódu a zbytečně to u nováčků nevyvolávalo WTF, kde se tam vzalo nějaké balance (to by asi zvládl odvodit) a kde se vykouzlilo fetchAll (přidává framework sám?), když tam nikde v předchozím kódu není. Vím, profíci si snadno domyslí vše potřebné, ale pro ně asi návod není určený ;) Jinak díky za návod, docela se mi hodí.

Vážně díky za vytrvalou práci na dokumentaci :). Mám jen takový drobný návrh – nebylo by lepší rovnou udělat zkratku na modely pomocí __get(), aby se k nim přistupovalo $this->balance? Totiž jakmile přijde okamžik, kdy se do presenterů budou předávat závislosti přímo (nikoliv přes $context) snadno, tak bude jednoduší tuhle metodu __get() zrušit a přistupovat přímo ke členským proměnným, a nebude třeba odstraňovat všude v kódu ono $this->models->. Což?

potapnik napsal(a):

to je paráda! konečně jsem to pochopil a začal používat pořádně a ne magicky :) díky Schmutzko!

Osobně jsem z té sekularizace trochu nesvůj, ale snad vše funguje. Já díky :)

vojtech.dobes napsal(a):

Vážně díky za vytrvalou práci na dokumentaci :). Mám jen takový drobný návrh – nebylo by lepší rovnou udělat zkratku na modely pomocí __get(), aby se k nim přistupovalo $this->balance? Totiž jakmile přijde okamžik, kdy se do presenterů budou předávat závislosti přímo (nikoliv přes $context) snadno, tak bude jednoduší tuhle metodu __get() zrušit a přistupovat přímo ke členským proměnným, a nebude třeba odstraňovat všude v kódu ono $this->models->. Což?

Díky za díky :).

Vypadá to líně, což se mi líbí, ale vidím tam moc magie:

• výše uvedený zápis bych volal jako $this->user (kolize)
• nutnost pojmenovat modely jinak než jako všechny proměnné dostupné přes $this
• pokud kouknu na kód, dojde mi až po načtení si models.neon, že vlastně (a kdy vždy) volám model

Nicméně při dodržení určité jmenné konvence, např. `mUser by to možné bylo. Klidně příklad rozepiš a mohu ho přidat jako alternativní přístup B (pokud nemáš práva). Napadá mne, že by stačilo použít místo:

services:
    models: Nette\DI\NestedAccessor(@container, "models")
    models.user: Models\User
    models.article: Models\Article

toto:

services:
    mUser: Models\User
    mArticle: Models\Article

Jestli jsem to nepochopil, tak mne oprav, ideálně příkladem.

Editoval Schmutzka (10. 4. 2012 0:53)

Schmutzka napsal(a):

Doufám, že jsem Majkla nepředběhl

Tahle otázka doufám nebyla myšlena vážně. Za slušnost bych považoval kdybys mě alespoň kontaktoval s dotazem, jestli jsem na tom např. už nezačal.

Považuji za nehoráznou drzost ukrást můj text, upravit ho a následně jej vydávat za svůj.

Třešničkou na dortu bylo zneužití URL, kterou jsem před pár dny přidal do původního článku.

Obsahově to stojí za pendrek.

• Neudělal jsi nic jiného, než že jsi slepil pár kusů kódu, kde navíc půlka je pro účel takového návodu nevhodná.
• Naprosto nesmyslně jsi použil NotORM, jehož použití je spíše rarita.
• Vůbec jsi nevysvětlil, co která část znamená. Zrovna to považuji za klíčovou část u něčeho, co má být v kuchařce/Planette.

tento slibovaný návod je velmi očekávaný :)

Nevšiml jsem si, že by ho někdo očekával.

Já bych to zase tak přísný nebyl, přece jenom publikováním zde na webu souhlasíš s nějakou tou licencí.

Co se mi moc nelíbí, tak opětovné použití service locatoru. Když už píšeme nové návody, bylo by super je psát dopředu-kompatibilně. Tedy plně využít DI.

Abych jenom nerejpal, tak až se pla.nette trochu uklidní, tak taky něco přihodím (nejspíš latte makra).

Schmutzka napsal(a):

@Majkl578: Omlouvám se, tvé mlčení jsem si chybně vyložil jako souhlas s postupem. Už to tam viselo poměrně dlouho i přes reakce na toto téma na fóru (= očekávání), tak jsem chtěl mezeru doplnit.

Asi máme různou představu o významu spojení dlouhou dobu. Přidal jsem to tam ve čtvrtek večer (viz log) a 4 dny mi opravdu nepřipadají dlouho.

HosipLan napsal(a):

Já bych to zase tak přísný nebyl, přece jenom publikováním zde na webu souhlasíš s nějakou tou licencí.

Ano, s Creative Commons CC-BY-SA (jak je uvedeno v patičce stránek). Pokud bychom byli důslední, tak explicitně vyžaduje uvedení autora a zdroje.

HosipLan napsal(a):

Co se mi moc nelíbí, tak opětovné použití service locatoru. Když už píšeme nové návody, bylo by super je psát dopředu-kompatibilně. Tedy plně využít DI.

Souhlasím. Obsahově je to hodně nešťastné a nepřináší to vpodstatě nic nového a ani poučného. Osobně bych volil jinou koncepci.

Majkl578 & Patrik Votoček: Tak dejte hlavy dohromady a udělejte lepší návod. Konkurence a rivalita nikdy neškodí ;)

Schmutzka: Díky za návod. Pomohlo mi to vysvětlit si další části Nette ;)