Intentando entender la clase mysql

[neodani]

Buenas,

A raiz del post de nuestro compañero eulerss

http://www.forosdelweb.com/f68/clase...1/#post3275707

Me sale una duda, que para no ensuciar su hilo abro este nuevo hilo, con una nueva clase mysql que utiliza el patron singleton al cual quiero aplicar el último comentario de maturano

class-mysql.php

Código PHP:

<?php
 
class DataBase {
 
    private $conexion;
    private $resource;
    private $sql;
    public static $queries;
    private static $_singleton;
 
    public static function getInstance(){
        if (is_null (self::$_singleton)) {
            self::$_singleton = new DataBase();
        }
        return self::$_singleton;
    }
 
    private function __construct($servidor, $user, $password, $db){
        $this->conexion = @mysql_connect($servidor, $user, $password);
        mysql_select_db($db, $this->conexion);
        $this->queries = 0;
        $this->resource = null;
    }
 
    public function execute(){
        if(!($this->resource = mysql_query($this->sql, $this->conexion))){
            return null;
        }
        $this->queries++;
        return $this->resource;
    }
 
    public function alter(){
        if(!($this->resource = mysql_query($this->sql, $this->conexion))){
            return false;
        }
        return true;
    }
 
    public function loadObjectList(){
        if (!($cur = $this->execute())){
            return null;
        }
        $array = array();
        while ($row = @mysql_fetch_object($cur)){
            $array[] = $row;
        }
        return $array;
    }
 
    public function setQuery($sql){
        if(empty($sql)){
            return false;
        }
        $this->sql = $sql;
        return true;
    }
 
    public function freeResults(){
        @mysql_free_result($this->resource);
        return true;
    }
 
    public function loadObject(){
        if ($cur = $this->execute()){
            if ($object = mysql_fetch_object($cur)){
                @mysql_free_result($cur);
                return $object;
            }
            else {
                return null;
            }
        }
        else {
            return false;
        }
    }
 
    function __destruct(){
        @mysql_free_result($this->resource);
        @mysql_close($this->conexion);
    }
}
?>
 

Ejemplo script llamando a la clase

Código PHP:

<?php
include('class-mysql.php');
 
// Hacemos la conexión
$db = new DataBase('localhost','root','pass12421','mibasededatos');
$db = DataBase::getInstance();
 
// Supongamos que tenemos una tabla de usuarios
// Hacemos la consulta:
$db->setQuery("SELECT * FROM wp_posts");
 
// La ejecutamos y al mismo tiempo obtenemos un arreglo de objetos
// con los campos especificados en la consulta como propiedades.
$usuarios = $db->loadObjectList();
 
// Los imprimimos directamente en pantalla...
foreach($usuarios as $usuario){
    echo 'ID: '.$usuario->ID;
    echo 'Nombre: '.$usuario->post_name;
    echo 'Grupo: '.$usuario->post_status;
    echo '<br />';
}
 
?>
 

Tengo que decir que las lineas 18,19 y 20 de la clase no estaban así sino que no se le pasaba ningun parametro... Los he puesto yo para que sea mas reusable. Entonces me preguntaba como llamar a la clase, estoy casi seguro que si no tuviese el patron singleton lo podría hacer así
$db = new DataBase('localhost','root','pass12421','mibaseded atos');

Pero lo tiene y no se como crear la clase y que se aplique el patron singleton... me pierdo un poco.

El error que me da al ejecutar el script, es que revise la linea 5 del script precisamente esta
$db = new DataBase('localhost','root','pass12421','mibaseded atos');

¿Qué me falta para acabar de encajar este rompezabezas?

Muchas gracias de antemano, estoy aprendiendo mucho con vuestros comentarios

[GatorV]

Lo que pasa es que tienes que pasar esos parámetros a la clase vía el Singleton, la idea del Singleton es solo tener una instancia, ve este ejemplo:

Código PHP:

class Singleton {
       private static $_instance;
       private $_var;
 
