seleccionar controladores y vistas en MVC

[mayid]

Que opinan de este FrontController? Lo que hace, basicamente, es elegir un controlador y una vista. Sin embargo aún me falta una buena parte del proyecto y ya siento que el FrontController se hace demasiado largo:

Código PHP:

  class FrontController{     
 
    var $controller; 
    var $dbObject; 
    var $phpvars; 
    var $contentsVars; 
     
    function __construct(){ 
                 
        include "controllers/BaseController.php";  
        $baseController = new BaseController();  
        $baseController->defineConstants(); 
         
        include "models/Model.php";  
        $this->dbObject = new dbConsults(); 
         
         
        /* Translations for the UI */ 
        $this->phpvars = $baseController->varsConsult($this->dbObject);  
                 
        // utf-8 fixer 
        include (APP_LIB."fixEncoding.php");  
         
        $this->includeControllers(); 
         
        /*** init View ****/ 
        include "views/View.php";             
        $whichView = $this->chooseView(); //$whichView = $this->chooseView(); // template name     
        $View = new View($whichView); // display view 
        $View->prepareContents($this->phpvars, $this->contentsVars); 
        $View->render(); 
    }     
     
    private function chooseView(){ 
         
        if ( isset($_GET['login']) )         
            $response = TEMPLATES."login.tpl.php"; 
         
        elseif ( isset($_GET['register']) )         
            $response = TEMPLATES."registration.tpl.php"; 
         
        elseif ( isset($_GET['lostpassword']) )         
            $response = TEMPLATES."lostpassword.tpl.php"; 
             
        elseif(isset($download))             
            $response = TEMPLATES."contents.download.tpl.php"; 
         
        elseif(!isset($_SESSION['name']))         
            $response = TEMPLATES."contents.NoLogued.tpl.php"; 
         
        elseif(isset($_SESSION['name']))         
            $response = TEMPLATES."contents.Logued.tpl.php";     
             
        return $response;     
    } 
 
    private function includeControllers() 
    { 
         
        if (isset($_GET['login'])) 
        {     
         
            include "controllers/LoginController.php";  
            $LoginController = new LoginController($this->dbObject); 
            $this->contentsVars['usersData'] = $LoginController->checkUser();                     
        }     
        elseif (isset($_GET['register'])) 
        {     
         
            include "controllers/RegistrationController.php";  
            $RegistrationController = new RegistrationController($this->dbObject); 
            $this->contentsVars['usersData'] = $RegistrationController->registerUser();     
        } 
        elseif (isset($_GET['logout'])) 
        {     
         
            include "controllers/LoginController.php";  
            $LoginController = new LoginController($this->dbObject); 
            $LoginController->logout(); 
        }     
        elseif (isset($_GET['lostpassword'])) 
        {     
         
            include "controllers/RememberPassController.php";  
            $RememberPassController = new RememberPassController($this->dbObject); 
            $this->contentsVars['usersData'] = $RememberPassController->rememberPassword(); 
        } 
        elseif (!isset($_GET['load']) and (FILENAME == "index")) 
        { 
             
            include "controllers/MainPageController.php";  
            $MainPageController = new MainPageController(); 
            $this->contentsVars['tutorialsList'] = $MainPageController->listTutorials($this->phpvars, $this->dbObject);                         
        } 
        else{ 
         
            include "controllers/PageController.php";  
            $PageController = new PageController($this->dbObject); 
            $this->contentsVars['theContents'] = $PageController->printPageContents();     
        } 
         
        include "controllers/MenusController.php";  
        $MenusController = new MenusController($this->dbObject); 
        $this->contentsVars['mainLinks'] = $MenusController->printMainMenu( $this->phpvars ); 
        @$this->contentsVars['relPages']  = $MenusController->printSecondaryMenu( $contentsVars['theContents']['relatedPages'] ); 
         
        include "controllers/TagsController.php";  
        $TagsController = new TagsController($this->dbObject); 
        $this->contentsVars['theTags'] = $TagsController->tag_cloud(); 
    }     
     
}  
   

