Názory ke článku Generujeme náhledy s PHP objektově – Programujte.com
 x   TIP: Přetáhni ikonu na hlavní panel pro připnutí webu
Reklama

Názory ke článku Generujeme náhledy s PHP objektově – Programujte.comNázory ke článku Generujeme náhledy s PHP objektově – Programujte.com

 

Názory ke článku Generujeme náhledy s PHP objektově

RR   NOVÝ
17. 1. 2010

Nepoužívejte private, kde stačí protected. Nenechte uživatele čekat na zpracování thumbu, ale předgenerujte mu data. Určitě ho nenechte parametricky určovat velikost thumbu. Je lepší používat generické výjimky, místo Thumb* výjimek.

Vašek   NOVÝ
18. 1. 2010

Bohužel ten výstup není validní. Width resp. height je nutné řešit v thumb.

Møstro   NOVÝ
18. 1. 2010

Autorovi se asi s odpovídáním do diskuze nenamáhá, proč taky, už se zde rozpitvávají detaily, článek je dobrý.

RR píše:

Nepoužívejte private, kde stačí protected. Nenechte uživatele čekat na zpracování thumbu, ale předgenerujte mu data. Určitě ho nenechte parametricky určovat velikost thumbu. Je lepší používat generické výjimky, místo Thumb* výjimek.



Ano, předgenerování dat by bylo dobré, ale i tak je článek dost obsáhlý. Jde o výuku programování, než o geniální hotové skripty. To samé s velikostí náhledu - skript pro načítání je nejspíše záměrně silně zjednodušen. Generické výjimky má PHP na tak mizerné úrovni, že není na škodu používat vlastní.

Vašek píše:

Bohužel ten výstup není validní. Width resp. height je nutné řešit v thumb.



Co se ti nezdá, Venco, validní?


RR   NOVÝ
10. 3. 2010

Reagoval na komentář od uživatele Møstro :

Generické výjimky má PHP na tak mizerné úrovni, že není na škodu používat vlastní.



Generické výjimky nemusí být obsažené v php. Znamená to druhové výjimky. Všimněte si, že pro tuto konkrétní třídu na thumbnaily vytvořil tři jednoúčelové výjimky, které navíc říkají jen to, že data pro thumb nejsou soubor, obrázek nebo mají neznámý formát. Proč nepoužít místo ThumbExceptionNotFile třeba IOException nebo něco podobného, co můžeme využít znova úplně jinde?

V čem s vámi zásadně nesouhlasím je, že rozpitvávám detaily. Na zjištění pár vhodných dunkcí stačí manuál. Programování není ale o lepení funkcí k sobě, nýbrž o hledání funkčních elegantních řešení. Zbytečný private je chyba, předgenerování thumbů je v praxi důležité (zkuste si generovat 150 thumbů na stranu). Kdybych pitval detaily, tak píšu třeba o $this->thumb = DIR_THUMBS.md5($img.$width.$height);

Møstro   NOVÝ
13. 3. 2010

Reagoval na komentář od uživatele RR :

Zapomínáte na jednu věc - proč tu ten článek vlastně je. Není tu jako chlouba autora, že umí psát dokonalé skripty (protože neumí, předpokládám :) ), ale jako sdělení pro ty, co se chtějí něčemu přiučit. A zároveň nežijí jen z tohoto článku, jinde si přečtou něco jiného, dají si dvě a dvě dohromady a vyberou si, co je lepší.

RR   NOVÝ
16. 3. 2010

Reagoval na komentář od uživatele Møstro :
A snad si ti, co se chtějí něčemu přiučit, přečtou i komentáře. Od toho tu jsou zase ty, aby doplnily nebo opravily autora. Napsat funkční kód je jen jedna strana mince, je potřeba přemýšlet i o jeho srozumitelnosti, struktuře, znovupoužitelnosti, integraci, datech a jejich formátech atd.

Vladimír Bělohradský   NOVÝ
16. 6. 2011

Dobrý den, breaky ve switch jsou zbytečné, pakliže tam voláte return. Na pár místech by se dal použít i stažený tvar. elseif se píše v PHP dohromady, urychluje to vyhodnocení podmínek. Nechci působit jako hnidopich, ale myslím že stojí za to se zamyslet nad čistotou kodu a podívat se na to očima parseru.

Poslední vítka jde k logice objektu. Incializovat objekt s tím že obsahuje sice nulové hodnoty, ale přesto je obsahuje, není zcela šťastný přístup. Zvolil bych spíše jako počáteční hodnoty NULL. Jednak nezabírají narozdíl od nul a prázdných řetězců v paměti "tolik" místa a druhak mi to přijde smysluplnější.

yetty, autor článku   NOVÝ
9. 7. 2011     web

#8 Vladimír Bělohradský
Dobrý den,

  • s breaky ve switch máte samozřejmě pravdu
  • stejně tak s elseif, patrně pouze překlep, omlouvám se, ale bohužel se občas stává.
  • s nulovými hodnotami tak docela nesouhlasím - samozřejmě by to bylo možné inicializovat s NULL, ale protože PHP není typové, je o něco těžší pak udržet úroveň kódu. Z prázdných uvozovek okamžitě poznám, že se jedná o string, z nuly že jde o číslo. S pamětí máte pravdu, ale myslím, že není potřeba tolik honit bajty ;)
czLukasss   NOVÝ
7. 10. 2012

Velmi pěkný článek, který mi pomohl s generováním náhledů. Děkuji autorovi!

Přidej svůj názor

×Vložení zdrojáku

×Vložení obrázku

Vložit URL obrázku Vybrat obrázek na disku
Vlož URL adresu obrázku:
Klikni a vyber obrázek z počítače:
 
Podporujeme Gravatara.
Zadej URL adresu Avatara (40 x 40 px) nebo e-mailovou adresu pro použití Gravatara.
Email nikam neukládáme, po získání Gravatara je zahozen.
-
Reaguješ na příspěvek:
Pravidla pro psaní příspěvků, používej diakritiku. ENTER pro nový odstavec, SHIFT + ENTER pro nový řádek.
Sledovat nové názory e-mailem (pouze pro přihlášené)
Sleduj názory ke článku a v případě přidání nového příspěvku o tom budeš vědět mezi prvními.



Hostujeme u Českého hostingu       ISSN 1801-1586       ⇡ Nahoru Webtea.cz logo © 20032016 Programujte.com
Zasadilo a pěstuje Webtea.cz, šéfredaktor Lukáš Churý