[mvc] Problem z obsługą błędów, + Ocena obecnego kodu Opcje

[Vengeance]

Oto mały przykład "wycięty" z mojego, powstającego jeszcze frameworka (jak widac pisanego mniej wiecej wg zasad MVC).
Przy starcie aplikacji wywołujemy metode Application::run() która wybiera akcje oraz widok. Uruchamia akcje, która z kolei może poprzez 'return' zwrócić:
+ Obiekt dziedziczący z klasy Action - wtedy kontroler uruchomi tę akcje (w ten sposób obsługiwane są łańcuszki akcji.
+ Inne, dowolne dane - wtedy kontroler traktuje je jako wynik działania akcji(łańcuszka) i przekazuje go do widoku(poprzez konstruktor). Na końcu kontroler wyświetla widok.

Po 1. Powiedzcie co tu może być źle przezemnie zrozumiane itd. Co można poprawić/zmienić. Jestem otwarty na propozycje ;]

Po 2. Mam problem z obsłużeniem błędów akcji. Takich jak np. w akcji 'showNews' brak newsa w bazie danych. Na razie obsługuje to jako łańcuszek jednak problem pojawia się, gdy akcja 'showError' korzysta z innego widoku niż akcja 'showNews'. Trzeba by było jakoś sensownie ten widok podmieniać. Jak ?
Albo jak rozwiązań to "fajnie". Mam wrażenie (pewnie uzasadnione), że to co istnieje teraz jest dość felernym rozwiązaniem.

ps. nie chce wykorzystywać czegoś ala 'fallback-action' gdyz to umożliwia generowanie tylko jednego komunikatu błędu gdy ogólnie akcja się "spsuje".

Prosze o pomoc (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Model:

[PHP] pobierz, plaintext 
<?php
class NewsContainer
{
   public function getNewsByID($iID)
   {
      if($iID <> 1) throw new NewsNotFoundException();
      return new News('Jakis Tytul', 'Jakas Tresc');
   }
}
 
class News
{
   public $title, $content;
   public function __construct($title, $content)
   {
      $this->title = $title;
      $this->content = $content;
   }
}
?>
[PHP] pobierz, plaintext 

Akcja:

[PHP] pobierz, plaintext 
<?php
 
   class showNews extends Action
   {
      public function display()
      {
         try {
            $model = new NewsContainer();   
            $news = $model->getNewsByID(1);
            return $news;
         } catch(NewsNotFoundException $e) {
            return new showError('Informacja', 'News o podanym ID nie istnieje');
         }
      }   
   }
 
?>
[PHP] pobierz, plaintext 

Widok:

[PHP] pobierz, plaintext 
<?php
 
   class showNewsAsHTML extends View
   {
      private $news, $_template;
 
      public function __construct($oNewsRecord)
      {
         $this->news = $oNewsRecord;
         $this->_template = new Smarty();
      }
 
      public function display()
      {
         $this->_template->assign('news', $news);
         $this->_template->display('html/showNews.tpl');
      }   
   }
 
?>
[PHP] pobierz, plaintext 

Kontroler:

[PHP] pobierz, plaintext 
<?php
 
class Application
{
      public function run()
      {
         $sActionName = $_GET['action'];
         $sViewName = $_GET['view'];
            
         $mDataForView = $this->performAction(new $sActionName());
         $oView = new $sViewName($mDataForView);
         $oView->display();
      }
 
      public function performAction(Action $oAction)
      {
         $result = $oAction->perform();
         /* Obsługa łańcuszków akcji */
         if($result instanceof Action)
            $this->performAction($result);
         else 
            return $result;
      }
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Vengeance 1.03.2005, 16:48:44

[Kinool]

tak na marginesie to:

public function __construct($title, $content)

konstruktor powienien byc raczej bez deklarowaia go jako public, domyslnie jest private, no chybaze ze to zamierzony zamiar wywolania ale nie bardzo rozumie do czego?