Routing we wlasnym MVC Opcje

[brzoza91]

Witam
Chciałbym stworzyć routing, do własnej implementacji MVC. Na początku przedstawię strukturę katalogów mojego projektu.
(IMG:http://img855.imageshack.us/img855/6812/struktura.gif)
W folderze app, będą pliki danego projektu. Każdy contoller będzie miał swój plik .yml w którym będą zapisane zasady routingu.

przykładowy plik index.yml
prefiks oznacza wpisany adres url, contoller i action wiadomo.

[PHP] pobierz, plaintext 
index: 
  prefix: /index/index/
  controller: Index
  action: index
 
show: 
  prefix: /index/show/
  controller: Index
  action: show
 
add: 
  prefix: /index/add/
  controller: Index
  action: add
[PHP] pobierz, plaintext 

i tak działa mój routing:
1. ktoś wpisuje adres na mojej stronie np. index/show/40
2. sprawdzane jest czy istnieje dany controller o nazwie index, akcja show i parametr 40
3. jesli tak to uruchamiany jest odpowiednia akcja z contollera

Routing.php

[PHP] pobierz, plaintext 
class Routing
{
    private $action;
    private $controller;
    private $parameter;
    private $routingFile;
    private $path=array();
 
    function __construct()
    {
        $this->createPath();
        $this->loadRoutingFile($this->path['contoller']);
        $path = $this->searchRouting($this->path['contoller'] . '/' . $this->path['action'], $this->
            routingFile);
 
        $this->setAction($path);
        $this->setContoller($path);
    }
 
    private function searchRouting($search, $array)
    {
        $array = (object)($array);
        $search = trim($search, '/');
 
        foreach ($array as $routing) {
            $r = trim($routing['prefix'], '/');
            if ($r == $search) {
                return $routing;
            }
        }
        header('Location: index.php');
        return false;
    }
 
    private function loadRoutingFile($name)
    {
        require_once ('/libs/yaml/Yaml.php');
        if (isset($name)) {
            $this->routingFile = Yaml::parse('c:/wamp/www/framework/app/routing/' . $name .
                '.yml');
        } else {
            $this->routingFile = Yaml::parse('c:/wamp/www/framework/app/routing/index.yml');
        }
    }
 
    private function createPath()
    {
        $url = explode('/', trim($_GET['url'], '/'));
 
        if (!isset($url[0])) {
            $url[0] = 'index';
        }
        if ($url[0] == "") {
            $url[0] = 'index';
        }
 
        if (!isset($url[1])) {
            $url[1] = 'index';
        }
        if ($url[1] == "") {
            $url[1] = 'index';
        }
 
        if(isset($url[2]) && filter_var($url[2],FILTER_SANITIZE_NUMBER_INT,FILTER_VALIDATE_INT)){
            $this->parameter=$url[2];
        }
        $this->path['contoller'] = $url[0];
        $this->path['action']= $url[1];
    }
 
    public function getAction()
    {
        return $this->action;
    }
 
    public function getContoller()
    {
        return $this->controller;
    }
 
    public function getParameter()
    {
        return $this->parameter;
    }
 
    public function setAction($path)
    {
        $this->action = $path['action'];
    }
 
    public function setContoller($path)
    {
        $this->controller = $path['controller'];
    }
 
 
}
[PHP] pobierz, plaintext 

Bootstrap.php

[PHP] pobierz, plaintext 
require_once 'core/Routing.php';
class Bootstrap
{
    public function __construct(){
        $routing=new Routing();
 
        require_once('/app/controllers/'.$routing->getContoller().'Controller.php');
        $contoller=$routing->getContoller().'Controller';
        $action=$routing->getAction().'Action';
 
        $page=new $contoller();
        $page->$action($routing->getParameter());
    }
 
}
[PHP] pobierz, plaintext 

Chciałbym aby mój kod był uniwersalny i był elastyczny dla wielu różnych projektów. Chciałbym też aby został napisany o dobre praktyki OOP. Głównie to właśnie zależy mi na wysokiej jakości kodu. Co myślicie o takim rozwiązaniu tego zagadnienia? Wszystko działa w tym routingu. Jednak bardzo mi zależy aby pisać przemyślany kod. Gdzie popełniłem jakiś błąd projektując ten routing ?

[!*!]

@up przeczytaj raz jeszcze mój ostatni post i to co napisał markonix "obrazkowo" ma to wyglądać tak:

[PHP] pobierz, plaintext 
class ControllerA
{
   public function index()
   {
        $getUser = new User();
        $arr = $getUser-> getAll(); // tworzysz zmienną ze zwróconą wartością, aby ją sprawdzić i później przesłać
        if(is_array($arr)) // sprawdzasz zmienną, a nie odwołanie bezpośrednie, zadanie dla Ciebie, aby wyjaśnić dlaczego (IMG:style_emoticons/default/tongue.gif) 
       {
          $w = new View($arr) // wysyłasz dane do widoku
       }
    }
 
}
[PHP] pobierz, plaintext 

Model:

[PHP] pobierz, plaintext 
class User
{
public function getAll()
{
    $PDO = // połączenie z bazą danych
    // jeśli chcesz coś z tymi danymi zrobić, np wywalić cokolwiek z tablicy, posortować to też w modelu i TYLKO w nim
    return $PDO->fetchAll() // zwracasz wynik jaki otrzymałeś od bazy z powrotem do kontrolera
}
 
}
[PHP] pobierz, plaintext 

Widok:

[PHP] pobierz, plaintext 
class View
{
    public function __construct($arr)
    {
       // tu robisz cokolwiek z danymi z tablicy
       echo $arr['name'];
       // tak jak napisał markonix, w widoku mogą znaleźć się pętle ify czy cokolwiek co zapewni ze widok ma być taki a nie inny, ALE nie rób w nim nie wiadomo jakich obliczeń, działań na bazie czy plikach (poza tym aby je wczytać ;))
    }
 
}
[PHP] pobierz, plaintext 

Ten post edytował !*! 17.01.2013, 10:28:45

[brzoza91]

[PHP] pobierz, plaintext 
class ControllerA
{
   public function index()
   {
        $getUser = new User();
        $arr = $getUser-> getAll(); // tworzysz zmienną ze zwróconą wartością, aby ją sprawdzić i później przesłać
        if(is_array($arr)) // sprawdzasz zmienną, a nie odwołanie bezpośrednie, zadanie dla Ciebie, aby wyjaśnić dlaczego (IMG:style_emoticons/default/tongue.gif) 
       {
          $w = new View($arr) // wysyłasz dane do widoku
       }
    }
 
}
[PHP] pobierz, plaintext 

wcześniej miałem prawie identycznie to zaprojektowane.
tylko doszedłem wtedy do pewnego problemu. W jednej akcji kontrolera potrzebuję wszystkich użytkowników, więc tak będzie to wyglądać(mowa jest o akcjach w kontrolerze)

[PHP] pobierz, plaintext 
public function AAction(){
    $getUser = new User();
    $arr = $getUser->getAll();
}
[PHP] pobierz, plaintext 

W powyższy sposób pobrałem wszystkich userów. A co zrobić jeśli chcę pobrać wszystkich userów z jakim s warunkiem+ JOIN z inna tabelą