NOVINKA - Online rekvalifikační kurz Python programátor. Oblíbená a studenty ověřená rekvalifikace - nyní i online.
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í.
Přejít k původní otázce
Avatar
Zdeněk Pavlátka
Tvůrce
Avatar
Zdeněk Pavlátka:16.4.2015 10:12

Ach ten čas... Tady máte konečně to hodnocení:
Neaktivní uživatel :
To co jsi odevzdal špatné není, ale jaksi jsi tam zapomněl dát editor, úpravy jsou pouze v kódu a nedají se používat.

Uživatel sítě :
Editor vypadá celkem pěkně, jen u úprav chybí odkaz zpět na úvodní stránku. A u změny velikosti by se mohly předvyplnit původní rozměry. Změna kontrastu ti nefunguje, místo kontrastu to mění jas. Kód vypadá velmi dobře.

pivovary :
Když jsem to zkusil spustit vypsalo to chybu (chyběl ti tam středník):

Parse error: syntax error, unexpected 'function' (T_FUNCTION) in C:\xampp\htdocs\machr\editor_image_pivovary\config\config.php on line 3

Po opravě této chyby mě to přesměrovává na stránku /error404. Co se týče kódu, tohle úplně nechápu:

const IMAGETYPE_JPEG = IMAGETYPE_JPEG;
const IMAGETYPE_GIF = IMAGETYPE_GIF;
const IMAGETYPE_PNG = IMAGETYPE_PNG;

Jinak kód není nejhorší.

Dominik Klapuch :
Mohl jsi se alespoň trochu pokusit nastylovat ten formulář, ale to tady není podstatné. Připadá mi trochu zbytečné dělat extra třídu pro každý efekt (hlavně když má jen jednu metodu), vzniká tím obrovské množství souborů o pár řádkách.

Filip Stryk :
Kód

foreach ($_POST['options'] as $o) {
        switch($o) {
                case 'toGrayScale':
                        $img->toGrayScale();
                        break;
                case 'changeBrightness':
                        $img->changeBrightness($_POST['changeBrightnessValue']);
                        break;
                // a tak dále
        }
}

by asi bylo lepší napsat jako podmínky. Tohle by se hodilo, kdybys měl frontu efektů - uživatel by mohl ovlivnit pořadí provedení úprav, ale tady nic takového nemáš. Jinak to není špatné.

Ondrca :
Bylo by fajn přidat historii upravených obrázků, nebo použití více efektů na jeden obrázek. Takhle musím po úpravě obrázek stáhnout a znovu nahrát abych mohl dál upravovat. Kontrast ti funguje převráceně - vyšší číslo znamená nižší kontrast, což je trochu zvláštní. Kód je na tom celkem dobře.

Škoda že nikdo nevytvořil frontu s úpravami, jak psal 1Pupik1989 . Ale ta v zadání nebyla. Placku vyhrává Uživatel sítě . Výtvor tedy publikuj a o placku si napiš do PM.

A teď extra hodnocení Machr na TypeScript - Úprava obrázků

Michal Žůrek - misaz :
Krásný objektový kód, skvěle fungující editor... Co dodat. Snad jen že placku si určitě zasloužíš, do PM mi napiš jakou chceš.

Akceptované řešení
+20 Zkušeností
+2,50 Kč
Řešení problému
Odpovědět
16.4.2015 10:12
Kolik jazyků umíš, tolikrát jsi programátor.
Avatar
Uživatel sítě
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
Uživatel sítě :16.4.2015 10:41

Díky, odkaz jsem dal pouze do hlavičky(v h1 a h2), nenapadlo mě kam ho jinam dát, ale místo by se jistě našlo. Změna kontrastu nefunguje, jelikož je tam stejný filtr jako na změnu jas IMG_FILTER_BRIG­HTNESS a má být IMG_FILTER_CON­TRAST(V publikované verzi to bude opravené).

Nahoru Odpovědět
16.4.2015 10:41
Chybami se člověk učí, běžte se učit jinam!
Avatar
Dominik Klapuch
Člen
Avatar
Odpovídá na Zdeněk Pavlátka
Dominik Klapuch:16.4.2015 11:38

Ahoj, co se ti nezdá na mém počtu souborů? Každá třída vykonává přesně jen jednu věc - v tomto případě efekt. V případě, že by si někdo chtěl napsat vlastní kód na daný filtr, kód by mohl být mnohem delší. Jedna metoda se mi nezdá špatná, rozhraní by mělo být co nejtenčí. Kdybych to vše dal do jednoho souboru, mohlo by nastat to, že bych měl třídu o 2000 řádkách. Na tuto třídu by bylo pak těžké vytvořit rozhraní, které by se neměnilo. Myslím si, že ve všech projektech, ve který se splňují SOLID principy je větší množství souborů. Pokud se pletu, vyvrať mi to, prosím.

Nahoru Odpovědět
16.4.2015 11:38
Kód a data patří k sobě.
Avatar
pivovary
Člen
Avatar
Odpovídá na Zdeněk Pavlátka
pivovary:16.4.2015 11:44

config.php zazipoval asi dřív než jsem ho odzkoušel. Proč se to přesměrovává na error404 zkusím zjisit jak stáhnu xampp ale myslím že by to mohl dělat .htaccess. Jelikož při psaní kódu používám takový styl že si to kopíruju hned na produkci tam mi to jede bez problémů.

A k těm constantám je to psané dle zdejších tutoriálů od David Hartinger, dle mně je to spíše k lepší práci s něma přímo v té třídě abych je nemusel v kódu převádět na konstanty, protože jako string se s nima nedá pracovat

constant($_SESSION["editor"]["flip"])
 
Nahoru Odpovědět
16.4.2015 11:44
Avatar
Ondrca
Tvůrce
Avatar
Odpovídá na Zdeněk Pavlátka
Ondrca:16.4.2015 17:51

Gratuluji vítězům :)

Nahoru Odpovědět
16.4.2015 17:51
Zase jsem o něco chytřejší
Avatar
KleoCZ
Člen
Avatar
KleoCZ:17.4.2015 8:29

OT: jak už tu bylo mnohokráte zmíněno obrázky lze velice snadno upravovat pomocí JS sám se učím tento framework http://fabricjs.com/image-filters/ ... píši to jen pro doplnění, kdyby si někdo chtěl upravovat.

Btw: jestli někdo pracuje s tímto frameworkem nebo by měl zájem o něm pokecat, pošlete mi zprávu do pošty :)

Nahoru Odpovědět
17.4.2015 8:29
Apeluji na všechny co odpovídají na dotazy, tak pokud je to jen trochu možné dávejte rovnou reálná řešení třeba v htt...
Avatar
Michal Žůrek - misaz
Tvůrce
Avatar
Odpovídá na KleoCZ
Michal Žůrek - misaz:17.4.2015 8:41

není to o ničem pointu programu vyřešit frameworkem?

 
Nahoru Odpovědět
17.4.2015 8:41
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 7 zpráv z 57.