Duda con PDO y mi clase

[GatorV]

Porque no mejor heredas tu clase Db de la clase de PDO y te quitas así los problemas de usar return ya que a la hora de en tu constructor regresar otro objeto no se puede.

Saludos.

[neodani]

Iniciado por GatorV

Porque no mejor heredas tu clase Db de la clase de PDO y te quitas así los problemas de usar return ya que a la hora de en tu constructor regresar otro objeto no se puede.

Saludos.

No se como realizar la herencia de forma correcta, tengo una idea pero no está completa y mucho menos buena...

Código PHP:

class DB extends PDO {
    private $_username;
    private $_password;
    private $_host;
    private $_db;
    private $_conexion;
    
    public function __construct($dsn,$username=null,$password=null,$driver_options=null)
   {
        $config_data = $config->getConfig();
        $this->_username = $config_data['username'];
        $this->_password = $config_data['password'];
        $this->_host = $config_data['host'];
        $this->_db = $config_data['db'];
   
      parent::__construct($dsn, $this->_username, $this->_password, $driver);
   }
 
}
 

Si sirve de referencia tengo una herencia de otro post que funciona muy bien. ¿Se puede aprovechar?

Código PHP:

<?php
/* 
    CLASE EXTENDIDA DE PDO para contar el número de consultas que se realizan 
*/
class CountPDO extends PDO
{
    private $_queryCount = 0;
    
    public function query()
    {
        $this->_increaseQueryCount();
        $args = func_get_args();
        
        return call_user_func_array(array('parent', 'query'), $args);
    }
    
    public function prepare($statement, $driver_options = array())
    {
        $this->_increaseQueryCount();
        return parent::prepare($statement, $driver_options);
    }
    
    private function _increaseQueryCount()
    {
        $this->_queryCount++;
    }
    
    public function getQueryCount()
    {
        return $this->_queryCount;
    }
    public function ResetCount()
    {
        $this->_queryCount = 0;
    }
}
?>
 

Gracias de antemano

[GatorV]

Te recomiendo leer un poco sobre que es y como funciona la herencia, si solo le vas a pasar un parámetro que es el config tienes que declararlo de esa forma.

[neodani]

Iniciado por GatorV

Te recomiendo leer un poco sobre que es y como funciona la herencia, si solo le vas a pasar un parámetro que es el config tienes que declararlo de esa forma.

Si he leído y visto tutoriales sobre la herencia, se trata de "sobreescribir" métodos que existen en la clase "padre".

Pero no se seguir

Se supone que estoy redeclarando el constructor de la clase PDO para que sea como hemos especificado anteriormente, que sea capaz de leer la variables del fichero .ini

Y luego debe existir por algún lado la conexión PDO que no sé si se tiene que definir a parte tal y como está o se necesita algo más?

Código PHP:

class DB extends PDO {
    private $_username;
    private $_password;
    private $_host;
    private $_db;
    private $_conexion;
    
    public function __construct(Config $config)
    {
        $config_data = $config->getConfig();
        $this->_username = $config_data['username'];
        $this->_password = $config_data['password'];
        $this->_host = $config_data['host'];
        $this->_db = $config_data['db'];
        
        return $this->Conectar();
    }
    
    private function Conectar(){
        try {
            $this->_conexion = new PDO("mysql:host=$this->_host;dbname=$this->_db", $this->_username, $this->_password);
            // echo a message saying we have connected
            echo 'Connected to database';
        }
        catch(PDOException $e){
            echo $e->getMessage();
        }
        return $this->_conexion;
    }
}
 

[GatorV]

Estas aplicando muy mal la herencia, mira este ejemplo:

Código PHP:

class Padre
{
          public function __construct($var1, $var2)
         {
                 // Esta clase acepta dos parámetros $var1 y $var2
          }
}
 
class Hija extends Padre
{
         public function __construct($var)
         {
                  // como la clase padre requiere 2 valores, le pasamos el $var de aqui y otro valor que definimos nosotros
                  parent::__construct($var, 2);
          }
}
 

Si te fijas, al crear una instancia de Hija, es una instancia de Padre y le estamos pasando al constructor de padre los datos que quiere, tu tienes que hacer lo mismo una clase Db que acepte de parámetro un objeto config y que internamente llame al constructor de su clase padre con los datos que requiere.

Saludos.

[portalmana]