       public static function getInstance($var = '')
       {
               if (self::$_instance == null ) {
                      self::$_instance = new self($var);
               }
 
               return self::$_instance;
       }
 
       private function __construct($var = '')
       {
              $this->_var = $var;
       }
 
       public function printVar()
       {
               echo $this->_var;
       }
}
 
$obj = Singleton::getInstance("hola");
$obj->printVar(); // imprime hola
 
$obj2 = Singleton::getInstance();
$obj2->printVar(); // imprime hola
 
$obj3 = Singleton::getInstance("bye");
$obj3->printVar(); // sigue imprimiendo hola
 

La idea de Singleton es que no se pueda construir así tienes solo una instancia global.

Saludos.

[maturano]

Iniciado por neodani

El error que me da al ejecutar el script, es que revise la linea 5 del script precisamente esta
$db = new DataBase('localhost','root','pass12421','mibaseded atos');

Primero que todo, cuando aparezcan errores, muestra exactamente cuales son. Sí te dice que veas la línea 5, pero no nos dices por qué, cuál es el error exacto.

Eso solo como consejo/petición al comunicar problemas.

Iniciado por neodani

Me sale una duda, que para no ensuciar su hilo abro este nuevo hilo, con una nueva clase mysql que utiliza el patron singleton al cual quiero aplicar el último comentario de maturano
(...)
Entonces me preguntaba como llamar a la clase, estoy casi seguro que si no tuviese el patron singleton lo podría hacer así
$db = new DataBase('localhost','root','pass12421','mibaseded atos');

Pero lo tiene y no se como crear la clase y que se aplique el patron singleton... me pierdo un poco.

En mi último comentario hablaba exclusivamente de la función del constructor, no aplicaba el patrón singleton. También mencioné leyeras los enlaces que he dejado. Extraído de www.php.net/singleton (que dejé cuando sí hablaba de Singleton):

Código PHP:

class Example
{
    // ...
    
    // OJO: A private constructor; prevents direct creation of object
    private function __construct() 
    {
        echo 'I am constructed';
    }
    // ...
}
 
 
// Y más adelante, en su implementación:
 
// OJO: This would fail because the constructor is private
$test = new Example;
 
// This will always retrieve a single instance of the class
$test = Example::singleton();
$test->bark();
 

Con singleton la idea es no utilizar el operador new, lo utilizas dos veces y tienes dos instancias, no tienes control sobre ello. Para tener el control se especifica como privado el método constructor y manejas la creación/recuperación de la instancia a través de un método.

Te falta fusionar las ideas. Tu código:

Código PHP:

// Hacemos la conexión
$db = new DataBase('localhost','root','pass12421','mibasededatos');
$db = DataBase::getInstance(); // ¡Sobre-escribes inmediatamente después!
 

¿No el método getInstance() es el que recupera/crea la instancia?. Fusionando:

Código PHP:

// Hacemos la conexión
$db = DataBase::getInstance('localhost','root','pass12421','mibasededatos');
 

¿Mejor?. Espero quede claro ahora.

[neodani]

Buenas,

Muchas gracias por las explicaciones, las intento aplicar pero no tengo éxito. Creo que me falta una cosa.

Este es el código de la clase. Si os fijáis he añadido en la linea 13 y 18 los parámetros y me dice lo siguiente:

Código:

Warning: mysql_select_db(): supplied argument is not a valid MySQL-Link resource in C:\AppServ\www\00-laboratorio_test\secciones\mysql\clase_mysql_1\class-mysql.php on line 20

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in C:\AppServ\www\00-laboratorio_test\secciones\mysql\clase_mysql_1\class-mysql.php on line 26

Warning: Invalid argument supplied for foreach() in C:\AppServ\www\00-laboratorio_test\secciones\mysql\clase_mysql_1\ejemplo.php on line 16

Clase

Código PHP:

<?php
 
class DataBase {
 
    private $conexion;
    private $resource;
    private $sql;
    public static $queries;
    private static $_singleton;
 
