Duda con clase para la DB(singleton).

[RIVERMILLOS]

buenas, tengo una clase que gestiona la base de datos(singleton) , Mi duda es la siguiente. tengo algo como esto.

Código PHP:

  
<?php 
 
class clase{ 
 
function __construct(){ 
 
$DB=$this->db::getInstance(); 
 
} 
 
public function llamar(){ 
 
$claseX = new ClaseX(); 
 
} 
 
} 
 
 
?>   

Mi duda es la siguenta , como hago para que en la ClaseX pueda utilizar la instancia de la base de datos sin tener que crearla dentro de esta misma.


saludos

[GatorV]

Guardalo en una propiedad de tu clase y se la pasas:

Código PHP:

class Foo
{
        private $_bar;
 
        public function __construct()
        {
                 $this->_bar = Something::getInstance();
        }
 
        public function other()
        {
                $obj = new Baz($this->_bar);
        }
}
 

Saludos.

[RIVERMILLOS]

Si había pensado algo así , pero pensé que no era buena idea.

saludos

[masterpuppet]

Lo que no es una buena idea es un Singleton para la bbdd, si googleas vas a encontrar mucha info al respecto.

Saludos.

[RIVERMILLOS]

ok , según lo que leí es mejor manejar un patrón registry ?.


saludos

[GatorV]

Así es.

Saludos.

[RIVERMILLOS]

Buscando ejemplo me tope con este , esta bien si lo sigo? o tiene alguna sugerencia de algun ejemplo complete de este patrón?

http://desphpparatodos.blogspot.com/...do-pdo-el.html


saludos

[GatorV]

ese ejemplo lo vi aqui mismo en el foro, pero sí, básicamente es algo así, un registro donde guardes objetos.

Saludos.

[abimaelrc]

Lo que no me gusto es que en uno de los metodos te añade el &. Cuando en realidad ya no es necesario. Por lo demas hay mas ejemplos en internet, por si quieres seguir aprendiendo y aqui en el foro, busca tambien los que estan en ingles.

[RIVERMILLOS]

Muchas gracias voy a seguir buscando mas ejemplos.

Pero primero quisiera comentarles como utilizo actualmente las clase singleton, para ver si al cambiarme al registry si sea de utilidad o no, yo actualmente programo en php , pero hago servicios para ser utilizados con amfphp , entonces tengo mis clases(servicios) , entonces en cada servicio tengo en el constructor de la clase el llamado al getInstance() del la clase DB y la utilizo segun el metodo que sea llamado. En este modelo con amfphp tendria algun inconveniente con registry.?


saludos

[GatorV]

No, simplemente reemplazas tu getInstance por Registry::get('db'); o algo similar.

[RIVERMILLOS]

Listo , ya estoy entendiendo mas o menos como funciona , Pero ahora tengo una duda como hago para que el archivo de configuracion .ini no sea visble por el navegador?

slaudos

[GatorV]

Pues simplemente dejalo fuera de la ruta documentroot y ya.

Saludos.

[RIVERMILLOS]

Ok era una pregunta tonta.

Mira estos son una parte de los archivos que estoy usando en los ejemplos de registry.

Código PHP:

<?php
error_reporting ( E_ALL );
ini_set("display_errors" , 1 );
 
include("Factory.php");
include("Registry.class.php");
include("Reader.class.php");
include("Db.class.php");
 
class DatabaseFactory implements Factory
{
    public static function create($sIniFile)
    {
        $config  = new Reader($sIniFile);        
        $config_data = $config->getConfig();
        $database_class = key($config_data);
        include($database_class.'.class.php');
        $db = new $database_class($config);
        registry::add($sIniFile, $db);       
        return $db;
    }
}
?>
 

Código PHP:

<?php
 
final class Mysql extends PDO
{
    private $_username;
    private $_password;
    private $_host;
    private $_db;
 
