Cenové hladiny, aneb „kam s nima“

Takže „b“, do toho se mě moc nechce, ale asi to bude nejčistější řešení.

No třídu na stanovení ceny, abych pravdu řekl, ani nevím jak bych si jí představil, ani kde bych jí přesně volal. V presenteru načtu zboží co se má zobrazit a v renderu ho vykresluji… tak jestli pak v render metodě volat tuto třídu, nebo v šabloně? (do třídy by se předalo například katalogové číslo a id zákazníka, pokud se jedná o nepřihlášeného, vrátila by se základní cena, pokud je přihlášenej, už by se rozhodovalo. To si představit umím, ale kde to přesně volat?)

Jen navazuji na to co psal petr.pavel

petr.pavel napsal(a):

Předpokládám, že některý zákazník má 5% slevu na všechno, jiný 10% na spotřební materiál, ale na zbytek jen 5%, nebo slevu 15%

V tomto doplňku je 4. parametr procentní upravení ceny viz ukázka

{!$money|currency:eur:usd:1.1}

a aby se ti nepřepočítávalo tak zachovej výchozí parametry metody format.

{!$money|currency:NULL:NULL:1.1} není potřeba přepočítávat

Přidaná hodnota je že ti to vyřeší formátování měny, oproti helperu number který je v nette má výhodu že si to globálně nastavíš pro celý web a nemusíš pokaždé znovu nastavovat viz ukázka .

Ale já bych to řešil na úrovni databáze a využil jen formátování čísla.

Editoval h4kuna (1. 8. 2012 10:47)

Jestli vždycky pracuješ se slevou, tak místo nastavCenu() bych napsal zjistiSlevu() a cenu počítal až třeba v šabloně. Cílem je vyčlenit do metody jen to, co je potřeba pro odlišení ceny.

Taky záleží na tom, podle čeho cenu/slevu sestavuješ. Jestli podle kategorie výrobku a sazby zákazníka, předávej metodě tyto dvě, ne prostředníky. Tj. ne cenu a id zákazníka, ale kategorii a sazbu. Bude to logicky průhlednější a líp se ti pak bude metoda testovat.

Proč chceš předávat celou identitu, když už teď víš, že potřebovat budeš jen hladinu? Znáš ten vtip o klavíru?

Jestli používáš Nette Database, tak by to nemuselo být 10 dotazů, ale jen jeden. Tohle lazy zpracování je jedna z jeho hlavních předností.

Existuje takové doporučení, nejdřív programovat čistě z hlediska návrhu, a teprve když to všechno funguje, tak začít optimalizovat výkon.

Něco jsem včera večer ještě spíchl:

/**
 * Vrátí přepočítanou cenu zboží. Nepřihlášený uživatel, nebo uživatel bez cenové hladiny obdrží zpět počáteční cenu.
 * Pokud má uživatel nastavenou cenovou hladinu, cena se nastaví:
 * a) pokud je v tabulce self::TABLE_SLEVAZBOZI zadané katalogové číslo se zvolenou cenovou hladinou, přiřadí se zadaná cena
 * b) vytáhne se procentní sleva z tabulky self::TABLE_CENOVAHLADINA a vynásobí se aktuální cena
 * c) pokud cenová hladina neexistuje, vrátí se počáteční cena
 * @param int|NULL $cenovaHladina
 * @param string $katalog
 * @param double $cena
 * @return double
*/
public function getProduktCena($cenovaHladina, $katalog, $cena)
{
    // pokud neexistuje cenová hladina, vracím zpět nezměněnou cenu
    if(!isset($cenovaHladina) || is_null($cenovaHladina)) {
        return $cena;
    }
    // pokud existuje pevná sleva na zboží se zvolenou cenovou hladinou
    $row = $this->getSlevaZbozi()->select('cena')->where('goods_katalog', $katalog)->where('cenovehladiny_id', $cenovaHladina)->limit(1)->fetch();
    if($row) {
        return $row['cena'];
    }
    // pokud neexistuje pevná cena, načtu si cenovou hladinu a přenastavím cenu
    $row = $this->getCenovaHladina()->select('sleva')->where('id', $cenovaHladina)->limit(1)->fetch();
    if($row) {
        return $cena * $row['sleva'];
    } else {
        // pokud hladina neexistuje, vracím nezměněnou cenu
        return $cena;
    }
}

v basePresenteru si dodám helper:

$_this = $this;
$this->template->registerHelper('nastavCenu', function($cena, $katalog) use ($_this) {
    if($_this->user->isLoggedIn()) {
        return $_this->modelShop->getProduktCena($_this->getUser()->getIdentity()->data['cenovehladiny_id'], $katalog, $cena);
    } else {
        return $cena;
    }
});

Kde následně mohu volat v šabloně:

přepočítáno: {$ceny[$value['goods_katalog']]['cenaSdani']|nastavCenu, $value['goods_katalog']}

Editoval tatyalien (2. 8. 2012 8:02)

• $_this->getUser()->getIdentity()->data['cenovehladiny_id'] – nějak toto používám, v data mám ostatní údaje po autorizaci uživatele. Ale prubnu to tvoje $_this->user->identity->cenovehladiny_id
• if(!isset($cenovaHladina) || is_null($cenovaHladina)), zde by „0“ neměla být nastavená, ale raději jsem testoval jestli existuje hladina, a není NULL tak šlapej dál, ale asi bz fakt stačilo empty($cenovaHladina)
• To pole ceny je pouze ve vykreslení košíku. V sessionu/DB košíku si pouze držím: users_id, goods_katalog, goods_nazev, ks, poznamka, datum. Pokud uživatel klikne na košík tak si jen vytáhnu aktuální ceny z DB proto je zde samostatné pole „ceny“. Kdekoliv jinde na stránkách je jinak cena přímo při výpisu z tabulky goods.

Editoval tatyalien (2. 8. 2012 11:23)