    public static function getInstance(){
        if (is_null (self::$_singleton)) {
            self::$_singleton = new DataBase($servidor, $user, $password, $db);
        }
        return self::$_singleton;
    }
 
    private function __construct($servidor, $user, $password, $db){
        $this->conexion = @mysql_connect($servidor, $user, $password);
        mysql_select_db($db, $this->conexion);
        $this->queries = 0;
        $this->resource = null;
    }
 
    public function execute(){
        if(!($this->resource = mysql_query($this->sql, $this->conexion))){
            return null;
        }
        $this->queries++;
        return $this->resource;
    }
 
    public function alter(){
        if(!($this->resource = mysql_query($this->sql, $this->conexion))){
            return false;
        }
        return true;
    }
 
    public function loadObjectList(){
        if (!($cur = $this->execute())){
            return null;
        }
        $array = array();
        while ($row = @mysql_fetch_object($cur)){
            $array[] = $row;
        }
        return $array;
    }
 
    public function setQuery($sql){
        if(empty($sql)){
            return false;
        }
        $this->sql = $sql;
        return true;
    }
 
    public function freeResults(){
        @mysql_free_result($this->resource);
        return true;
    }
 
    public function loadObject(){
        if ($cur = $this->execute()){
            if ($object = mysql_fetch_object($cur)){
                @mysql_free_result($cur);
                return $object;
            }
            else {
                return null;
            }
        }
        else {
            return false;
        }
    }
 
    function __destruct(){
        @mysql_free_result($this->resource);
        @mysql_close($this->conexion);
    }
}
?>
 

Script

Código PHP:

<?php
include('class-mysql.php');
 
// Hacemos la conexión
$db = DataBase::getInstance('localhost','root','pass12421','mibasededatos');
 
// Supongamos que tenemos una tabla de usuarios
// Hacemos la consulta:
$db->setQuery("SELECT * FROM wp_posts");
 
// La ejecutamos y al mismo tiempo obtenemos un arreglo de objetos
// con los campos especificados en la consulta como propiedades.
$usuarios = $db->loadObjectList();
 
// Los imprimimos directamente en pantalla...
foreach($usuarios as $usuario){
    echo 'ID: '.$usuario->ID;
    echo 'Nombre: '.$usuario->post_name;
    echo 'Grupo: '.$usuario->post_status;
    echo '<br />';
}
 
?>
 

Qué es lo que falta para que acabe de funcionar esto? :S

Muchas gracias por vuestra ayuda!

[_ssx]

Te falta agregar los parámetros al método getInstance por que no le estas pasando las variables al constructor.


Tienes esto.

Código PHP:

  public static function getInstance(){ 
        if (is_null (self::$_singleton)) { 
            self::$_singleton = new DataBase($servidor, $user, $password, $db); 
        } 
        return self::$_singleton; 
    }  
   

Podría ser así.

Código PHP:

  public static function getInstance($servidor, $user, $password, $db){ 
        if (is_null (self::$_singleton)) { 
            self::$_singleton = new DataBase($servidor, $user, $password, $db); 
        } 
        return self::$_singleton; 
    }  
   

[pateketrueke]

otra opción, es crear un método alternativo y dejar el singleton tal cual...

Código PHP:

  function connect($host, $user, $pass, $db)
{
  // ...
}

// ...

$foo = bar::instance();
$foo->connect(/*... */);  
   

[neodani]

Iniciado por _ssx

Te falta agregar los parámetros al método getInstance por que no le estas pasando las variables al constructor.


Tienes esto.

Código PHP:

  public static function getInstance(){ 
        if (is_null (self::$_singleton)) { 
            self::$_singleton = new DataBase($servidor, $user, $password, $db); 
        } 
        return self::$_singleton; 
    }  
   

Podría ser así.

Código PHP:

  public static function getInstance($servidor, $user, $password, $db){ 
        if (is_null (self::$_singleton)) { 
            self::$_singleton = new DataBase($servidor, $user, $password, $db); 
        } 
        return self::$_singleton; 
    }  
   

Ostras es verdad! que tonto no lo vi