Cuando hice un cursillo de java nos dieron una lista aditiva de cuando se podría heredar, como una guía de cuando esta bien que se herede y que no se use por el mero hecho de reutilizar código...

+ Cuando está seguro de que cumple la relación ... es un ...
p.ej.: un zafral no es un empleado (le faltan beneficios, no tiene aumentos de sueldo, etc.)

+ Cuando está seguro de que toda la funcionalidad heredada tiene sentido
p.ej.: si “Documento” incluye un método “entregarMercadería” los “Recibos” no podrán heredar de dicha clase

+ Si la superclase es abstracta se heredarán varias subclases diferentes.
p.ej.: si tubieramos una única clase de Mascota y fuera Perro no tendría sentido que se separaran en dos clases diferentes (Mascota – Perro).

+ Cuando el cuadro de clases obtenido tienen funcionalidades diferentes
(respecto a la superclase y entre subclases)
p.ej.: Si todas las especializaciones de Mascota “ladraran”, “hicieranPiruetas”
y se “rascaranLasPulgas”, aunque fueran “Iguanas”, podrían representarse con la
misma clase agregando un atributo “especie”.

o Cuando piensa que tiene una especialización y nadie le dice que no (¡¡¡ buena suerte !!!)

Y un enlace muy bueno de Enrique

Saludos

[neodani]

Iniciado por GatorV

Estas aplicando muy mal la herencia, mira este ejemplo:

Código PHP:

class Padre
{
          public function __construct($var1, $var2)
         {
                 // Esta clase acepta dos parámetros $var1 y $var2
          }
}
 
class Hija extends Padre
{
         public function __construct($var)
         {
                  // como la clase padre requiere 2 valores, le pasamos el $var de aqui y otro valor que definimos nosotros
                  parent::__construct($var, 2);
          }
}
 

Si te fijas, al crear una instancia de Hija, es una instancia de Padre y le estamos pasando al constructor de padre los datos que quiere, tu tienes que hacer lo mismo una clase Db que acepte de parámetro un objeto config y que internamente llame al constructor de su clase padre con los datos que requiere.

Saludos.

Así mejor? sobreescribo el constructor de PDO y luego dentro de él llamo a la clase padre con los datos que requiere mas el nuevo parámetro?

Código PHP:

class DB extends PDO {
    private $_username;
    private $_password;
    private $_host;
    private $_db;
    private $_conexion;
    
    public function __construct(Config $config)
    {
        $config_data = $config->getConfig();
        $this->_username = $config_data['username'];
        $this->_password = $config_data['password'];
        $this->_host = $config_data['host'];
        $this->_db = $config_data['db'];
        
        parent::__construct($dsn, $username="", $password="", $driver_options=array(), Config $config);
    }
}
 

[neodani]

¿ O debería llamar al padre pasandole lo parámetros originales, es decir sin el Config?

parent::__construct($dsn, $username="", $password="", $driver_options=array());

[GatorV]

Debes de llamar al constructor pasandole los parámetros que tu quieras definir, te dejo otro ejemplo más claro:

Código PHP:

class Auto
{
         public function __construct($marca, $color)
         {}
}
 
class VWAzul
{
         public function __construct() // nota: no hay parámetros
         {
                $marca = 'VW';
                $color = 'Azul';
                parent::__construct($marca, $color);
         }
}
 

Saludos.

[neodani]

Iniciado por GatorV

Debes de llamar al constructor pasandole los parámetros que tu quieras definir, te dejo otro ejemplo más claro:

Código PHP:

class Auto
{
         public function __construct($marca, $color)
         {}
}
 
class VWAzul
{
         public function __construct() // nota: no hay parámetros
         {
                $marca = 'VW';
                $color = 'Azul';
                parent::__construct($marca, $color);
         }
}
 

Saludos.

Ajam, entonces esta es la manera correcta de heredar en este caso, no?

Código PHP:

class DB extends PDO {
    private $_username;
    private $_password;
    private $_host;
    private $_db;
    private $_conexion;
    
    public function __construct(Config $config)
    {
        $config_data = $config->getConfig();
        $this->_username = $config_data['username'];
        $this->_password = $config_data['password'];
        $this->_host = $config_data['host'];
        $this->_db = $config_data['db'];
        
        parent::__construct($dsn, $username="", $password="", $driver_options=array());
    }
}
 

