Ala MVC framework, jak usprawnić, poprawić itp Opcje

[kangurmk]

Na laborkach z programowania dostaliśmy za zadanie stworzyć stronę internetową oparta o wzorzec MVC, postanowiłem nie korzystać z dostępnych powszechnie frameworków, tylko napisać własny prosty ala framework oparty na idei wzorca MVC (przy okazji ucząc się dogłębniej idei OPP i wzorców projektowych). Ponieważ ten framework nie jest jeszcze kompletny i mimo że będzie dalej rozbudowywany chciałem zapytać Was o pomysły, poprawki odnosie jego rozwoju.
Podstawowe informacje o budowie tego frameworka:
- zastosowanie mod_rewrite
- przekazywanie przez url modeli i akcji (wzorowane na ZendFramework)
- wykorzystanie bibliotek xajax i savant
Na razie brak dokładnej implementacji błędów i zabezpieczeń przed XSS, SQLInjection i innymi ...Injection ;] (częściowe są)
Kod frameworka wraz z przykładowymi kontrolerami, modelami i widokami możecie ściągnąć z http://rapidshare.com/files/168371642/miko.rar lub http://www.ppiw.ovh.org/miko.rar

Ten post edytował kangurmk 29.11.2008, 10:45:11

[AxZx]

- przekazywanie przez url modeli i akcji (wzorowane na ZendFramework)

w większości aplikacji (o ile nie we wszystkich z MVC:) przekazuje się w adresie nazwe kontrolera nie modelu.

implementacji błędów

jakie błędy chcesz implementować w szkielecie działania aplikacji?

[kangurmk]

w większości aplikacji (o ile nie we wszystkich z MVC:) przekazuje się w adresie nazwe kontrolera nie modelu.

u mnie początek nazwy kontrolera i modelu są identyczne (tylko ten pierwszy ma na końcu nazwy Controller a ten drugi Model) i oprócz tego że w url wpisanie np index-index.html oznacza że zmienne model=index, action=index, to ta zmienna model oznacza nazwę kontrolera który będzie użyty, ale ponieważ jest to trochę pogmatwane to zmienię jej nazwę z model na controller.
Przy okazji kontrolerów i modeli to mój framework wymusza istnienie modelu odpowiadającego konkretnemu kontrolerowi, a mianowicie jeśli istnieje plik np. IndexController.php (a w nim klasa IndexController) to musi istnieć IndexModel.php (a w nim klasa IndexModel, która może być pusta). Czy jak na mała złożoność tego framwerok zostawić ten wymóg, czy jednak dodać kod który pozwoli też tworzyć kontrolery bez potrzeby tworzenia im odpowiadających modeli.

jakie błędy chcesz implementować w szkielecie działania aplikacji?

Chodzi mi o implementacje wszystkich możliwych błędów w klasach dostarczonych wraz z frameworkiem, np jeśli użytkownik przekaże parametr który powoduje że aplikacja się wysypie, to wtedy powinien pojawić się stosowny komunikat że aplikacja zawiera błąd (odnośnie wyświetlania błędów i np ich zapisu do logów, albo wysyłania na maila chcę stworzyć jakaś klasę). Co do obsługi błędów w konkretnych kontrolerach i modelach to są one specyficzne dla "kodu jaki został w nich wpisany" więc to osoba która programuje dany kawałek martwi sie o obsługę błędów, ja tylko dostarczam zgrabny sposób ich zapisu, wyświetlenia itp. (wspomniana klasa, którą mam zamiar napisać)

[pyro]

wrzuc gdzie indziej niz rapid...

[kangurmk]

Proszę bardzo: http://www.ppiw.ovh.org/miko.rar

[AxZx]

u mnie początek nazwy kontrolera i modelu są identyczne (tylko ten pierwszy ma na końcu nazwy Controller a ten drugi Model) i oprócz tego że w url wpisanie np index-index.html oznacza że zmienne model=index, action=index, to ta zmienna model oznacza nazwę kontrolera który będzie użyty, ale ponieważ jest to trochę pogmatwane to zmienię jej nazwę z model na controller.
Przy okazji kontrolerów i modeli to mój framework wymusza istnienie modelu odpowiadającego konkretnemu kontrolerowi, a mianowicie jeśli istnieje plik np. IndexController.php (a w nim klasa IndexController) to musi istnieć IndexModel.php (a w nim klasa IndexModel, która może być pusta). Czy jak na mała złożoność tego framwerok zostawić ten wymóg, czy jednak dodać kod który pozwoli też tworzyć kontrolery bez potrzeby tworzenia im odpowiadających modeli.

modele powinny być ładowane wg uznania programisty. w jednym kontrolerze możesz nie lądować wcale modeli bo ci nie są potrzebne, a w drugim będziesz potrzebował 10 modeli.

