Ward, dankjewel, maar die stack trace die krijg jij toch alleen maar te zien omdat jij de Exception niet hebt opgevangen? Als je de Exception correct opvangt krijg je die stack trace niet te zie.
Maar je moet het meer zien in de lijn met het voorbeeld van Wouter. Bijv. een InvalidInputException. Als je die Exception gooit, wil je eigenlijk maar 2 parameters doorgeven: throw new InvalidInputException('foo', 'integer');
De message moet dan uiteindelijk zoets worden:
Invalid input was given: 'foo' is not an integer.
Exception occured in file ... on line ...
En het opstellen van die message, daar wil ik dus een setMessage method voor gebruiken. Om af te dwingen dat ik die method niet vergeet, wilde ik 'm dus opnemen als abstract private method.
Ja, maar dan is het heeeeeel erg fout om dat met interfaces of abstracte methods te doen. Die zijn voor OO contracten, dingen die niet met programmeren te maken hebben. Dat jij je zelf wilt afdwingen iets in een method te doen, wat trouwens volkomen zinloos is, moet je jezelf gewoon net zo hard afdwingen als dat je classnamen met een hoofdletter begint. Dat moet je niet in de code afdwingen, maar gewoon in je hoofd.
Zo is het in veel gevallen ook volkomen doelloos om het mooi maken van een message af te dwingen. Bijv. in het geval die jij hier gaf. De exception is "The file ' . $file . ' could not be written." dat het een FileSystemException is weet je al dat het een exception van de FileSystem is. Dat je er dan in de logs "The FileSystem experienced an exception:" erbij wilt hebben staan moet je in het catch gedeelte (het handle gedeelte) oplossen.
Je zit een exception nog steeds verkeerd naar mijn mening en ik vind het je heel moeilijk om je wel op het goede spoor te krijgen. Laten we nog maar een vergelijking starten, kijken of je het dan wel door krijgt:
Een Exception is vertaald "uitzondering". Er gebeurd iets wat een uitzondering is op de verwachte gebeurtenissen. In programmeren is iets wat anders is dan dat je verwachte fout.
Deze fout wil je doorgeven aan de rest van je applicatie. Dat doe je dus door alleen die uitzondering naar iedereen door te gooien, niet door allerlei andere informatie erbij te stoppen. Pas degene die de uitzondering van jouw aanhoort (catched) gaat beslissen wat hij er mee gaat doen.
Stel ik heb mijn been gebroken en ik lig op straat. Dan zit ik niet te schreeuwen "Wouter heeft zijn been gesproken op de gemeenlandslaan in Huizen voor huisnummer 58." Nee, het enige wat je mij hoort schreeuwen is "Ik heb mijn been gebroken!". Dat is de Exception.
Vervolgens komt er een voorbijganger aangehold die mijn geschreeuw heeft gehoord en mij wil helpen (de catcher). Deze weet dat ik Wouter heet (wat iedereen heeft natuurlijk een naamkaartje, in de OO wereld in elk geval wel). Vervolgens gaat deze voorbijganger 112 bellen voor een ambulance, daarin geeft hij door "Ik ben op de gemeenlandslaan 58 en iemand heeft hier zijn been gebroken." Vervolgens belt hij ook mijn ouders en zegt: "Wouter heeft zijn been gebroken." Zie je het grappige? Deze voorbijganger (de catcher) geeft deze extra informatie wel door, aangezien degene aan wie hij het doorgeeft deze informatie nodig heeft.
Zo werkt het ook in de OO wereld. De FileSystem class ligt op straat te schreeuwen "Ik kan bestand $file niet ophalen!" Een catcher die voorbij komt denkt: "Hé, ik weet hoe ik hem kan helpen." Vervolgens zorgt hij dat de logger informatie krijgt over welke klasse de exception gooide, wat voor exception en wat het bericht was. Ook geeft hij, indien hij in de debug mode zit, door aan een exception printer dat er een exception is geweest, welke exception, de message en de stacktrace.
Ha, kijk Wouter. Dat is een duidelijk voorbeeld. Alleen het heeft voor mij nog even wat verduidelijking nodig. Stel ik gooi een exception:
throw new Exception('The file ' . $file . ' could not be written'.);
Stel dat ik dit zou loggen. Dan zie ik in de log alleen staan: The file 'foo.php' could not be written.
Dan moet ik toch ook zien welke exception class die exception heeft gegooid? En moet ik dan vanuit de catch aangeven vanuit welke class dat is gebeurd?
Kun je anders eens een (heel simpel) voorbeeldje geven van wat jij in het catch-blok zou zetten?
throw new Exception('The file ' . $file . ' could not be written\nStack: '. $e->getTraceAsString()); ?
Edit: waarom zou je de exceptie uberhaupt willen catchen? Als hij niet kan schrijven naar een bestand, en die logica is belangrijk voor het vervolg van mijn applicatie, zou ik juist willen dat hij op zn bek gaat
De stack trace kun je op halen uit de exception klasse, die hoort niet bij se exception. Zien welke klasse de exception gooit kun je uit de stack trace halen en als het goed is kun je dit zien aan de classname van de exception.
Deze exception wil je catchen omdat ik niet een hele app wil laten crashen omdat de cache niet werkte. Ik zal alleen willen opslaan dat het niet werkte, zodat ik het later kan gaan debuggen.
De stack trace kun je op halen uit de exception klasse, die hoort niet bij se exception. Zien welke klasse de exception gooit kun je uit de stack trace halen en als het goed is kun je dit zien aan de classname van de exception.
Deze exception wil je catchen omdat ik niet een hele app wil laten crashen omdat de cache niet werkte. Ik zal alleen willen opslaan dat het niet werkte, zodat ik het later kan gaan debuggen.
Ozzie, voorbeeldje komt eraan!
Eigenlijk was het hele punt dat elke exceptie een getTrace methode heeft.
En als je goed leest dan zie je dat ik zeg "en die logica is belangrijk voor het vervolg van mijn applicatie". Ik wist niet dat het om een cache verhaal ging
@Ozzie waarom je je uberhaupt bezig houdt met het cachen van dingen terwijl je de basis van oo nog niet eens onder de knie hebt snap ik niet. Weet je wel dat zeker 90% van de vragen die je stelt opgelost zouden kunnen worden als je eens een weekje de tijd nam om object oriented programming te leren? Daarna zou je dan eens verder kunnen gaan met vette dingen maken
public function doCache($data)
{
try {
$this->filesystem->write($this->filename, $data);
} catch (FileSystemException $e) {
// that's bad, all my speed optimalisation topics on phphulp are now useless...
// let's log it, so Ozzie can start fixing this in a hurry
$this->logger->warning(
sprintf(
'The FileCacher experienced a problem with dataset %s and file %s: %s',
json_encode($data),
$this->filename,
$e->getMessage()
)
);
ExceptionHandler::handle($e);
}
}
}
class ExceptionHandler
{
static public function handle(\Exception $e)
{
if (self::$debug) {
echo $this->templating->render('Ozzie:Core:exception.html.twig', array(
'message' => $e->getMessage(),
'stacktrace' => $e->getTraceAsString(),
'exception_class' => get_class($e),
));
@Wouter: thanks voor het voorbeeld. Ik ben even benieuwd waar jij $this->logger vandaan tovert. Ik zou verwachten dat je een log functie in je "default" Exception class inbouwt en dat je dan dit doet:
<?php
} catch (FileSystemException $e) {
$e->log('... hier de log message');
}
?>
Als kritische opmerking vraag ik me wel af... waarom verwerk je "ExceptionHandler::handle($e);" niet in de log functie van de Exception class? Dus normaal gesproken loggen... maar op het moment dat je in debug-mode zit, toon je direct de error-message. Is dat niet handiger dan bij iedere Exception nog een keer die exception handler aan te roepen?
@NOLot: jammer dat je zegt dat ik de basis dingen niet onder de knie heb. Je kunt beter iets opbouwends en stimulerends zeggen. Ik heb nooit met Exceptions gewerkt, en wil graag het principe onder de knie krijgen. En aan je eigen voorbeeld te zien, heb je dat zelf ook nog niet helemaal onder de knie. Iedereen komt hier om iets te leren. Je advies zal best goed bedoeld zijn, maar ik schiet er niet veel mee op.
>> Ik zou verwachten dat je een log functie in je "default" Exception class inbouwt en dat je dan dit doet:
Oe, ozzie. Je begrijpt het nog steeds niet, dan maar over op de radicale methode: De exception klasse mag geen 1 actie uitvoeren, hij mag alleen data vasthouden over de error.
Ik die krijsend op de grond lig door mijn gebroken been ga je toch niet gebruiken om 112 te bellen? Nee, je gebruikt je eigen mobiel, die je zojuist bij het vertrekken van huis in je broekzak geïnjecteerd hebt.
Dus hoe kom ik aan de logger service? Juist ja, door te injecteren via de controller.
>> Je advies zal best goed bedoeld zijn, maar ik schiet er niet veel mee op.
Dan begrijp je zijn advies verkeerd, want er zit een hele grote kern van waarheid in. Je stelt nu al zo'n 1.5 jaar vragen over OO en eigenlijk schiet je niet heel veel op. De forum gebruikers steken veel tijd om jou 1 OO aspect te leren, waarna we dit over een half jaar weer precies hetzelfde mogen doen. Ik kan bijna al je OO vragen van nu beantwoorden met een linkje van een topic van jou vroeger, dat doe ik niet omdat de zoekfunctie van het forum zo slecht is.
Als je nou eens een maantje steekt in het lezen van een goed oo boek en het doorhebben van de basis principen en design patterns zal je over een jaar je framework afhebben. Op deze manier blijft het alleen bij nadenken.
