Diskuze: Machr na PHP - Základ eshopu

Neaktivní uživatel

Člen

Neaktivní uživatel:19.1.2014 10:42

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

Odpovědět

19.1.2014 10:42

Neaktivní uživatelský účet

Honza Bittner

Tvůrce

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.

Neaktivní uživatel

Člen

Neaktivní uživatel:19.1.2014 11:54

To jsem chtěl dnes udělat

Nahoru Odpovědět

19.1.2014 11:54

Neaktivní uživatelský účet

mkub

Tvůrce

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

Kit

Tvůrce

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ů.

mkub

Tvůrce

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

Kit

Tvůrce

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ů.

mkub

Tvůrce

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

Kit

Tvůrce

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ů.

mkub

Tvůrce

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

Zdeněk Pavlátka

Tvůrce

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.

Zdeněk Pavlátka

Tvůrce

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.

Zdeněk Pavlátka

Tvůrce

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.

Ondřej Štorc

Tvůrce

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..

Milan Gallas

Tvůrce

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

Michal Martinek

Tvůrce

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

Michal Žůrek - misaz

Tvůrce

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

Nahoru Odpovědět

19.1.2014 20:40

Zdeněk Pavlátka

Tvůrce

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.

Zdeněk Pavlátka

Tvůrce

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.

Neaktivní uživatel

Člen

Neaktivní uživatel: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

Neaktivní uživatelský účet

Ondřej Štorc

Tvůrce

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..

Zdeněk Pavlátka

Tvůrce

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

Editováno

Nahoru Odpovědět

19.1.2014 21:35

Kolik jazyků umíš, tolikrát jsi programátor.

Neaktivní uživatel

Člen

Neaktivní uživatel: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

Neaktivní uživatelský účet

Pavel Mareš

Tvůrce

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 ...

Martin Konečný (pavelco1998)

Tvůrce

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

idc

Člen

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

Unregistered User

Neregistrovaný

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

Nahoru Odpovědět

20.1.2014 0:13

Zdeněk Bauer

Tvůrce

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? Jde o machra od Sdraca, jako podlézání bych to moc neřešil...

Nahoru Odpovědět

20.1.2014 1:11

Kit

Tvůrce

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ů.

Zdeněk Pavlátka

Tvůrce

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.

Michal Žůrek - misaz

Tvůrce

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

mkub

Tvůrce

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

Michal Žůrek - misaz

Tvůrce

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

Martin Konečný (pavelco1998)

Tvůrce

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

mkub

Tvůrce

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

mkub

Tvůrce

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

Honza Bittner

Tvůrce

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.

Honza Bittner

Tvůrce

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.

Zdeněk Bauer

Tvůrce

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

Michal Žůrek - misaz

Tvůrce

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

Zdeněk Bauer

Tvůrce

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

Zdeněk Bauer

Tvůrce

Zdeněk Bauer:20.1.2014 14:20

//Hops

Editováno

Nahoru Odpovědět

20.1.2014 14:20

Michal Žůrek - misaz

Tvůrce

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

Martin Konečný (pavelco1998)

Tvůrce

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

Zdeněk Bauer

Tvůrce

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

Michal Žůrek - misaz

Tvůrce

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

Nahoru Odpovědět

20.1.2014 14:36

michalkasparec

Tvůrce

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

Nahoru Odpovědět

20.1.2014 15:11

Zdeněk Pavlátka

Tvůrce

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.

David Hartinger

Vlastník

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

Petr Nymsa

Tvůrce

Petr Nymsa:20.1.2014 17:43

Wow 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

Nahoru Odpovědět

20.1.2014 17:43

Pokrok nezastavíš, neusni a jdi s ním vpřed

Nejčastěji vyhledáváné

Diskuze: Machr na PHP - Základ eshopu