PHP OO Conexion PDO con patron Registro

[ramonw]

Buenas amigos, he leído en el foro que la conexion a la base de datos es mejor hacerla con el patron Registry que con Singleton.
Acabo de terminar una conexion que es con pdo y utiliza una clase que es un patron registro, que obviamente "registra" la conexion, para poder utilizarla posteriormente en cualquier otra clase, aqui les dejo las clases y me gustaría que me corrijieran a ver si está bien implementado el uso del patron, o está faltando algo (mas alla de que funcione lo que me interesa es el modelado en general y el uso del patron).

//clase que hace la conexion a pdo

Código PHP:

class conectarpdo{
    
 
    private  $base;
    private  $usuario;
    private  $pass;
 
    function __construct(){
 
 
        $this->base="mibase";
        $this->usuario="miuser";
        $this->pass="mipassword";
 
    }
 
    public function conectar(){
 
        $conectar=new PDO('mysql:host=localhost;dbname='.$this->base,$this->usuario,$this->pass);
                 return $conectar;
 
    }
}
 

// la interface del patron registry y su implementacion en la clase a utilizar

Código PHP:

interface patron_registro{
 
public function agregar();
public function remover();
public function obtener();
public function existe();
 
}
 
class registro_conexion implements patron_registro {
 
    private $registro;
 
    public function  agregar() {
       if(!isset($this->registro))
       {
 
         $conn = new conectarpdo();
         $valor_conexion = $conn->conectar();
         $this->registro = $valor_conexion;
       }
    }
 
    public function  remover() {
        if(!empty($this->registro))
         {
           unset($this->registro);
         }
    }
 
    public function  obtener() {
 
       if(!empty($this->registro))
         {
           return $this->registro;
         }
    }
 
 
    public function  existe() {
        if(!empty($this->registro))
        {
          return true;
        }
        else{
          return false;
        }
   }
 
 
 
}
 

Su utilización:

Código PHP:

//dentro de un metodo de cualquier clase
 
$crearconn = new registro_conexion();
$existe = $crearconn->existe();
 
if(TRUE == $existe){
    $conectar = $crearconn->obtener();
}
else{
    $crearconn->agregar();
    $conectar = $crearconn->obtener();
}
 
//en $conectar tengo una conexion PDO y puedo utilizarla por ej. para hacer un select.
$consulta=$conectar->prepare('select * from paraobj where id =:id');
$consulta->bindParam(':id',$this->dato);
 
$consulta->execute();
 

Bueno, el que pueda echar una miradita a ver si esta bien implementado esto le agradezco.

[GatorV]

No, el patrón Registry, no se implementa así, te recomiendo veas ejemplos en internet, algo muy simple sería asi:

Código PHP:

class Registry {
    private $_cache;
    
    public function __construct() {
        $this->_cache = array();
    }
    public function set($key, &$item) {
        $this->_cache[$key] = &$item;
    }
    public function get($key) {
        return $this->_cache[$key];
    }
    public function has($key) {
        return ($this->get($key) !== null);
    }
}
 

Y lo usas como:

Código PHP:

$Registry = new Registry();
if (!$Registry->has('conexion')) {
      $Registry->set('conexion', $conexion);
}
// Mas adelante
$conexion = $Registry->get('conexion');
 

Saludos.

[Hidek1]

en la linea

Código PHP:

return ($this->getEntry($key) !== null);
 

es error de tipeo.. o me perdí en alguna parte ?

Código PHP:

return ($this->get($key) !== null);
 

saludos
tnks por el ejemplo !

[GatorV]

Cierto tienes toda la razón lo programé de rápido jeje

[ramonw]

Hola, gracias por la corrección.
Por lo que entiendo en tu ejemplo, la clase Registro "debe" recibir la conexion a la db por parametro y no como lo hice yo tipo "asociación" (por referirnos a un supuesto diagrama de clases), o sea creando el objeto dentro del metodo "agregar".

Estoy en lo cierto?

Otra cosa, en tu ejemplo en la linea 7, en el metodo "set" ingresas un parametro por referencia "&", tenía entendido que en php5 todos los parametros pasan automaticamente por valor... si no estoy en lo cierto que quiere decir el "&".
gracias.

[carlos_belisario]

el echo de usar el registry en este tipo de clases para el manejo de base de datos es que puedas instanciar diferentes manejadores en una misma pagina cuando sea necesario y el echo de no tener que instanciar varias veces la clase para realizar las consultas, yo haciendo consultas en este foro llegue a realizar una clase extendiendo de PDO si gustas en visualizarla saludos

[HerSAn]