Chodzi mi o implementacje wszystkich możliwych błędów w klasach dostarczonych wraz z frameworkiem, np jeśli użytkownik przekaże parametr który powoduje że aplikacja się wysypie, to wtedy powinien pojawić się stosowny komunikat że aplikacja zawiera błąd (odnośnie wyświetlania błędów i np ich zapisu do logów, albo wysyłania na maila chcę stworzyć jakaś klasę). Co do obsługi błędów w konkretnych kontrolerach i modelach to są one specyficzne dla "kodu jaki został w nich wpisany" więc to osoba która programuje dany kawałek martwi sie o obsługę błędów, ja tylko dostarczam zgrabny sposób ich zapisu, wyświetlenia itp. (wspomniana klasa, którą mam zamiar napisać)

hmm czyli może chodzi o implementacji obsługi błędów? jak to można nazwać fachowo?

[bim2]

w większości aplikacji (o ile nie we wszystkich z MVC:) przekazuje się w adresie nazwe kontrolera nie modelu.

Hmm, jak zwał tak zwał. ja nazywam "twoje kontrolery" akcjami, a "twój frontConroler" kontrolerem.

[AxZx]

ale chyba przyznasz że model i kontroler są głównymi składnikami MVC ?
raczej nie można mówić jak zwał tak zwał bo są to 2 odrębne kwestie.
model to model, kontroler to kontroler - czy to front czy jakiś inny.

[Cysiaczek]

Niewygodne i nieco mylące tworzenie zmiennych w akcji. Uzywasz vars, a dalej jest assign. Powinno wystarczyc

[PHP] pobierz, plaintext 
<?php
$this->vars->nazwa=1;
//lub
$this->nazwa='cos'
?>
[PHP] pobierz, plaintext 

Potem cały kontener z danymi jest ekstraktowany automatycznie w widoku

[PHP] pobierz, plaintext 
<?php
if(isset($this->vars['logOut']))
        {
            if($this->vars['logOut'] == 1){
                $this->vars['auth']->LogOut();
            }
        }
?>
[PHP] pobierz, plaintext 

no nie wiem, ale wydaje mi się, że

[PHP] pobierz, plaintext 
<?php
$this->context->user->logout();
?>
[PHP] pobierz, plaintext 

Byłoby lepszą implementacją niż dostęp przez tablicę, choć jeszcze lepsze by było, gdyby istniała osobna akcja wylogowująca użytkownika.

Pozdrawiam

[kangurmk]

Niewygodne i nieco mylące tworzenie zmiennych w akcji. Uzywasz vars, a dalej jest assign. Powinno wystarczyc

[PHP] pobierz, plaintext 
<?php
$this->vars->nazwa=1;
//lub
$this->nazwa='cos'
?>
[PHP] pobierz, plaintext 

vars przechowuje zmienne przekazane metodą GET i zmienne dodane funkcją AddMKObj klasy MKBootStrap, a metoda assign służy do kopiowania zmiennej do szablonu (jest to opakowanie funkcji assign z systemu szablonów savant), czyli zmienne przekazane metodą assign będzie można używać w kodzie widoku (w szablonie).
Ogólnie koncepcja polega na tym że w pliku index.php tworzony jest obiekt klasy MKBootStrap, który domyślnie tworzy obiekty klasy savant i xajax + zmienne z $_GET i dodaje je to tablicy vars (dzieje się to w konstruktorze klasy MKBootStrap). Obiekt tej klasy tworzymy tak:

Kod

$boot = MKBootStrap::getInstance($load->GetAppDir());

Następnie możemy, utworzyć obiekty innych klas zawartych w folderze lib, np obiekt klasy Auth jeśli wymagamy aby nasza aplikacja obsługiwała logowanie użytkowników (na razie w lib jest tylko jedna klasa Auth którą możemy a nie musimy dodawać, w przyszłości pewnie powstanie więcej, np do rejestracji użytkowników, wysyłania e-maili itp)

Kod

$auth = MKAuth::getInstance();
$boot->AddMKObj('auth', $auth);

i teraz w każdym kontrolerze mamy dostęp do obiektu auth poprzez zmienną $this->vars.

Potem cały kontener z danymi jest ekstraktowany automatycznie w widoku

[PHP] pobierz, plaintext 
<?php
if(isset($this->vars['logOut']))
        {
            if($this->vars['logOut'] == 1){
                $this->vars['auth']->LogOut();
            }
        }
?>
[PHP] pobierz, plaintext 

no nie wiem, ale wydaje mi się, że

[PHP] pobierz, plaintext 
<?php
$this->context->user->logout();
?>
[PHP] pobierz, plaintext 

Byłoby lepszą implementacją niż dostęp przez tablicę, choć jeszcze lepsze by było, gdyby istniała osobna akcja wylogowująca użytkownika.

Pozdrawiam

Co do osobnej akcji wylogowującej to dobry pomysł, ale to już kwestia gustu osoby piszącej dany kontroler.

[Cysiaczek]