Utilizo esta llamada porque es la que aparece en la documentacion de PDO...
parent::__construct($dsn, $username="", $password="", $driver_options=array());

¿Es correcto?

Gracias de antemano!

[GatorV]

No, es incorrecto, por favor medita el ejemplo que te puse, al tu hacer esto:

Código PHP:

function foo($bar) {
 
}
 
foo($bar = "");
 

El interprete de PHP ve primero la asignacion dentro del paréntesis $bar = "" y asigna a $bar un valor vacio, posterioremente llama a foo.

Te recomiendo leas un libro de PHP básico para que entiendes como es que se tienen que llamar las funciones y como hacer el paso de parámetros.

Saludos!

[neodani]

Iniciado por GatorV

No, es incorrecto, por favor medita el ejemplo que te puse, al tu hacer esto:

Código PHP:

function foo($bar) {
 
}
 
foo($bar = "");
 

El interprete de PHP ve primero la asignacion dentro del paréntesis $bar = "" y asigna a $bar un valor vacio, posterioremente llama a foo.

Te recomiendo leas un libro de PHP básico para que entiendes como es que se tienen que llamar las funciones y como hacer el paso de parámetros.

Saludos!

Creo que ahora te entiendo, precisamente, eso era lo que no me cuadraba, que tuviese que pasarle esos parametros:

parent::__construct($dsn, $username="", $password="", $driver_options=array());

Sino mas bien los que toca que son ni mas ni menos que los que aparecen aqui, verdad?

$this->_username = $config_data['username'];
$this->_password = $config_data['password'];
$this->_host = $config_data['host'];
$this->_db = $config_data['db'];

Sin embargo, mi duda es que no se poner en su sitio la variable $dsn, segun el ejemplo de PDO debería tener esta forma

'mysql:host=127.0.0.1;dbname=codelab'

Debería hacerlo así, declarar una variable llamada $_dsn;

$this->_$dsn="mysql:host=$this->_host;dbname=$this->_db";

parent::__construct($this->_$dsn, $this->_username, $this->_password);

Mejor? :D

[GatorV]

Exacto ahora sí, te digo sería mucho mejor para ti y para que aprendas más que leas un libro de PHP, así puedes entender como funciona la sintaxis y como crear funciones y declarar los valores.

Saludos.

[neodani]

Iniciado por GatorV

Exacto ahora sí, te digo sería mucho mejor para ti y para que aprendas más que leas un libro de PHP, así puedes entender como funciona la sintaxis y como crear funciones y declarar los valores.

Saludos.

Entonces no hace falta que utilice el método Conectar() dentro de la clase no?

Código PHP:

private function Conectar(){
        try {
            $this->_conexion = new PDO("mysql:host=$this->_host;dbname=$this->_db", $this->_username, $this->_password);
            // echo a message saying we have connected
            echo 'Connected to database';
        }
        catch(PDOException $e){
            echo $e->getMessage();
        }
        return $this->_conexion;
    }
 

Con esto sería suficiente no?

<?php

Código PHP:

class DB extends PDO {
    private $_username;
    private $_password;
    private $_host;
    private $_db;
    private $_conexion;
    private $_dsn;
 
    public function __construct(Config $config)
    {
        $config_data = $config->getConfig();
        $this->_username = $config_data['username'];
        $this->_password = $config_data['password'];
        $this->_host = $config_data['host'];
        $this->_db = $config_data['db'];
        $this->_dsn="mysql:host=$this->_host;dbname=$this->_db";
        
        try {
            parent::__construct($this->_dsn, $this->_username, $this->_password);
            // echo a message saying we have connected
            //echo 'Connected to database';
        }
        catch(PDOException $e){
            echo $e->getMessage();
            exit;
        }
        
    }
}
 
?>
 

ejemplo1.php

Código PHP:

<?php
 
require('config_reader.php');
require('db.php');
require('registry.php');
 
class Categoria{
    private $nombreCategoria;
    
    // Listar todas las categorias de la base de datos
    public function listarCategorias($db){
        $query = "SELECT nombre FROM categorias";
        $comando = $db->prepare($query);
        $comando->execute();
 
        while( $datos = $comando->fetch() ) {
        // Imprimimos resultados
        echo $datos[0].'<br />';
        }
    }
    
    // Listar todas los partidos de la base de datos
    public function listarPartidos($db){
        $query = "SELECT home FROM partidos";
        $comando = $db->prepare($query);
        $comando->execute();
 
        while( $datos = $comando->fetch() ) {
        // Imprimimos resultados
        echo $datos[0].'<br />';
        }
    }
    
}
 