    public function __construct(Reader $config)
    {
        $config_data = $config->getConfig();
        $this->_username = $config_data['Mysql']['username'];
        $this->_password = $config_data['Mysql']['password'];
        $this->_host     = $config_data['Mysql']['host'];
        $this->_db       = $config_data['Mysql']['dbname'];
        $this->_dsn      = 'mysql:dbname='.$this->_db.';host='.$this->_host;
        parent::__construct($this->_dsn, $this->_username, $this->_password);     
    }
    
}
?>
 

Código PHP:

<?php
 
/*
 * Clase encargada de gestionar las conexiones a la base de datos
 * @author Esteban Quintero
 */
Class Db{
 
    /*establese la conexion*/
    protected $_conect;
    /*trabaja los query*/
    protected $_query;
    /*resultados de las consultas*/
    protected $_row;
    protected $_result; 
 
/**
 * La función construct es privada para evitar que el objeto pueda ser creado mediante new
 **/
function __construct(){
   
    try{
    
        $site_path = '../../includes/DB/config.ini';
        $this->_conect=DatabaseFactory::create($site_path);  
 
    }  catch (Exception $e){
       
        
        
    }
}
/**
 * Método para ejecutar una sentencia sql
 * @param String $sql sentencia sql para ejecutar en la BD. 
 *
 **/
public function ejecutar($sql){
   
  try{
       
       $this->sql=$sql;
       
       $this->resulset = $this->_conect->query($sql);
       
       if(!$this->resulset){
           
           error_log("error al ejecutar la consulta ".$sql." funcion ejecutar ",3,"errorlog/Db_log.log");    
       
           
       }
       
   }catch(Exception $e){
        error_log("error al ejecutar la consulta ".$sql." funcion ejecutar ".$e->getMessage(),3,"errorlog/Db_log.log");
   }
   return $this->resulset;
}
/**
 *Método para obtener una fila de resultados de la sentencia sql
 *@param String $resulset resulset del resultado de la sentencia.
 */
public function obtener_fila($resulset){
try{
   if(!$resulset){
     $this->array=array();
   }else{
     $this->array=$resulset->fetchObject();
   }
}catch(Exception $e){
    error_log($e->getMessage()." Clase Db Funcion obtener_fila parametro".$resulset,3,"errorlog/user_log.log");
}
return $this->array;
}
/*
 *Método para obtener las filas de resultados de la sentencia sql
 *@param String $resulset resulset del resultado de la sentencia.
 */
public function obtener_filas($resulset){
    if(!$resulset){
        error_log("Error en la consulta sql :".$this->sql." funcion obtener_filas",3,"errorlog/Db_log.log");
    }else{
    $this->array=$resulset->fetchAll();
    }
return $this->array;
}
/**
 *Método para obtener el numer de  filas del resultado de la sentencia sql
 *@param String $resulset resulset del resultado de la sentencia.
 */
public function numero_filas($resulset){
 try{
     if($resulset){
        
         return $resulset->rowCount();
     
     }else{
        
         return false;
     
     }
 }catch(Exception $e){
   error_log($e->getMessage()." Error al contar los datos sql : ".$this->sql."funcion numero_filas -> @param ".$resulset,3,"errorlog/Db_log.log");
 }
}
/**
 *Inicia una trasaccion
 **/
public function startTransaction(){
  $this->_conect->beginTransaction();
}
/**
 * Confirma una trasaccion
 **/
public function commit(){
   $this->_conect->commit();
}
/**
 * Elimina una trasaccion
 **/
public function rollBack(){
  $this->_conect->rollBack();
}
/**
 *Ejecuta una trasaccion
 *@param String $sql sentencia a ejecutar en le servidor.
 */
public function exec($sql){
 try{
  $this->_conect->exec($sql);
 }catch (Exception $e){
  error_log("Error al ejecutar sql :".$sql."funcion exec",3,"errorlog/Db_log.log");
 }
}
/**
 *retorna el ultimo id ingresado.
 **/
public function lastInsertId(){
return $this->_conect->lastInsertId();
}
/**
 * Prepara un a sentencia SQL
 */
public function prepare($sql){
    
 $stmt = $this->_conect->prepare($sql); 
 return $stmt;
}
 
}
?>
 

