cargar modelos en un solo lugar

[destor77]

Hola,
en mis controladores actualmente tengo que crear una instancia del modelo en cada metodo que tengo que usar por ejemplo:

Código PHP:

class ejemploController extends Controller{
public function index(){
$modelo = new ClaseModelo();
$modelo->llamoaunmetodo();
}
public function otroMetodo(){
$modelo = new ClaseModelo();
$modelo->llamoaotrometodo();
}
 

y anda todo bien
Bien en mi clase controller tengo esto:

Código PHP:

//Incluimos los modelos        
 $this->Utilidades->recursiveInclude($config->get('modelsFolder'));
 

el codigo de recursiveInclue es :

Código PHP:

/**
     * funcion para cargar recursivamente clases.
     * 
     * @access  public
     * @param string $path
     */
    public function recursiveInclude($path) {
        if(!empty($path)){  
            $dir    = scandir($path);
            if ( (is_array($dir)) && (count($dir) > 0) )
            foreach($dir as $k => $v)
            if ( !(is_dir($path.$v)) && ( file_exists($path.$v) ) && ($v != ".")  && ($v != "..") && (strtolower(end(explode(".", $v))) == 'php') )
            include_once($path.$v);
            elseif ( is_dir($path.$v) && ($v != ".") && ($v != "..") && ($v != 'smarty') )
            recursiveInclude($path.$v."/");
        }
    }
 

que lo que hace es un include de todas las clases de un directorio este caso la carpeta modelos.

Bien lo que he intentado hacer (sin exito) es esto:

Código PHP:

class ejemploController extends Controller{
private $modelo;
        public function __construct(){
            $this->modelo = new ClaseModelo();
        }
 
public function index(){
$this->modelo->metodo;
}
}
 

pero cuando hago esto me sale este error:
Fatal error: Class 'ClaseModelo' not found in D:\xampp\htdocs\ampa\admin\controladores\ejemplo\e jemploController.php on line 14
la linea 14 es:

Código PHP:

$this->modelo = new ClaseModelo();
 

como puedo instanciar un modelo y que se pueda usar en todos los metodos de un controlador?
desde ya muchas gracias

[GatorV]

Si te sale ese error es porque no haz incluido la clase, seguramente ejecutas eso de recursiveInclude antes de crear el controlador, revisa que ejecutes el recursiveInclude después de instanciar el controlador.

Saludos.

[rodrigo_00]

deberias reemplazar el recursiveInclude por un autoload =)

http://php.net/manual/en/language.oop5.autoload.php

luego tambien podrias hacer algun tipo de factory a la que le pides el modelo, si no lo creo nunca te devuelve una nueva instancia y si ya lo habia creado te devuelve la instancia anteriormente creada

entonces en vez de hacer un new a cada rato para cada modelo puedes hacer directamente

$this->getModel('Usuarios')->update(...);

Saludos

[destor77]

ok muchas gracias por la data voy a investigar un poco mas sobre lo de factory, yo lo estaba implementando asi:

Código PHP:

abstract class Controller{
    
    //protected $Smarty;
    protected $Email;
    protected $Paginador;
    protected $Session;
    protected $Utilidades;
    protected $Imagen;
    protected $Upload;
    protected $Captcha;
    protected $BreadCrumb;
    protected $Vistas;
    protected $Mensajes;
    
 
    /**
     * Constructor
     * 
     * @access public
     * @version 0.1
     *
     */
    public function __construct()
    {   
        
        
        //creo una instancia de la configuración               
        $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 PHPMailer();
        $this->Email->IsSMTP();
        $this->Email->SMTPAuth   = true;                  // enable SMTP authentication
        $this->Email->Host       = $config->get('host');
        $this->Email->Username   = $config->get('usuario');
        $this->Email->Password   = $config->get('pass');
        $this->Email->Port       = $config->get('puerto');
 
        $this->Email->From       = $config->get('email');
        $this->Email->FromName   = $config->get('nombre');
                    
        //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();
              
        //Incluimos los modelos        
        $this->Utilidades->recursiveInclude($config->get('modelsFolder'));
        
        
        
        
 
    }
    
    
}
 

estaría correcto si el recursive lo pongo fuera del constructor como un metodo de la clase controller?

salu2

[rodrigo_00]

Si hablamos realmente de lo que este bien o este mal, el recursive include esta mal, en negrita y subrayado u.u

Por otro lado, si ponerlo como un metodo aparte o no, estarian bien ambas opciones, aunque lo mejor me parece llamarlo desde el constructor

Saludos

[GatorV]

Y el problema es ese, en tu constructor de tu controller es cuando haces el include, si quieres ejecutar el constructor del padre que heredas tienes que explicitamente llamarlo usando parent::__construct().

Saludos.

[destor77]

gracias a los dos por tanto conocimiento es que uno nunca termina de aprender todo jajaja, por no decir que uno es "bruto".
Por el momento y como decimos los argentinos "lo atamos con alambre", lo solucione asi:

Código PHP:

private $CategoriasNoticias;
        function __construct(){
            parent::__construct();
            $this->CategoriasNoticias = new CategoriasNoticias();
        }
        public function index(){
            $this->Utilidades->validar($this->Session->get('admin'));           
            $paginador = $this->Utilidades->paginador($this->CategoriasNoticias->listarCategoriasNoticias(),5);
            $data['paginador'] = $paginador;
            $data['breadCrumb'] = $this->BreadCrumb->listarBreadCrumb($_REQUEST);
            if($this->Session->get('mensaje')){
                $data['mensaje'] = $this->Session->get('mensaje');
                $this->Session->del('mensaje');
            }
            $this->Vistas->show('index.html',$data);
        }
 

no creo que sea la mejor solución pero por el momento hasta que tenga mas tiempo de investigar como lo hacen otros framework y modificarlo.
feliz año nuevo para los dos!!!!