Session v @layout počet zboží v košíku

Ideální je plnit data pro layout v obecném abstraktním presenteru ze kterého finální presentery dědí. Na plnění layoutu nepoužít metodu render<Action>, nýbrž beforeRender(). Pozor, metody beforeRender() by měly volat beforeRender() parent presenteru, pro případ, že by se o úroveň výše také plnily nějaká data do layoutu.

use Nette\Application\UI\Presenter

abstract class BasePresenter extends Presenter
{
	.....

	public function beforeRender()
	{
		parent::beforeRender();		// není úplně nutné vždy, ale je dobré to tam mít
		$this->template->data = $data   // k dispozici v @layout.latte
	}
}

a finální presenter

final class MyPresenter extends BasePresenter
{
	.....

	public function renderDefault()
	{
		$this->template->actionData = $actionData   // k dispozici v šabloně akce
	}
}

Editoval m.brecher (13. 10. 2022 16:21)

Ahoj, já víceméně všechny aplikace kde je veřejná část (Front) a administrace (Admin) stavím tak, že mám 3 layouty:

@layout.latte - obecný pro celý web

	@front-layout.latte - dědí z @layout.latte, určen pro Front část
	@admin-layout.latte - dědí z @layout.latte, určen pro Admin část

Layoutům odpovídají abstraktní presentery:

BasePresenter je na vrcholu - data do @layout.latte

	FrontPresenter extends BasePresenter - data pro @front-layout.latte
	AdminPresenter extends BasePresenter - data pro @admin-layout.latte

Když to postavím takhle, tak potom si můžu naplnit data do libovolného layoutu v odpovídajícím abstraktním presenteru.

Současně je vhodné i aplikaci rozčlenit na moduly Front a Admin a potom má každý modul svůj layout a svůj abstraktní společný presenter. Ale moduly není nutné mít, ale layouty a abstraktní presentery doporučuji takto zorganizovat. Ono se to vždycky hodí.

@mimcala

Abstraktní presenter se dá skvěle využít na tři věci:

• pohlídat přístupová práva do všech odvozených finálních presenterů – metoda startup()
• zajistit data pro layout všech odvozených presenterů, dá se zde layout nastavit – metoda beforeRender()
• registrovat komponenty které se používají v odvozených finálních presenterech – metoda CreateComponentXxx()

Takhle Ti to bude asi stačit, nepoužívej v nich metody action<Action>, nebo render<View>, ty se používají ve finálních presenterech.

Ve finálních presenterech se data jednotlivých akcí, nebo pro konkrétní šablony <View> do šablony dávají ideálně v render<View>. I ve finálním presenteru můžeš použít metodu beforeRender(), pokud potřebuješ dodat do všech šablon v tom presenteru nějaká společná data – to je ale spíše dost vyjímečné.

Pokud potřebuješ vykreslovat v layoutu košík uložený v session, tak doporučuji si košík vytvořit jako komponentu – tj. samostatnou class se svojí vlastní šablonou na vykreslování. Komponentu košík zaregistruješ v odpovídajícím abstraktním presenteru a do layoutu vykreslíš tagem {control kosik}. Komponenta si sama dodá do svojí vlastní template svoje data a žádný presenter se o to nestará. To je nejlepší postup. Měl bys tak komponentu košík, která nemá signály a jenom vykresluje košík ze session a komponentu přidat do košíku, která přidává ze stránek s produkty konkrétní produkt do košíku. Komponenta přidat do košíku by pracovala se signálem a vykreslovala by link pro přidání do košíku.

Snad jsem to popsal dost srozumitelně. Kdyby ne, pošlu nějaké ukázky kódů.

Editoval m.brecher (13. 10. 2022 23:10)

Aha, ,skvěle vysvětleno chápu, udělal jsem to podle tohoto, ale stejně mi to háže chybu, že nezná proměnnou :/
mám to řešené takto

<?php

namespace App\Presenters;

use Nette;
use Nette\Application\UI\Presenter;



abstract class HlavniPresenter extends Presenter
{

    public function beforeRender()
    {
        $this->setLayout('HlavnistranaLayout');
        $this->template->kosik = "3";
  }

tady mám HlavnistranaLayout, kde je proměnná, kterou mi hlásí, že to nezná

