[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

[chmolu]

Na początek, jako że jestem nowy na tym forum, chciałbym przywitać wszystkich forumowiczów (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Widzę, że i tu ludzie zmagają się z 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.

Osobiście uważam, że łańcuszki akcji prawie nigdy ci się nie przydadzą. Lepiej operacje typu 'Usuń newsa' -> 'usuń artykuł' umieszczać w jednej akcji wywołując kilka klas modelu. W swoim frameworku mam metodę forward($action) kontrolera, którą może wywołać inna akcja. Tak realizuję łańcuszek. To akcja wywołuje następne akcje. Plus jest taki, że można na przykład wywołać inną akcję przed uruchomieniem właściwej. Można to zrobić tak jak u ciebie - to też jest dobre rozwiązanie. U mnie by się ono nie sprawdziło. Ale, tak jak powiedziałem, w praktyce 'łańcuszki akcji' spotyka się bardzo rzadko.

+ 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.

To jest ciekawe rozwiązanie. Można w ten sposób na przykład przekazać listę błędów Validatora formularzy.

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.

W moim frameworku nie ma akcji typu showNews/showArticle/showXX. To jest zadanie widoków. Przez akcję rozumiem coś, co zmienia model (addArticle, addNews, deleteNews). A zatem, jeśli http://www.mysite.com/index.php?view=showNews to wtedy nie ma potrzeby uruchamiania akcji. Odpalam tylko odpowiedni widok. Jeśli natomiast http://www.mysite.com/index.php?action=deleteNews to odpalam akcję, która decyduje, który widok wyświetlić (np. $controller->setView($name)).

Widoki w moim frameworku chcę rozwiązać podobnie jak to jest w WACT. Też będą komponenty, tagi, z tym, że to wszystko będzie oparte na drzewie DOM (tyle, że bez używania DOM::save/load - to muszę niestety napisać sam :/).

Jak rozwiązać problem błędów? Nie ma akcji typu showNews, więc sprawdzenia, czy dany news istnieje możesz dokonać w widoku. Nie wiem jak to będzie w smarty, bo odczuwam niechęć do tego systemu. Zobacz, jak to jest rozwiązane w WACT - IMHO znakomity sposób (z wykorzystaniem komponentów). A jeśli błąd wystąpi w akcji typu deleteNews to możesz przerwać akcję i wywołać setView('Error', $dane /* komunikaty o błędach */);

Są różne sposoby. Właśnie przez takie drobne wydawałoby się problemy MVC staje się strasznie trudne do implementacji :/. Ja też na razie nie mogę wymyślić, jak radzić sobie np. z błędami Validatora, czy z autoryzacją. Odezwij się na gg to może razem coś wymyślimy (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)