validacion de instanciacion, se puede hacer esto?

[malakian]

Hola muchachos, tengo una duda....
hice una clase para conectarme a la bd, hacer consultas, etc

Código PHP:

class database {
        var $id_conexion;
        function database(){
            $this->id_conexion = "";
            return ($this->conectardatabase("localhost","3306","root","1234"));
        }
        function conectardatabase($server, $pto, $user, $clave){
                $this->id_conexion = @mysql_connect($server.":".$pto, $user, $clave);
                return ($this->id_conexion);
        }
                function abrirdatabase($database){
                 return(mysql_select_db($database, $this->id_conexion));
        }
         } //fin clase
 

hasta ahi todo bien, funciona!
en el metodo conectar deberia retornarme true o false, eso lo recibe el metodo constructor y lo retorna (true / false)
ahora miren como instancio

Código PHP:

include("clase.php");
if ($obj=new database()){
  echo "sin problemas!"
}else{
  echo "no existe conexion";
}
 

pero por alguna razon siempre me muestra el mensaje "sin problemas", si cambio la clave o el usuario sigue saliendo el mensaje "sin problemas" pero deberia salir que no existe conexion...
hago un print_r($obj) y me muestra las propiedades correctamente...

Esta validacion en el momento de instanciar estaria bien hecha?? o como deberia hacerla? gracias

[GatorV]

Para eso son las excepciones, el constructor no regresa nada, simplemente regresa una instancia de la clase, por lo que el return que pongas ahí es nulo, no sirve de mucho, tienes que hacer algo así:

Código PHP:

class db
{
      public function __construct()
     {
                if(!$this->conectarDb()) {
                       throw new Exception('Imposible conectar');
                }
      }
}
 
try {
      $db = new database();
} catch (Exception $e) {
      echo $e->getMessage();
}
 

Saludos.

[malakian]

Muchas Gracias gatorV, a estudiar excepciones

[GatorV]

Ojo que las excepciones son precisamente para eso, para casos excepcionales, o cuando la clase no pueda trabajar de la forma que necesita debido a que no hay los datos, no las uses como control de flujo de tu programa, ya que no son para eso, sí lo que quieres es por ejemplo checar que haya una conexión, dale un método a tu clase que lo haga:

Código PHP:

class db
{
          private $conexion;
          public function __construct(/* config *)
          {
                   $this->config = $config;
          }
 
          public function connect()
          {
                   $connect = foo();
                   $this->conexion = $connect;
                   return ($connect->id ? true : false);
          }
 
           public function query($sql)
           {
                    if (!$this->conexion) {
                             throw new Exception('no hay conexión');
                    }
 
 
                     // stuff
           }
}
 
try {
      $db = new db(/* */);
      if (!$db->connect()) {
                echo "no se pudo conectar";
      } else {
               $result = $db->query($sql);
      }
} catch (Exception $e) {
      echo $e->getMessage();
}
 

Saludos.

[malakian]

GatorV, algo asi tenia, una propiedad llamada id_conexion, en esta quedaba almacenada el resultado de mysql_connect, luego hacia

Código PHP:

$db=new database;
if ($db->id_conexion){
  echo "OK";
}
 

solo que queria hacerlo en una sola linea

Código PHP:

if ($db=new database){
//bla bla
}
 

pense que podia establecer una condicion implicita y evaluarla segun el valor retornado desde la instancia(true / false)
Ahora, me confundo, osea que no se "deberia" usar exepciones para instanciar...

[GatorV]

No entiendo a que te refieres con "excepciones para instanciar", sí puedes lanzar excepciones en tu constructor, solo te digo que no controles el flujo con las excepciones, ya que son para precisamente, casos excepcionales.

Saludos.

[malakian]

Ok muchas gracias, igual seguire leyendo