  <nav class="navbar navbar-dark navbar-expand-md sticky-top navbar-shrink py-3" id="mainNav">
        <div class="container"><a class="navbar-brand d-flex align-items-center" n:href="Hlavnistrana:"><span class="bs-icon-sm bs-icon-circle bs-icon-primary shadow d-flex justify-content-center align-items-center me-2 bs-icon" style="width: 40px;height: 40px;"><svg xmlns="http://www.w3.org/2000/svg" width="1em" height="1em" fill="currentColor" viewBox="0 0 16 16" class="bi bi-cpu" style="width: 26px;height: 26px;">
                        <path d="M5 0a.5.5 0 0 1 .5.5V2h1V.5a.5.5 0 0 1 1 0V2h1V.5a.5.5 0 0 1 1 0V2h1V.5a.5.5 0 0 1 1 0V2A2.5 2.5 0 0 1 14 4.5h1.5a.5.5 0 0 1 0 1H14v1h1.5a.5.5 0 0 1 0 1H14v1h1.5a.5.5 0 0 1 0 1H14v1h1.5a.5.5 0 0 1 0 1H14a2.5 2.5 0 0 1-2.5 2.5v1.5a.5.5 0 0 1-1 0V14h-1v1.5a.5.5 0 0 1-1 0V14h-1v1.5a.5.5 0 0 1-1 0V14h-1v1.5a.5.5 0 0 1-1 0V14A2.5 2.5 0 0 1 2 11.5H.5a.5.5 0 0 1 0-1H2v-1H.5a.5.5 0 0 1 0-1H2v-1H.5a.5.5 0 0 1 0-1H2v-1H.5a.5.5 0 0 1 0-1H2A2.5 2.5 0 0 1 4.5 2V.5A.5.5 0 0 1 5 0zm-.5 3A1.5 1.5 0 0 0 3 4.5v7A1.5 1.5 0 0 0 4.5 13h7a1.5 1.5 0 0 0 1.5-1.5v-7A1.5 1.5 0 0 0 11.5 3h-7zM5 6.5A1.5 1.5 0 0 1 6.5 5h3A1.5 1.5 0 0 1 11 6.5v3A1.5 1.5 0 0 1 9.5 11h-3A1.5 1.5 0 0 1 5 9.5v-3zM6.5 6a.5.5 0 0 0-.5.5v3a.5.5 0 0 0 .5.5h3a.5.5 0 0 0 .5-.5v-3a.5.5 0 0 0-.5-.5h-3z"></path>
                    </svg></span><span>me75</span></a><button data-bs-toggle="collapse" class="navbar-toggler" data-bs-target="#navcol-1"><span class="visually-hidden">Toggle navigation</span><span class="navbar-toggler-icon"></span></button>
            <div class="collapse navbar-collapse" id="navcol-1">
                <ul class="navbar-nav mx-auto">
                    <li class="nav-item"><a class="nav-link active" n:href="Hlavnistrana:">Domů</a></li>
                    <li class="nav-item"><a class="nav-link active" n:href="Produkty:">Obchod</a></li>
                    <li class="nav-item"><a class="nav-link active" n:href="Cenik:">Ceník</a></li>
                    <li class="nav-item"><a class="nav-link active" n:href="Jaktofunguje:">Jak to funguje</a></li>
                </ul><a class="btn btn-primary shadow" role="button" n:href="Cenik:">Registrovat</a><a class="btn btn-primary shadow" role="button" n:href="Prihlaseni:" style="margin-left: 17px;background: rgba(55,99,244,0);">Přihlásit</a>
                <a href="#" class="btn btn-primary mx-2" type="button" style="padding-right: 14px;padding-left: 14px;background: rgba(55,99,244,0);"><svg xmlns="http://www.w3.org/2000/svg" width="1em" height="1em" fill="currentColor" viewBox="0 0 16 16" class="bi bi-cart3" style="font-size: 26px;">
                        <path d="M0 1.5A.5.5 0 0 1 .5 1H2a.5.5 0 0 1 .485.379L2.89 3H14.5a.5.5 0 0 1 .49.598l-1 5a.5.5 0 0 1-.465.401l-9.397.472L4.415 11H13a.5.5 0 0 1 0 1H4a.5.5 0 0 1-.491-.408L2.01 3.607 1.61 2H.5a.5.5 0 0 1-.5-.5zM3.102 4l.84 4.479 9.144-.459L13.89 4H3.102zM5 12a2 2 0 1 0 0 4 2 2 0 0 0 0-4zm7 0a2 2 0 1 0 0 4 2 2 0 0 0 0-4zm-7 1a1 1 0 1 1 0 2 1 1 0 0 1 0-2zm7 0a1 1 0 1 1 0 2 1 1 0 0 1 0-2z"></path>
                    {$kosik}</svg></a>
            </div>
        </div>
    </nav>

Vyřešeno, děkuji, další presenter musí dědit HlavniPresenter .. moje školácká chyba :))

