spájame
slovenskú
IT komunitu
pridaj sa
Registrácia · Login
Jiri Knesl

Jiri Knesl


Overený používateľ, členom od 10.8.2011

(zatiaľ žiadne)

Jiri Knesl nenapísal žiadne články/novinky

Jiri Knesl napísal 1 komentár

0 0 Jiri Knesl 2011-08-10 09:14:53
Tento kód je úplně špatně.
1) není důvod extendovat třídu framework
2) podstrčím-li do URL název nějaké třídy, která není controller, dostanu 500
3) metody jsou příliš dlouhé
4) není zde žádná štábní kultura, občas je název třídy velký, občas malý
5) volani die() způsobí, že se na kód nedají napsat testy, správně by se měla vyhodit vyjímka, controller by měl mít nějaký dispatch, který ji odchytí
6) není důvod, proč neloadovat modely a controllery přes autoloading
7) extract($view_data, EXTR_PREFIX_SAME, "wddx"); - má zkontrolovat spíš zakázané indexy a vyhodit vyjímku, takhle se člověk nedozví, proč se jeho proměnná nepřevedla (resp.prefixovala nějakým nesmyslem, jaký nikdy neuhodne)
8) to if(!defined("BASE_PATH")) exit("No direct script access allowed"); tam taky nemá co dělat, aplikace má být mimo Document Root
9) komentáře nejsou často správně použity, nejhorší je snad: // metóda ktorá vypíše dáta - ta metoda nic nevypisuje, jen vrací

Páčilo sa mi...

Zajtra.sk > Ľudia > Jiri Knesl