Komentare ku vlastnemu eshopu

Překlady adres by měl dělat router, místo presenteru s anglickým názvem, který přesměruje na presenter s českým názvem.

David Matějka napsal(a):

v productRepository mas sql injection diru

mozem poprosit konkretnejsie?

dale jen tak namatkove:

$session = $this->presenter->context->getService("session");
$cartSession = $session->getSection('cart');

1. v presenteru $this->presenter=== $this
2. na context bys nemel sahat (skoro nikde)
3. v presenteru muzes session ziskat pres $this->getSession('cart')

…

https://gitlab.websupport.sk/…resenter.php#L33 to cislo 11, neni vubec dobry se takhle spolehat na ID v databazi. a kdyzuz, tak bys to mel mit v nejake konstante..

…

https://gitlab.websupport.sk/…resenter.php#L98 generovat takhle nejaky kody neni bezpecne. pouzij radeji treba https://api.nette.org/…/Random.html

…

https://gitlab.websupport.sk/…resenter.php#L219
na sestavovani url muzes pouzit treba https://api.nette.org/…ttp/Url.html nebo aspon http://cz2.php.net/…ld-query.php

…

checkoutpresenter a objednavkapresenter maji skoro stejny kod.

…

https://gitlab.websupport.sk/…resenter.php#L38 count() nad selection je neefektivni. to vybere vsechny zaznamy z db. pokud chces pouzit promo count v sql, dej tam parametr *, tedy count('*)

David Matějka napsal(a):

dale jen tak namatkove:

$session = $this->presenter->context->getService("session");
$cartSession = $session->getSection('cart');

1. v presenteru $this->presenter=== $this
2. na context bys nemel sahat (skoro nikde)
3. v presenteru muzes session ziskat pres $this->getSession('cart')

…

https://gitlab.websupport.sk/…resenter.php#L33 to cislo 11, neni vubec dobry se takhle spolehat na ID v databazi. a kdyzuz, tak bys to mel mit v nejake konstante..

…

https://gitlab.websupport.sk/…resenter.php#L98 generovat takhle nejaky kody neni bezpecne. pouzij radeji treba https://api.nette.org/…/Random.html

…

https://gitlab.websupport.sk/…resenter.php#L219
na sestavovani url muzes pouzit treba https://api.nette.org/…ttp/Url.html nebo aspon http://cz2.php.net/…ld-query.php

…

checkoutpresenter a objednavkapresenter maji skoro stejny kod.

…

https://gitlab.websupport.sk/…resenter.php#L38 count() nad selection je neefektivni. to vybere vsechny zaznamy z db. pokud chces pouzit promo count v sql, dej tam parametr *, tedy count('*)

dakujem opravil som vsetky tieto nedostatky

inac ak by sa to dotiahlo uplne, mohlo by sa to niekde hodit aj tu na stranku pre inspiraciu pre ostatnych

MKI-Miro napsal(a):

Ahojte

takze implementoval som vsetky pripomienky

https://gitlab.websupport.sk/…/tree/master

napada niekoho este nejake vylepsenie ?

myslite ze by sa to mohlo niekde publikovat aby to pomohol zaciatocnikom ?

online sa to da vidiet tu

http://www.hrackykosice.sk/

Vdaka

Ahoj,

http://www.hrackykosice.sk/4m

Jaký má význam ten levý sloupec?

Upřímně co se mě týče, v roli zákazníka bych ten eshop asi vypl, a pokračoval dál. Vyhledávání by určitě mělo našeptávat jak produkty, tak kategorie. A levý panel by určitě neměl být vypsán výrobci na prvním místě.

Když jdu pro hračky, chci dejme tomu třeba tank, protože je rád lepím. No tady musím použít ctrl+f, abych tu kategorii vůbec našel.

Editoval GEpic (21. 8. 2016 19:57)

Editoval jsem původní post, nevšiml jsem si, že si odpovídal.

Plus za mě další věc:

• Zobrazení na mobilních telefonech / tabletech

A nemůžu se zbavit pocitu, že i přesto že to je na Nette, je to nějaké pomalé (a to nemám pomalý internet). Nekoukal jsem do kódu, ale ty výrobce asi nemáš nacachované, že? Veškeré kategorie v levém sloupci vypisuješ při každém requestu.

Editoval GEpic (21. 8. 2016 20:01)

@GEpic nemam nacachovane (hned aj idem pozriet ak sa to robi)

na tablete sa ti to zle zobrazuje ? celkovo aleob len nejaka cast ?

@Myiyk

kde by som tu licenciu mal dat ?
(skusim tie commity pomenuvavat)

@ all

