Rychlý dotazník pro verzi 2.0

David Grudl napsal(a):

Měl bych několik rychlých a jednoduchých dotazů na téma „co změnit ve verzi 1“. Ocenil bych vaše stručné odpovědi.

1. mají se logy výjimek v Debug komprimovat?

Volitelně. Ideální je, pokud při zprovozňování webu se to komprimovat nebude, těch chyb bývá víc a rozbalování otravuje. Při provozu se to klidně komprimovat radši může.

2. defaultní Html::$xhtml má být TRUE nebo FALSE?

Nevim, asi false. Hlavně ať je to stejné jako ve skeletonech a tak.

3. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?

Asi bych nechal get.

4. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?

Určitě.

5. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)

Radši ne, těch rozbitých webů…

6. jsou třeba IHttpRequest, IHttpResponse a IUser?

Nepoužil jsem, ale IUser by se možná někomu hodit mohlo.

7. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))

Těžko říct, je to čitelnější, ale víc psaní.

9. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?

Určitě. Normální člověk beztak Html echuje a toStringuje.

10. ponechat ve frameworku Nette\Web\Ftp?

Extras asi stačí.

11. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback?

Nevim.

12. zrušit abstraktní třídu AutoLoader?

Ano.

13. hodí se k něčemu FreezableObject?

Já z toho dědim v Ormionu recordy a collection.

14. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?

Proč ne.

15. je k něčemu ArgumentOutOfRangeException?

Tak ve zdrojáku jsou to dva řádky, zrušením týhle výjimky FW moc neodtučníš.

16. podporovat FireLogger? (či nahradit jím FirePHP?)

Asi jo. FirePHP vlastně ani nevim, jestli mi funguje.

1. určitě by to minimálně chtělo možnost komprimaci vypnout.
2. TRUE. Je to restriktivnější a s nastaveným TRUE je vygenerovaný kód validní i v HTML. Naopak to neplatí.
3. Je mi jedno.
4. Jo.
5. V ideálním finálním stavu by se cols mělo stejně nastavovat v šabloně, takže bych ho klidně přesunul až jako poslední parametr. Ale maxLength je spíš věcí validátoru, takže sem podle mě také tolik nepatří. Takže je mi to jedno.
6. IUser jo, ostatní já nepotřebuji, ale tím neříkám že by se měly vyhodit.
7. Asi jo, teď stejně vždycky píšu komentář.
8. Asi jo.
9. Do extras s ním.
10. Spíš jo, ale je mi to celkem jedno, stejně nepoužívám.
11. Jo.
12. Mě ne, ale umím si představit využití.
13. Já bych nechal
14. Určitě.
15. Já nepoužívám na vývoj Firefox, takže je mi to jedno.

1. Defaultně zapnuto, možnost vypnout.
2. Rozhodně TRUE.
3. Nevím? Asi je mi to jedno.
4. Ano, to vypadá rozumně.
5. Spíš ano…
6. Nepoužívám. Snad se dá možná využít IUser, ale ani to nepoužívám…
7. Možná, těžko říct. Se současnou implementací problém nemám. (EDIT: asi ano, je to čitelnější)
8. Ano.
9. Ve FW netřeba.
10. Nevím, nepoužívám.
11. Asi ano.
12. Nepoužívám.
13. Možná je to zbytečné. Ale na druhou stranu, proč ne…takže bych je asi ponechal.
14. Ano, to je poměrně zbytečné. Ale je to podobný problém jako 13. Zásadní problém v tom nevidím.
15. Nevím…tyto feature moc nepoužívám.

1. uvital bych prepinac
2. TRUE
3. Ano
4. Ano
5. Ano
6. Nepoužívám, ale IUSer se může hodit
7. proč ne, víc psaní mi nevadí, ruku na srdce, kolikrát se to v aplikaci napíše? :)
8. Ano
9. Já jsem pro ponechání, osobně používám v několika aplikacích
10. nevím, zatím jsem nepoužil
11. klidně
12. zatím jsem nepoužil
13. klidně bych nechal

15. mě zatím stačí FirePHP, ale změně se nebráním

1. mají se logy výjimek v Debug komprimovat?
   • Ne
2. defaultní Html::$xhtml má být TRUE nebo FALSE?
   • True (možná mít toto nastavení v configu?)
3. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?
   • Ale jo …
4. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?
   • Ano
5. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)
   • Ne
6. jsou třeba IHttpRequest, IHttpResponse a IUser?
   • N/A
7. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))
   • Hmmmm … delší, ale přehlednější …
8. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?
   • Ano
9. ponechat ve frameworku Nette\Web\Ftp?
   • Klidně v extras
10. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback?
    • N/A
11. zrušit abstraktní třídu AutoLoader?
    • N/A
12. hodí se k něčemu FreezableObject?
    • N/A
13. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?
    • Stačí jedna výjimka
14. je k něčemu ArgumentOutOfRangeException?
    • N/A
15. podporovat FireLogger? (či nahradit jím FirePHP?)
    • N/A

Editoval Foowie (7. 6. 2010 11:34)

1. mají se logy výjimek v Debug komprimovat?

ne

2. defaultní Html::$xhtml má být TRUE nebo FALSE?

