[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

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

[ActivePlayer]

[PHP] pobierz, plaintext 
<?php
 
die(&#092;"Brak newsa\");
 
?>
[PHP] pobierz, plaintext 

jestes w stanie zarzucic mi w celach edukacyjnych cały kod ?

[bela]

W Oceny jest

[Vengeance]

http://forum.php.pl/index.php?showtopic=26844

Tu jest cały kod jednak w starszej wersji. Tzn tam jest rozdzielenie na akcje widok/logiki. Teraz jednak pracuje nad oddzieleniu tego wszystkie i stowrzeniu poprawnych akcji i widoków Jak ktoś ma pomysł i może pomóc to prosze.

[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?

[Vengeance]

No Coś ty ! Od kiedy kontruktor jest 'private'!

Takie coś stosuje się tylko w Singletonie. Przcież ustawnienei 'private'
uniemożliwi ci zrobienie '$obj = new Object();' hehe

Dzieki i tak za uwage ;] czekam na kolejne ;p

[chmolu]

Na początek, jako że jestem nowy na tym forum, chciałbym przywitać wszystkich forumowiczów

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

[bela]

Łańcuszki rzadko ? To zależy czy akcję są podzielone na akcje widoku i logiki. Te pierwsze mają za zadanie wyświetlić dane z modelu, a drugie wprowadzić zmiany w modelu. Przy takim układzie łańcuszki to rzecz naturalne: dodajemy news a później wyświetlamy informacje, albo przy niepowodzeniu, akcje logiki wyrzuca fallback(wyjatek), czyli w gruncie też akcje widoku.

Co sie w smarty moze się nie podobac

Problem z łańcuszkami jest gdy chcemy wrzucić 2 akcje logiki do łańczuszka, bo co zrobić gdy jedna przejdzie a druga wyrzuci fallback.

[Vengeance]

@chmolu:
Łańcuszki właśnie przydają się często Przynajmniej ja widze, że będzie mi to przydatne.

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

Wiesz, ja wychodze z założenia, że podel sam z siebie nie może być "uruchomiony". Tzn _zawsze_ musi wystąpić akcja, nawet jeśli jest działanie ogranicza się do zrobienia 'return true;'. Wtedy dopiero kontroler inicjuje widok. Pozatym w widoku nie pobieram żadnych danych bezpośrednio z modelu. Tego dokonuje akcja przekazując następnie wynik do modelu. Z tego właśnie wynika, że stosuje akcje typu showNews/showArticle/showXX.

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 */);

Także miałem pomysł, aby wykorzystać "coś" co jest w stanie zmienić widok. Metoda setView czy też jaki dodatkowy parametr zwracany w 'return'. Jednak wydawało mi się to jakieś "dziwne". Po to właśnie storzyłem ten wątek aby uzyskać inne sposoby

[chmolu]

A może model zamiast wywalać wyjątek powinien po prostu zwrócić "pusty" obiekt, albo obiekt typu Error:

[PHP] pobierz, plaintext 
<?php
 
class Error
{
   private $message;
 
   public function __construct($message)
   {
	   $this->message = $message;
   }
 
   public function getMessage()
   {
	   return $this->message;
   }
}
 
?>
[PHP] pobierz, plaintext 

Wtedy twoj kod moglby wygladac tak:

[PHP] pobierz, plaintext 
<?php
 
class NewsContainer
{
   public function getNewsByID($iID)
   {
	  if(!news_exists) 
	  {
		  return new Error('News o podanym ID nie istnieje');
	  }
	  return new News('Jakis Tytul', 'Jakas Tresc');
   }
} 
 
class showNews extends Action
{
   public function display()
   {
	   $model = new NewsContainer();   
	   $news = $model->getNewsByID(1);
	   return $news;
   }   
}
 
class Application
{
	private $view = DEFAULT_VIEW;
 
	public function run()
	{
	   $sActionName = $_GET['action'];
	   $this->view  = $_GET['view'];
 
		 $mDataForView = $this->performAction(new $sActionName());
		 $oView = new $this->view($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
		 {
			if ($result instanceof Error) $this->view = 'Error';
			return $result;
		 }
	  }
}
 
?>
[PHP] pobierz, plaintext 

[Vengeance]

Zauwaz, że akcja czesto dokonuje operacji na danych z modelu
A wiec wole wyjatki (ladniej wylapuje sie blad).

Jedyne co moge potwierdzic, to ze juz myslalem aby w performAction()
dodac jeden warunek 'instanceof Error' co powodowalo by np. zmiane widoku, czy zdefiniowanej przez uzytkownika akcji, obslugujacej bledy.