Además, tengo ya muuchas plantillas y una sola vista. Así que creo que estoy equivocando los conceptos en algún punto.

[destor77]

creo que estas mezclando algunas cosas, el frontcontroller solo debería encargarse de saber a que controlador tiene que llamar, este es el frontcontroller que uso en mi framework:

Código PHP:

/**
     * @package Punk Framework
     * @copyright Copyright (C) 2009 Onírico Sistemas. Todos los derechos reservados.
     * @version 0.1
     * @author Lucas M. Sastre
     * @link http://www.oniricosistemas.com
     *
     * @name FrontController.php
     */
    /**
     * Modificaciones realizadas
     *
     * 28/02/2010
     * - se modificó el código, ahora valida si la aplicación esta siendo ejecutada
     * en el servidor del cliente o en un servidor local, y a partir de esa
     * validación setear las ruta_absolutas de los directorios de la apliación.
     *
     * 07/03/2010
     * - se agrego un ruta_absoluta relativa para el directorio de las imagenes.
     *
     * 23/03/2010
     * - Se agrego el redireccionamiento error404.
     * - Se agrego el guardado de errores en un archivo errores.log
     *
     * 30/03/2010
     * - se agrego al método loader que evite los directorios .svn
     *
     * 02/04/2010
     * - Se modifico en el FrontController.php los include y la validación para el idioma.
     */
 
    class Router {
    private static $instance;
    private $servidor;
    private $ubicacion;
    private $ruta;
    private $ruta_absoluta;
    private $path;
 
    /**
     * Constructor
     *
     * @name _construct
     * @param $registry
     * @access public
     * @version 0.1
     * @author Lucas M. Sastre
     *
     *
     */
    function __construct($param) {
        $this->servidor = $_SERVER['HTTP_HOST'];
        $ubicacion=explode('/',$_SERVER['PHP_SELF']);
        array_pop($ubicacion);
        $this->ubicacion = $ubicacion;
        $this->path = implode('/',$this->ubicacion);
        if(array_search('admin',$this->ubicacion)) {
        array_pop($this->ubicacion);
        $ruta = implode('/',$this->ubicacion);
        $this->ruta_absoluta = $ruta.'/' ;
        }
        else {
        $ruta = implode('/',$this->ubicacion);
        $this->ruta_absoluta = $ruta.'/' ;
        }
        $this->ruta = $_SERVER['DOCUMENT_ROOT'].$ruta.'/';
    }
 
    /**
     *  router
     * @name router
     * @access public
     * @version 0.1
     * @author Lucas M. Sastre
     */
    public function route() {
        include_once($this->ruta.'configuracion.php');
        $config->set('root',$this->ruta);
        //configuro la ruta_absoluta de las imagenes
        $config->set('imagenes',$config->get('root').'images/');
        $config->set('urlImagenes','http://'.$this->servidor.$this->ruta_absoluta.'images/');
 
        //seteo el path
        $path=$config->get('root');
 
        //valido si no estoy en el root y seteo el nuevo path
        if(strpos($this->path,'admin')) {
        $path=$config->get('root').'admin';
        //seteo la ruta_absoluta url
        $config->set('urlRoot','http://'.$this->servidor.$this->ruta_absoluta);
        }
        else {
        $path=$config->get('root').'website';
        //seteo la ruta_absoluta url
        $config->set('urlRoot','http://'.$this->servidor.$this->ruta_absoluta);
        }
 
        //seteo el path definitivo
        $config->set('path',$path);
 
        //cargo todas las clases
        $this->loader($config->get('root').$config->get('core'));
 
        //obtengo el controlador que se envia por $_GET
        $controller = $_GET['controlador'];
 
        //obtengo el controlador que se envia por $_GET
        if(isset($_GET['accion'])) {
        $action = $_GET['accion'];
        }
 
 
        //seteo el controlador si esta vacio seteo el controlador por default
        if( empty($controller) ) {
        $controller = 'indexController';
        }
        else {
        $controller = $controller.'Controller';
        }
 
        //unset($_COOKIE['leng']);
        //cargo el lenguaje del sistema
        if($config->get('multi')==1) {
        $leng = Language::singleton();
        $leng->get_session_handler();
        $idiomaActual = $config->get('lenguaje');
        setcookie ("leng", 'es', time () + 7*24*60*60);
        session_start();
        //echo "<pre>";print_r ($_COOKIE['leng']);echo "</pre>";exit();
 
        $_SESSION['leng'] = $idiomaActual;
 
        if(isset($_GET['leng'])) {
            setcookie ("leng", $_GET['leng'], time () + 7*24*60*60);
            $idiomaActual = $_GET['leng'];
            $_SESSION['leng'] = $_GET['leng'];
        }
        elseif(isset($_COOKIE['leng'])) {
            if(file_exists($config->get('root')."/lenguajes/".$_COOKIE['leng'].".php")) {
            $idiomaActual = $_COOKIE['leng'];
            $_SESSION['leng'] = $_COOKIE['leng'];
            }
        }
        else {
            if(file_exists($config->get('root')."/lenguajes/".$_SESSION['leng'].".php")) {
            $idiomaActual = $_SESSION['leng'];
 
            }
        }
 
        // Incluimos el archivo del idioma seleccionado
        // o el archivo por defecto si no se seleccionó
        // idioma o si no se encuentra el archivo
        include $config->get('root')."/lenguajes/".$idiomaActual.".php";
        }
 
        //extraigo el nombre del controlador.
        $nombre=explode('Controller',$controller);
 
        //seteo la accion, si esta vacia seteo la acción por default
        if( empty($action) ) {
        $action = 'index';
        }
        else {
        $action = $action ;
        }
 
        //seteo la ruta_absoluta del controlador
 
        $controllerLocation =  $config->get('path') . '/controladores/' . $nombre[0] . '/'.$controller.'.php';
        $vista=$config->get('path') . '/controladores/' . $nombre[0] . '/templates/';
        $config->set('vista',$vista);
 
        //creo una instancia de los errores
        //$errores= Errores::singleton();
 
        //valido si existe el archivo sino ejecuta la excepcion
        if( file_exists( $controllerLocation ) ) {
        include_once( $controllerLocation );
        } else {
        //trigger_error("No se encuentra el controlador $controllerLocation<br/>",E_USER_NOTICE);
        error_log("[".date("F j, Y, G:i")."] [E_USER_NOTICE] [tipo Controlador] No se encuentra el controlador: {$controllerLocation}\n", 3,$config->get('root').'/errores.log');
        header("Location:index.php?controlador=error404");
        }
 
        //valido si existe la clase sino ejecuta la excepcion
        if( class_exists( $controller, false ) ) {
        $cont = new $controller();
        } else {
        //trigger_error( "No se encuentra la clase en el controlador $controller en $controllerLocation<br/>",E_USER_NOTICE);
        error_log("[".date("F j, Y, G:i")."] [E_USER_NOTICE] [tipo Clase] No se encuentra la clase en el controlador $controller en $controllerLocation \n", 3,$config->get('root').'/errores.log');
        header("Location:index.php?controlador=error404");
        }
 
        //valido si existe el método sino ejecuta la excepcion
        if( method_exists( $cont, $action ) ) {
        $cont->$action();
        } else {
        //trigger_error( "La Acción $action no existe $action en la clase $controller en $controllerLocation<br/>",E_USER_NOTICE);
        error_log("[".date("F j, Y, G:i")."] [E_USER_NOTICE] [tipo Accion] La Acción $action no existe $action en la clase $controller en $controllerLocation \n", 3,$config->get('root').'/errores.log');
        header("Location:index.php?controlador=error404");
        }
 
 
 
    }
    /**
     * Cargador de clases
     *
     * @name loader
     * @access public
     *  @version 0.2
     *  @since 0.1
     *  @author Lucas M. Sastre
     */
    public function loader() {
        //separo los parametros de la url
        /*$ubicacion=explode('/',$path);
        $this->servidor=$_SERVER['HTTP_HOST'];
 
        //creo el path para las clases
        if($this->servidor!='localhost' && $this->servidor!='192.168.1.105' && $this->servidor!='127.0.0.1:4001'){
            $path=$_SERVER['DOCUMENT_ROOT'].'/core/clases/';
        }
        else{
            $path=$_SERVER['DOCUMENT_ROOT']."/".$ubicacion[1].'/core/clases/';
        }*/
 
        $path = str_replace('\\','/',dirname(__FILE__)).'/';
        //echo "<pre>";print_r ($path);echo "</pre>";exit();
        //echo "<pre>";print_r (dirname(__FILE__));echo "</pre>";exit();
 
        //leo el directorio
        $dir = scandir($path);
        //valido si es un array y que tenga algun contenido
        if ((is_array($dir)) && (count($dir) > 0))
        foreach ($dir as $k=>$v)
        //recorro el directorio y valido que sea un archivo php
            if ( !(is_dir($path.$v)) && ( file_exists($path.$v) ) && ($v != ".") && ($v != "FrontController.php") && ($v != "..") && (strtolower(end(explode(".", $v))) == 'php') )
            //incluyo el archivo
            include_once($path.$v);
            elseif ( is_dir($path.$v) && ($v != ".") && ($v != "..") && ($v != 'smarty') && ($v != '.svn')&& ($v != 'ajax'))
            loader($path.$v."/");
    }
    }
 