mam na vas este jednu otazku
v base presenteri nastavujem $this->lang
ale v translatorovi mi nefunguje if ($this->lang == "") {
ako to spravne zapisat ?

vdaka

Jen takova technicka, proc nikde v presenterech nepouzivas konstruktor? (neber to jako pripominku, jen cira zvedavost)

@MKI-Miro ve finálních presenterch, který už nikdo nedědí (abys nemusel předávat všechny závislosti z předků)

namiesto konsturktoru robim mam inject nie ?

/** @var \App\Model\category_manufacturerRepository @inject */
    public $category_manufacturerRepository;

1.stale nerozumiem co ine by malo byt v tom konstruktore

2.
mam na vas este jednu otazku
v base presenteri nastavujem $this->lang
ale v translatorovi mi nefunguje if ($this->lang == "") {
ako to spravne zapisat ?

MKI-Miro napsal(a):

namiesto konsturktoru robim mam inject nie ?

/** @var \App\Model\category_manufacturerRepository @inject */
    public $category_manufacturerRepository;

1.stale nerozumiem co ine by malo byt v tom konstruktore

2.
mam na vas este jednu otazku
v base presenteri nastavujem $this->lang
ale v translatorovi mi nefunguje if ($this->lang == "") {
ako to spravne zapisat ?

S tim bych rekl ze to bude proto, ze $this->lang v base presenteru je v uplne jinem contextu nez $this->lang v translatoru. (nededi z basePresenteru. ale nejsem si jistej jestli $lang neni nejaka „globalni“ promenna)

ASi bych v tomto pripade nespolehal na to ze je nejaka promena prazdna, a snazil bych se ji davat obsah vzdy. Takze budto bude $this->lang == „svk“ nebo $this->lang == „en“ a pak to muzes oddebugovat v translatoru.

Editoval Muhahe (7. 9. 2016 14:12)

@Muhahe To tvoje mi přijde, že není úplně podle DI pravidel (vytváříš třídu ručně, nikoliv aby sis ji nechal předat již vytvořenou).

@MKI-Miro o jazyku by měl vědět asi translator, tak mu nějakej jazyk nastav.

Pavel Kravčík napsal(a):

@Muhahe: Já mám právě za to, že je tohle ošklivé zneužívání containeru. Na google se bude dát něco najít, třeba pěkný článek od Davida.

Nejhezčí varianta samozřejmě je mít presentery, co nejčistější. Buď komponenty s továrničkami nebo modely, které si svoje závislosti řeší sami. Případně předek s injectem.

Dobre. Diky za odkaz a novou myslenku ;)

Netvrdim ze to ma vyhody, prave proto se ptam, proc on to nepouziva, abych pripadne nasel lepsi cestu. Puvodne jsem predaval modely do parametru konstruktoru, ale casto jsem se dostal do problemu, ze jich bylo docela dost. Tak predavam jen database, a modely vytvarim prave v konstruktoru

@Muhahe Za mě je určitě lepší předávat přímo modely. Třída potřebuje crawlerModel tak se k němu v konstruktoru hlásí a ne že si ho sama bude v konstruktoru vytvářet.

Co když někdy v budoucnu budeš chtít používat jinou implementaci crawlerModel? Například z důvodu unit testování.. Budeš potřebovat závislosti mockovat a u tvého způsobu to nepůjde.

Problém s příliš mnoha závislostmi v konstruktoru lze určitě řešit lépe – buď třídu rozdělit do více menších tříd, které nebudou potřebovat tolik závislostí, a nebo několik závislostí obalit novou třídou-modelem, která bude té tvé třídě poskytovat rozhraní. Místo 9 závislostí pak budeš mít třeba 3 (ty nové třídy-modely) z nichž každá bude mít třeba taky 3.

Editoval Barvoj (7. 9. 2016 15:04)

@PavelKravčík To už je myslím jen otázka priorit :-) Já konstruktor používám jen u komponent, protože jsou ve většině případů finální, případně přes setup+decorator nastavím potřebné závislosti do Base* tříd.
U BasePresenterů se mi teda spíš osvědčily inject* metody, protože u public property nikdy nevíš kdo je bude chtít použít někde úplně mimo (v komponentě třeba), nebo nějakej potomek ji bude taky využívat (to se mi nelíbí tak používám private v BasePresenterech).

Myslim, že se moc odkláníme od tématu… :-/

vidite

existuje x otazok ako nieco spravit aby to bolo „best practise“, nikde nie je urobene nejake komplexnejsie riesenie z ktoreho by sa to dalo okukat a prave to ma prividlo k tomu aby som zverjnil nieco co moze pomoct ostatnym :)

takze uz len sa zhodnut ako to upravit aby bolo vsetko vporiadku :)

