NOVINKA: Získej 40 hodin praktických dovedností s AI – ZDARMA ke každému akreditovanému kurzu!
Hledáme nové posily do ITnetwork týmu. Podívej se na volné pozice a přidej se do nejagilnější firmy na trhu - Více informací.

Diskuze: Machr na PHP - Základ eshopu

V předchozím kvízu, Online test znalostí PHP, jsme si ověřili nabyté zkušenosti z kurzu.

Aktivity
Přejít k původní otázce
Avatar
MadaraCZech
Člen
Avatar
Odpovídá na mkub
MadaraCZech:19.1.2014 10:42

Dobře, díky. Už se blížím ke konci :)

Odpovědět
19.1.2014 10:42
...
Avatar
Honza Bittner
Tvůrce
Avatar
Honza Bittner:19.1.2014 11:05

Někteří z vás už to mají pěkné, jen vám poradím, že se ještě v e-shopech dává kolikrát tu položku chcete, tzn tam nebude

samsung
samsung

ale

samsung 2x

Také by bylo dobré doplnit to o AJAX. :)

Nahoru Odpovědět
19.1.2014 11:05
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
MadaraCZech
Člen
Avatar
Odpovídá na Honza Bittner
MadaraCZech:19.1.2014 11:54

To jsem chtěl dnes udělat :)

Nahoru Odpovědět
19.1.2014 11:54
...
Avatar
mkub
Tvůrce
Avatar
Odpovídá na MadaraCZech
mkub:19.1.2014 14:10

a este nieco... radsej si na localhost nahod WAMP (ak mas Windows) a vyvijaj to offline, nie online, lebo s tymi chybkami si prezradzas aj adresarovu strukturu...
a ak bude to plnefunkcne, az potom to zaves dakde na nete a spristupni to ostatnym

 
Nahoru Odpovědět
19.1.2014 14:10
Avatar
Kit
Tvůrce
Avatar
Odpovídá na mkub
Kit:19.1.2014 14:18

WAMP je dnes už zbytečný. Stačí samotné PHP, funguje jako webserver.

Nahoru Odpovědět
19.1.2014 14:18
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
mkub
Tvůrce
Avatar
Odpovídá na Kit
mkub:19.1.2014 14:25

vcera som cital o tom... ale niektore tie baliky este stale obsahuju webovy server Apache

 
Nahoru Odpovědět
19.1.2014 14:25
Avatar
Kit
Tvůrce
Avatar
Odpovídá na mkub
Kit:19.1.2014 14:29

Těch webových serverů je celá řada. I Python má svůj. Některé fungují i jako proxy, samotné FastCGI se dá použít také jako webserver, Nginx, Lighttpd, ...

Nahoru Odpovědět
19.1.2014 14:29
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
mkub
Tvůrce
Avatar
Odpovídá na Kit
mkub:19.1.2014 14:31

s tym suhlasim... tie dva co si spomenul su neporovnatelne mensie nez Apache a rychlejsie...

 
Nahoru Odpovědět
19.1.2014 14:31
Avatar
Kit
Tvůrce
Avatar
Odpovídá na mkub
Kit:19.1.2014 14:34

Našel jsem i pár opravdu miniaturních webserverů, měly něco kolem 30 KB.

Nahoru Odpovědět
19.1.2014 14:34
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
mkub
Tvůrce
Avatar
Odpovídá na Kit
mkub:19.1.2014 14:59

existuje tolko roznych webovych serverov, dokonca aj Facebook ma svoju verziu weboveho servera

 
Nahoru Odpovědět
19.1.2014 14:59
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Odpovídá na David Hartinger
Zdeněk Pavlátka:19.1.2014 15:23

V jakém formátu chceš ten export databáze? phpMyAdmin mi dává strašně možností.

Nahoru Odpovědět
19.1.2014 15:23
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
Zdeněk Pavlátka:19.1.2014 17:34

V čem se obvykle odevzdává export databáze?

Nahoru Odpovědět
19.1.2014 17:34
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
Zdeněk Pavlátka:19.1.2014 17:37

To tu nikdo není???

Nahoru Odpovědět
19.1.2014 17:37
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
Ondřej Štorc
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
Ondřej Štorc:19.1.2014 17:41