//Cargo los datos del fichero .ini
$config_slave = new Config('params_slave.ini');
$config_master = new Config('params_master.ini');
 
//Creo el objeto de la base de datos
$db_slave = new db($config_slave);
$db_master = new db($config_master);
 
//Guardo las instancias diferentes con Registry
Registry::save('db_slave', $db_slave);
Registry::save('db_master', $db_master);
 
//Recupero las instancias con Registry
$db_slave=Registry::get('db_slave');
$db_master=Registry::get('db_master');
 
$cat= new Categoria();
$cat->listarCategorias($db_slave); // Leo de la BBDD slave
$cat->listarPartidos($db_master); // Ejemplo leyendo de otra BBDD master
 
?>
 

[GatorV]

Así es, aunque como tip, yo no manejaría la excepcion de conectar dentro de tu clase DB, se la dejaría a la clase donde la implementas para que tengas un mejor control del flujo de tu aplicación.

Saludos.

[neodani]

Iniciado por GatorV

Así es, aunque como tip, yo no manejaría la excepcion de conectar dentro de tu clase DB, se la dejaría a la clase donde la implementas para que tengas un mejor control del flujo de tu aplicación.

Saludos.

Es decir, dentro de cada clase que implemente, en este ejemplo Categorias() y Partidos()?

La clase MyPDO.class.php la dejarías así??

Código PHP:

<?php
/* 
    CLASE EXTENDIDA DE PDO:
    - Cuenta el número de consultas que se realizan
    - Utiliza la carga los datos de conexión desde la clase Config
*/
class MyPDO extends PDO {
    private $_username;
    private $_password;
    private $_host;
    private $_db;
    private $_conexion;
    private $_dsn;
    private $_queryCount = 0;
    
    public function __construct(Config $config)
    {
        $config_data = $config->getConfig();
        $this->_username = $config_data['username'];
        $this->_password = $config_data['password'];
        $this->_host = $config_data['host'];
        $this->_db = $config_data['db'];
        $this->_dsn="mysql:host=$this->_host;dbname=$this->_db";
        
        parent::__construct($this->_dsn, $this->_username, $this->_password);
    }
    
    public function query()
    {
        $this->_increaseQueryCount();
        $args = func_get_args();
        
        return call_user_func_array(array('parent', 'query'), $args);
    }
    
    public function prepare($statement, $driver_options = array())
    {
        $this->_increaseQueryCount();
        return parent::prepare($statement, $driver_options);
    }
    
    private function _increaseQueryCount()
    {
        $this->_queryCount++;
    }
    
    public function getQueryCount()
    {
        return $this->_queryCount;
    }
    public function ResetCount()
    {
        $this->_queryCount = 0;
    }
}
?>
 

No se de qué manera hacer para controlar la excepción dentro de la clase Categoria()

Código PHP:

class Categoria{
    private $_con;
    
    public function __construct($DbConexion){
        try {
            $this->_con=$DbConexion;
            echo "Conectado a la base de datos<br/>";
        }
        catch(PDOException $e){
            echo $e->getMessage();
            echo "Falló la conexión a la base de datos<br/>";
            exit;
        }
    }
    
    // Listar todas las categorias de la base de datos
    public function listarCategorias(){
        $query = "SELECT nombre FROM categorias";
        $comando = $this->_con->prepare($query);
        $comando->execute();
 
        while( $datos = $comando->fetch() ) {
            echo $datos[0].'<br />';
        }
    }
}
 

¿Puedes indicarme?

Muchas gracias de antemano!

PD: pongo el ejemplo para que se vea de donde sale el parametro que le paso a la clase...

Código PHP:

//Cargo los datos del fichero .ini
$config_slave = new Config('params_slave.ini');
$config_master = new Config('params_master.ini');
 
//Creo el objeto de la base de datos
$db_slave = new MyPDO($config_slave);
$db_slave->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);
$db_master = new MyPDO($config_master);
$db_master->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);
 
//Guardo las instancias diferentes con Registry
Registry::save('db_slave', $db_slave);
Registry::save('db_master', $db_master);
 
//Recupero las instancias con Registry
$db_slave=Registry::get('db_slave');
$db_master=Registry::get('db_master');
 
