Kde zachytáváte výjimky?

Školení, která pořádám

Představte si aplikaci podle návrhového vzoru MVC nebo podobného, napsanou třeba v Nette, ve které chceme někde v postranním menu zobrazit třeba výběr článků. Pokud se tento výběr nepodaří načíst, neměl by zamezit zobrazení hlavního obsahu stránky. Model může vypadat třeba takhle:

<?php
class Article {
    static function findBests() {
        return dibi::fetchAll("SELECT * FROM [article] WHERE [best]");
    }
}
?>

Prezenter:

<?php
class ArticlePresenter {
    function renderDetail($id) {
        // tady bude načtení hlavního obsahu stránky - detailu článku
        // kromě toho načteme i výběr článků pro postranní menu
        $this->template->bestArticles = Article::findBests();
    }
}
?>

Šablona postranního menu:

{if $bestArticles}
	<h3><a href="{link Article:bests}">Výběr článků</a></h3>
	<ul>
	{foreach $bestArticles as $article}
		<li><a href="{link Article:detail $article->id}">{$article->title}</a></li>
	{/foreach}
	</ul>
{/if}

Zdrojový kód je k dispozici na GitHubu.

Jak je vidět, model používá Dibi, které v případě chyby vyvolá DibiException. A mě zajímá, jak s touto výjimkou naložíte. Ošetříte ji už v modelu, v prezenteru, nebo ji snad necháte probublat až na nejvyšší úroveň? A pokud ji nezachytíte už v modelu, použijete dokumentační komentář @throws? A nebude divné, že se prezenter prostřednictvím typu výjimky dozví o implementaci modelu? Nebo ji třeba v modelu přeložíte na jinou výjimku? A jak vlastně bude vypadat blok catch?

Všimněte si také toho, že v šabloně je odkaz na Article:bests – to bude stránka, kde je hlavním obsahem právě seznam nejlepších článků a kde je tedy žádoucí, aby skončila pětistovkou, pokud se seznam nepodaří načíst. A aby vypsala informaci o prázdném seznamu, pokud žádné články jako výběr ještě označené nebudou. Přijde mi logické, aby tato stránka vevnitř volala stejnou metodu modelu Article::getBests.

Ptám se proto, že jsem stejné otázky v osobním rozhovoru položil hlavnímu vývojáři významného českého frameworku a jeho odpovědi se mi zrovna dvakrát nelíbily. A pokud vám možnost výskytu chyby při čtení dat z databáze přijde nepravděpodobná, tak si představte, že se ptáte nějaké nespolehlivé služby (třeba Twitteru). Pokud byste v tom případě své řešení upravili, tak si ještě odpovězte na otázku, jakou chyba musí mít pravděpodobnost, aby stálo za to s ní nezacházet jako s fatální.

Viz řešení.

Jakub Vrána, Řešení problému, 9.8.2010, diskuse: 110 (nové: 0)

Diskuse

Sniper:

Dibi vyjimku zachytit primo v modelu a prelozit na nejakou jinou, napr. ArticleException, kterou pak zpracovat v controlleru. @throws je samozrejmosti.

Pripadne jeste z modelu do controlleru vratit prazdnej datovej objekt, to by pak zalezelo na konkretnich pozadavcich.

Kde zachytáváte výjimky?

Školení, která pořádám

Diskuse

Sniper:

Jakub Vrána :

Sniper:

Jakub Vrána :

Sniper:

Jakub Vrána :

martin:

Jakub Vrána :

martin:

Jakub Vrána :

martin:

Jakub Vrána :

martin:

Jakub Vrána :

martin:

Jakub Vrána :

martin:

Jakub Vrána :

martin:

Jakub Vrána :

martin:

Martin Hruška:

v6ak:

martin:

v6ak:

martin:

v6ak:

martin:

v6ak:

Franta:

v6ak:

Jakub Vrána :

TheTom:

Jakub Vrána :

TheTom:

Jakub Vrána :

TheTom:

Jakub Vrána :

TheTom:

Jakub Vrána :

v6ak:

bzuK:

Jakub Vrána :

bzuK:

Jakub Vrána :

Džoukr:

martin:

Džoukr:

Jakub Vrána :

Džoukr:

Jakub Vrána :

Džoukr:

martin:

cooler:

Džoukr:

Džoukr:

martin:

Džoukr:

martin:

Džoukr:

martin:

Džoukr:

Lemrouch:

cooler:

v6ak:

Michal Prynych:

Jakub Vrána :

Michal Prynych:

Džoukr:

cooler:

Jakub Vrána :

cooler:

Jakub Vrána :

Peca:

Jakub Vrána :

Peca:

Jakub: