Duda con clase mysql, saber si se ejecuta 2 veces la consulta

[neodani]

Buenas,

Tengo una pequeña duda,

Si tengo esta clase mysql

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($servidor, $user, $password, $db){
        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;
        }
    }
    
    //Numero de columnas    
    function num_rows(){
        return @mysql_num_rows($this->execute());
    }
    
    function __destruct(){
        @mysql_free_result($this->resource);
        @mysql_close($this->conexion);
    }
}
?>
 

Y lanzo este script de prueba

prueba.php

Código PHP:

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

Mi pregunta es si en la linea del
echo $db->num_rows();

¿Se vuelve a lanzar la consulta a la base de datos? de ser así, no es óptimo que la lance dos veces, no?

Se como solucionarlo si hago un count($datos) me dirá cuantas filas me arrojó la consulta SQL, pero si quisiera sacar ese número con la clase mysql sin tener que hacer de nuevo la consulta, ¿habría alguna forma? :S

Muchas gracias de antemano!

[GatorV]

A tu pregunta, sí ya que tu método num_rows manda a llamar a $this->execute(); Lo que tendrías que hacer es checar dentro de tu método execute() si ya existe un $this->resource, y en dado caso que exista usar ese en lugar de volverlo a lanzar.

Saludos.

[neodani]

Iniciado por GatorV

A tu pregunta, sí ya que tu método num_rows manda a llamar a $this->execute(); Lo que tendrías que hacer es checar dentro de tu método execute() si ya existe un $this->resource, y en dado caso que exista usar ese en lugar de volverlo a lanzar.

Saludos.

Es decir, que debería quedar así?

Código PHP:

public function execute(){
        if (!$this->resource){
            if(!($this->resource = mysql_query($this->sql, $this->conexion))){
                return null;
            }
        }
        $this->queries++;
        return $this->resource;
    }
 

Qué es lo que se guarda en el $this->resource? el resultado del mysql_query?

Si es así, porque no utilizar directamente en la función del número de columnas?

Código PHP:

function num_rows(){
        return @mysql_num_rows($this->resource);
    }
 

Muchas gracias de antemano!

[eulloa]

Según como lo tienes y creo entender, en el $this->resource lo que guardas es eso, el resource de la query q estás haciendo.
Si quieres más info sobre el resource type
Ahora lo que tendrías que hacer es más como isset($this->resource)

[GatorV]

O usar mejor is_resource y en efecto es mejor si usas directamente "return mysql_num_rows($this->resource);".

PD. no es tan recomendable que uses @ es mejor que tengas una mejor forma de controlar tus errores que solamente suprimirlos.

Saludos.

[neodani]

Iniciado por GatorV

O usar mejor is_resource y en efecto es mejor si usas directamente "return mysql_num_rows($this->resource);".

PD. no es tan recomendable que uses @ es mejor que tengas una mejor forma de controlar tus errores que solamente suprimirlos.

Saludos.

He entendido bien...?
Para ese caso del mysql_num_rows no haría falta usar el is_resource y es mejor utilizar directamente el return mysql_num_rows($this->resource);

Pero para este caso si que lo ideal sería que quedase así?

Código PHP:

private function __construct($servidor, $user, $password, $db){
        $this->conexion = mysql_connect($servidor, $user, $password);
        if (!is_resource($this->conexion)) {
            die('Can\'t connect : ' . mysql_error());
        }
        mysql_select_db($db, $this->conexion);
        $this->queries = 0;
        $this->resource = null;
    }
 

Y en lugar de

Código PHP:

public function execute(){
        if (!$this->resource){
            if(!($this->resource = mysql_query($this->sql, $this->conexion))){
                return null;
            }
        }
        $this->queries++;
        return $this->resource;
    }
 

Quedaría así?

Código PHP:

public function execute(){
        if (!is_resource($this->resource)) {
            if(!($this->resource = mysql_query($this->sql, $this->conexion))){
                return null;
            }
        }
        $this->queries++;
        return $this->resource;
    }
 

Muchas gracias de antemano

[eulloa]

Nop, se hablan de dos cosas diferentes

Código PHP:

public function execute(){
        if (!is_resource($this->resource)) {
                return null;
        }
        $this->queries++;
        return $this->resource;
    }
 
function num_rows(){
        return mysql_num_rows($this->resource); //SIN LA @
    }
 

[neodani]

Iniciado por eulloa

Nop, se hablan de dos cosas diferentes