// Creo el primer objeto el cual hace una conexion con la BBDD slave
$cat= new Categoria($db_slave);
$cat->listarCategorias(); 
$cat->listarCategorias(); 
 
// Creo el segundo objeto el cual hace una conexion con la BBDD master
$par= new Partidos($db_master);
$par->listarPartidos(); 
 
// Cuenta el total de consultas que hubieron en cada BBDD
echo "Consultas realizadas en BD Slave: ".$db_slave->getQueryCount()."<br/>"; 
echo "Consultas realizadas en BD Master: ".$db_master->getQueryCount()."<br/>";
 

[GatorV]

Donde implementes me refiero a donde estes usando las clases, por ejemplo:

Código PHP:

try {
        $cat= new Categoria();
        $cat->listarCategorias($db_slave); // Leo de la BBDD slave
        $cat->listarPartidos($db_master); // Ejemplo leyendo de otra BBDD master
} catch (PDOException $pe) {
        echo $pe->getMessage();
}
 

Así puedes controlar mucho mejor el flujo si se ocasiona algún problema o cachar diferentes tipos de excepciones.

Saludos.

[neodani]

Iniciado por GatorV

Donde implementes me refiero a donde estes usando las clases, por ejemplo:

Código PHP:

try {
        $cat= new Categoria();
        $cat->listarCategorias($db_slave); // Leo de la BBDD slave
        $cat->listarPartidos($db_master); // Ejemplo leyendo de otra BBDD master
} catch (PDOException $pe) {
        echo $pe->getMessage();
}
 

Así puedes controlar mucho mejor el flujo si se ocasiona algún problema o cachar diferentes tipos de excepciones.

Saludos.

Lo he añadido donde me comentas pero es como si no hiciese caso, ya que el mensaje de error que muestra no el "correcto".

Código PHP:

try {
    // Creo el primer objeto el cual hace una conexion con la BBDD slave
    $cat= new Categoria($db_slave);
    $cat->listarCategorias(); // Leo de la BBDD slave
} catch(PDOException $e){
    echo $e->getMessage();
    echo "Hubo un error";
}
 

He modificado el nombre de la base de datos en el fichero de .ini por uno que no existe y me produce el siguiente error:

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42000] [1049] Unknown database 'code3lab'' in C:\wamp\www\00-laboratorio_test\secciones\class_categorias\Nueva\ ConectarCon2_BD_version2\MyPDO.class.php:25 Stack trace: #0 C:\wamp\www\00-laboratorio_test\secciones\class_categorias\Nueva\ ConectarCon2_BD_version2\MyPDO.class.php(25): PDO->__construct('mysql:host=loca...', 'root', 'jupiter') #1 C:\wamp\www\00-laboratorio_test\secciones\class_categorias\Nueva\ ConectarCon2_BD_version2\ejemplo1.php(59): MyPDO->__construct(Object(Config)) #2 {main} thrown in C:\wamp\www\00-laboratorio_test\secciones\class_categorias\Nueva\ ConectarCon2_BD_version2\MyPDO.class.php on line 25

La linea 25 hace de la clase MyPDO.class.php hace referencia a aquí

parent::__construct($this->_dsn, $this->_username, $this->_password);

¿Por qué no captura correctamente la excepción?
Ademas tengo puesto justo despues de la creación de los objetos de base de datos el setAttribute, por lo que el error tb debería ser diferente no?

$db_slave = new MyPDO($config_slave);
$db_slave->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);
$db_master = new MyPDO($config_master);
$db_master->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);

¿Sabes por qué?

[GatorV]

Si te fijas el error ya es que no conoce una db llamada code3lab' (nota que tienes una comilla de más).

Saludos.

[neodani]

Iniciado por GatorV

Si te fijas el error ya es que no conoce una db llamada code3lab' (nota que tienes una comilla de más).

Saludos.

Pero pensaba que debía ser como el de clase PDO, si elimino el try y catch anterior y se lo pongo a la clase MyPDO así:

Código PHP:

public function __construct(Config $config)
    {
        $config_data = $config->getConfig();
        $this->_username = $config_data['username'];
        $this->_password = $config_data['password'];
        $this->_host = $config_data['host'];
        $this->_db = $config_data['db'];
        $this->_dsn="mysql:host=$this->_host;dbname=$this->_db";
        try {   
            parent::__construct($this->_dsn, $this->_username, $this->_password);
        } catch(PDOException $e){
            echo $e->getMessage();
            echo "Hubo un error";
        }
    }
 