si te fijas en ningun momento instancio la vistas u otras clases, solamente hago la instancia del controlador a llamar.

Después lo que hago es crear una clase abstracta para que todos los contoladores hereden de ella, y una clase vista para el manejo de las vistas (obvio no jejeje) cuya instancia la hago en la clase abastracta para que este disponible para todos los controladores.

[mayid]

Veo. Gracias por publicarlo.

Lo que estas haciendo con el constructor y con el metodo route() es definir rutas locales y absolutas para distintas carpetas. Yo a eso lo hago con define() para que esten disponibles en todas las clases. Pero veo que vos usas metodos magicos. Esos metodos son tácitos y no hay que redeclararlos? Van a estar presentes dentro de las demás clases o hay que pasarlos como parámetros?

Luego seteas un controlador dependiendo de una variable get:
$controller = $_GET['controlador'];

Mismo con una acción:
$action = $_GET['accion'];

A eso lo he visto en el framework Zend. Y quisiera aprender a implementarlo.

Más adelante definis idioma en session y en cookie. E incluís una pagina de idioma que supongo que contiene variables especificas para el idioma.

Y finalmente hay un método loader() que supongo que carga las clases automáticamente en vez de hacer includes para cada una?.

En fin, lo que me haría falta es definir un Controller abstracto, e implementarlo con varios otros controladores que serían llamados por una variable get (que siempre debe estar definida, no?).