Código PHP:

public function execute(){
        if (!is_resource($this->resource)) {
                return null;
        }
        $this->queries++;
        return $this->resource;
    }
 
function num_rows(){
        return mysql_num_rows($this->resource); //SIN LA @
    }
 

Buenas,

El que código que has puesto eulloa falta la consulta no? :S

Código PHP:

public function execute(){
        if (!is_resource($this->resource)) {
            if(!($this->resource = mysql_query($this->sql, $this->conexion))){
                return null;
            }
        }
        $this->queries++;
        return $this->resource;
    }
 
private function __construct($servidor, $user, $password, $db){
        $this->conexion = mysql_connect($servidor, $user, $password);
        if (!is_resource($this->conexion)) {
            die('Can\'t connect : ' . mysql_error());
        }
        mysql_select_db($db, $this->conexion);
        $this->queries = 0;
        $this->resource = null;
    }
 

Correcto?

[GatorV]

En efecto faltaría ejecutar la consulta. Es siempre no recomendable usar @ a menos que estes ya manejando el error, si no solo se silencia el error y no puedes ver que pasa.

Saludos.

[neodani]

Iniciado por GatorV

En efecto faltaría ejecutar la consulta. Es siempre no recomendable usar @ a menos que estes ya manejando el error, si no solo se silencia el error y no puedes ver que pasa.

Saludos.

Si quito la @ y muestro el mensaje de error

Código PHP:

$this->conexion = mysql_connect($servidor, $user, $password);
        if (!is_resource($this->conexion)) {
            die('No se pudo conectar a la base de datos: ' . mysql_error());
        }
 

No se pudo conectar a la base de datos: Access denied for user 'roo4t'@'localhost' (using password: YES)

No estoy dando información al visitante del usuario de la base de datos?

Y lo mismo a la hora de un fallo al seleccionar la base de datos

Código PHP:

$db_selected = mysql_select_db($db, $this->conexion);
        if (!$db_selected) {
            die ('No se pudo seleccionar la base de datos: ' . mysql_error());
        }
 

No se pudo seleccionar la base de datos: Unknown database 'hmgr4oupcat'


¿Debería eliminar el mysql_error() del final de cada die, para hacerlo más seguro, no?

Muchas gracias de antemano

[eulloa]

NO, lo que deberías hacer es solventar los errores. De todas formas estás en fase de desarrollo y ver los errores es importante es esta fase.
Luego, lo que te ha sugerido GatorV, es manejar los errores tu mismo. Puedes consultar excepciones
No te preocupes tanto por estas medidas de seguridad al principio. Más tarde lo harás

[neodani]

Iniciado por eulloa

NO, lo que deberías hacer es solventar los errores. De todas formas estás en fase de desarrollo y ver los errores es importante es esta fase.

Hola eulloa, no estoy en fase de desarrollo de la aplicación, sino más bien en fase de retoques de la clase mysql.

Los errores de arriba son provocados a consciencia, para que indicar eso, que con mysql_error() aparece información que el visitante podría "utilizar"

[eulloa]

Iniciado por neodani

Hola eulloa, no estoy en fase de desarrollo de la aplicación, sino más bien en fase de retoques de la clase mysql.

Retoques, desarrollo, en fin que no está oficialmente en línea ¿no?. Estás "probando", luego: fase de desarrollo

Iniciado por neodani

Los errores de arriba son provocados a consciencia

Provocados a conciencia (fase de desarrollo)

Iniciado por neodani

para que indicar eso, que con mysql_error() aparece información que el visitante podría "utilizar"

Entonces lo que te ha sugerido GatorV, maneja tu mismo los errores. Visita el link q te dejé, es del mismo manual

[GatorV]

En efecto es para cuando estas en fase de desarrollo en producción no debes de mostrar esos errores si no solamente "un error ocurrió", y manejar log de errores o algo por el estilo (usar excepciones es una muy buena forma).

Saludos.

[neodani]

Muchas gracias a ambos por las aclaraciones

Estoy intentando entender toda la clase de pies a cabeza, he añadido nuevos métodos y la estoy "documentando", pero hay algunas funciones que no acabo de entender por completo.