todo es funcionado , bien lo unico es que no entiendo como recojo la instancia de la clase ya creada. es registry::get() , pero no comprendo el parametro que le deberia pasar.


porque lo coloco asi y me retorna la instancia de Mysql

$x = registry::get('../../includes/DB/config.ini');


saludos

[RIVERMILLOS]

Si hago esto otra vez.

Código PHP:

$site_path = '../../includes/DB/config.ini';
        $this->_conect=DatabaseFactory::create($site_path);
 

estaria creando otra instancia?


saludos

[GatorV]

Si, porque a tu clase DbFactory le faltaría revisar en el registro si existe o no la instancia antes de crearla.

Saludos.

[RIVERMILLOS]

Listo lo que hice ahora fue esto.

DbFactory

Código PHP:

public static function create($sIniFile)
    {
        $config  = new Reader($sIniFile);   
        
        $config_data = $config->getConfig();
        
        $database_class = key($config_data);
        
        include($database_class.'.class.php');
        
        if(Registry::has($sIniFile))
        {
            
            return Registry::get($sIniFile);
            
        }else{
            
            $db = new $database_class($config);
            
            Registry::set($sIniFile, $db);  
            
            return $db;  
            
        }
        
 
    }
 

Y Cambie el registry por uno que tu hiciste en otro post

Código PHP:

<?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);
        }
    }
 ?>
 

hay creeria que ya no se crea una nueva.

Si?

saludos

[GatorV]

Sí se ve mejor aunque yo no haría el parsing de la configuración si ya tienes el elemento en el registro...

[RIVERMILLOS]

Si claro no seria necesario. Muchas gracias


saludos

[RIVERMILLOS]

Lo modifique asi para que no hiciera el parsing,

Código PHP:

if(Registry::has($sIniFile))
        {
            
            return Registry::get($sIniFile);
            
        }else
         {
            
            $config  = new Reader($sIniFile);   
        
            $config_data = $config->getConfig();
 
            $database_class = key($config_data);
 
            include($database_class.'.class.php');
            
            $db = new $database_class($config);
            
            Registry::set($sIniFile, $db);  
            
            return $db;  
            
         }
 

pero tengo el siguiente error.



[09-Aug-2011 09:53:53] PHP Fatal error: Using $this when not in object context in Registry.class.php on line 19



saludos

[GatorV]

Pues si ves la clase Registry no es estatica, tienes que instanciarla para leer las propiedades de la clase, por eso te marca error.

Saludos.

[RIVERMILLOS]

Ok , ahora la instancie , pero tengo una duda si vuelvo a llamar a eso el array $this->_cache quedaria nulo y siempre me retornaria una nueva.

Código PHP:

$Registry = new Registry();
        
        if ( ! $Registry->has( 'conexion' ) ) {
            
            $db = new $database_class( $config );
            
            $Registry->set( 'conexion' , $db );
            
        }
 
        $db = $Registry->get( 'conexion' );
        
        return $db;
 

saludos.

[GatorV]

Cierto, es mejor que la hagas estatica, algo así:

Código PHP:

<?php
class Registry
{
    private static $_registry = array();
    
    private function __construct(){}
    
    public static function has($entry)
    {
        return (isset(self::$_registry[$entry]));
    }
    
    public static function set($entry, $value)
    {
        self::$_registry[$entry] = $value;
    }
    
    public static function get($entry)
    {
        if (self::has($entry)) {
            return self::$_registry[$entry];
        } else {
            return null;
        }
    }
    
    public static function clear()
    {
        self::$_registry = array();
    }
}
 

Saludos.