[mepeisen]

Zitat von kampfgnom

Kann man das so machen?? Oder nicht. Denn so hätt ich zumindest schonmal eine möglichkeit zuÜberprüfen ob es wirklich "meine" Exceptions sind

Das ist aus dem genannten Grund schon OK. Die Namensgebung solltest du - wie schon gesagt - überdenken. ExceptionHandler ist da ziemlich irreführend. Eher sowas wie "MyException" oder "KampfGnomException" oder "BaseException".

Aber funktioniert der Code denn so? IMHO muss eine Exception immer von Exception abgeleitet sein. Das, was in einem Catch-Block steht ebenso. Sprich: Prüfe mal, ob der deinen Catch-Block überhaupt anspringt.

[kampfgnom]

Ähm, myException ist doch von der Exceptionsklasse abgeleitet...

und das zweite Codebespiel war letztendlich der wirklcihe Exceptoinshandler, also meine Errorklasse die dann weiß was mit den Exceptions zutun ist.

Das müsste eigentlich so heißen

PHP-Code:

function exception_handler($exception) {
     $error = new Errorclass($exception); #Errorklasse
     $error->execute(); #Ausführen der Aktion der jeweiligen Exceptoin
}

set_exception_handler('exception_handler'); 


[kampfgnom]

Ok ich hab weitergebastelt.

Habe jetzt auch einen ErrorHandler der anhand einer Config entscheidet, was mit einer Exception passiert.
Aber die Config ist genau mein Problem.
Man muss ja für einen eigenen Exceptionshandler eine eigene funktion schreiben
das sieht bei mir so aus:

PHP-Code:

function exceptionshandler($exception)
{
        $handler = new Error_ExceptionsHandler();
        $handler->handleException($exception);
}

set_exception_handler('exceptionshandler'); 


Mein Problem ist aber, das ich noch eineConfig setzen muss. Man ja aber keine Config-Variable übergeben kann.

Sollte man dann die Konfiguration in diese Funktion verlegen also so:

PHP-Code:

function exceptionshandler($exception)
{
        $config = array( array('type' => "Error_Exceptions_DbException", 'action' => 'log'));
        $handler = new Error_ExceptionsHandler();
        $handler->setExceptionsConfig($config);
        $handler->handleException($exception);
}

set_exception_handler('exceptionshandler'); 


oder wie könnte ich das lösen. Vorallem will ich gewährleisten, dass man auch die Config aus einer Ini-Datei holen kann, aber dazu bräuchte ich ja wieder eineklasse und ich glaube dann wird da zuviel logig in die Funktion gebracht. Irre ich mich da?

zur Info mein Errorhandler sieht so aus, vlt könnt ihr da noch eine Verbesserung finden:

PHP-Code:

<?php
class Error_ExceptionsHandler
{
    protected $exceptionsConfig = null;
    public function __construct()
    {
        
    }
    
    public function setExceptionsConfig(array $config)
    {
        if (key_exists('type', $config) and key_exists('action', $config))
        {
            $this->exceptionsConfig = $config;
        }else throw new Exception('Unknown build of an Array!');
        
    }
    
    public function addExcptionConfig(array $config)
    {
        $this->exceptionsConfig[] = $config;
    }
    
    public function handleException(Error_Exceptions_iErrorException $exception)
    {
        if ($this->exceptionsConfig !== null)
        {
            foreach ($this->exceptionsConfig as $config)
            {
                if ($config['type'] == $exception->getType())
                {
                    switch ($config['action'])
                    {
                        case 'log':
                            echo $this->writeLog($exception->getMessage(), $exception->getCode());
                            break;
                        
                        #other ways to handle an Exception...
                        default:
                            throw new Exception('Unknown Exception! The Script has to cancel!');
                    }
                }
            }
        }else throw new Exception('You have to Configure the Exceptionshandler!');
        
    }
    
    protected function writeLog($message, $code)
    {
        //code to write a message in a log!
       //NUR ZUM TESTEN:
        return "[".date()."] #".$code." ".$message;
    }
}
?>

Es sollten ja keine Exceptions in der Exceptionsklasse geworfen werden, das lößt ja eine endlosschleife aus. War aber die einzige Möglichkeit das ganze abzubrechen...oder sollte man da lieber die(); nehmen??

[bobby]

Hi Gabriel,

die Funktion 'exceptionshandler' ist, zumindest derzeit, überflüssig. Du kannst auch einfach eine Instanz + Methode an set_exception_handler übergeben.
(Geht mit den meisten Funktionen, die eine Callback-Funktion erwarten)

PHP-Code:

<?php
class Error_ExceptionsHandler
{
    public function something(Exception $e)
    {
        die('Ooohh no! ' . $e->getMessage());
    }
}


set_exception_handler(array(new Error_ExceptionsHandler(), 'something'));

throw new Exception('Someting went wrong.');
?>

bobby.

[kampfgnom]

könnte ich ihm auch eine Instanz übergen. Also so:

PHP-Code:

$Exceptionsconfig     = array(
                                    array ('type'=> 'default', 'action' => 'all'), 
                                    array('type' => "Error_Exceptions_DbException", 'action' => 'all')
                                );
        $config             = array (
                                    'logfile' => 'log.log', 
                                    'adress' => 'errors@example.tld'
                                );
        $handler = new Error_ExceptionsHandler($config);
        
        $handler->setExceptionsConfig($Exceptionsconfig);
set_exception_handler(array($handler, 'handleException' )); 


gut danke. Geht!
Habs getestet. Vielen dank.
Damit wir das ganze Überschaubarer!

[kampfgnom]

Hab jetzt mal mein "Errorhandling" online gestellt.
Wer möchte kann es sich ja mal anschauen und mir sagen was ihm gefällt bzw. fast besser wäre was nicht so doll ist:
hier ist die SVN-Repositroy zu finden:
http://svn2.assembla.com/svn/aedo/


grüße
Gabriel