A continuación la clase al completo y un ejemplo de implementación.

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($servidor, $user, $password, $db){
    // Implementación del patrón SINGLETON para no repetir instancias
        if (is_null (self::$_singleton)) {
            self::$_singleton = new DataBase($servidor, $user, $password, $db);
        }
        return self::$_singleton;
    }
 
    private function __construct($servidor, $user, $password, $db){
    // Conecta al servidor
        $this->conexion = mysql_connect($servidor, $user, $password);
        if (!is_resource($this->conexion)) {
            die('No se pudo conectar a la base de datos: ' . mysql_error());
        }
     // Selecciona la base de datos
        $db_selected = mysql_select_db($db, $this->conexion);
        if (!$db_selected) {
            die ('No se pudo seleccionar la base de datos: ' . mysql_error());
        }
        $this->queries = 0;
        $this->resource = null;
    }
 
    public function execute(){
        if (!is_resource($this->resource)) {
            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 getArrayResult(){
    // Devuelve un array asociativo de la consulta (por nombres de campo en vez de por indice de campo)
        if (!($cur = $this->execute())){
            return null;
        }
        $array = array();
        while ($row = @mysql_fetch_object($cur)){
            $array[] = $row;
        }
        return $array;
    }
    
    public function setQuery($sql){
    // Carga la consulta SQL
        if(empty($sql)){
            return false;
        }
        $this->sql = $sql;
        return true;
    }
 
    public function freeResults(){
        // Libera el resultado de la memoria y continua con el script
        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;
        }
    }
    
    public function getNumFilas(){
        // Devuelve el numero total de filas (horizontal) de la consulta
        return mysql_num_rows($this->resource);
    }
    function getNumColumnas(){
        // Devuelve el numero total de columnas (vertical) de la consulta
        return mysql_num_fields($this->resource);
    }
    public function getTotalConsultas(){
        // Devuelve el numero total de consultas realizadas
        return $this->queries;  
    }
    
    public function depura($data){
    // Funcion para evitar el SQL INJECTION
        // Elimina los espacios en blanco
        $data = trim($data); 
        // Aplicará stripslashes si está habilitado magic_quotes_gpc
        if(get_magic_quotes_gpc())
        {
            $data = stripslashes($data);
        }
        // Hace falta una conexión a mySQL para usar esta funcion
        $data = mysql_real_escape_string($data);
        return $data;
    }
 
 
    public function getUltimoID(){
        if(!($this->resource = mysql_query(mysql_insert_id(), $this->conexion))){
            return false;
        }
        $this->queries++;
        return $this->resource;
    }
    
    public function fetch_array(){   
        return mysql_fetch_array($this->resource);  
    }
    
    function __destruct(){
        // Libera el resultado de la memoria y cierra la conexión
        mysql_free_result($this->resource);
        mysql_close($this->conexion);
    }
}
?>
 

Implementación

Código PHP:

<?php
include('class-mysql.php');
 
// Hacemos la conexión
$db = DataBase::getInstance('localhost','root','password','hmgroupcat');
 
// Prueba de valores
$valor1=1;
$valor2='2009-10-06 19:03:49';
 
$valor1=$db->depura($valor1);
$valor2=$db->depura($valor2);
 
// Hacemos la consulta segura:
$query = sprintf("SELECT ID,post_date,post_content FROM wp_posts WHERE ID='%s' OR post_date='%s'",
    $valor1, $valor2);
$db->setQuery($query);
 
// La ejecutamos y al mismo tiempo obtenemos un arreglo de objetos
// con los campos especificados en la consulta como propiedades.
$datos = $db->getArrayResult();
 
//$datos = $db->fetch_array();
//echo $db->getUltimoID();
 
// Otro ejemplo de depuración de datos de entrada
$_POST['username']='n-1"\/%aDan';
$_POST['password']='23/""\1235a*!';
echo $username = $db->depura($_POST['username']);
echo "<br/>";
echo $password = $db->depura($_POST['password']);
 
echo $db->getNumFilas();
echo "<br/>";
echo $db->getNumColumnas();
echo "<br/>";
echo $db->getTotalConsultas();
 
echo "<pre>";
print_r($datos);
echo "</pre>";
 
// Los imprimimos directamente en pantalla...
/*foreach($datos as $campo){
    echo 'ID: '.$campo->ID;
    echo 'Nombre: '.$campo->post_name;
    echo 'Grupo: '.$campo->post_status;
    echo '<br />';
}
*/
?>
 

Y ahora las TRES funciones que me gustaría que me explicarais que es lo que hacen...

Código PHP:

public function execute(){
        if (!is_resource($this->resource)) {
            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 loadObject(){
        if ($cur = $this->execute()){
            if ($object = mysql_fetch_object($cur)){
                mysql_free_result($cur);
                return $object;
            }
            else {
                return null;
            }
        }
        else {
            return false;
        }
    }
 

Esta no he conseguido que funcione:

Código PHP:

public function getUltimoID(){
        if(!($this->resource = mysql_query(mysql_insert_id(), $this->conexion))){
            return false;
        }
        $this->queries++;
        return $this->resource;
    }
 

Y esta de aquí function getArrayResult() pensaba que se podría hacer de la siguiente manera pero no funciona, es muy diferente?

Código PHP:

public function fetch_array(){   
        return mysql_fetch_array($this->resource);  
    }
 

Muchas gracias de antemano!

[GatorV]

Si lees el manual de PHP, solamente debes de usar mysql_insert_id() con el parámetro a la conexión, no la debes de pasar a mysql_query, o sea que tu función quedaría así:

Código PHP:

public function getUltimoID()
{
    $ultimoId = mysql_insert_id($this->conexion);
    return $ultimoId;
}
 

Tu función fetch_array lo que hace es regresar un array de tu ultimo resultado.

Saludos.

[neodani]

Iniciado por GatorV

Si lees el manual de PHP, solamente debes de usar mysql_insert_id() con el parámetro a la conexión, no la debes de pasar a mysql_query, o sea que tu función quedaría así:

Código PHP:

public function getUltimoID()
{
    $ultimoId = mysql_insert_id($this->conexion);
    return $ultimoId;
}
 

Lo ejecuté, y me mostró 0 como ultimoId

No debería mostrar el último ID autonumerico del último valor?

Código PHP:

// Hacemos la consulta segura:
$query = sprintf("SELECT ID,post_date,post_content FROM wp_posts WHERE ID='%s' OR post_date='%s'",
    $valor1, $valor2);
$db->setQuery($query);
 
// La ejecutamos y al mismo tiempo obtenemos un arreglo de objetos
// con los campos especificados en la consulta como propiedades.
$datos = $db->getArrayResult();
//echo $db->countAffectedRows();
//$datos2 = $db->fetchAll();
echo $db->getUltimoID();
 

Iniciado por GatorV

Tu función fetch_array lo que hace es regresar un array de tu ultimo resultado.

Saludos.

Si la consulta me devuelve 10 filas solo mostrará la última fila en formato array, correcto?

Entonces esta función debería mostrar todas las filas en el array $rows no?

Código PHP:

public function fetchAll()
    {
          $rows = array();      
          while ($row = mysql_fetch_array( $this->resource, MYSQL_ASSOC )) {
              array_push($rows, $row);
          }
          return $rows;
    }
 

Cuando lo ejecuto, me da error en la linea:
while ($row = mysql_fetch_array( $this->resource, MYSQL_ASSOC )) {


Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in C:\AppServ\www\00-laboratorio_test\secciones\mysql\clase_mysql_1\cla ss-mysql.php on line 142

Por qué los argumentos no son correctos, si segun el manual son
( resource $result [, int $result_type = MYSQL_BOTH ] )

La siguiente pregunta que tengo es, que diferencia habría en conseguir hacerlo de la manera anterior o esta de aquí.

Código PHP:

public function getArrayResult(){
    // Devuelve un array asociativo de la consulta (por nombres de campo en vez de por indice de campo)
        if (!($cur = $this->execute())){
            return null;
        }
        $array = array();
        while ($row = @mysql_fetch_object($cur)){
            $array[] = $row;
        }
        return $array;
    }
 

Muchas gracias de antemano!

[GatorV]

Te devuelve 0 ya que necesita ser llamado después de llamar a un INSERT() o un Stored Procedure que llame a un INSERT().

Del otro problema te lo muestra ya que $this->resource no es un recurso valido, prueba imprimir antes el valor de mysql_error() para que veas si tu consulta fallo.

Saludos.

[neodani]

Iniciado por GatorV

Te devuelve 0 ya que necesita ser llamado después de llamar a un INSERT() o un Stored Procedure que llame a un INSERT().

Del otro problema te lo muestra ya que $this->resource no es un recurso valido, prueba imprimir antes el valor de mysql_error() para que veas si tu consulta fallo.

Saludos.

Me diste la clave :D

Faltaba ejecutarlo, ahora funciona

Código PHP:

public function fetchAll()
    {
        if (!($cur = $this->execute())){
            return null;
        }
        $rows = array();      
        while ($row = mysql_fetch_array( $this->resource, MYSQL_ASSOC )) {
            array_push($rows, $row);
        }
        return $rows;
    }
 

Esta linea de aquí lo que hace es intentar ejecutar la consulta, si da problemas retornará null no?

Código PHP:

if (!($cur = $this->execute())){
  return null;
}
 

Otra pregunta,

¿Por qué cuando ejecuto los dos métodos a la vez, solo me muestra datos en el que ejecuto primero, en este caso $datos1 contiene información y $datos2 está vacio...?

Código PHP:

// Hacemos la consulta segura:
$query = sprintf("SELECT ID,post_date,post_content FROM wp_posts WHERE ID='%s' OR post_date='%s'",
    $valor1, $valor2);
    
$db->setQuery($query);
 
$datos1 = $db->getArrayResult();
$datos2 = $db->fetchAll();
 
echo "<pre>";
print_r($datos1);
echo "</pre>";
echo "<pre>";
print_r($datos2);
echo "</pre>";
 

Muchas gracias de antemano!

[GatorV]

Eso es porque tu clase esta mal diseñada desde un principio y solo funciona para 1 sola consulta, si quieres ver una implementación múltiple te recomiendo leer como funciona PDO que precisamente separa su clase en PDO y PDOStatment, o Zend_Db que tiene Zend_Db_Adapter y Zend_Db_Statement.

Saludos.

[neodani]

Iniciado por GatorV

Eso es porque tu clase esta mal diseñada desde un principio y solo funciona para 1 sola consulta, si quieres ver una implementación múltiple te recomiendo leer como funciona PDO que precisamente separa su clase en PDO y PDOStatment, o Zend_Db que tiene Zend_Db_Adapter y Zend_Db_Statement.

Saludos.

Me dejas a cuadros O_o! tan mal está que no se puede retocar la parte "errónea"??

Me recomiendas que use en vez de mi clase, la clase PDO?

Saludos y gracias

[GatorV]

Claro es mucho más recomendado usar PDO, por muchas razones, la primera, que es nativa (programada en C), otra es que es múltiple, puedes usar muchas bases de datos (mysql, mssql, sqlite, oracle, postgresql, odbc).

Otra es porque tiene un diseño mucho más abstracto, que te permite mantener más limpio tu implementación.

Para "arreglar" tu clase, debes de partirla en 2, una que sea el Connection Manager, y otro el Result Manager, es decir, una clase se encarga de manejar lo que es la conexión al servidor, enviar querys/etc. Tu otra clase se encarga de manejar un resultset de mysql, o sea, descargar resultados, número de filas, etc.

Esto es para que puedas tener una implementación más limpia, por ejemplo:

Código PHP:

$db  = Database(/** .. **/);
//...
$result1 = $db->query(/** query **/);
$result2 = $db->query(/** query **/);
while ($row = $result1->fetchRow()) {
       // $row
}
while ($row = $result2->fetchRow()) {
       // $row
}
 

Saludos.

[atrianaster]

Iniciado por GatorV

Si lees el manual de PHP, solamente debes de usar mysql_insert_id() con el parámetro a la conexión, no la debes de pasar a mysql_query, o sea que tu función quedaría así:

Código PHP:

public function getUltimoID()
{
    $ultimoId = mysql_insert_id($this->conexion);
    return $ultimoId;
}
 

Tu función fetch_array lo que hace es regresar un array de tu ultimo resultado.

Saludos.

GatorV distes en el punto clave.

En este caso no seria necesario pasarle $this->conexion como argumento, ya que es un parámetro opcional que de no ser especificado el método asume la ultima conexión abierta, la cual siempre será la misma porque la clase utiliza el patrón Singleton.

Código PHP:

int mysql_insert_id  ([  resource $link_identifier  ] )
 

Código PHP:

public function getUltimoID()
{
  $ultimoId = mysql_insert_id();
  return $ultimoId;
}
 

En resumen menos código más claridad.

PDO es más recomendable.

[GatorV]

@atrianaster

De hecho no, precisamente es el problema, su clase simplemente es un "wrapper" a unas funciones de MySQL y no le va a brindar completamente lo que necesita.

Aunque te "ahorres" el link, es recomendable que lo ponga, ya que de hecho no es recomendable el uso de Singleton para clases del tipo conexiones a DB, ya que limitan el poder usar tu clase más de una vez en una sola ejecución.

[atrianaster]

Iniciado por GatorV

ya que limitan el poder usar tu clase más de una vez en una sola ejecución.

El patrón de diseño"Singleton" está diseñado para restringir la creación de objetos pertenecientes a una clase.

Esta clase DataBase hace uso de este patrón, donde en el constructor abre la conexión a la base de datos MySQL utilizando la librería de funciones php_mysql.dll.

No se limita usar la clase, la clase la puedes usar cuantas veces quieras, así como invocar todos sus métodos indefinidamente una vez teniendo una instancia de la misma.
Lo que si se limita es la conexión a la base de datos que siempre será la misma, es decir cuando invocas al método getInstance este devuelve un enlace identificativo el cual será reutilizado una y otra vez.

Ventas:

1.) Que nuestra aplicación web ahorre recursos del servidor.
2.) Tenemos bien controlado la creación de objetos, al no permitir que nuestra aplicación tenga innumerable y descontroladas conexiones innecesarias abiertas a una misma base de datos.

Iniciado por GatorV

ya que de hecho no es recomendable el uso de Singleton para clases del tipo conexiones a DB

No entiendo porque planteas esto.

Atento a tus comentarios.
Saludos y mis respetos como desarrollador.

[GatorV]

Lo que pasa es un problema clave, que pasa cuando quieres trabajar en dos bases de datos diferentes en el mismo archivo?, es ahi cuando se ve una deficiencia de usar Singleton para clases que sean conexiones.

Si bien como dices:

Iniciado por atrianaster

Ventas:

1.) Que nuestra aplicación web ahorre recursos del servidor.
2.) Tenemos bien controlado la creación de objetos, al no permitir que nuestra aplicación tenga innumerable y descontroladas conexiones innecesarias abiertas a una misma base de datos.

