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

Diskuze – PHP generátor obrázků na základě klíče

Zpět

Upozorňujeme, že diskuze pod našimi online kurzy jsou nemoderované a primárně slouží k získávání zpětné vazby pro budoucí vylepšení kurzů. Pro studenty našich rekvalifikačních kurzů nabízíme možnost přímého kontaktu s lektory a studijním referentem pro osobní konzultace a podporu v rámci jejich studia. Toto je exkluzivní služba, která zajišťuje kvalitní a cílenou pomoc v případě jakýchkoli dotazů nebo projektů.

Komentáře
Avatar
Michal Žůrek - misaz:21.8.2014 11:55

odhadoval jsem že z klíče vezmeš otisk a něco podle toho vygeneruješ. Pak jsem otevřel kód a první co si říkám, WoW krátký kód, to je dobrý. Pak si řeknu FUJ neobjekotvý, nekomentovaný a nepřehledný kód.

Po povzdechu začnu číst kód a zjišťuji že otisk byl dobrý tip. Dívám se dál, čaruješ s hashem a zde postrádám ty komentáře. Nakonec si už snad jen všimnu čecho-angličtiny, máš tam proměnné $color1, $scalea (nemělo by být a velké?) a zároveň $pocetKruhu, $sirka (paradoxně definoval si WIDTH) a podobné. Nicméně výsledek je relativně zajímavý.

 
Odpovědět
21.8.2014 11:55
Avatar
Odpovídá na Michal Žůrek - misaz
Neaktivní uživatel:21.8.2014 15:19

no,jak jsem psal,tak jsem to dělal hlavně z důvodu úspory prostředků, takže jsem nějak s OOP nepočítal protože je několikanásobně pomalejší než procedurální kód.

ohledně čecho-angličtiny máš tak nějak pravdu,ale používám to,co vím že si vzpomenu co znamená, i když chápu že by se to tak nemělo.

ale když narážíš na tu proměnnou sirka a definici width,tak to má své opodstatnění -> sirka určuje parametr velikosti generovaých kruhů,zatímco definice WIDTH a HEIGHT obsahují velikost výsledného obrázku

a ohledně komentářů, mám pocit že pokud to chci vydat jen jako program,tak není žádná povinnost komentáře uvádět

Odpovědět
21.8.2014 15:19
Neaktivní uživatelský účet
Avatar
Odpovídá na Neaktivní uživatel
Michal Žůrek - misaz:21.8.2014 17:55

to nevadí, že pokaždé je to jiná šířka. Je lepší použít třeba $shapeWidth. V okamžiku kdy jakýkoliv program uvedeš i se zdrojovým kódem, tak ho nejspíše někdo bude číst, respektive snažit se o to. Špatně se čte kód, kde je něco co není na první pohled jasně pochopitelné co dělá. Šetření výkonu na úkor OOP mi přijde jako blbost. Jak často generuješ avatary? Ne tak často jako třeba něco vkládáš do databáze a to trvá mnohem déle. Pokud bychom však vzali takové mikrooptimalizace do posledního detailu, tak celý ten kód vyhoď z té funkce avatár, protože i volání funkcí je nějaký čas navíc.

 
Odpovědět
21.8.2014 17:55
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 3 zpráv z 3.