Zobrazení chyby ve stránce

Laděnka standardně noticky neodchytává – pouze pokud používáš FireBug s FirePHP, odchytí je a zobrazí v něm. Pokud jde o logování do souboru, tipuji, že Laděnka detekovala vývojový režim a proto chyby neloguje, ale vypisuje.

já jsem totiž nějak žil v domění, že se zachytávají i notice, před časem jsem četl o laděnce na root.cz a utkvěla mě v hlavě věta o chybách notice „Vnímám je jako upozornění na nejzávažnější chyby v kódu. Je proto obrovským omylem začátečníků se domnívat, že tyto chyby mohou zahazovat“, a k pokusům s Nette jsem se dostal až teď a jaksi jsem předpokládal, že laděnka odchytává i notice.
Článek mě potěšil, neboť k chybám přistupuji obdobně.
Takže, aby mi laděnka odchytávala i notice, tak jsem v debug.php za řádek 437 připsal další položku pole
E_NOTICE=>1,

Editoval Mirek (11. 6. 2009 11:26)

Zkusil jsem to tam dát, zapněte Debug::$strictMode = TRUE. Uvidíme, jak se to osvědčí.

David Grudl napsal(a):

Zkusil jsem to tam dát, zapněte Debug::$strictMode = TRUE. Uvidíme, jak se to osvědčí.

Pokud je na stránce chyb víc, tak se laděnka nakreslí několikrát přes sebe (vidět je pouze poslední chyba). Zvážil bych, zda při striktním módu nevolat při zachycení první chyby rovnou exit(). Chce ale pořádně zvážil všechny následky, protože mě se nad tím teď přemýšlet nechce :)

nojo, ten exit jsem tam zapomněl…

David Grudl napsal(a):

Zkusil jsem to tam dát, zapněte Debug::$strictMode = TRUE. Uvidíme, jak se to osvědčí.

Tak můžu říct, že mě se s tímhle nastavení pracuje skvěle. Přeci jen textík „Undefined variable…“ někde v layoutu se dá snadno přehlédnout. Takhle je to fakt paráda…

Co kdyby si Debug všechny menší chyby ukládal někam bokem a pokud dojde k nějaké přísnější byly by zobrazeny na bluescreenu taky? občas se stává že výjimku nebo chybu způsobí nějaká menší ale laděnka ji zakryje (firebug tohle řeší ale…). Zkusil jsem to implementovat náhled zde, jde o blok Previous errors.

Nespoléhej na warning, ta funkce musí mít i nějakou návratovou hodnotu… takže otestuj tu a podle toho vyhazuj výjimku. Podle PHP dokumentace by mělo fungovat něco takového:

if(@ociexecute($s, OCI_DEFAULT) === FALSE)
	    throw new Exception("Something failed...");

Editoval gmvasek (27. 10. 2010 10:17)

No hlavně v Nette 2.0 se už strictMode nedá použít v produkčním módu (což je dobře), takže bych ti radil spolehnout se na test návratové hodnoty ;)

K čemu chceš poslední výjimku? Nějak nechápu… vždyť tam akorát stačí vyhodit vlastní výjimku kterou si zpracuješ ErrorPresenterem… nic víc nepotřebuješ ;)

Na to by mohlo pomoct tryError a catchError.

gmvasek napsal(a):

No hlavně v Nette 2.0 se už strictMode nedá použít v produkčním módu (což je dobře), takže bych ti radil spolehnout se na test návratové hodnoty ;)

Je k tomu nějaký důvod?

Mimojiné vzhledem k tomu, že strictMode automaticky ukončí vykonávání scriptu (vyjímka je ihned odchycena), což není přesně to chování, které bych očekával a hlavně chtěl, uvažoval jsem, že za inicializaci debug si přidám vlastní převádění na vyjímky (viz. níže). Byla na to již vedená diskuze, aby strictMode vyjímky vyhazoval a neodchytával je rovnou, ale David se k tomu tuším nevyjádřil, takže pokud tohle v Nette 2 nechce zavést, docela by mě zajímal důvod.

něco ve stylu:

function exception_error_handler($errno, $errstr, $errfile, $errline ) {
    throw new ErrorException($errstr, 0, $errno, $errfile, $errline);
}
set_error_handler("exception_error_handler");

Zkoušel to už někdo nebo má s tím někdo nějaké zkušenosti.

Mé hlavní důvody pro toto chování:
Přístup k neinicializovanému prvku v poli může způsobit v aplikaci obrovské problémy, zvlášť pokud se ukládá do databáze. Proto je nutné chyby převádět na vyjímky. Pokud je ale vyjímka ihned odchycena Debugem, nemohu ji odchytit já a naložit s chybou podle svého.

Dík

S tím se nedá nesouhlasit. Ale ruku na srdce, „bezchybná“ aplikace „neexistuje“. Zvlášť pokud jde o velkou aplikaci na které se podílí více lidí.
Osobně si myslím, že strictMode by měl chyby transformovat na vyjímky a programátor by se měl rozhodnout, zda ji odchytí nebo nechá aplikaci padnout do 500 internal server error.

Proč se bráníš přepsání:

} elseif (self::$strictMode && !self::$productionMode) {
    self::_exceptionHandler(new \FatalErrorException($message, 0, $severity, $file, $line, $context));
    exit;
}

na

} elseif (self::$strictMode) {
    throw new \FatalErrorException($message, 0, $severity, $file, $line, $context);
}

?

Mám to odzkoušené a funguje to.

Určitě by nejedno srdce programátora zaplesalo. V PHP by se pracovalo jako v jiných vyspělých jazycích. Dokonce už taková myšlenka padla, aby se chyby převáděly na vyjímky. Takto by se toho docílilo a mohl by to používat jen ten, kdo by chtěl.

To že na serveru by měla být aplikace bez chyb není argument. A krok, kdy se strictMode zakáže na produkčním serveru je spíš házení klacků pod nohy a velké bezpečnostní riziko, že se aplikace zhroutí, znehodnotí se datbáze, atp. Spíš to beru jako krok zpět než krok dopředu.

Krok dopředu by byla výše uvedená změna. Kdo nechce, tak si to nezapne. Kdo si to zapne, nic se pro něj nezmění. A pokud opravdu někdo chce aby zůstalo stávající chování, PROSÍM, ať je do Debug přidána jedna statická proměnná, která mi dovolí zapnout vyhazování vyjímek.

Pokud existuje pádný argument, že výše uvedená změna je špatná, samozřejmě ji akceptuji, ale pokud je to jen názor, že je toto řešení správné bez pádného argumentu, je to pak škoda, že na toto rozhodnutí doplatí ostatní.

Ješte aditační poznámka: Tím že se vypnul strictMode v produkčním režimu, tím vlastně pozbil smyslu. Protože když aplikaci ladím ve vývojovém režimu, tak strictMode nepotřebuji, protože chyby vidím v debug baru.

Editoval bene (28. 10. 2010 15:45)

StrictMode znamená „náhlá smrt“ a je to alternativní způsob ladění chyb, kdy se zobrazení v Debug baru nahradí za zobrazení červené obrazovky. Asi by bylo vhodné $strictMode přejmenovat na $immediateDeath, ale nevím, to druhé se špatně píše :-)

Každopádně tohle nemá s převáděním na výjimky skutečně nic společného. Ten „převod“ na FatalErrorException je tam čistě z interních důvodů, protože to zjednodušuje implementaci Debug::_exceptionHandler.

Převádění chyb na výjimky v Nette implementováno není a asi nebude, je to moc velký zásah do PHP.