Esto mismo lo puedes ocupar al usar algún otro patron de diseño, como Registry, con el cual puedes crear tu instancia de tu clase, y registrarla en el registro para usarla mucho más adelante, y no te limitas a usar solo una instancia por ejecución.

Saludos.

[atrianaster]

Iniciado por GatorV

Lo que pasa es un problema clave, que pasa cuando quieres trabajar en dos bases de datos diferentes en el mismo archivo?, es ahi cuando se ve una deficiencia de usar Singleton para clases que sean conexiones.
Saludos.

Positivo, aunque existen alternativas:

Código PHP:

class DataBase {
private $conexion;
private $resource;
private $sql;
public static $queries;
private static $_singleton;
private static $_db;
private static $_servidor;
 
public static function getInstance($servidor, $user, $password, $db) {
   if ( (!isset (self::$_singleton)) || ($db != self::$_db) || ($_servidor != self::$_servidor) ) {
          $c = __CLASS__;
          self::$_singleton = new $c($servidor, $user, $password, $db);
          self::$_db  = $db;  
          self::$_servidor = $servidor;        
        }
        return self::$_singleton;
}
 

[GatorV]

De todos modos, muestrame como se trabajaría en el mismo script con 2 bases de datos al mismo tiempo, de la forma que tu expones tendría que trabajar en 1 base de datos, cerrar la conexíon y abrir otra para la segunda, no pueden interactuar entre ellas, siempre es mas limpio algo así:

Código PHP:

$db = new db($datosConexionPrincipal);
Registry::save('defaultDb', $db);
$db2 = new db($datosSegundaConexion);
 
$result = $db->query("SELECT * FROM foo");
while ($row = $result->fetch()) {
         $db2->insert($sql);
}
 

Al usar un patrón como Registry puedes guardar la instancia por defecto de tu clase Db y así te evitas estar creando objetos de forma innecesaria y se consigue lo que en teoría se haría con el Singleton y aparte queda la limpieza y el poder de poder usar otras bases de datos.

Saludos.

[neodani]

Iniciado por GatorV

Al usar un patrón como Registry puedes guardar la instancia por defecto de tu clase Db y así te evitas estar creando objetos de forma innecesaria y se consigue lo que en teoría se haría con el Singleton y aparte queda la limpieza y el poder de poder usar otras bases de datos.

¿Necesitas usar el Registry en PDO para poder conectarte a dos base de datos distintas?

Muchas gracias

[GatorV]

No, el Registry simplemente es para llevar un registro de cosas y evitar estar creando objetos de forma innecesaria, en este caso puede ser una conexíon a una base de datos, u otro tipo de cosas.

Para PDO basta con instanciar dos objetos diferentes y no tienes problema.

Saludos.