Tambien puedes hacer uso de la clase SplObjectStorage de SPL( standar php library) que es un registry. no implementas y vas con el standard

http://www.php.net/manual/en/class.splobjectstorage.php

[masterpuppet]

Iniciado por HerSAn

Tambien puedes hacer uso de la clase SplObjectStorage de SPL( standar php library) que es un registry. no implementas y vas con el standard

http://www.php.net/manual/en/class.splobjectstorage.php

No creo que SplObjectStorage sea el mas adecuado para el trabajo, utiliza como clave objetos, preferible hacer como Zend y utilizar ArrayObject.

[HerSAn]

Iniciado por masterpuppet

No creo que SplObjectStorage sea el mas adecuado para el trabajo, utiliza como clave objetos, preferible hacer como Zend y utilizar ArrayObject.

Si tienes razon es mas bien un mapper pero bien puede usarse para hacer el registry pero se tiene que trabajar mas, mejor usar el componente de ZF lastima me gustaba la idea del SplObjectStorage. pero bueno se puede utlizar de otras formas igual con el oberver o composite. cosa de verlo

[ramonw]

Bueno he hecho las correcciones, ademas lo he problado y funciona perfectamente.

Código PHP:

//clase que hace la conexion
 
class conectarpdo{
 
 
    private  $base;
    private  $usuario;
    private  $pass;
 
    function __construct(){
 
 
        $this->base="mibase";
        $this->usuario="myuserl";
        $this->pass="mypassword";
 
    }
 
    public function conectar(){
 
        $conectar=new PDO('mysql:host=localhost;dbname='.$this->base,$this->usuario,$this->pass);
                 return $conectar;
 
    }
}
 

La clase Registry con algun cambio en el metodo has (me gusta mas asi)

Código PHP:

class Registry {
    private  $_cache;
 
    public function __construct() {
        $this->_cache = array();
    }
    public function set($key, $item) {
        $this->_cache[$key] = $item;
    }
    public function get($key) {
        return $this->_cache[$key];
    }
    public function has($key) {
        if(!empty($this->_cache[$key]))
        {
          return true;
        }
        else{
          return false;
        }
    }
}
 

Asi es su uso:

Código PHP:

$uno = new conectarpdo();
$conexion = $uno->conectar();
 
 
$Registry = new Registry();
if (!$Registry->has('conexion')) {
      $Registry->set('conexion', $conexion);
}
// Mas adelante
$conexion = $Registry->get('conexion');
 
$datito = 27;
//en $conexion tengo una conexion PDO y puedo utilizarla por ej. para hacer un select.
$consulta=$conexion->prepare('select * from administradores where id =:id'); 
$consulta->bindParam(':id',$datito);
 
$consulta->execute();
while($fila=$consulta->fetch()){
 
etc. etc.
 
//en la misma pagina otra 
$lamismaconexion = $Registry->get('conexion');
 
 
//en $lamismaconexion tengo ahora la conexion PDO y puedo utilizarla por ej. para hacer otro select.
 
$consulta=$lamismaconexion->prepare( etc. etc.
 

Bueno asi ha quedado, si ven algo a corregir por favor avisar.
Saludos.

[Dany_s]

¿en vez de preguntar si no existe entonces setear no es mejor en el método set verificar si existe?

o antes de crear la conexión verificar si existe entonces si existe no crear la conexión?

[Hidek1]

 
 

Código PHP:

$Registry = new Registry();
if (!$Registry->has('conexion')) {
      $uno = new conectarpdo();
      $conexion = $uno->conectar();
      $Registry->set('conexion', $conexion);
}
 

eso debería ir dentro para q tuviera sentido o me equivoco ?

saludos.

[carlos_belisario]

pues pienso que pudieras aprovechar un poco mas de las bondades que ofrece la POO, utilizando la herencia extendiendo tu clase conectarpdo directamente de PDO, ademas de en el constructor directamente instanciar a PDO, de esta manera te ahorrarias el estar colocando un metodo extra solo para instanciar a pdo utilizando tu codigo pudieramos hacer algo como esto:

Código PHP:

//clase que hace la conexion
 
class conectarpdo extends PDO
{ 
    private  $base;
    private  $usuario;
    private  $pass;
 
    function __construct()
   {
         $this->base="mibase";
        $this->usuario="myuserl";
        $this->pass="mypassword";
        parent::__construct('mysql:host=localhost;dbname='.$this->base,$this->usuario,$this->pass);
   }
}
 

de esta manera solo con instanciar la clase estarias creando ya la instancia de PDO ya que la misma extiende de esta, saludos espero te sirva