El error es menos agresivo
SQLSTATE[42000] [1049] Unknown database 'code3lab'Hubo un error

Pensaba que quedaría así, pero si me comentas que no, que al poner el try y catch en:

Código PHP:

try {
        $cat= new Categoria();
        $cat->listarCategorias($db_slave); // Leo de la BBDD slave
        $cat->listarPartidos($db_master); // Ejemplo leyendo de otra BBDD master
} catch (PDOException $pe) {
        echo $pe->getMessage();
}
 

queda distinto, entonces es una pena...

[GatorV]

Si te fijas es el mismo error, mi comentario a manejar el try/catch en donde implementas es para que tu puedas controlar el flujo de tu código, si no se va aparar en el primer exit() y de nada sirve la excepción como tal.

Ahora para que "repares" el error tienes que poner el nombre de la base de datos correcta, por eso te lanza la excepcion.

[neodani]

Iniciado por GatorV

Si te fijas es el mismo error, mi comentario a manejar el try/catch en donde implementas es para que tu puedas controlar el flujo de tu código, si no se va aparar en el primer exit() y de nada sirve la excepción como tal.

Ahora para que "repares" el error tienes que poner el nombre de la base de datos correcta, por eso te lanza la excepcion.

Jejeje no nos entendiamos, para que el try y catch me mostrase el mismo error que si lo ponia dentro de la clase MyPDO, debía ponerlo no cuando creaba la Categoria sino antes, justo cuando hago la creación de la instancia de la base de datos.

Aquí

Código PHP:

try {
    $db_slave = new MyPDO($config_slave);
    $db_slave->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);
} catch(PDOException $e){
    echo $e->getMessage();
    echo "Hubo un error";
}
 

Y ahora sí, el error que produce era el que estaba intentando ver :D
Me refiero que queda mostrado de la misma manera

SQLSTATE[42000] [1049] Unknown database 'code3lab'Hubo un error

Sin embargo, no consigo que el script termine y no siga mirando cuando no se puede crear la instancia si los datos de conexión no son validos.

Ejemplo:

Código PHP:

try {
    $db_slave = new MyPDO($config_slave);
} catch(PDOException $e){
    echo $e->getMessage();
    echo "Hubo un error";
    exit;
}
$db_slave->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);
 

El script muestra el error pero sigue ejecutandose fuera del TRY y CATCH, porque viene el setAttribute que claro, falla pq no se ha creado la instancia. Pero si funcionase el exit, no debería haber llegado al setAttribute no?

SQLSTATE[42000] [1049] Unknown database 'cod3elab'Hubo un error
Fatal error: Call to a member function setAttribute() on a non-object in C:\wamp\www\00-laboratorio_test\secciones\class_categorias\Nueva\ ConectarCon2_BD_version2\ejemplo1.php on line 59

¿La pregunta es... por qué no finaliza el script dentro del CATCH si hay un exit?

Gracias de antemano!

[GatorV]

Como te comento nuevamente, no me estas entendiendo tu puedes realizar todo dentro del mismo try para evitar eso precisamente, por ejemplo:

Código PHP:

try {
//Cargo los datos del fichero .ini
$config_slave = new Config('params_slave.ini');
$config_master = new Config('params_master.ini');
 
//Creo el objeto de la base de datos
$db_slave = new MyPDO($config_slave);
$db_slave->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);
$db_master = new MyPDO($config_master);
$db_master->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);
 
//Guardo las instancias diferentes con Registry
Registry::save('db_slave', $db_slave);
Registry::save('db_master', $db_master);
 
//Recupero las instancias con Registry
$db_slave=Registry::get('db_slave');
$db_master=Registry::get('db_master');
 
// Creo el primer objeto el cual hace una conexion con la BBDD slave
$cat= new Categoria($db_slave);
$cat->listarCategorias(); 
$cat->listarCategorias(); 
 
// Creo el segundo objeto el cual hace una conexion con la BBDD master
$par= new Partidos($db_master);
$par->listarPartidos(); 
 
// Cuenta el total de consultas que hubieron en cada BBDD
echo "Consultas realizadas en BD Slave: ".$db_slave->getQueryCount()."<br/>"; 
echo "Consultas realizadas en BD Master: ".$db_master->getQueryCount()."<br/>";
 
} catch (Exception $e) {
         echo "Ocurrio un error";
}
 