Podle mně v SQL..

Nahoru Odpovědět
19.1.2014 17:41
Život je příliš krátký na to, abychom bezpečně odebírali USB z počítače..
Avatar
Milan Gallas
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
Milan Gallas:19.1.2014 17:44

Export - a můžeš nechat klidně možnost SQL, kterou budeš mít vybranou automaticky. A to je vše. Já to tak už kolikrát do soutěže odevzdal.

 
Nahoru Odpovědět
19.1.2014 17:44
Avatar
Michal Martinek
Tvůrce
Avatar
Michal Martinek:19.1.2014 20:22

Ahoj. Účastním se také soutěže s mým dnešním na rychlo splácaným výtvorem. Je to napsané s databází SQLite3, takže jestli ji máte na svém počítači, tak vám to pojede ihned. Jestli máte pouze MySQL a nechcete SQLite, tak odkomentujte v index.php připojení k MySQL,zakomentujte to s SQLite a rozjeďte si ten sql příkaz. (MySQL je tam rychle předělané, tak po něm moc nechtějte). Je to můj první pokus o MVC, tak pište prosím jakékoliv připomínky.
http://leteckaposta.cz/904968463

 
Nahoru Odpovědět
19.1.2014 20:22
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Michal Žůrek - misaz:19.1.2014 20:40

Ahoj, přidám svůj výtvor.
Je to zatím bez dokumentace a řeším online hosting co podporuje .htacces .
http://misaz.moxo.cz/…r-na-php.zip
export tabulek je zde http://misaz.moxo.cz/…dbexport.sql počíta s tím že databáze se jmenuje eshop, pokud chcete jinak, tak to pozměňte v eshopsettings.xml.

Je to poměrně róbusní (strávil jsem s tím celý týden) a je to navrženo, tak aby to šlo snadno rozšířit, alespoň já to zvládnu a až dopíšu komentáře, dokumentaci, tak by to mohli zvládnout i ostatní, uvidím jak to do zítřejších 10:00 stihnu.

Je tam i narychlo splácaná šablona v CSS, omlouvám se a odsazení zdrojového kódu, protože PHP tools mám jen ve Visual Studiu 2010, tak místo tabulátoru jsou mezery (neměl jsem náladu hledat to v nastavení bez vyhledávacího políčka) a CSS je odsazené úplně otřesně.

Do budoucna to budu rozšiřovat, snažil jsem se to navrhnout tak aby přidáni funkcionality bylo co nejsnadnější.

Router jsem si napsal celý sám, takže se od běžně používaných možná bude mírně lišit, chtěl jsem aby si našetl třídu podle URL, zjisitl jestli ta třída implementuje IPage (kvůli injekci) a pokud implementuje nechal ji připravit se a vypsat. Protože to skoro celé funguje na AJAXu, tak Router umí pracovat i s IAjaxItem, funguje obdobně jako u IPage.

Pro jistotu jsem to dal už dnes (i s nějakými menšími nedostatky), protože poslední dobou rád spím i do 11:00.

Editováno 19.1.2014 20:42
 
Nahoru Odpovědět
19.1.2014 20:40
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Odpovídá na David Hartinger
Zdeněk Pavlátka:19.1.2014 20:54

Tady dávám svůj výtvor:
http://nuz15-own-web.9e.cz/index.php
Divné znaky v textu ignorujte, špatně se načítají z databáze. na localu mi to fungovalo, asi se to špatně načetlo na server.
Stránka umožňuje hodnocení knih, hledání mezi nimi a snažil jsem se o co nejlepší design. Při objednávání se vypíše seznam objednaných knih. Je to můj první projekt v PHP a tento jazyk spolu s MySQL jsem se učil během tvorby. Kód tedy není nejlepší, ale snažil jsem se o objektový návrh.
Zdroják:
https://dl.dropboxusercontent.com/…r_na_php.zip
Export databáze:
https://dl.dropboxusercontent.com/…database.sql

Nahoru Odpovědět
19.1.2014 20:54
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
Zdeněk Pavlátka:19.1.2014 21:18