También debería aprender a ligar los controladores con las vistas. En tu FrontController vos tenés esto:

$vista=$config->get('path') . '/controladores/' . $nombre[0] . '/templates/';
$config->set('vista',$vista);

Pero no estás instanciando la vista aún, no?

[GatorV]

Recuerden que uno de los principios básicos de la programación orientada a objetos es el correcto diseño de sus clases y el de delegar responsabilidades.

Me refiero que intenten hacer sus clases en componentes independientes, que se ocupen a la tarea específica.

El FrontController, inicia y llama al controlador, el Router rutea las rutas correctas desde la URL, la vista renderea, el controller realiza las acciones etc.

Si hacen una clase todo en uno luego es más complicado el mantenimiento, recuerden la clave en el buen diseño, es la abstracción y la delegación de tareas.

Saludos.

[mayid]

Bueno, este es el FrontController según lo que pude concebir hasta ahora:

Código PHP:

  class FrontController{     
 
    var $controller; 
     
    function __construct(){ 
         
        $this->defineConstants(); // url and path constants 
        $this->callController();         
    }     
     
    public function defineConstants(){  
         
        // If moving to another server, we must update this! 
        if (strpos($_SERVER["HTTP_HOST"], "localhost") !== false){ 
            define('APP', 'http://localhost/AoI/AoITutorials/application/'); 
            define('WEB', 'http://localhost/AoI/AoITutorials/web/'); // localhost for development 
        } 
        else {     
            define('APP', 'http://www.artofillusion.com.ar/application/'); 
            define('WEB', 'http://www.artofillusion.com.ar/AoITutorials/'); 
        } 
         
        // fix for domain and subdomain issue: aoi-castellano / artofillusion.com.ar 
        if (strpos($_SERVER["HTTP_HOST"], "castellano") !== false){ 
            header("Location: ".WEB);  // redirect. 
        }; 
         
 
        define('WEB_REL_PATH', $_SERVER['DOCUMENT_ROOT']."AoI/AoITutorials/"."web/");         
        define('APP_REL_PATH', $_SERVER['DOCUMENT_ROOT']."AoI/AoITutorials/"."application/"); 
        define('APP_LIB', APP_REL_PATH.'Library/');     
 
        // html <head> related constants (absolute path) 
        define('LIBRARY_DIR', 'Library/'); 
        define('LINK_CSS', 'Library/styles/'); 
        define('IMG_DIR', 'Library/images/'); 
                 
        // html <body> related constants (for including ui images, etc)                 
        define('CONTROLLERS', APP_REL_PATH.'controllers/'); 
        define('TEMPLATES', APP_REL_PATH.'templates/');             
        define('LIBRARY', WEB_REL_PATH.'Library/');  
        define('CSS_DIR', LIBRARY.'styles/'); 
     
    } 
     
    private function callController(){ 
         
        // get controller 
        if(! empty($_GET['controller'])) 
              $controllerName = $_GET['controller'] . 'Controller'; 
        else 
              $controllerName = "MainPageController"; 
         
        // get action 
        if(! empty($_GET['action'])) 
            $actionName = $_GET['action']; 
        else 
            $actionName = "index"; 
     
        // check controller existence     
        if(is_file(CONTROLLERS.$controllerName.".php")) 
            require CONTROLLERS.$controllerName.".php"; 
        else 
            die('Unexisting page - Go to index?'); 
         
        // check action existence     
        if (is_callable(array($controllerName, $actionName)) == false) 
        { 
             die('Unexisting action - Go to index?'); 
        } 
         
        // instantiate the controller 
        $controller = new $controllerName(); 
        $controller->$actionName(); 
        $controller->callView();     
    } 
 
}  
   