Saludos.

[neodani]

Haciendolo como me decías me salia el mismo error y era pq en la clase MyPDO tenía tb un TRY CATCH, al quitarlo conseguí que funcionase! Yuhu!

Por cierto, una última duda...

Ahora mi clase MyPDO no utiliza ningún patrón singleton como hacia antes, ¿significa que se me pueden duplicar instancias sin que yo lo sepa, o al utilizar el Registry ya se encarga de alguna manera para que no se dupliquen instancias, aunque me temo que el Registry es simplemente para llevar eso un registro de instancias...?

Pongo un ejemplo:

Código PHP:

$config_slave = new Config('params_slave.ini');
$db_slave = new MyPDO($config_slave);
$db_slave->setAttribute(MyPDO::ATTR_ERRMODE, MyPDO::ERRMODE_EXCEPTION);
Registry::save('db_slave', $db_slave);
$db_slave=Registry::get('db_slave');
 

Hasta aquí sería el proceso inicial que tengo que hacer para hacer toda la conexión... ¿no sé si se podría simplificar en una sola linea... en lugar de tener que poner siempre las 5, quizás una función?

Luego hago uso de las clases que tenga en mi aplicación, las cuales hacen uso de la conexión a la bbdd, para ello les tengo que pasar en todas las clases la instancia de la conexión como parámetro, correcto?

$cat= new Categoria($db_slave);
$cat->listarCategorias();

¿Existe alguna manera de duplicar conexiones de forma "errónea" o todo está creado de tal manera que si se intenta cargar otro conexión con los mismos parámetros que una que esté en uso, no se creará? me refiero a que si todo esta bien para utilizar la clase sin temor a tener instancias de bases de datos por ahí... (vamos lo que bien evita el singleton y que ya no implementa en la clase MyPDO...)

Muchas gracias de antemano!

[GatorV]

Por si solas no se pueden duplicar, es imposible, más bien tu vía código tienes que controlar eso, ¿como? usando el registro siempre, esa es tu tarea de programador.

Ahora resumir todo eso en una sola línea? Pues puedes hacer otra clase que pasandole el parámetro del config cree el objeto y te regrese los datos, esa es precisamente la tarea del patrón Factory.

De hecho ahora pensando podrías usar un patrón Factory con un cache interno y así omites tu Registry (o lo usas internamente), por ejemplo:

Código PHP:

class dbFactory
{
         public static function create($sIniFile) {
                  if (Registry::exists($sIniFile)) {
                          return Registry::get($sIniFile);
                  }
 
                  $Config = new Config($sIniFile);
                  $db = new db($Config);
                  Registry::set($sIniFile, $db);
                  
                  return $db;
         }
}
 
$db_master = dbFactory::create('master.ini');
$db_slave = dbFactory::create('slave.ini');
 

Saludos.

[neodani]

Iniciado por GatorV

Por si solas no se pueden duplicar, es imposible, más bien tu vía código tienes que controlar eso, ¿como? usando el registro siempre, esa es tu tarea de programador.

Ahora resumir todo eso en una sola línea? Pues puedes hacer otra clase que pasandole el parámetro del config cree el objeto y te regrese los datos, esa es precisamente la tarea del patrón Factory.

Será por patrones! madre mia! xDDDDDDDDDDDDDDDDD

Este es el código final del ejemplo. Funciona perfecto.

Código PHP:

<?php
require('config_reader.php');
require('MyPDO.class.php');
require('registry.php');
 
/************************************************************************/
class dbFactory
{
    public static function create($sIniFile) {
        if (Registry::exists($sIniFile)) {
            return Registry::get($sIniFile);
        }
 
        $Config = new Config($sIniFile);
        $db = new MyPDO($Config);
        Registry::save($sIniFile, $db);
 
        return $db;
    }
}
/************************************************************************/
class Categoria{
    private $_con;
    
    public function __construct($DbConexion){
            $this->_con=$DbConexion;
    }
    
    // Listar todas las categorias de la base de datos
    public function listarCategorias(){
        $query = "SELECT nombre FROM categorias";
        $comando = $this->_con->prepare($query);
        $comando->execute();
 
        while( $datos = $comando->fetch() ) {
            echo $datos[0].'<br />';
        }
    }
}
/************************************************************************/
class Partidos{
    private $_con;
    