MKI-Miro napsal(a):

CZechBoY napsal(a):

@Muhahe To tvoje mi přijde, že není úplně podle DI pravidel (vytváříš třídu ručně, nikoliv aby sis ji nechal předat již vytvořenou).

@MKI-Miro o jazyku by měl vědět asi translator, tak mu nějakej jazyk nastav.

cize navrhujes aby to co sa robi v Base preseneteri:

$httpRequest = $this->context->getService('httpRequest')->getUrl();
        if ($httpRequest->host == "www.hrackykosice.sk" || $httpRequest->host == "hrackykosice.sk")
            {$this->lang = "";$this->site = "hracky";}
        else if ($httpRequest->host == "www.gameskosice.com" || $httpRequest->host == "gameskosice.com")
            {$this->lang = "_en";$this->site = "games";}
        else if ($httpRequest->host == "www.wholesalekosice.eu" || $httpRequest->host == "wholesalekosice.eu")
            {$this->lang = "_en";$this->site = "wholesale";}

aby sa robil aj v Translator ?

Osobne bych detekoval jazyk prave v tom translatoru, ale jestli to neni prasarna, to ti nepovim. Ale urcite bude problem v tom ze promena $this->lang v basePresenteru je uplne jina promenna nez $this->lang v translatoru.

A myslenka translatoru mi prijde takova, ze prave on se stara o to, v jakem jazyku ma byt vysledek, takze proc zatezovat zbytek kodu jazykem? at se pekne postara translator :)

@MKI-Miro já bych si zaregistroval ten tvůj překladač do DIC a po vytvoření bych zavolal nějakou metodu na detekci jazyku.

class Translator implements \Nette\Localization\ITranslator
{
	public function detectLanguage()
	{
		// nejak zjisti jazyk..
		// muzes si do parametru i nechat predat nejakou sluzbu
	}

	public function translate($message, $count = null)
	{
		// tvoje implementace ITranslator::translate
	}
}

services:
	-
		class: App\Translator
		setup:
			- detectLanguage

jinak to kontrolování jazyka při každém volání translate je nic moc… co použít třídní proměnnou, kterou naplníš v tom detectLanguage?

btw. můžeš si udělat vlastní latte.templateFactory abys měl v každý šabloně už translator nastavený

Editoval CZechBoY (7. 9. 2016 16:26)

CZechBoY napsal(a):

@MKI-Miro já bych si zaregistroval ten tvůj překladač do DIC a po vytvoření bych zavolal nějakou metodu na detekci jazyku.

>

jinak to kontrolování jazyka při každém volání translate je nic moc… co použít třídní proměnnou, kterou naplníš v tom detectLanguage?

btw. můžeš si udělat vlastní latte.templateFactory abys měl v každý šabloně už translator nastavený

tak k tomuto mam viacero otazok

1. toto sa teda nema nachadzat v base presenter ?

$translator = new \App\Language\Translator($this->lang);
        $this->template->setTranslator($translator);

2. Kedy volat detectLanguage?

3. ako v tom datectLanguage zistim aktualnu domenu ?

dakujem

@MKI-Miro
překladač je služba, kterou můžeš používat i jinde než v presenterech (v komponentách, latte filtrech nebo i modelech a obyč třídách), takže bys to měl taky jako službu používat (zaregistruj do DIC v konfiguráku).
ad detekce jazyka podle domény: můžeš si předat třeba instanci Nette\Http\IRequest a z ní vyčíst doménu (jak už si ukazoval nahoře)

public function detectLanguage(\Nette\Http\IRequest $httpRequest)
{
	if ($httpRequest->host === "www.hrackykosice.sk" || $httpRequest->host === "hrackykosice.sk") {
		$this->lang = "Slovak";
	} elseif ($httpRequest->host === "www.gameskosice.com" || $httpRequest->host === "gameskosice.com") {
		$this->lang = "English";
	} elseif ($httpRequest->host === "www.wholesalekosice.eu" || $httpRequest->host === "wholesalekosice.eu") {
		$this->lang = "English";
	} else {
		$this->lang = "Czech";
	}
}

pak translate:

public function translate($message, $count = null)
{
	return constant("App\Language\\{$this->lang}::{$message}");
}

kdy volat detectLanguage jsem ti napsal minule, zaregistruj si ten překladač jako službu a jako setup uveď tu metodu detectLanguage (parametry se doplní z DIC).

Editoval CZechBoY (7. 9. 2016 17:02)

Zachytim si tu výjimku v ErrorPresenteru a zákazníkovi ukážu 404, většinou si 404 loguju max jako info, ať mi to nepřekáží mezi exception/error.