[destor77]

Iniciado por mayid

Veo. Gracias por publicarlo.

Lo que estas haciendo con el constructor y con el metodo route() es definir rutas locales y absolutas para distintas carpetas. Yo a eso lo hago con define() para que esten disponibles en todas las clases. Pero veo que vos usas metodos magicos. Esos metodos son tácitos y no hay que redeclararlos? Van a estar presentes dentro de las demás clases o hay que pasarlos como parámetros?

esos metodos los uso una sola vez, va en realidad se usa cada vez que se llama a un controlador, es decir en mi index.php tengo esto:

Código PHP:

ob_start();
    require 'core/clases/FrontController.php';
    
    $router = new Router($_SERVER['PHP_SELF']);
    $router->loader();
    $router->route();
    ob_flush();
 

que lo que hace es instanciar el frontcontroller, llamar al cargador de clases y despues ejecutar el ruteo (esto no quiere decir que este bien, si ves los index.php de otros framework pueden tener mucho mas código, pero eso va en cada uno). También esta armado de esta manera, porque en mi framework puedo tener multiple aplicaciones que comparten el mismo core.

Iniciado por mayid

Luego seteas un controlador dependiendo de una variable get:
$controller = $_GET['controlador'];

Mismo con una acción:
$action = $_GET['accion'];

A eso lo he visto en el framework Zend. Y quisiera aprender a implementarlo.