La opción de pateketrueke tb es válida pero ya son dos lineas, creo que así queda mas compacto...

Lo dejaré así en 1 sola linea

Muchas gracias por vuestra ayuda!

[GatorV]

Yo dejaría los parámetros como opcionales así solo los usas una vez para crear el objeto y las proximas veces solo usas getInstance() sin parámetros para ahorrar código

[neodani]

Iniciado por GatorV

Yo dejaría los parámetros como opcionales así solo los usas una vez para crear el objeto y las proximas veces solo usas getInstance() sin parámetros para ahorrar código

Uhm estoy un poco espeso a estas horas GatorV, no acabo de entender a lo que te refieres con dejar los parámetros como opcionales...

¿Te refieres al patron singleton dejarlo sin $servidor, $user, $password, $db?

Código PHP:

public static function getInstance($servidor, $user, $password, $db){
        if (is_null (self::$_singleton)) {
            self::$_singleton = new DataBase($servidor, $user, $password, $db);
        }
        return self::$_singleton;
    }
 

Y crear el metodo como decia pateketrueke?

Salu2!

[GatorV]

No, me refiero a esto:

Código PHP:

public static function getInstance($servidor = '', $user = '', $password = '', $db = ''){
        if (is_null (self::$_singleton)) {
            self::$_singleton = new DataBase($servidor, $user, $password, $db);
        }
        return self::$_singleton;
    }
 

Así lo puedes llamar la primera vez:

Código PHP:

$conn = DB::getInstance($server, $user, $password, $db);
// Mas adelante
$conn = DB::getInstance();
 

Saludos.

[neodani]

Iniciado por GatorV

No, me refiero a esto:

Código PHP:

public static function getInstance($servidor = '', $user = '', $password = '', $db = ''){
        if (is_null (self::$_singleton)) {
            self::$_singleton = new DataBase($servidor, $user, $password, $db);
        }
        return self::$_singleton;
    }
 

Así lo puedes llamar la primera vez:

Código PHP:

$conn = DB::getInstance($server, $user, $password, $db);
// Mas adelante
$conn = DB::getInstance();
 

Saludos.

Pero si lo haces así, la primera vez guay porque le pasas los parámetros y gracias al constructor te conectará con la base de datos automáticamente, pero con la segunda llamada que consigues? si no le pasas los parámetros no podrá conectarse a la base de datos...

Si he entendido bien el patron singleton haría mas o menos así:

Con la primera linea mira si existe el recurso sino existe ninguna instancia abierta la crea por primera vez, trabajas con ella, etc... y si vuelves hacer la misma llamada pasandole los mismos datos, comprueba que ya existe una instancia que se llama así, por lo que la aprovechará. Ahora bien, si los datos de conexión a la hora de llamar a la nueva instancia han cambiado entiendo que te crea una segunda instancia diferente no?

Que consigues llamando a la instancia con la segunda forma, sin pasarle los datos? quieres decir que es capaz de detectar que existe una instancia lanzada y por eso no hace falta pasarle los datos? en ese caso que sentido lanzar esa linea, porque no utilizas algun metodo de la clase para hacer cosas...

No acabo de entender la utilidad de volverla a llamar sin opciones...

Gracias!

[maturano]

No puedo creer que a esta altura, se siga sin entender.

Singleton:
- Tiene como propósito manejar UNA MISMA instancia de un objeto.
- ¿Cómo funciona?
- A través de un método que cuando es llamado comprueba si una propiedad interna es una instancia de la propia clase. Si no lo es, crea una nueva instancia y la guarda en dicha propiedad, si sí lo es, retorna esa instancia.

Hasta ahí, sin parámetros ni nada; ahí queda singleton.

En tu clase, quieres pasar argumentos en tu constructor, es solo por eso que los pasas una primera vez, porque cuando cree la instancia los necesitará, pero para un segundo llamado, la instancia ya estará creada, solo la va a retornar ...

Es realmente simple, un patrón sencillo.

