[php] MVC controller benaderen vanuit andere Controll

[Freemann]

Na alle feedback op mijn vorige topics ben ik overgestapt op een MVC pattern en probeer ik daar mijn weg in te vinden.

Nu ben ik al een eind gekomen om het principe van een MVC pattern te doorgronden, heb ik zelf een DB class toegevoegd en deze weten te benaderen (is misschien niet moeilijk voor andere maar voor mij is het allemaal nieuw).
Nu zit ik dat ik een 2de controller op heb gemaakt en deze werkt ook. Nu wil ik de outpot van deze controller gebruiken in een andere controller.

In de code heb ik een:

• router
• registry
• models
• views

nu heb ik 2 controllers;
indexController
kostenControler

In deze controllers zit zo ie zo een index functie en dan nog functies die je zelf aanmaakt.

Nu wil ik vanuit indexController de kostenController aanroepen maar dit lukt me niet.

Voor mijn gevoel zijn de registry.class en de router.class van belang voor mijn probleem en daarom
hieronder wat code en de foutmelding.

bij de volgende aanroep:
aanroep:
$registry = new registry;
$test = $registry->kostenController();

foutmelding:
Fatal error: Call to undefined method Registry::kostenController() in D:\.....\mvc_registraties\controller\indexController.php on line 233

aanroep:
$kosten = $this->registry->kostenController->index();

foutmelding:
Notice: Undefined index: kostenController in D:\......\mvc_registraties\application\registry.class.php on line 39

Fatal error: Call to a member function index() on a non-object in D:\......\mvc_registraties\controller\indexController.php on line 231

registry.php

PHP:

<?php

Class Registry {

 /*
 * @the vars array
 * @access private
 */
 private $vars = array();


 /**
 *
 * @set undefined vars
 *
 * @param string $index
 *
 * @param mixed $value
 *
 * @return void
 *
 */
 public function __set($index, $value)
 {
    $this->vars[$index] = $value;
 }

 /**
 *
 * @get variables
 *
 * @param mixed $index
 *
 * @return mixed
 *
 */
 public function __get($index)
 {
    return $this->vars[$index];
 }


}

?>

PHP:

<?php

class router {
 /*
 * @the registry
 */
 private $registry;

 /*
 * @the controller path
 */
 private $path;

 private $args = array();

 public $file;

 public $controller;

 public $action; 

 function __construct($registry) {
        $this->registry = $registry;
 }

 /**
 *
 * @set controller directory path
 *
 * @param string $path
 *
 * @return void
 *
 */
 function setPath($path) {

    /*** check if path i sa directory ***/
    if (is_dir($path) == false)
    {
        throw new Exception ('Invalid controller path: `' . $path . '`');
    }
    /*** set the path ***/
    $this->path = $path;
}


 /**
 *
 * @load the controller
 *
 * @access public
 *
 * @return void
 *
 */
 public function loader()
 {
    /*** check the route ***/
    $this->getController();

    /*** if the file is not there diaf ***/
    if (is_readable($this->file) == false)
    {
        $this->file = $this->path.'/error404.php';
                $this->controller = 'error404';
    }

    /*** include the controller ***/
    include $this->file;

    /*** a new controller class instance ***/
    $class = $this->controller . 'Controller';
    $controller = new $class($this->registry);

    /*** check if the action is callable ***/
    if (is_callable(array($controller, $this->action)) == false)
    {
        $action = 'index';
    }
    else
    {
        $action = $this->action;
    }
    /*** run the action ***/
    $controller->$action();
 }


 /**
 *
 * @get the controller
 *
 * @access private
 *
 * @return void
 *
 */
private function getController() {

    /*** get the route from the url ***/
    $route = (empty($_GET['rt'])) ? '' : $_GET['rt'];

    if (empty($route))
    {
        $route = 'index';
    }
    else
    {
        /*** get the parts of the route ***/
        $parts = explode('/', $route);
        $this->controller = $parts[0];
        if(isset( $parts[1]))
        {
            $this->action = $parts[1];
        }
    }

    if (empty($this->controller))
    {
        $this->controller = 'index';
    }

    /*** Get action ***/
    if (empty($this->action))
    {
        $this->action = 'index';
    }

    /*** set the file path ***/
    $this->file = $this->path .'/'. $this->controller . 'Controller.php';
}


}

?>

al met al snap ik niet hoe ik een andere controller aan kan roepen terwijl het wel lukt om de db class aan te roepen met de onderstaande code:
$results = $this->registry->db->query('select * from ......... ');

Wie kan mij verder helpen?

[Freemann]

niemand ?

[prototype]

Even kort:
Controllers dienen geen output te hebben. Ze dienen alleen de interactie af te handelen tussen eindgebruiker en de modellen. En dat brengt me meteen dan ook op 't volgende punt: volgens mij wil jij hier gewoon je model voor aanroepen als je kosten moet ophalen. In je controller wil je dan zoiets:

PHP:

class MijnController extends ApplicationController {
    //....

    public function index() {
        $this->kosten = Kosten::find('all');
        // of $this->Kosten->find('all'); of $this->registry('Kosten')->find('all); als late static binding je niet ActiveRecord zo laat implementeren.
    }
}

Tot slot, pak een bestaand PHP MVC framework erbij ipv "nog een rails/struts clone" maken ;-)

Voor al het andere:



En:

[ Voor 11% gewijzigd door prototype op 26-07-2009 18:38 ]

[sky-]

Class Registry ?

Moet dat niet class registry zijn.

Je router komt trouwens rechtstreeks uit een boek, titel is me even ontschoten. (Hmm: PHP 5 Power Programming misschien ? ;-)) Volgens het voorbeeld uit dat boek hoef je je controller niet te registreren in je register, je router roept je controller aan. In je register kan je dan wel weer Database classes, templating (voor je views) etc registreren.

[ Voor 13% gewijzigd door sky- op 26-07-2009 19:12 ]

[Freeaqingme]

Zoals Prototype al aangeeft wil je waarschijnlijk de logica die je nu aanroept in je ene controller in een model stoppen. Daarnaast hoef je dat model dan helemaal niet aan te roepen vanuit de controller, maar gewoon vanuit de view.

Een controller gebruik je om user input af te handelen. Als er geen userinput is kan je in principe je controller leeglaten

[Freemann]

sky- schreef op zondag 26 juli 2009 @ 19:12:
Je router komt trouwens rechtstreeks uit een boek, titel is me even ontschoten. (Hmm: PHP 5 Power Programming misschien ? ;-)) Volgens het voorbeeld uit dat boek hoef je je controller niet te registreren in je register, je router roept je controller aan. In je register kan je dan wel weer Database classes, templating (voor je views) etc registreren.

De source die ik gebruik komt van:
http://www.phpro.org/tuto...-View-Controller-MVC.html
en waar die hem vandaan hebben?! Dus zou zomaar kunnen dat die het uit een boek hebben

freakingme schreef op maandag 27 juli 2009 @ 02:55:
.....

Een controller gebruik je om user input af te handelen. Als er geen userinput is kan je in principe je controller leeglaten

Kijk dat is een leuke een nuttige opmerking!

Ben zoekende en heb nog weinig gevonden over wat je nu in de M,V,C, 's zet.Nu ben ik weer een stapje verder

Denk dat ik maar is in een kast ga duiken om te kijken of er nog een patterns boek ligt van mijn opleiding om te zien of dat me nu wel kan boeien :):$

[ Voor 49% gewijzigd door Freemann op 27-07-2009 08:07 ]