De PHPhulp kroeg
service container truc?
Pagina: « vorige 1 2 3 volgende »
Roept ook wel weer een paar vragen op. Dat try/catch blok, kun je dat ook overkoepelend maken? Of moet ik bij iedere input validatie dan een try/catch blok gebruiken? Kan ik bijv. op 1 plek al die verschillende catch-es instellen?
In jouw voorbeeld maak jij van $getal een positief getal, maar geef je dan ook nog een foutmelding naar de gebruiker, en zo ja... hoe? Echo je dan hardcoded een melding vanuit het catch-gedeelte?
Code (php)
1
throw new NaNException(sprintf('Het argument "%s" is geen getal.', $getal));
Stel ik heb 50 classes waarin ik input van de gebruiker wil valideren. Moet ik dan ook 50 keer een try/catch blok maken? Is dat de bedoeling? Of maak ik 1 keer een try/catch blok en zorg ik dat al die 50 classes hierbinnen vallen?
Ik snap wat je bedoelt met de foutmelding naar de gebruiker, maar op welke plek echo je die foutmelding? Doe je dat vanuit het catch-blok? Ik kan me voorstellen dat een foutmelding een bepaalde css meekrijgt. Krijg je dan zoiets:
Code (php)
1
2
3
4
5
6
7
2
3
4
5
6
7
<?php
try {
} catch (Exception $e) {
echo '<span class="exception">' . $e->getMessage() . '</span>';
}
?>
try {
} catch (Exception $e) {
echo '<span class="exception">' . $e->getMessage() . '</span>';
}
?>
Is dat de bedoeling?
Gewijzigd op 02/10/2013 10:43:21 door Ozzie PHP
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
<?php
try {
/********************************************
* In deze try gaan we *
* - input verwerken, *
* - daarmee een berekening uitvoeren en *
* - het resultaat opslaan in een database. *
********************************************/
} catch (InputException $e) {
} catch (MathException $e) {
} catch (DatabaseException $e) {
} catch (Exception $e) {
}
?>
try {
/********************************************
* In deze try gaan we *
* - input verwerken, *
* - daarmee een berekening uitvoeren en *
* - het resultaat opslaan in een database. *
********************************************/
} catch (InputException $e) {
} catch (MathException $e) {
} catch (DatabaseException $e) {
} catch (Exception $e) {
}
?>
Maar vroeg of laat zul je zien dat deze code te veel doet. Ik zou er vooral geen hele applicatie in stoppen met 50 klassen. Zoiets als de DatabaseException in dit voorbeeld kun je prima elders onderbrengen.
Foutmeldingen voor de gebruiker toon je alleen als de gebruiker het probleem kan verhelpen. Dat is een van de redenen om daarvoor een aparte Exception-klasse toe te voegen.
Gelukkig heeft php daar ook al iets voor, de set_exception_handler die alle excepties opvangt die niet opgevangen worden in een catch blok. Op die manier kan je gewoon overal excepties gooien, en kun je in de handler er iets leuks mee doen.
Code (php)
1
2
3
4
5
2
3
4
5
set_exception_handler(function($e) {
echo '<span class="exception">' . $e->getMessage() . '</span>';
});
throw new RuntimeException('boom');
echo '<span class="exception">' . $e->getMessage() . '</span>';
});
throw new RuntimeException('boom');
@NOLot: ik zou niet voor alle exceptions een foutmelding aan de gebruiker willen geven (denk ik nu), maar bijv. bij een verkeerde user input kan ik me voorstellen dat je aangeeft dat de gebruiker een getal moet intypen ipv een string. Dan zou je dus per catch-blok moeten aangeven of er iets moet worden ge-echoot. Ik vind het alleen raar/onnatuurlijk om iets te echoën in een catch-blok. Ik echo liever iets in een view, maar dat kan op deze manier dus niet?
Code (php)
Zelf gebruik ik excepties nooit op die manier, maar meer om programmeer problemen tegen te gaan
Vergeet niet dat php niet echt een exception friendly taal is. In andere talen kun je (op syntax errors na) ALLE errors afvangen met een try catch, in php niet.
Hoe oud ben je eigenlijk ozzie?
Voorheen werkte ik gewoon met if-jes en elsejes. Maar nu wil ik dus eens met exceptions gaan werken. Ik snap dat het voordelen heeft, en nu wil ik weten hoe ik het op de juiste manier moet implementeren.
Even weer het voorbeeldje van Ward (iets aangepast)... stel ik zet dit in de core/kernel class:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
2
3
4
5
6
7
8
9
10
11
12
13
<?php
try {
// Hier gaan we de complete request afhandelen.
} catch (InputException $e) {
} catch (MathException $e) {
} catch (DatabaseException $e) {
} catch (Exception $e) {
}
?>
try {
// Hier gaan we de complete request afhandelen.
} catch (InputException $e) {
} catch (MathException $e) {
} catch (DatabaseException $e) {
} catch (Exception $e) {
}
?>
Nu hoef ik maar 1x een try en catch in te stellen. Stel nu dat ik in class Foo een InputException wil gooien, dan komt ie in de try/catch van de core/kernel terecht. Is dat een optie? Of gaat de code dan ook niet meer verder in die Foo class?
Offtopic:
"Hoe oud ben je eigenlijk ozzie?"
Privé vragen graag via PM :)))
Ozzie PHP op 02/10/2013 14:52:21:
Ik snap wat je bedoelt, maar ik probeer vooral te begrijpen hoe en wanneer je exceptions moet afvangen.
Voorheen werkte ik gewoon met if-jes en elsejes. Maar nu wil ik dus eens met exceptions gaan werken. Ik snap dat het voordelen heeft, en nu wil ik weten hoe ik het op de juiste manier moet implementeren.
Even weer het voorbeeldje van Ward (iets aangepast)... stel ik zet dit in de core/kernel class:
Nu hoef ik maar 1x een try en catch in te stellen. Stel nu dat ik in class Foo een InputException wil gooien, dan komt ie in de try/catch van de core/kernel terecht. Is dat een optie? Of gaat de code dan ook niet meer verder in die Foo class?
Voorheen werkte ik gewoon met if-jes en elsejes. Maar nu wil ik dus eens met exceptions gaan werken. Ik snap dat het voordelen heeft, en nu wil ik weten hoe ik het op de juiste manier moet implementeren.
Even weer het voorbeeldje van Ward (iets aangepast)... stel ik zet dit in de core/kernel class:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
2
3
4
5
6
7
8
9
10
11
12
13
<?php
try {
// Hier gaan we de complete request afhandelen.
} catch (InputException $e) {
} catch (MathException $e) {
} catch (DatabaseException $e) {
} catch (Exception $e) {
}
?>
try {
// Hier gaan we de complete request afhandelen.
} catch (InputException $e) {
} catch (MathException $e) {
} catch (DatabaseException $e) {
} catch (Exception $e) {
}
?>
Nu hoef ik maar 1x een try en catch in te stellen. Stel nu dat ik in class Foo een InputException wil gooien, dan komt ie in de try/catch van de core/kernel terecht. Is dat een optie? Of gaat de code dan ook niet meer verder in die Foo class?
Als jij in je Foo class een exceptie gooit, en hem niet daarbinnen opvangt, crasht je code net zolang totdat je bij de eerstvolgende exception vanger komt. Als dat dan toevallig in de kernel class is, zal hij daar in komen.
Dat is zeker een optie, kijk maar eens naar een HttpKernel implementatie van symfony: https://github.com/symfony/HttpKernel/blob/master/HttpKernel.php#L63
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
Je kunt een deel van de verantwoordelijkheid echter ook delegeren naar de klasse die de exception genereert. Dan heb je dezelfde core/kernel als hierboven, maar wordt de klasse:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
Het kan dus beide, want je kunt exceptions doorgeven.
Stel, iemand voert een int in ipv een string. Ik wil dan een melding tonen, maar de rest van de code wil ik wel nog laten uitvoeren. Dus de errormelding moet in beeld komen, maar daaronder moet het formulier weer tevoorschijn komen. Kan dat gewoon? Of stopt de code zodra ie in een catch-blok komt?
In de catch van een Input_Exception, zoals in het voorbeeld, kun je dus een foutmelding opstellen via $e->getMessage() en die daarna ergens anders in een view stoppen.
Jouw voorbeeld:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
Hoe kan ik hier dan in het catch-gedeelte de foutmelding opvangen en weer teruggeven aan de Foo class?
Code (php)
Als er in de constructor van Foo dus een exception wordt gegooid zal deze klasse niet worden aangemaakt. Stel ik heb geen try catch blokken, dan zal mijn hele applicatie stoppen. Maar dat wil ik niet, ik wil dat hij dan even dit overslaat maar het contact formulier weer geeft. Daar plaats ik een try catch blok om het aanmaken van deze klasse, de rest heeft er dan geen last van.
Zodra er iets fout gaat in een klasse kun je natuurlijk niet meer terug naar de klasse, er ging immers iets fout...
@Wouter: kijk, da's een duidelijke uitleg! Maar dat zou dus betekenen, dat als ik een InputException in de kernel opvang, dat de overige code dus ook niet meer wordt uitgevoerd. En als er dan onder het try/catch blok in de kernel geen code meer staat, dan houdt het script dus gewoon op. Correct? Dus wat ik dacht te kunnen doen, kan dus niet.
Ward van der Put op 02/10/2013 16:32:23:
In de catch van een Input_Exception, zoals in het voorbeeld, kun je dus een foutmelding opstellen via $e->getMessage() en die daarna ergens anders in een view stoppen.
Ward gaat er hier dan vanuit dat de view pas "onder" het try/catch blok in de kernel wordt aangeroepen zodat de foutmelding uit de catch aan de view kan worden doorgegeven?
Pagina: « vorige 1 2 3 volgende »