Editoval mimacala (15. 10. 2022 21:11)

Tak ale zrovna košík, který bude potřeba úplně všude je přesně ten UseCase, kdy BasePresenter použít ne? :D

Jakože chápu, že se může stát, že časem přibyde sekce, kde třeba košík nebudeš chtít, ale hej, řešit to takto všechno a za chvíli mi z těch trait exploduje hlava. Protože to máš traitu pro patičku s kontaktním formem, to máš traitu pro menu, to máš traitu pro košík, jinde zase pro zákaznický chat…

5 trait který musíš kopírovat do každýho presenteru jak vyšinutých. A jasně, uděláš jednu traitu, co je bude mít všechny v sobě, ale ruku na srdce. Jaký je pak rozdíl mezi tou traitou, kterou použiješ všude a BasePresenterem?

Já ho třeba používám minimálně na ověření, jestli má přístup uživatel do dané sekce (modulu).

Tedy v App\AdminModule\Presenters\BasePresenter mám asi toto:

<?php declare(strict_types=1);

namespace App\BackModule\Presenters;

use Nette\Application\UI\Presenter; as UiPresenter;
use Nette\Http\IResponse;

abstract class Presenter extends UiPresenter
{
    protected function startup()
    {
        $user = $this->getUser();

        if (!$user->isAllowed('Backend', 'view')) {
			// Or some other code like 401...
            $this->getHttpResponse()
                ->setCode(IResponse::S303_SEE_OTHER, 'Logged in as a client, admin was automatically logged out, or visited the page via a link without previous login');

            $this->redirect(':Sign:in', ['backLink' => $this->storeRequest('+30 minutes')]);
        }

        parent::startup();
    }
}

Editoval Bulldog (16. 10. 2022 0:11)

@Bulldog
Naopak, s narůstajícím počtem víc a víc traity oceníš.
Jaký je rozdíl mezi BasePresenterem a traitou použitou všude? Právě v tom, že jednou (a nemusíme řešit, zda se to stane nebo ne, řešme jen kdy) to „všude“ přestane platit. Navíc dílčí traity umožňují do presenteru nakombinovat jen to potřebné, BasePresenter nalije vždy všechno.

@m.brecher Traita je samozřejmě součást OOP, jako třeba interface. Jaké nedostatky v architektuře signalizuje?

Komponentami zdaleka nepokryješ vše, např. tu změnu layoutu, ověření přihlášeného uživatele, továrničku na menu. Přitom to vše může mít své traity.
Nějaká analogie s include je docela mimo, i přes snahu o zjednodušení. Chápu, že hájíš své workflow, ale přiznejme si, že „použití první traity po roce váhání“ není úplně nejlepší předpoklad pro objektivní zhodnocení daného přístupu.

Mrkni třeba sem: https://youtu.be/yIhKrIlhtGY?…
Zkus si tak napsat ze 3 weby a pak bude Tvé srovnání objektivní.

P.S. nedostatek v architektuře jsem nezachytil žádný…

@KamilValenta

asi hodně záleží na tom, jak aplikaci od základů stavíš a jak je rozsáhlá. Já od začátku počítám s hierarchií presenterů a komponentami a proto asi u mě potřeba trait nevzniká. Ale už Jsi mě malinko zviklal, že u malých aplikací se dá do presenterů přidávat funkcionalita traitami, které mohou částečně nahradit abstraktní presentery, nebo komponenty.

Já jsem si traitou jsem si vylepšil Presentery Nette, podobně jako v Nette je traita Nette\SmartObject, která vylepšuje některé drobné nedostatky PHP.

Nahradit abstraktní presentery traitami ale rychle narazí na omezení trait. Některé funkce, které já realizuji BasePresenterem traity inteligentně nezvládnou.

V BasePresenteru v startup() přesměrovávám na kanonickou url, protože interní přesměrování z /www/ na / .htaccessem vytvoří duplicitní url.

Kanonizace url BasePresenterem

abstract class BasePresenter extends Nette\Application\UI\Presenter
{
    #[Inject]
    public Canonizer $canonizer;

    startup()
    {
        parent::startup();
    	$this->canonizer->canonize();
    }
}

final class ArticlePresenter extends BasePresenter
{
        startup()
    {
        parent::startup();
    	..... 				// specific authorization for this presenter
    }
}

Kanonizace url traitou

trait Canonizer
{
    #[Inject]
    public Canonizer $canonizer;

    startup()
    {
        parent::startup();
    	$this->canonizer->canonize();
    }
}

final class ArticlePresenter extends Nette\Application\UI\Presenter
{
	use Canonizer;

	startup()				// shit! this will not work with trait :(
    {
        parent::startup();
    	..... 				// specific authorization for this presenter
    }
}

Traita kterou místo BasePresenteru vkládáme do všech presenterů má zásadní omezení – nelze v této architektuře škálovat setup presenterů – tj. provést společný setup v traitě a pak specifické setupy v presenteru – tak jako se běžně používá v architektuře BasePresenteru.

Další příklad. Setupuji parametr $appDir z .neon do BasePresenteru – pro pohodlné vkládání jakýchkoliv latte šablon či layoutů.

// konfigurace

decorator:
	App\Presenters\BasePresenter:
		setup:
			- $appDir = %appDir%

setupováné $appDir BasePresenterem

abstract class BasePresenter extends Nette\Application\UI\Presenter
{
    public string $appDir;

    .....

    public function beforeRender()
    {
		parent::beforeRender();
        $this->template->appDir = $this->appDir;  // $appDir for all templates or layouts :)
   }

}

final class ArticlePresenter extends BasePresenter
{
    .....

    public function beforeRender()
    {
		parent::beforeRender();
        $this->template->var = $this->var;  // $var for all presenter templates (actions)
   }

}

setupováné $appDir traitou

trait AppDir
{
    public string $appDir;

    .....

    public function beforeRender()
    {
		parent::beforeRender();
        $this->template->appDir = $this->appDir;  // $appDir for all templates or layouts :)
   }

}

final class ArticlePresenter extends Nette\Application\UI\Presenter
{
	use AppDir;

	public function beforeRender()
    {
		parent::beforeRender();
        $this->template->var = $this->var;  // shit! with trait this will not work :(
   }
}

Z příkladů je jasné, že některé funkce BasePresenteru nelze nahradit traitou.

Editoval m.brecher (16. 10. 2022 23:15)

m.brecher napsal(a):
Z příkladů je jasné, že některé funkce BasePresenteru nelze nahradit traitou.

Ne, z příkladů je jasné, že jsi buď neviděl odkazované video, nebo jsi ignoroval můj příklad v tomto vlákně, nebo oboje.
Všechny uvedené příklady traitou napsat lze.

@mimacala

v První řadě hodilo mi to error „Cannot use a scalar value as an array“ na řádku

Protože to vypadá, že máš historicky v té session uložený nějaký scalár. Takže to ten scalár načte a pak nový kód se z toho snaží přistupovat ke skaláru jako k poli. Musíš nejdřív celou session vymazat, než začneš zkoušet novou strukturu kódu.

Prosím tedy „(string)“, v php 8.0 je nově typový systém proměnných, toto definuje text nebo co přesně to dělá před $idProduktu?

Pokud se podíváš na definici tvé funkce, která má za úkol přidávat produkty do košíku public function actionPridatkosik(int $idProduktu, int $mnozstvi), tak uvidíme, že bere jako parameter $idProduktu a před ním je klíčové slovíčko int, což znamená, že v proměnné $idProduktu bude uloženo celé číslo.
Pokud se ale podíváme, co bere nad SessionSection metoda get, kterou voláš, tak uvidíme její předpis takový: public function get(string $name)
Což říká, že jako první parametr to bere string, což je řetězec, tedy série znaků.