Rozumiem. Zobacz jednak, że nie połapałem się od razu, a widziałem już wiele frameworków. W moim rozwiązaniu $this->vars przechowywała właśnie zmienne, które potem zły do szablonu. Użycie assign('nazwa', 'wartosc') jest po prostu niewygodne. NIe możesz dal dać implementacji __set() ?
Pozdrawiam

[mike]

Użycie assign('nazwa', 'wartosc') jest po prostu niewygodne. NIe możesz dal dać implementacji __set() ?

Nie zgadzam się.
W symfony możesz przekazać dane z akcji do widoku na dwa sposoby:

[PHP] pobierz, plaintext 
<?php
 
$this->var = $var;
// lub
$this->setVar('var', $var);
 
?>
[PHP] pobierz, plaintext 

Pomimo tego, że pierwszy sposób jest szybszy to jednak ja preferuję drugi sposób. Dlaczego? Ponieważ jest dużo bardziej czytelny i świadomy. W akcji, która ma sporo kodu można szybko się pogubić co poszło do widoku jeśli działasz tylko na metodach magicznych.

[Cysiaczek]

Heh, kwestia gustu - ja nie chcę, żeby zlikwidował assign, tylko dodał możliwość ustawiania przez magiczne. Ja jednak w SF używam magicznych

[bim2]

Hmm, ja jednak byłbym nadal z setVar Pana Mike. Kiedy zobaczył bym Cysiaczek twój kod nie od razu połapałbym się co ląduję do szablonu a co ustawiasz na potrzeby klasy. : - ) Nie upraszczajmy wszystkiego na siłę, bo w końcu zabłądzimy.

[kangurmk]

Ok, pozostanę przy obecnym rozwiązaniu (jak będę miał czas to spróbuje też zaimplementować drugi sposób z metoda magiczną __set()), ale na razie mam do was pytanie czy da się zrobić tak żeby poniższy kod:

Kod

$this->vars['auth']->LogOut();

zastąpić

Kod

$this->vars->auth->LogOut();

czyli zamiast tablicy vars musiałby być jakiś obiekt vars, tylko jak wtedy utworzyć dynamicznie pole w tym obiekcie. Tzn. Dodając do vars obiekt auth w mam do niego dostęp przez vars->auth (czyli chyba de facto muszę mieć publiczne pole auth), i tak dalej dodając jakiś obiekt mam do niego dostęp przez vars->[nazwa_tego_obiektu]?

[erix]

Magiczne metody __set() i __get()

[kangurmk]

Czy ten nowy obiekt MKVarsTransver (wyżej w postach nazywa się vars) może wyglądać tak:

[PHP] pobierz, plaintext 
<?php
class MKVarsTransver {
    private $vars = array();
    public function __set($name, $value)
    {
         $this->vars[$name] = $value; 
    }
     public function __get($name)
    {
         return $this->vars[$name];
    }
$
?>
[PHP] pobierz, plaintext 

Czy przydało by się dodać jakieś inne metody do tej klasy?
Czy warto zastosować wzorzec singleton to tego obiektu? Chociaż drugiej strony chyba sie nie utworzy więcej niż raz, jeśli tworzymy go w metodzie obiektu który jest singletonem, dokładniej w konstruktorze klasy MKBootStrap. A może zamiast tworzyć nowy obiekt dodać te metody do klasy MKActionController?


---
dodałem te metody magiczne do klasy MKActionController

[PHP] pobierz, plaintext 
<?php
 
abstract class MKActionController
{
    
    protected $model;
  protected $view;
  protected $vars;
  private $model_file_name;
  private $model_class_name;
  
    function __construct($vars) 
   {
         $this->model_file_name = '../models/'.$vars['controller'].'Model.php';
         if(file_exists($this->model_file_name))
        {
            require_once($this->model_file_name);
              $this->model_class_name = $vars['controller'].'Model';
                $this->model = new $this->model_class_name();
                $this->vars = $vars;
                $this->vars['presentation']->addPath('template','../views/');
            }
   }
   
  public function __set($name, $value)
  {
        $this->vars[$name] = $value;
  }
  public function __get($name)
  {
      return $this->vars[$name];
  }
   
  function getView() 
  {
    return $this->view;
  }
}
?>
[PHP] pobierz, plaintext 

zmieniłem kod w kontrolerze IndexController na:

[PHP] pobierz, plaintext 
<?php
if(isset($this->vars['logOut']))
        {
            if($this->vars['logOut'] == 1){
                $this->vars['auth']->LogOut();
            }
        }
?>
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
<?php
if(isset($this->vars->logOut))
        {
            if($this->vars->logOut == 1){
                $this->vars->auth->LogOut();
            }
        }
?>
[PHP] pobierz, plaintext 

i wywala mi błąd

Notice: Trying to get property of non-object in C:\xampp\htdocs\mvc\branches\miko\application\controllers\IndexController.php on line 9
(w kodzie wyżej oznacza to linię 5)

Ten post edytował kangurmk 29.11.2008, 17:02:16

[erix]

Metoda __get jest wywoływana, gdy zmienna w klasie nie istnieje.