[SOLUCIONADO] Implementar patron singleton

giuli956 · #1 (**permalink**) 24/02/2014, 11:26

Bueno el caso es que realize un script Singleton.php, para conexion a la base de datos, pero ahora me piden implementarlo en un sistema que habiamos hecho previamente, en dicho sistema tenia un script de conexion:

Código PHP:

Ver original<?php
    function conectarDB(){
        $conexion=mysql_connect("localhost","root","");
        if (!$conexion){
            die('Error al conectarse a mysql: ' . mysql_error());   
        }else{
            return $conexion;
        }
    }
    function desconectarDB(){
        mysql_close($conexion);        
    }
 
?>

Y solo hacia

Código PHP:

Ver original$con= conectarDB()

Pero quiero saber como utilizar es te archivo:

Código PHP:

Ver original<?php
 
class conectarDB{
    static private $instancia=NULL;
    private $servidor="localhost";
    private $user="root";
    private $pass="";
    private $bd="sgu";
    private $link;
    private $stmt;
    private $array;
    
    static $_instance;
    
    private function __construct(){
        $this->conectar();
    }
    public static function getInstance(){
      if (!(self::$_instance instanceof self)){
         self::$_instance=new self();
      }
      return self::$_instance;
   }
   private function conectar(){
      $this->link=mysql_connect($this->servidor, $this->user, $this->pass);
      mysql_select_db($this->bd,$this->link);
}
    public function ejecutar($sql){
      $this->stmt=mysql_query($sql,$this->link);
      return $this->stmt;
   }
 
   public function obtener_fila($stmt,$fila){
      if ($fila==0){
         $this->array=mysql_fetch_array($stmt);
      }else{
         mysql_data_seek($stmt,$fila);
         $this->array=mysql_fetch_array($stmt);
      }
      return $this->array;
   }
 
 
   public function lastID(){
      return mysql_insert_id($this->link);
   }
}
?>

por ejemplo en este archivo:

Código PHP:

Ver original<?php
include("../php/modelo/Singleton.php");
$con=conectarDB::getInstance();
     function get_user_porid ($id){
    if(isset($id)){
        $sql="select *
        from personas, usuarios
        where
        personas.idpersona=$id
        ";   
        
        mysql_select_db("sgp",$con);
        $resultado=mysql_query($sql,$con) or die('Mi error es: '.mysql_error()); 
        $registro=mysql_fetch_object($resultado); 
      
        return $registro; 
}
}
if ($_GET['id']!=''){
$persona= get_user_porid($_GET['id']);
 
}
 
 
 
 
    
?>
<html>
    <head>
        <meta charset="UTF-8"/>
        <title>Modificar usuario seleccionado</title>
    </head>
    <body>
        <p align="center"><?php include("../php/header.php"); ?></p>
        <form action="includes/modificoPersona.php" method="post" name="form">
        <input type="hidden" name="id" value="<?php echo $persona->idpersona ?>" />
            <table align="center" width="50%">
            <tr>
            <td>
                <fieldset>
                    <legend>Relaize los cambios</legend>
                    <table width="50%" cellpadding="5" border="0">
                        
                        <tr>                
                            <td><p>Tipo de documento</p></td>
                            <td><input type="text" name="tipo_documento" value="<?php echo $persona->idTipoDocumento?>"/></td>
                        </tr>
                        <tr>
                            <td><p>Numero</p></td>
                            <td><input type="text" name="numero" value="<?php echo $persona->numeroDocumento?>"/></td>
                        </tr>
                        <tr>
                            <td><p>Apellido</p></td>
                            <td><input type="text" name="apellido" value="<?php echo $persona->apellido?>"/></td>
                        </tr>
                        <tr>                            
                            <td><p>Nombres</p></td>
                            <td><input type="text" name="nombres" value="<?php echo $persona->nombres?>"/></td>
                        </tr>                            
                            <td><p>Fecha de nacimiento</p></td>
                            <td><input type="text" name="fechanac" value="<?php echo $persona->fechaNacimiento?>"/></td>
                        <tr>
                            <td><p>Sexo</p></td>
                            <td><input type="text" name="sexo" value="<?php echo $persona->sexo?>"/></td>
                        </tr>
                        <tr>    
                            <td><p>Celular</p></td>
                            <td><input type="text" name="Cel" value="<?php echo $persona->telefonoMovil?>"/></td>
                        </tr>
                        <tr>
                            <td><p>Telefono</p></td>
                            <td><input type="text" name="Tel" value="<?php echo $persona->telefono?>"/></td>
                        </tr>
                        <tr>
                            <td><p>email</p></td>
                            <td><input type="text" name="email" value="<?php echo $persona->email?>"/></td>
                        </tr>
                        <tr>
                            <td><p>Domicilio</p></td>
                            <td><input type="text" name="dom" value="<?php echo $persona->domicilio?>"/></td>
                        </tr>
                        <tr>
                            <td><p>Provincia</p></td>
                            <td><input type="text" name="prov" value="<?php echo $persona->provincia?>"/></td>
                        </tr>
                        <tr>
                            <td><p>Localidad</p></td>
                            <td><input type="text" name="loc" value="<?php echo $persona->localidad?>"/></td>
                        </tr>
                        <tr>
                            <td><p>Pais</p></td>
                            <td><input type="text" name="pais" value="<?php echo $persona->pais?>"/></td>
                        </tr> 
                   </table>
                </fieldset>
                     <fieldset>
                    <legend>Datos de usuario</legend>
                    <table width="50%" cellpadding="5" border="0">
                        <tr>
                            <td><p>User</p></td>
                            <td><input type="text" name="user" value="<?php echo $persona->username?>"/></td>
                        </tr> 
                        <tr>
                            <td><p>Pass</p></td>
                            <td><input type="text" name="pass" value="<?php echo $persona->password?>"/></td>
                        </tr> 
                        <tr>
                            <td><p>Tipo de Usuario</p></td>
                            <td><input type="text" name="tipouser" value="<?php echo $persona->idTipoUsuario?>"/></td>
                        </tr> 
                       
                        <tr>
                            <td><input type="submit" name="bt_enviar" value="Listo"/></td>
                        </tr>
                   </table>
 
                    </fieldset>
             
                    </td>
                    </tr>
                </table>
       
        </form>
    </body>
    