false

3. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?

proč ne?

4. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?

proč ne?

5. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)

nemám názor

6. jsou třeba IHttpRequest, IHttpResponse a IUser?

ne

7. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))

proč ne?

9. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?

určitě

10. ponechat ve frameworku Nette\Web\Ftp?

to je IMHO extras

11. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback?

nemám názor

12. zrušit abstraktní třídu AutoLoader?

nemám názor

13. hodí se k něčemu FreezableObject?

chvíli jsem ho používal, ale pak jsem přestal

14. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?

nemám názor

15. je k něčemu ArgumentOutOfRangeException?

je

16. podporovat FireLogger? (či nahradit jím FirePHP?)

nemám názor

1. ne nebo volitelně
2. uvítal bych FALSE, ale je mi to celkem jedno
3. ano
4. I když se budou jmenovat stejně, tak se budou chovat odlišně, to může být matoucí. Ale i tak bych to přejmenoval.
5. ano
6. IHttpRequest, IHttpResponse – ano (důvod) a IUser – nevím, nepoužil jsem, i když jsem o tom uvažoval. Spíš bych zvážil přejmenování třídy User na UserManager.
7. ano
8. ano
9. ne, přesunout do extras
10. asi ano, nemám praktickou zkušenost
11. implementační věc, je mi to fuk
12. mně ne
13. není to sice potřeba, ale zachoval bych je
14. nepoužil jsem
15. ano, zkusil bych nahradit FirePHP, se kterým byly často problémy

1. mají se logy výjimek v Debug komprimovat?

Dal bych to jako volitelné nastavení.

2. defaultní Html::$xhtml má být TRUE nebo FALSE?

Určitě FALSE, vše teď směřuje k HTML 5.

3. přejmenovat HttpUploadedFile::getImage() na vhodnější toImage() ?

Nechat getImage(), toImage() spíš říká, že se s tím objektem něco uvnitř stane, což není pravda.

4. sjednotit názvy obdobných metod formulářů setEmptyValue() a skipFirst() (asi na setEmptyValue)?

Řekl bych, že to jsou dvě různé věci, takže nesjednocovat. Spíš bych nějak upravil chování skipFirst() ve spojení s addRule(Form::FILLED), protože se to chová magicky (jedno je potřeba zavolat před druhém, aby to fungovalo) a lidi s tím mají problémy.

5. dát u addText() a addPassword() jako třetí parametr $maxLength (nyní je čtvrtý)

Klidně.

6. jsou třeba IHttpRequest, IHttpResponse a IUser?

Čím víc věci ve frameworku mohu nahradit vlastní implementací, tím lépe. Nechat.

7. nahradit bool parametry metod za bitové příznaky (např. $user->logout(TRUE) na $user->logout(User::CLEAR_IDENTITY))

To dává smysl ve chvíli, kdy by ta metoda mohla přijímat víc takových příznaků. Pokud je jen jeden, jsem jen pro boolean.

8. změnit Html::render() tak, aby obsah vykreslil element namísto vrátil?

Sjednotil bych chování napříč frameworkem, takže ano, vykreslit.

9. ponechat ve frameworku Nette\Web\Ftp?

Za dobu, co dělám v Nette, jsem to nepotřeboval, ale předtím jsem se z PHP několikrát na FTPčko potřeboval připojovat, takže bych to tam ponechal.

10. změnit čtvrtý parametr Permission::allow() z IPermissionAssertion na callback?

Určitě, bude to dávat větší smysl.

11. zrušit abstraktní třídu AutoLoader?

Nedokážu posoudit, jen vidím, že ji využívá jak NetteLoader, tak RobotLoader, tak bych jí asi ponechal.

12. hodí se k něčemu FreezableObject?

Nevím.

13. je třeba pro chybějící soubor/adresář disponovat hned třema výjimkama IOException & FileNotFoundException & DirectoryNotFoundException?

Nechal bych tři výjimky, mají aspoň výmluvné názvy.

14. je k něčemu ArgumentOutOfRangeException?

Může se hodit, pokud je parametr sice správného typu, ale je to např. vyšší nebo nižší číslo, než je dovoleno. Ale zatím jsem ji nikde nevyhodil :)

15. podporovat FireLogger? (či nahradit jím FirePHP?)

Určitě bych neodstraňoval podporu pro FirePHP.

1. nie
2. true
3. áno
4. áno
5. áno
7. áno
9. extras
13. áno

Editoval PaBi3 (7. 6. 2010 16:52)

Je to v době Nette Debug Baru potřeba?

Je. V tuto chvíli jediný způsob, jak ladit AJAXové požadavky nebo akce které po dokončení přesměrují na jinou stránku.

Ty Google Docs budeme používat častěji, je to opravdu fajn věc! :-)

Nebo bychom mohli ladící informace ukládat do SESSION a naučit DebugBar je číst (přes nějaký předpřipravený Presenter, který bude odpovídat pouze ve vývojovém prostředí). To by se mi líbilo nejvíce.

Zbytečné je znovu vynalézat kolo!

Hlavní problém FirePHP / FireLoggeru je v tom, že chyby vzniklé po odeslání hlaviček nevidí.