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

[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