</html>

Lo que hice fue:

Código PHP:

Ver originalinclude("../php/modelo/Singleton.php");
$con=new conectarDB();
$con->getInstance();

Obtengo el siguiente error:

Fatal error: Call to private conectarDB::__construct() from invalid context
Probe hacer publica el metodo constructor, pero igual da otros errores, no sera que tengo que pasar toda la aplicacion a objetos?

eits · #2 (**permalink**) 24/02/2014, 12:03

no puedes llamar al constructor directamente porque es privado, lo que debes de hacer es llamar al método estatico que es el que se encarga de devolver la misma instancia en caso de que ya exista una conección, de lo contrario te creará una

Código PHP:

Ver originalinclude("../php/modelo/Singleton.php");
    $instCon=conectarDB::getInstance();

en todo caso agrega un metodo público que te devuelva el link de conección

Código PHP:

Ver originalpublic function getLink()
{
   return $this->link;
}

y lo que tendrías que hacer a continuación es

Código PHP:

Ver originalinclude("../php/modelo/Singleton.php");
    $instCon=conectarDB::getInstance();
    $con = $instCon->getLink()

esto sería un modo de salir del problema pero no es lo optimo, lo ideal, es que, si ya tienes tu objeto de conección, pues, usala, así en lugar de usar el mysql_query usa el método ejecutar.

saludos.

giuli956 · #3 (**permalink**) 25/02/2014, 02:24

Bueno eits gracias por responder, probé hacer como me indicaste, lo de la conexion parece estar resuelto, por lo menos no da problemas.

Probe hacer luego:

Código PHP:

Ver original$resultado=conectarDB::ejecutar($sql);

para ejecutar la consulta, pero me dice:

Fatal error: Using $this when not in object context in H:\xampp\htdocs\sgu2\includes\php\modelo\Singleton .php on line 29

Es como que no esta creando un objeto conectarDB. Lo que observo es que la variable $sql no esta declarada en Singleton.php.

No se si puedes ayudarme te agradezco.

eits · #4 (**permalink**) 27/02/2014, 18:00

con el código que te dejé anteriormente erá todo lo que necesitabas, ahora bien, si quieres usar el método ejecutar deberás hacerlo de la siguiente manera

Código PHP:

Ver originalinclude("../php/modelo/Singleton.php");
$con=new conectarDB();
$con->getInstance();
$resultado=$con->ejecutar($sql);

Triby · #5 (**permalink**) 27/02/2014, 18:45

Hay cosas que no entiendo:

Programar una clase (eso está bien), pero... para:

1- Conectar a base de datos usando una librería obsoleta y que desaparecerá en PHP 5.5: http://www.forosdelweb.com/f18/anunc...oleta-1008145/

2- Hacer un spaghetti en el resto del código, mezclando php y HTML

Te sugiero buscar info sobre los siguientes temas:
- Programación orientada a objetos y patrones de diseño
- Patrón de diseño MVC

Si no te sientes cómodo con lo anterior, también podrías buscar sistemas de plantillas para facilitarte tanto el desarrollo de tu proyecto como cambios de diseño (html, css, etc.)

giuli956 · #6 (**permalink**) 28/02/2014, 05:15

Muchas gracias a los dos!! Y gracias @Triby por tus sugerencias..cualquier otra duda hago un nuevo tema.