Algunos lo manejamos asi, otros usan $_REQUEST y otro hacen una "limpiado" de los $_get, $_post, etc.

Iniciado por mayid

Más adelante definis idioma en session y en cookie. E incluís una pagina de idioma que supongo que contiene variables especificas para el idioma.

Exacto si en la configuración seteo que el sitio es multilenguaje incluyo el idioma seteo la cookie y la session.

Iniciado por mayid

Y finalmente hay un método loader() que supongo que carga las clases automáticamente en vez de hacer includes para cada una?.

En fin, lo que me haría falta es definir un Controller abstracto, e implementarlo con varios otros controladores que serían llamados por una variable get (que siempre debe estar definida, no?).

Exacto con el loader(); "incluyo" todos los archivos de x carpeta para no tener que hacer un include por cada uno, te imaginas si tenes que hacer 50 includes a mano.... En cuanto al controller abstracto yo lo uso así (no quiere decir que sea la mejor forma... antes que me rete gatorv jejeje)

Código PHP:

/**
     * @package Punk Framework
     * @copyright Copyright (C) 2009 Onírico Sistemas. Todos los derechos reservados.
     * @version 0.1
     * @author Lucas M. Sastre
     * @link http://www.oniricosistemas.com
     *
     * @name Control.php
     *
     * Modificaciones
     * 14/02/10 - Se agrego la clase email para enviar correos electronicos.
     */
 
    abstract class Controller {
 
    protected $Config;
    protected $Email;
    protected $Paginador;
    protected $Session;
    protected $Utilidades;
    protected $Imagen;
    protected $Upload;
    protected $Captcha;
    protected $BreadCrumb;
    protected $Vistas;
    protected $Mensajes;
    protected $Debug;
    protected $Calendario;
    protected $PermisosUsuarios;
    protected $Log;
    protected $Fechas;
    protected $Librerias = array();
 
    
 
    /**
     * Constructor
     *
     * @access public
     * @version 0.1
     *
     */
    public function __construct() {
 
 
        //creo una instancia de la configuración
        $this->Config = Config::singleton();
 
        //creo la instancia para las sesiones.
        $this->Session  = new Session();
        $this->Session->Session();
 
        //creo las instancias para el email
        $this->Email = new Email();
 
        //creo las instancias para las utilidades varias
        $this->Utilidades   = new Utilidades();
 
        //creo la instancia para el captcha
        $this->Captcha = new Captcha();
 
 
        //creo la instancia para las imagenes
        $this->Imagen   = new SimpleImage();
 
        //creo la instancia para el upload
        $this->Upload   = new file_upload();
 
        //creo la instancia para el breadcrumb
        $this->BreadCrumb   = new breadCrumbs();
 
        //creo la instancia de la vista
        $this->Vistas   = new Vistas();
 
        //creo la instancia de los mensajes
        $this->Mensajes = new Mensajes();
 
        //creo la instancia del calendario
        $this->Calendario = new calendar();
 
        //creo la instancia de los permisos
        $this->PermisosUsuarios = new PermisosUsuarios();
 
        //crea la instancia para el log
        $this->Log = new Log();
 
        //crea la instancia para las fechas
        $this->Fechas = new Fechas();
 
        //creo la instancia del FirePHP
        if($this->Config->get('entorno')!='pro') {
        $this->Debug = FirePHP::getInstance(true);
        }
        else {
        FirePHP::getEnabled(false);
        }
 
        //Incluimos los modelos
        $this->Utilidades->recursiveInclude($this->Config->get('modelsFolder'));
 
    }
 

Iniciado por mayid

También debería aprender a ligar los controladores con las vistas. En tu FrontController vos tenés esto:

$vista=$config->get('path') . '/controladores/' . $nombre[0] . '/templates/';
$config->set('vista',$vista);

Pero no estás instanciando la vista aún, no?

Exacto lo que estoy haciendo ahí es setear el atributo 'vista' con la ruta donde estan los html, ya que mi estructura de directorio es:

admin
--------controladores
-------------nombrecontrolador
-------------------templates
-----------------------index.html
-----------------------addUpdate.html
-------------------nombrecontrolador.php

La vista la instancio en el controlador abstracto.
Espero te sirva de algo... salu2

[mayid]

Me sirve, claro.

No entiendo bien tu clase abstracta. Ya teniendo un FrontController, yo lo que estoy haciendo ahora es definir un BasicController que:
- maneja cadenas en utf-8
- setea un estilo para la pagina
- consulta las variables de idioma
- llama a una vista que se le pasa por parametro.

Luego, genero al menos dos ramas que heredan de ese controlador. Una para mostrar las pagina, y otra para gestionar el login. Tendría que hacer algunas otras ramas para gestionar otras situaciones.

Mi inquietud es que cantidad de controladores y vistas debiera escribir. Ya tengo unos 10 controllers solo para mostrar las paginas (contenidos+menues+tagCloud), y hacer el login. Por ejemplo, para el login tengo:

-registrationController.php
-loginController.php
-lostPasswordController.php

Esos tres van asociados a una vista (loginView), y usan unas cuatro plantillas.

Me falta convertir una buena parte de mi projecto a MVC y no se bien si hacer pocos controladores superpoderosos, o varios con tareas puntuales. Por ejemplo, para controlar uploads, generar paginas nuevas, mostrar un panel de administrador, una seccion de notas para editores, etc, etc.

Quizás lo que me pasa es que aún no se usar las acciones, y que con eso se me compactaría el numero de controladores a escribir. Por ahora, todos los controladores tienen una acción index por default, que es llamada si no se pasa un parametro $_GET en especial.

Además, viendo tu codigo, que solo busca un controlador y una accion, no se bien como adaptar mi proyecto porque yo usaba la url para cambiar de lenguage también, y de ámbito. Mismo, necesitaría la url para hacer loads de diferentes paginas. Por eso en mi primera publicación hacía mucho esto:

if (isset($_GET['login']))
{

}
elseif (isset($_GET['register']))
{
}
elseif (isset($_GET['logout']))
{

}
elseif (isset($_GET['lostpassword']))
{

}
elseif (!isset($_GET['load']))
...

Vuelvo a decir que ya descarté ese método y que estoy implementando el uso de variables get que se corresponden con controladores y acciones.

En fin, gracias!

[_ssx]

Creo que te salia mucho mejor usar Zend Framework o parecidos para que veas como aplican varios patrones de diseño en su MVC tales como

"Two step view" y "Composite view"

http://martinfowler.com/eaaCatalog/twoStepView.html

http://java.sun.com/blueprints/corej...ositeView.html

Para resolver varios de los problemas que mencionas aquí, yo se que deseas aprender pero creo que el problema de aprender así es que empiezas por el lado equivocado cuando en realidad necesites hacer cambios en tu código te costara mucho trabajo ya que todo lo construiste sin una planeación mas robusta del código.

Yo por ejemplo vi sus ruteadores y sus Front controllers de Code Igniter, Cake PHP, Solar y Zend Framework y me gusto mucho la elegancia con la cual lo hace Zend y despues aplicar los conceptos a tu MVC es sencillo pero si no entiendes correctamente la delegación de funciones a objetos que les conciernen dificilmente vas a lograr algo más que mal código y tu intento por migrar tu aplicación a MVC sera meramente un "intento lleno de trucos parches y mañas"

[mayid]

Gracias por comentar. Si, temo esto de andar parchando y arreglando.

También es un problema avanzar "sin una planeación robusta". Y sin embargo, no puedo planear hasta no comprender del todo el modelo MVC y algún framework. Es por eso que la migración me sirve como campo de juego en donde ir resolviendo y comprendiendo la complejidad del patron.

Gracias por los links. Curiosamente son los mismos que dan en la quickstart de Zend. Que estoy estudiando desde hace algunos días:
http://framework.zend.com/manual/en/...te-layout.html

Puedo decir que tengo a Zend como tutor. Pero todavía no lo conozco enteramente. Y mi problema principal es migrar.