PHP: Добавление к брошенному сообщению об исключении

Question

Возьмут следующую функцию в качестве примера того, что я хочу сделать:

public function save() { 
    $this->connect('wb'); 
    try { 
     if(!$this->lock()) 
      throw new Exception("Unable to acquire configuration locks"); 
     if (!$backup = $this->backup()) 
      throw new Exception("Failed to create configuration backup"); 
     try { 
      if(!fwrite($this->_pointer, $this->dump("string"))); 
       throw new Exception("Error occured while writing to configuration"); 
      $this->unlock(); 
      $this->disconnect(); 
     } catch (Exception $e) { 
      if(rename ($backup, $this->_file)) 
       $e .= PHP_EOL."Successfully restored configuration from backup"; 
      else 
       $e .= PHP_EOL."Failed to restore configuration from backup"; 
      $this->unlock(); 
      $this->disconnect(); 
      throw $e; 
     } 
    } catch (Exception $e) { 
     echo PHP_EOL, $e->getMessage(); 
    } 
} 

Я вложенный try() и catch() заявления. Исключение выбрано из самого внутреннего и поймано, я затем выполняю некоторые функции и выдаю другое исключение. Обратите внимание, где я пишу $e .=, я понимаю, что это неправильный синтаксис. Я хочу добавить строку в исключение $e->getMessage().

Как бы я это сделал?

Answer 1

Создайте свой собственный класс исключения и создайте метод добавления строки в сообщение.

<?php 
class SuperException extends Exception 
{ 
    public function AppendToMessage($msg) 
    { 
     // $this->message is inherited from Exception class, 
     // where it is protected field (member) of the class 
     $this->message .= $msg; 
    } 
} 
?> 

Answer 2

Почему бы не использовать отдельную переменную для сохранения сообщений?

public function save() { 
    $this->connect('wb'); 
    $exceptionMessage = ""; 
    try { 
     if(!$this->lock()) 
      throw new Exception("Unable to acquire configuration locks"); 
     if (!$backup = $this->backup()) 
      throw new Exception("Failed to create configuration backup"); 
     try { 
      if(!fwrite($this->_pointer, $this->dump("string"))); 
       throw new Exception("Error occured while writing to configuration"); 
      $this->unlock(); 
      $this->disconnect(); 
     } catch (Exception $e) { 
      if(rename ($backup, $this->_file)) 
       $exceptionMessage .= PHP_EOL."Successfully restored configuration from backup"; 
      else 
       $exceptionMessage .= PHP_EOL."Failed to restore configuration from backup"; 
      $this->unlock(); 
      $this->disconnect(); 
      throw $e; 
     } 
    } catch (Exception $e) { 
     echo PHP_EOL. $exceptionMessage . PHP_EOL . $e->getMessage(); 
    } 
} 

Answer 3

Большинство объектно-ориентированных языковых фреймворков, которые поддерживают обработку исключений, осознают необходимость в этой теме. Вы должны понимать, что Исключения не журналы того, что произошло. Скорее, они там, чтобы точно определить, где произошла ошибка.

Таким образом, не только этот синтаксис неверен, вся идея нарушает дюжину принципов OOD. Вам следует ввести класс регистратора, который собирает информацию об ошибках по ходу дела и использует исключения для точного определения событий , которые необходимо записать.

class ErrorLogger { 

    private $log; 

    public function __construct() { 
     $this->log = array(); 
    } 

    public function log(Exception $e) { 
     array_push($this->log, $e->getMessage()); 
    } 

} 

и далее в коде:

$logger = new ErrorLogger(); 
try { 
    : 
} catch (Exception $e) { 
    $logger->log($e); 
}