Pokud bychom tedy zkusili udělat něco jako $kosik->get(4); tak dostaneme error

TypeError

Nette\Http\SessionSection::get(): Argument #1 ($name) must be of type string, int given, called in ...

To znamená, že my bereme jako identifikátor integer, ale session chce string.
No a přetypování integeru na string lze v PHP několika způsoby:

"$idProduktu"
"" . $idProduktu
(string) $idProduktu

takže zápis (string) $idProduktu je pouze explicitní přetypování integeru na string. Osobně doporučuju dělat jen tu poslední možnost. Samozřejmě se více napíšeš, ale aspoň je to hned na první pohled vidět a neuděláš chybu. Ostatní 2 způsoby se používají spíše na vložení proměnné do řetězce jako:

"There is $count apple"
"There is " . $idProduktu . "apple"

Ale i tyto způsoby nejsou úplně ok:

V prvním případě můžou nastat problémy jako že potřebuješ přidat těsně za proměnnou nějakou příponu, například potřebuješ říct, kolikátý daný závodník skončil:

"Skončil jsi na $placetém místě" // Proměnná je $place

Tady vidíš, že PHP není schopno rozeznat které písmena ještě jsou proměnná a která už ne. Na to se vytvořily závorky:

"Skončil jsi na {$place}tém místě" // Proměnná je $place

Tady už to v pohodě rozezná.
Co když ale potřebuješ napsat kolik sis vydělal dolarů?

"Vydělal sis ${$amount}"

tady taky nastane problém, protože nejdřív se vykoná vnitřek závorak a ten se vezme díky znaku dolaru jako další proměnná, takže v případě, že obsah proměnné amount je 10, tak se PHP bude snažit najít proměnnou s názvem $10. Takže musíme escapovat dolary:

$foo = 'ahoj';
$bar = 'foo';

echo "Your string ${$bar}"; 	// Vypíše  "Your string ahoj"
echo "Your string \${$bar}";	// Vypíše  "Your string $foo"

Díky tomuto fenoménu můžeme vytvořit proměnnou, která se jmenuje null :D

$a = null;
$$a = 'This is string saved to variable named null';

Problém může nastat i se závorkami, takže musíš zkoumat kdy kde jak co escapovat a to je zbytečný peklo.

Druhý způsob tedy vypadá lépe, protože můžeme místo uvozovek " použít apostrofy ', které berou vše jako znaky, takže se nic nemusí escapovat:

'Skončil jsi na ' . $place . 'tém místě'	// Skončil jsi na 15tém místě
'Vydělal sis $' . $amount					// Vydělal sis $50

Problém tady ale nastane při překladu těch řetězců představ si to ve formátu, kde máš proměnné třeba 3 v tom řetězci

'Tlupa ' . $animal . ' seděla uprostřed ' . $place . ' asi v ' . $time . 'hodin.' // Tlupa koček seděla uprostřed domu asi v 10hodin.

v tomhle případě musíš mít uloženy 4 různé řetězce a překládat je postupně, což je sebevražda. Takže konkatenovat řetězce by se mělo přes funkci sprintf:

sprintf('Tlupa %s seděla uprostřed %s asi v %shodin.', $animal, $place, $time); // Tlupa koček seděla uprostřed domu asi v 10hodin.

Všimni si, že zde je uvnitř řetězce povoleno %shodin, jelikož sprintf definuje zástupné symboly, které mají vždy znak % a jeden znak za tím, případně specifický formát výpisu, jako formátování čísla, nebo explicitní změnu pořadí argumentů, ale to se dočteš v odkaze, takže je jasně rozpoznatelné, že %s vypíše řetězec. A jediné, co zde musíme escapovat je znak %, pokud jej chceme přímo vypsat a to tak, že %% vypíše prostě znak %.
No a jako výhoda je, že máš 1 řetězec, který můžeš házet do překladů.

Takže obecně pro přetypování proměnné na řetězec bych doporučil zápis (string) $prom a pro výpis proměnné do řetězce funkci sprintf

Tedy tvoje poslední úpravy by ve zjednodušené verzi mely vypadat asi takto:

public function actionPridatkosik(int $idProduktu, int $mnozstvi, int $cena)
{
    if ($mnozstvi < 1) { //pokud přidává málo kusů
        $this->error('Menuzete pridat tak malo kusu');
    }

    $kosik = $this->session->getSection('kosik'); // načte sekci
    $vKosiku = $kosik->get((string) $idProduktu) ?? [
        'mnozstvi' => 0,
        'cena' => 0.0,
    ];

    // Tady víme, že v proměnné $vKosiku vzdy bude pole s cenou a mnozstvim z minula (z minula muze byt i vychozi hodnota, tedy nulove pole)
    $vKosiku['mnozstvi'] += $mnozstvi;  // += říká, že je to ekvivalent se zápisem        $vKosiku['mnozstvi'] = $vKosiku['mnozstvi'] + $mnozstvi;
    $vKosiku['cena'] += $cena;  // V případě, že bude cena, nebo mnozstvi 0 jako vychozi hodnota, tak se udela prostě 0 + $cena coz je stejne jako obsah puvodniho tveho elsu

    $kosik->set((string) $idProduktu, $vKosiku); // nastaví session.

    $this->flashMessage("Zboží bylo přidáno do košíku");
    $this->redirect("Produkty:");
}

a

public function ZobrazKosik()
{
    $mnozstvi = 0;
    $section = $this->session->getSection("kosik");

    foreach ($section as $produkt => $hodnoty) {
        if ($hodnoty['mnozstvi'] === 0) {
            continue;   // Muže se stát, že někdo odebere z košíku produkty a u takových nechceš aby se zobrazovaly jako 0 kusů daného produktu. Ale lepší je mazat rovnou ze session už při odebírání ;)
        }

        echo (
            sprintf('ProduktID:%d Množstí:%d Cena:%f', $produkt, $hodnoty['mnozstvi'], $hodnoty['cena'])
        );
        $mnozstvi += $hodnoty['mnozstvi'];
    }

    return $mnozstvi; // počet položek v košíku
}

Jenom prosím ještě počítej s 2 věcmi:

1. statické řetězce je lepší nezapisovat jako "mnozstvi" ale jako 'mnozstvi', jelkož jak jsem již napsal, tak řetězce v uvozovkách se snaží vykonávat inline kód a tedy jsou statické řetězce na vykreslení pomalejší, jelikož se nejdřív projdou a otestuje se, jestli obsahují proměnné/výrazy. Řetězce v apostrofech tohle nedělají, takže se vypisují okamžitě, jelikož s nimi je nakládáno přímo jako s řetězci. Takže je dobré na to myslet. Budeš pak mít nepatrně rychlejší aplikaci a když náhodou přibyde něco, co musíš escapovat, tak na to nemusíš myslet.
2. Metoda by neměla dělat žádné side efekty. Respektive tvůj kód by neměl provádět výpis pomocí echo, pokud zároveň něco počítá atp. Na to se udělaly návrhy kódů, aby byla jedna třída, která volá další a jejich odpovědi vypisuje. Té třídě, co to vypisuje se říká View. všechny ostatní by měly jen něco počítat (model) nebo spojovat view a model (Controller/Presenter)

Každé zboží, resp. jeho množství může mít v košíku svůj form a ten můžeš odesílat ajaxově.
Nebo vypsat všechna množství do jednoho formu a pod tím submit ve stylu „přepočítat“.

Varianta, která mně hodně žere, když někdo odesílá form při onchange. Protože než se stránka přenačte, stihnu změnit ještě množství u 1–2 zboží a to musím dělat pak znova…

@Bulldog jasně, to je v podstatě jedna z ajaxových variant… Já myslel ty případy, kdy někdo při onchange odesílá celý form s redirectem. Aby si tazatel nevybral tuhle variantu :)

Takže vzhledem k tomu, že to teď mám v jednom formu, takže nejjednodušší řešení je přidat tlačítko na upravení.
Každopádně šlo by třeba přes ajax zavolat na php funkci v nette, kde přidávám zboží do session ? to by mohlo vyřešit problém s tám aby uživatel zbytečně klikal na talčítko navíc, akorát by se musela zase obnovit stránka, ale uživatel by to měl snažší.

Tedy šlo by zavolat na ajax aby zavolal na fukci pro přidání do session ? Jak by šel vyřešit preloader ? to zase obstarává js ?
Děkuji