Presenter – rozbor, názorná ukázka chyb

Podle mě by jsi mel mít všechno v render.
Do action patří přesměrování, zobrazení jiného renderu apod.

No na vysvetlenie co robit kde a ako pouzivat Render a Action doporucujem poriadne precitat nasledovne

• Action vs. View
• Nette\Presenter

K tvojmu kodu, naozaj by dane veci kludne mohli ist do Render metody. Celkovo by to slo trochu aj zjednodusit, ale kazdy mame svoj coding style a nechcem rypat do toho tvojho len preto, ze ja by som to napisal trosilinku odlisne.

Nechapem tak trochu tej metode getNumOfRecordsInColumns – teda chapem co robim, ale nedalo by sa vyuzit toto: Grid renderind ? Resp. pride mi, ze to co robis tou metodou je uz naozaj starost view, presenter by nemal apriori vediet, ze chces v sablone nieco delit do 3 stlpcov a pomahat jej s tym, on jej ma len predat data a ona si ich sama spracuje. Kebyze casom prides na to, ze potrebujes 4 stlpce, musis hned prepisovat aj presenter, pricom za pouzitia Grid renderingu len jedno cislo v template.

Osobně si myslím, že:

1. úvod, fotogalerie a nabídka služeb by měli mít každá svůj view.
2. to co dělá metoda getNumOfRecordsInColumns by se asi mělo dělat v šabloně, případně rovnou v modelu.
3. rozlišovat jaké kategorie menu ti vrátí getCategories z modelu, takhle natvrdo podle parametru 0 nebo nic, no nevim. Takhle napohled mi to přijde dost neintuitivní, ale když se v tom vyznáš.

Body 2 a 3 nejsou až tak důležité, ale bod jedna by mol znamenat nepochopení základních principů, tak se nad tím zkus zamyslet ;)

Podle toho kousku se to těžko posuzuje.

Ani napsal(a):

Osobně si myslím, že:

1. úvod, fotogalerie a nabídka služeb by měli mít každá svůj view.

a není to zbytečné, protože ve všech třech případech je kód šablony stejný, jsou pouze jiná data, která view dostane… proto to mám dohromady a ovládá to ContentPresenter…

2. to co dělá metoda getNumOfRecordsInColumns by se asi mělo dělat v šabloně, případně rovnou v modelu.

ano ano, souhlasím, jak naznačil kolega nahoře, zbytečně jsem si to zkomplikoval :)

3. rozlišovat jaké kategorie menu ti vrátí getCategories z modelu, takhle natvrdo podle parametru 0 nebo nic, no nevim. Takhle napohled mi to přijde dost neintuitivní, ale když se v tom vyznáš.

0 = false, 1= true a 1 je implicitně… vzhledem k tomu že ta metoda má jen jeden parametr a že jsem k ní napsal dokumentaci a používám IDE mi to přijde postačující a myslím, že bych to pochopil i za rok :-) možná by bylo lepší tam psát true a false a nenechávat prázdné kvůli rozlišení, na to možná dojde ale až při refaktoringu :-)
>
díky za zhodnocení ;-) co ta 1) i tak by to mělo být oddělené? nevznikne mi tím hodně duplicitního kódu?

Editoval Ofi (6. 9. 2010 16:55)

je to v podstatě tak že se pošle šabloně description pro daný menu item a buď všechny položky „nabídka služeb“ (pro úvod a nabídka služeb), nebo fotogalerie (pro fotogalerie) každá stránka pak vypíše nadpis, popis, a stejný formát pro výběr v nabídce služeb nebo fotogalerii… dyžtak postnu i kód šablony :)

{block title}
<title>{$description->menuItem}</title>
{/block}
{block content}


    {if isset($description) && $description != null}
        <h3>
            {$description->menuItem}
        </h3>
        <p>{!$description->description}</p>
    {/if}

{if isset($categories)}
    <hr class="multi-separator">
    <h3>
        {$headerOfCategories}
    </h3>
    <ul>
    {foreach $categories as $key => $value}
        {if $key==0 ||
	    $key==$numInColumns[0] ||
	    $key==$numInColumns[0]+$numInColumns[1]
	}
              <li>
        {else}
              <hr class="separator">
        {/if}
              <strong>
                  <a href="{link detail,
			String::webalize($headerOfCategories),
			$value->seoUrl}">
			{$value->title}
		  </a>
              </strong>
              <img
	src="images/uploaded/{$value->seoUrl}/{$value->firstImage}"
	alt="alt"
	width="170">
    {/foreach}
    </ul><hr class="clear">
{/if}

{/block}