    public function __construct($DbConexion){
        $this->_con=$DbConexion;
    }
    
    // Listar todos las partidos de la base de datos
    public function listarPartidos(){
        $query = "SELECT home FROM partidos";
        $comando = $this->_con->prepare($query);
        $comando->execute();
 
        while( $datos = $comando->fetch() ) {
        echo $datos[0].'<br />';
        }
    }
}
/************************************************************************/
try {
    $db_master = dbFactory::create('bd_master.ini');
    $db_slave = dbFactory::create('bd_slave.ini');
 
    $cat= new Categoria($db_slave);
    $cat->listarCategorias(); // Leo de la BBDD slave
    $cat->listarCategorias(); // Vuelvo a leer de la BBDD slave
 
    // Creo el segundo objeto el cual hace una conexion con la BBDD master
    $par= new Partidos($db_master);
    $par->listarPartidos(); //Leo de la BBDD 
 
    // Cuenta el total de consultas que hubieron en cada BBDD
    echo "Consultas realizadas en BD Slave: ".$db_slave->getQueryCount()."<br/>"; 
    echo "Consultas realizadas en BD Master: ".$db_master->getQueryCount()."<br/>"; 
    
} catch (Exception $e) {
    echo $e->getMessage();
    echo "<br/>Ocurrio un error";
    exit;
}
?>
 

Saludos y mil gracias por las lecciones del hilo

[atrianaster]

Bueno y para los que desconocen sobre este patrón o desean investigar:

http://desarrolladorsenior.blogspot....y-en-php5.html

[atrianaster]

Iniciado por GatorV

Porque no mejor heredas tu clase Db de la clase de PDO y te quitas así los problemas de usar return ya que a la hora de en tu constructor regresar otro objeto no se puede.

Saludos.

Si te retorna la instancia PDO dentro del constructor de la clase, pero como un atributo privado.

Código PHP:

require('config_reader.php');
require('db.php');
require('registry.php');
 
$config = new Config('params.ini');
 
$db_slave = new db($config);
echo '<pre>'; var_dump($db_slave); echo '</pre>'; exit();
 

Esto imprimiría:

Connected to database

Código PHP:

object(db)#2 (5) {
  ["_username":"db":private]=>
  string(4) "root"
  ["_password":"db":private]=>
  string(12) "jupiter"
  ["_host":"db":private]=>
  string(9) "localhost"
  ["_db":"db":private]=>
  string(6) "codelab"
  ["_conexion":"db":private]=>
  object(PDO)#3 (0) {
  }
}
 

Por lo que tendrías que tener un método publico getConexion por ejemplo que te devuelva la instancia de PDO en vivo y poder utilizar posteriormente todos los métodos de PDO.

Aunque como dice GatorV es mejor heredar de PDO directamente.

[atrianaster]

Iniciado por portalmana

Cuando hice un cursillo de java nos dieron una lista aditiva de cuando se podría heredar, como una guía de cuando esta bien que se herede y que no se use por el mero hecho de reutilizar código...

+ Cuando está seguro de que cumple la relación ... es un ...
p.ej.: un zafral no es un empleado (le faltan beneficios, no tiene aumentos de sueldo, etc.)

+ Cuando está seguro de que toda la funcionalidad heredada tiene sentido
p.ej.: si “Documento” incluye un método “entregarMercadería” los “Recibos” no podrán heredar de dicha clase

+ Si la superclase es abstracta se heredarán varias subclases diferentes.
p.ej.: si tubieramos una única clase de Mascota y fuera Perro no tendría sentido que se separaran en dos clases diferentes (Mascota – Perro).

+ Cuando el cuadro de clases obtenido tienen funcionalidades diferentes
(respecto a la superclase y entre subclases)
p.ej.: Si todas las especializaciones de Mascota “ladraran”, “hicieranPiruetas”
y se “rascaranLasPulgas”, aunque fueran “Iguanas”, podrían representarse con la
misma clase agregando un atributo “especie”.

o Cuando piensa que tiene una especialización y nadie le dice que no (¡¡¡ buena suerte !!!)

Y un enlace muy bueno de Enrique

Saludos

Excelentes comentarios

Saludos.

[historiasdemaria]

No entiendo por que Singleton no es bueno para la bd?

Y, por que no crear una clase que herede de PDO e implmentear Singleton dentro para "servir" la connexion

?