Kódování opraveno :) Nalezená chyba (neodeslání objednávky) opravena.

Nahoru Odpovědět
19.1.2014 21:18
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
MadaraCZech
Člen
Avatar
MadaraCZech:19.1.2014 21:30

ONLINE : www.machrphp.tode.cz

ZDROJÁK : http://uloz.to/…machrphp-zip

heslo : sdraco

DB : http://uloz.to/…89732075-sql¨

heslo : sdraco

Nahoru Odpovědět
19.1.2014 21:30
...
Avatar
Ondřej Štorc
Tvůrce
Avatar
Odpovídá na MadaraCZech
Ondřej Štorc:19.1.2014 21:35

Přečti si pravidla pro soutěže..

Budou nahrány na shareserver leteckaposta.cz, dropbox nebo Google drive. Ne uloz.to nebo mega.
Nahoru Odpovědět
19.1.2014 21:35
Život je příliš krátký na to, abychom bezpečně odebírali USB z počítače..
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
Zdeněk Pavlátka:19.1.2014 21:35

Opravil jsem pár zbývajících chyb. Upravený zdroják je na stejné adrese.
pozn. Že je kniha skladem znamená, že jí mám doma :D

Editováno 19.1.2014 21:36
Nahoru Odpovědět
19.1.2014 21:35
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
MadaraCZech
Člen
Avatar
MadaraCZech:19.1.2014 21:42

OPRAVA :

ZDROJÁK : http://leteckaposta.cz/583258292

DB : http://leteckaposta.cz/969382375

Nahoru Odpovědět
19.1.2014 21:42
...
Avatar
Pavel Mareš
Tvůrce
Avatar
Odpovídá na MadaraCZech
Pavel Mareš:19.1.2014 21:54

Madara stejně ti to nefunguje:

Kontrola objednávky
  2x Tričko - 359 Kč Odebrat -> 718
  1x Boty - 692 Kč Odebrat -> 718 + 692 = ?1051? -> 1410 Kč
    Celkem : 1051 Kč

Je hezké, že ti to zobrazuje že mám 2x triko, ale počítat s tím to moc neumí. Musím ale uznat, že tvůj eshop je pro zákazníka asi nejvýhodnější. Pravda že oprava asi není nijak nákladná, ale ta chyba tam je.

Nahoru Odpovědět
19.1.2014 21:54
Však ono půjde ...
Avatar
Martin Konečný (pavelco1998)
Tvůrce
Avatar
Martin Konečný (pavelco1998):19.1.2014 22:26

Mohli byste, prosím, zasílat i ukázky? Rád totiž hledám způsoby, jak podstrčit špatná data a poškodit tak systém. :-)

Pozn.: Ale pro dobré účely! Chyby většinou autorovi nahlašuji, a tím mu pomáhám do budoucna k lepší ochraně.

Nahoru Odpovědět
19.1.2014 22:26
Aktuálně připravuji browser RPG, FB stránka - https://www.facebook.com/AlteiraCZ
Avatar
idc
Člen
Avatar
Odpovídá na Zdeněk Pavlátka
idc:19.1.2014 23:04

Když dám koupit a poté se chci vrátit na předchozí stránku, tak se mi košík vyprázdní. To je korektní chování? :) Promiň, jestli to je tak úmyslně, jen mě to překvapilo :)

 
Nahoru Odpovědět
19.1.2014 23:04
Avatar
Unregistered User
Neregistrovaný
Avatar
Unregistered User:20.1.2014 0:13

Jak jsi to dělal dlouho? Víš, že se hodnotí design a uděláš to co jsi udělal? :` A co to podlézání? Eshop u sdraca.. chudák sdcraco :P

 
Nahoru Odpovědět
20.1.2014 0:13
Avatar
Zdeněk Bauer
Tvůrce
Avatar
Odpovídá na idc
Zdeněk Bauer:20.1.2014 1:11

Ano, chová se to správně.
Protože pokud se vrátíš o stránku zpět, načte se ta stránka z cache prohlížeče a tam jsi ty předměty ještě koupené neměl.

@Neregistrovanej: Kde jsi četl že se hodnotí i design? :O Jde o machra od Sdraca, jako podlézání bych to moc neřešil...

 
Nahoru Odpovědět
20.1.2014 1:11
Avatar
Kit
Tvůrce
Avatar
Odpovídá na idc
Kit:20.1.2014 8:27

Není to korektní chování.

Nahoru Odpovědět
20.1.2014 8:27
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Odpovídá na Kit
Zdeněk Pavlátka:20.1.2014 8:30

Už jsem psal, že je to první, co jsem v php napsal. Funguje to a to je úspěch.

Nahoru Odpovědět
20.1.2014 8:30
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Michal Žůrek - misaz:20.1.2014 10:42

konečně mi moxo zprovoznilo DNS a funguje http://misazeshop.moxo.cz/ . Při nasazování jsem zjistil že na serveru kontroluje velikost písmen a proto mi to vždy hází nějaké chyby, protože v zdrojácích je to malžmi písmeny pay, cash, cart ale ty třídy jsou velkými Pay, Cash, Cart, nevím proč mi to neházelo ty chyby na localhostu. Kdyby vám to nejelo vůbec tak opravená varianta je zde: http://misaz.moxo.cz/…repaired.zip

 
Nahoru Odpovědět
20.1.2014 10:42
Avatar
mkub
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
mkub:20.1.2014 10:53

je to dane tym, ze Windows nerozlisuje velke a male pismena, ale Unix like OS rozlisuju a to je casta chyba pri prenose kazdeho weboveho projektu z Windowsu na Linux/Unix

 
Nahoru Odpovědět
20.1.2014 10:53
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Odpovídá na mkub
Michal Žůrek - misaz:20.1.2014 10:55

jo, tak aspoň že David Hartingervi bude fungovat i ta první verze co jsem stihl poslat před deadline.

 
Nahoru Odpovědět
20.1.2014 10:55
Avatar
Martin Konečný (pavelco1998)
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
Martin Konečný (pavelco1998):20.1.2014 11:04

Ty v tom eshopu můžeš odeslat oběd?

Nahoru Odpovědět
20.1.2014 11:04
Aktuálně připravuji browser RPG, FB stránka - https://www.facebook.com/AlteiraCZ
Avatar
mkub
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
mkub:20.1.2014 11:30

ked si pouzil OOP, kludne si mohol vyuzit aj auload, tym by si si usetril nacitavanie tried (trieda by sa nacitala az vtedy, ak by bola potrebna - pri vytvarani instancie), pri viacerych triedach velmi to zjednoduhsi kod.
dalej v tom kode hrozi, ze budez neho gulas...

vo sa tyka funkcnosti, pokial by bol db stroj nedostupny, tak by sa zobrazila hlaska, ze sa nepodarilo k databazi pripojit (co by bolaspravna cinnost), ale pokracovalo by to vyberom databazy, co by skoncilo vygenerovanim typickej chybovej hlasky, ze neexistuje ziadna databaza

ale inak na to, ze to je tvoj prvy PHP kod, sa mi paci

 
Nahoru Odpovědět
20.1.2014 11:30
Avatar
mkub
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
mkub:20.1.2014 11:40

vzdy treba dodrziavat nazvy presne take velke, ako to mas ulozene vo filesysteme... napr. u mna by to bez drobnej upravy nefungovalo, kedze moj system rozlisuje velkost pismena a ze: index.php != INDEX.PHP != Index.php
vo svete Windows plati: index.php == INDEX.PHP == Index.php
cize treba na to davat pozor, lebo system Windows aj automaticky konvertuje velkost pisma v nazve suborov a ak mas niekde volanie napr.:

include ("functions.php"};

a subor index.html, tak pri ukladani sa ti moze stat, ze Windows to skonvertuje na Index.html a vo Windowse to bude fungovat bezproblemovo,
ale ak to prenesies na Linuxovy, ci Unxovy, stroj (ci na Mac, resp. Solaris), tak ti to potom prestane fungovat...

u sdraca by to fungovalo, ale na freehostingu nie, kedze freehosting bezi na LAMPP

 
Nahoru Odpovědět
20.1.2014 11:40
Avatar
Honza Bittner
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
Honza Bittner:20.1.2014 11:41

Škoda jen že nejde přidat vícekrát položka z obchodu, ale jenom z košíku... Ale jinak super... :)

Nahoru Odpovědět
20.1.2014 11:41
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
Honza Bittner
Tvůrce
Avatar
Odpovídá na MadaraCZech
Honza Bittner:20.1.2014 11:45

To tvé se mi líbí ... pro zákazníka výhodné : http://prntscr.com/2kxqtb :)

Nahoru Odpovědět
20.1.2014 11:45
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
Zdeněk Bauer
Tvůrce
Avatar
Zdeněk Bauer:20.1.2014 14:13

Dneska se to nějak protáhlo, že? :)

 
Nahoru Odpovědět
20.1.2014 14:13
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Odpovídá na Zdeněk Bauer
Michal Žůrek - misaz:20.1.2014 14:17

těch příspěvků je strašně moc, nehledě na to že některé nejsou ani tady ve vlákně ale v PM. Navíc byl oběd, pak si sdraco musel dát nějakou pauzu aby mu z toho nehráblo. Nehledě na to že hodinu a půl trval druhý machr.

 
Nahoru Odpovědět
20.1.2014 14:17
Avatar
Zdeněk Bauer
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
Zdeněk Bauer:20.1.2014 14:20

Jojo, máš recht :)
Taky musím na oběd.
Možná by se hodilo tlačítko pro označení příspěvku jako "odevzdávací" aby se to pak nepletlo s obyčejnýma příspěvkama.

 
Nahoru Odpovědět
20.1.2014 14:20
Avatar
Zdeněk Bauer
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
Zdeněk Bauer:20.1.2014 14:20

//Hops :)

Editováno 20.1.2014 14:21
 
Nahoru Odpovědět
20.1.2014 14:20
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Odpovídá na Zdeněk Bauer
Michal Žůrek - misaz:20.1.2014 14:22

celkově by se pro to hodil nějaký systém, on možná nějaký časem přijde.

 
Nahoru Odpovědět
20.1.2014 14:22
Avatar
Martin Konečný (pavelco1998)
Tvůrce
Avatar
Martin Konečný (pavelco1998):20.1.2014 14:30

Bere se v úvahu i bezpečnost? Snad u všech eshopů jsem si dokázal objednat neexistující věc, smazat neexistující věc (v košíku mám -2 produktů), nebo dokonce kupovat věci za -17000 Kč.

Nahoru Odpovědět
20.1.2014 14:30
Aktuálně připravuji browser RPG, FB stránka - https://www.facebook.com/AlteiraCZ
Avatar
Zdeněk Bauer
Tvůrce
Avatar
Odpovídá na Martin Konečný (pavelco1998)
Zdeněk Bauer:20.1.2014 14:31

Určitě, toto je základ ;)
Já to mám celkem vyladěné, jen pochybuju že to bude uznaný či vůbec výherní :)

 
Nahoru Odpovědět
20.1.2014 14:31
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Odpovídá na Martin Konečný (pavelco1998)
Michal Žůrek - misaz:20.1.2014 14:36

jo, umě jde taky nakupovat za -1700, ale najde třeba smazat databázi. Ty záporné počty jsou pak jen o přidání jednoho řádku kódu. Čili já jsem tak nějak počítal, že se tam něco najde, ale ty bezpečnostní věci by měly být OK.

Editováno 20.1.2014 14:36
 
Nahoru Odpovědět
20.1.2014 14:36
Avatar
michalkasparec
Tvůrce
Avatar
michalkasparec:20.1.2014 15:11

Škoda, že sem se nestihl zúčastnit, ale po prohlédnutí soutěžních e-shopů některé nevypadají špatně. Až na kontrolu existujícího zboží a vkládání validních dat do políček atd. :)
Smekám před lidma, co to "splácali" za pár hodin a je to vlastně nejspíš jejich první eshop

edit: ani nevím, jestli bych se mohl zúčastnit, když dělam e-shopy v Nette

Editováno 20.1.2014 15:12
 
Nahoru Odpovědět
20.1.2014 15:11
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Odpovídá na Michal Žůrek - misaz
Zdeněk Pavlátka:20.1.2014 16:09

U mě dát záporný počet nejde, ale není nejlepší uchovávat objednávku v url.

Nahoru Odpovědět
20.1.2014 16:09
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
David Hartinger
Vlastník
Avatar
David Hartinger:20.1.2014 17:36

Pánové, úroveň výsledků je opravdu příšerná, hodnotil jsem to dost dlouho a jsem z toho zklamaný a unavený. Machr na PHP bude určitě častěji, protože potřebujeme trénovat. PHP je opravdu špatně navržený jazyk. Pokud ho člověk dobře umí, dělá hezké věci. Ale pokud ho neumí nebo i když ho umí jen průměrně, tak se ten kód nedá číst.

Heredoc
Použití Heredoc není cool, a to ani trochu. Je to naprosto zbytečná věc, která se v praxi nepoužívá. Pro šablony je naprosto nevhodná, protože umožňuje vkládat pouze proměnné, nelze zavolat metodu. A ne, ani hackování tohoto nedostatku pomocí getterů není cool. PHP má naprosto skvělou šablonovou syntaxi, která je určena přesně k tomu, aby byla v šablonách (a ne, není to, že něco echujete, PHP má přímo konstrukce pro tvorbu šablon, stačí se podívat na místní tutoriály). Za to, že tuto syntaxi neznáte a místo ní znáte tento nesmysl, může dost možná Kit, tímto ho tedy žádám, aby to tu přestal propagovat. Pokud vám PHP šablony z nějakého důvodu nestačí nebo se vám nelíbí, existují věci jako Smarty nebo Latte, které jsou přímo vytvořené pro tvorbu složitých šablon. Heredoc k tomu není a již ho takto použitý nechci vidět. Dokonce mi připadá, že si myslíte, že když se HTML echuje v modelu v heredoc, tak je to v pořádku. Ne, není, HTML patří do šablony.

Narazil jsem dokonce na Kontroler, který vypadal takto:

foreach ($items as $item => $count) {
  $name = $productManager->getNameOfItem($item);
  $tbody[] = "<tr><td>".$name["jmeno_vyrobku"]."</td><td>".$count."</td><td>".
  ($bag->getPriceOfItem($item)*$count)."</td></tr>";
}
$tableRow = implode("\n", $tbody);

$this->data['table'] = @$table;

Tato technika je opravdu postavená na hlavu, kontroler je potom zasviněný HTML kódem a implode. MVC ztrácí smysl, takhle ne.

SQL injekce
V několika řešeních jsem našel dokonce SQL injekce. Toto je naprostý základ, je to jako byste psali sloh a nevěděli, že věta začíná velkým písmenem. Všechna řešení s SQL injekcemi v těchto soutěžích budou odteď automaticky diskvalifikována a kód nebudu vůbec dále číst.

Pořadí hodnocení neodpovídá umístění v soutěži. Nejvíce se mi líbila řešení Zdeněk Bauer a Michal Martinek, ti získávají placku. Nevím, jestli to měli úplně dobře, jsem z ostatních opravdu odrovnaný.

Hodnocení

  • zdenda204 - Tak se konečně setkávám s Nette. Řešení vypadá na první pohled dobře, zde máš asi výhodu oproti ostatním, jelikož v Nette nedělám.
  • fuulll - Nevím, proč se metoda renderující pohled jmenuje setView. Další případ nečitelných šablon s Heredoc a znečištěných kontrolerů a modelů.
  • Misaz - Nešlo mi přidat od jednoho zboží více položek. Návrhově je to zajímavé, ale když už jsi se s tím tak piplal, proč jsi to rovnou neudělal jako MVC? Sice to máš krásně objektové, ale to hlavní (oddělení logiky a výstupu) tam není, takže to potom ztrácí smysl. A ne, heredoc tomu neprospěje.
  • mm8 - Opravdu nevím, proč v HTML šabloně otevřeš PHP, ve kterém následně vypíšeš HTML. To nedává vůbec žádný smysl. Na rozdíl od fuullla to máš alespoň v šabloně a ne v kontroleru. Máš tam metody co se jmenují vypiš, ale ve skutečnosti něco vracejí, sekvence jako return true else return false a podobně. Nicméně tohle se mi líbí asi nejvíc, takže máš placku. Podívej se prosím tě do místních tutoriálu, jak má vypadat šablona.
  • patrik.valkovic - Docela hezké řešení. Nevím jaký má smysl ukládat query do pole pod číselné indexy. Je to strašně nepřehledné, SQL kód dotazu má být v metodě, ve které se ta query volá. Bindování v tom DB wrapperu je teda dost husté, pochybuji, že je taková příšernost potřeba, každopádně bych použil PDO místo MySQLi.
  • Facedown - Přemýšlel jsem proč nejde diakritika a ty tam nemáš vůbec žádnou HTML strukturu. Řešení je úplně neobjektové, ale pořád lepší, než špatné objektové. Allowid bys neměl plnit v indexu, ale v každém skriptu. Nějak mi uniká výhoda heredoc syntaxe, předpokládám, že tě to naučil Kit. Nepoužívej ji, zaprvé ji nikdo nepoužívá, zadruhé to IDE nebarví, zatřetí věci jako <<<EOT jsou zbytečné balasty navíc. Můžeš přeci PHP vypnout a napsat HTML jako HTML.
  • Deiv - Chválím MVC, validace patří do modelu, ne do controlleru. Pohledy jsou horší, vůbec nepoužíváš šablonovou syntaxi a dokonce v nich tvoříš modely. Pohled pouze vypisuje hotová data z nějaké kolekce, kterou mu připraví model a předá kontroler. Model košíku ti tam úplně chybí. Chybí uložení objednávky do DB.
  • nuz15 - Přijde mi divné předávat si objednávku v URL, takhle musíš parametr propagovat úplně všude a když ti zákazník vyťuká znovu URL nějaké stránky, přijde o obsah košíku. K tomu je přeci session. Co se týče kódu, tak na index.php resignuji, v tom se prostě nevyznám, nezabalený kód, miliony cyklů, v dalších souborech jsem našel SQL injekce. Přečti si nějaké základy práce s OOP v PHP.
  • MadaraCzech - První věc - SQL injekce, řešení s touto chybou automaticky nemůže vyhrát. Chválím seskupování položek v košíku dle typu. Řešení je naprosto neobjektové a ani se o objektovost nesnaží, což je možná lepší, než když se někdo snaží o OOP a vznikne patlanice. PHP má šablonou syntaxi, je zbytečné psát do šablon echo nebo za while ukončovat direktivu.
  • kubp - Výstup vypadá skvěle, ale v kódu mícháš objektový a procedurální přístup, hlavně ck.php je dost divné včetně názvu souboru, měl by se jmenovat jako třída. V ed.php máš dost dalších zásadních chyb, hlavně SQL injekce, používání SELECT místo COUNT a obecně konvence. Tohle budeš muset dopilovat.

Příště již nebudu vůbec hodnotit řešení s tak zásadními chybami, jako jsou SQL injekce nebo nefunkční diakritika. Pevně doufám, že se úroveň soutěží v tomto jazyce časem zlepší, oproti ASP to je teda masakr, je vidět, že v dobře navrženém jazyce jsou prostě diametrálně kvalitnější výsledky.

Vítězové prosím publikujte a napište x. Ostatní si přečtěte nějaké místní seriály.

Nahoru Odpovědět
20.1.2014 17:36
New kid back on the block with a R.I.P
Avatar
Petr Nymsa
Tvůrce
Avatar
Odpovídá na David Hartinger
Petr Nymsa:20.1.2014 17:43

Wow :D Možná bych zkusil dát repete s podobným zadáním, tj i řešení spojené se SESSION, MVC apod :) Kdybych našel čas, přidal bych se. Myslím že bych také nedopadl lépe v PHP 8|

Nahoru Odpovědět
20.1.2014 17:43
Pokrok nezastavíš, neusni a jdi s ním vpřed
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Tato diskuze byla označena moderátorem jako nekonstruktivní a myslíme si, že nemá pro veřejnost příliš vysokou hodnotu. Členy můžete samozřejmě stále oslovit soukromě a to formou zpráv nebo chatu.

Zobrazeno 50 zpráv z 106.