Al comienzo GatorV dejó un código más que claro y sencillo. Ejecuta el método que hace el trabajo con otros argumentos y sin ellos y muestra como siempre regresa la misma instancia. Dale su tiempo para entenderlo.

[_ssx]

Solo como dice maturano date el tiempo de entender el getInstance en el if esta todo lo que necesitas entender.

Código PHP:

  public static function getInstance($servidor, $user, $password, $db){ 
        if (is_null (self::$_singleton)) { 
            self::$_singleton = new DataBase($servidor, $user, $password, $db); 
        } 
        return self::$_singleton; 
    }  
   

[neodani]

Iniciado por _ssx

Solo como dice maturano date el tiempo de entender el getInstance en el if esta todo lo que necesitas entender.

Código PHP:

  public static function getInstance($servidor, $user, $password, $db){ 
        if (is_null (self::$_singleton)) { 
            self::$_singleton = new DataBase($servidor, $user, $password, $db); 
        } 
        return self::$_singleton; 
    }  
   

Sí he entendido el significado de singleton, pero mi última duda era (si queréis cojo el ejemplo de GatorV)

Código PHP:

class Singleton {
       private static $_instance;
       private $_var;
 
       public static function getInstance($var = '')
       {
               if (self::$_instance == null ) {
                      self::$_instance = new self($var);
               }
 
               return self::$_instance;
       }
 
       private function __construct($var = '')
       {
              $this->_var = $var;
       }
 
       public function printVar()
       {
               echo $this->_var;
       }
}
 
$obj = Singleton::getInstance("hola");
$obj->printVar(); // imprime hola
 
$obj2 = Singleton::getInstance();
$obj2->printVar(); // imprime hola
 
$obj3 = Singleton::getInstance("bye");
$obj3->printVar(); // sigue imprimiendo hola
 

En los 3 casos imprime hola porque fue como se llamo la primera vez. Esto es lo mismo si la primera vez te conectas a una base de datos especifica.
Mi duda era que pasa si quieres conectarte a otra base de datos diferente (o en el ej. de GatorV, que pasa si quieres te te escriba bye en lugar de hola?)

Esa era mi duda mi reflexión del último post... como dijo a la clase que lo que estoy llamando es una nueva conexión a otra base de datos. Por que me extrañaría que no se pudiera. ¿Se haría con un metodo? ¿No habría dos conexiones abiertas a dos base de datos distintas?

Salu2!

[maturano]

Iniciado por neodani

Ahora bien, si los datos de conexión a la hora de llamar a la nueva instancia han cambiado entiendo que te crea una segunda instancia diferente no?

Eso es una duda puntual sobre el comportamiento del patrón Singleton, y la respuesta es un NO. Es muy diferente a cuando dices esto:

Iniciado por neodani

Esa era mi duda mi reflexión del último post... como dijo a la clase que lo que estoy llamando es una nueva conexión a otra base de datos. Por que me extrañaría que no se pudiera. ¿Se haría con un metodo? ¿No habría dos conexiones abiertas a dos base de datos distintas?

Este ya estás planteando el "deseo" de un comportamiento de tu clase, es otro tema muy aparte, fuera de Singleton; y no, en tu último post dijiste otra cosa.

¿Se puede?, ¡Claro!. ¿Un nuevo método?, ¿diferenciar por parámetros?, ¿se ha de cerrar la conexión si había una?, ¿se mantienen las dos?, ¿un nuevo parámetro para indicar que es una nueva conexión? ... Ya tienes que resolver esos problemas, el cómo, el comportamiento que tenga dependerá de ti.

Puedes, por ejemplo (algo pensando no querer salir de la clase que ya se tiene) tener un array de instancias, y guardar los parámetros con los que se creó esa instancia manteniendo la asociación. Luego podrás comparar si son los mismos parámetros y crear o recuperar la conexión, o identificarlas por algún otro parámetro, o crear otra clase para manejar tus conexiones.

Adivino ahora tienes en mente la primera posibilidad que plantee. Te invito a intentarlo y a observar el comportamiento de clases ya existentes que abstraen las conexiones a BDs.