duda con conecciones a una BD

[RIVERMILLOS]

hola quiciera saber si es malo tener un metodo publico que se conecte a una base de datos y dentro de este metodo llamar a uno privado que tambn se conecte a la base de datos.

EJ:

Código PHP:

class Clase{
 
private $conn;
 
public function metodo1(){
 
$this->conn=get_db_conn();
 
$this->metodo2();
 
mysql_close($this->conn);
}
 
private function metodo2(){
 
$conn=get_db_conn();
 
mysql_close($conn);
 
}
 
}
 

saludos

[GatorV]

Tema movido desde PHP a Frameworks y PHP orientado a objetos

[dargorg]

¿Por qué va a ser malo? De hecho, si no lo haces así, no podrás obtener datos de ninguna base de datos. De hecho, el ejemplo que has puesto es justamente cómo se debería hacer, para que así no se pueda acceder desde ningún otro sitio más que ese.

La única puntualización que yo pondría (que no sé si lo has hecho así) es que ese código debería ejecutarse en una clase padre (que sería la principal de todas, y por tanto, es heredada), más concretamente en el constructor de la clase, para así tener la conexión abierta desde el inicio de la ejecución del programa.

Un saludo y a programar!!

[RIVERMILLOS]

hola justamente hice algo asi , encontre una clase que maneja el patro singleton y creo una instancia de esta clase en el constructor.. para hacer lo que dijiste pero tengo una duda es cierto que no es necesario el mysql_close();?


saludos

[dargorg]

No es estrictamente necesario, pero si recomendable si quieres optimizar los recursos de tu máquina (servidor donde se ejecuten los scripts). También que liberes recursos con mysql_free_result.

Un saludo, y si necesitas algo más aquí me tienes.

[RIVERMILLOS]

Hola muchas gracias por la ayuda , estoy aprendiendo a optimizar mis clases php y me has ayudado arto , tengo otra duda si uso __destruct() en este metodo puedo colocar mysql_free_result y mysql_close?


saludos

[RIVERMILLOS]

hola mira los metodos que mirando por hay en varios lados construi...

Código PHP:

public function freeResults(){
 
    switch ($this->tipo){
         case 'mysql':mysql_free_result($this->stmt);
                           break;
         case 'postgress': pg_free_result($this->stmt);
                           break;
         break;
     }
        
    return true;
    }
 
   /*Destructor de la clase    */
   private function  __destruct() {
  switch ($this->tipo){
         case 'mysql':
 
    if (is_resource($this->link)) {
    mysql_close($this->link);
 
      }
   break;
         case 'postgress': 
   if (is_resource($this->link)) {
    pg_close($this->link);
 
   }
 
                           break;
         break;
     }
 
   
 
   }
 

son metodos para liberar recusos.


si pueden servir?


saludos

[dargorg]

Sí, teoricamente sí es lo correcto, pero yo no lo haría ahí puesto que muchas veces los destructores en PHP no llegan a ejecutarse, no funcionan como deberían. Acuérdate antes de hacer mysql_close y pg_close de llamar a la función freeResults y listo ;)