PHP OO Ayuda con clase PHP

[fcmonier85]

Hola Foreros estoy implementando el siguiente codigo php y no me funciona el desconectarme de la BD espero me puedan ayudar en que es lo que estoy haciendo mal:

Cuando trato de ejecutar el metodo ejecutar_consulta($consulta), me da error en $this->desconectarme;

que podra ser

clase para conectarme y desconectarme a la BD

Código PHP:

<?php
    class class_mysql
    {   
        private $conexion;      
        private static $_instance = null;
 
        public static function getInstance() {
            if( self::$_instance == null ) 
            {
                self::$_instance = new self();
            }
            return self::$_instance;
         } 
        //metodo para conectarme a la BD
        public function conectarme_mysql()
        {
            if(!isset($this->conexion))
            {
                $this->conexion = (mysql_connect("localhost","root","xxx")) or die(mysql_error());
                mysql_select_db("ext_aprendiando",$this->conexion) or die(mysql_error());
            }
        }
        //cerrar conexion
        public function cerrar_conexion() 
        {
            return mysql_close($this->conexion); 
        }   
    }
?>
 

clase para hacer las onsultas a la BD

Código PHP:

<?php
    require_once "class_conexion_bd.php";
    
    class class_mysql_consultas
    {   
        private $total_consultas;
        
        function __construct()
        {
            $class_mysql = class_mysql::getInstance(); 
            $this->conecion = $class_mysql->conectarme_mysql(); 
            $this->desconectarme = $class_mysql->cerrar_conexion(); 
        }
        //se ejecuta una consulta
        public function ejecutar_consulta($consulta)
        {
            $this->conecion;
            $this->total_consultas++;
            $resultado = @mysql_query($consulta);
            if(!$resultado)
            {
                echo 'MySQL Error: ' . mysql_error();
                exit;
            }
            $this->desconectarme;
            return $resultado; 
        }
        //debuelbe un array con los resultados de la consulta donde la cual se ve los resultados por el nombre del campo de la BD
        public function fetch_assoc($consulta)
        { 
            //$this->coneccion();
            return mysql_fetch_assoc($consulta);
        }
        //me da la cantidad de duplas de un una consulta a la BD
        public function num_rows($consulta)
        {
            //$this->coneccion();
            return mysql_num_rows($consulta);
        }
        //
        public function getTotalConsultas()
        {
            return $this->total_consultas;
        }
    }
    
?>
 

[JaimeSavines]

Puedes escribir el error que te manda php, por favor

[pateketrueke]

estas bastante perdido, en ningún momento estas asignando la función aquí...

Código PHP:

  $this->desconectarme = $class_mysql->cerrar_conexion(); 
   

lo que haces es asignar el resultado de dicha función, osea que la ejecutas ahí mismo!!

[fcmonier85]

me pone esto:

MySQL Error: Access denied for user 'ODBC'@'localhost' (using password: NO)

[fcmonier85]

Pregunto pq no se que es lo que esta pasando... y como podria solucionar el problema ...

Iniciado por pateketrueke

estas bastante perdido, en ningún momento estas asignando la función aquí...

Código PHP:

  $this->desconectarme = $class_mysql->cerrar_conexion(); 
   

lo que haces es asignar el resultado de dicha función, osea que la ejecutas ahí mismo!!

[carlos_belisario]

los metodos te dan una respuesta cuando los llamas el echo es que quieres guardar la llamada a un metodo en un atributo para luego llamarlo cuando lo desees desde el atributo y eso no es correcto lo que te indico el amigo pateketrueke es la respuesta que necesitas que tienes que hacer llamar a $class_mysql->cerrar_conexion(); donde desees cerrar la conexion, porque de la manera que lo estas haciendo abres y cierras la conexion de una sola vez, espero que haber explicado bien, saludos

[pateketrueke]

Iniciado por fcmonier85

Pregunto pq no se que es lo que esta pasando... y como podria solucionar el problema ...

¿y no te das cuenta de lo que sucede?

¡estas conectando y desconectando en el constructor de tu clase!!

te sugiero ampliamente que repases la referencia del lenguaje, porque en si tu problema es meramente humano, no de código...

[Patriarka]

Lo que te indica el error es que tenes mal los datos de acceso a la base.

revisalos y cambialos por los que correspondan.

y si, estas despelotado............

pd: la conexion se cierra al final del srcipt.......

[fcmonier85]

grasias a todos ya vi lo que me decias pateketrueke, carlos_belisario y Patriarka. pero lo quito y lo pongo d ela siguiente manera y no me funciona: como deberia quedar:

Código PHP:

function __construct()
        {
            $class_mysql = class_mysql::getInstance(); 
            $this->conecion = $class_mysql->conectarme_mysql(); 
            //$this->desconectarme = $class_mysql->cerrar_conexion(); 
        }
        //se ejecuta una consulta
        public function ejecutar_consulta($consulta)
        {
            $this->conecion;
            $this->total_consultas++;
            $resultado = @mysql_query($consulta);
            if(!$resultado)
            {
                echo 'MySQL Error: ' . mysql_error();
                exit;
            }
            //$class_mysql->cerrar_conexion(); 
            return $resultado; 
        }
 

PD: Soy nuevo en esto.

[fcmonier85]

para explciar mejor hago la llamada pero no me funciona es decir pongo el code de la siguiente forma:

Código PHP:

function __construct()
        {
            $class_mysql = class_mysql::getInstance(); 
            //$this->conecion = $class_mysql->conectarme_mysql(); 
            //$this->desconectarme = $class_mysql->cerrar_conexion(); 
        }
        //se ejecuta una consulta
        public function ejecutar_consulta($consulta)
        {
            $class_mysql->conectarme_mysql();
            $this->total_consultas++;
            $resultado = @mysql_query($consulta);
            if(!$resultado)
            {
                echo 'MySQL Error: ' . mysql_error();
                exit;
            }
            $class_mysql->cerrar_conexion();
            return $resultado; 
        }
 

me da este error:
Fatal error: Call to a member function conectarme_mysql() on a non-object in dir_consultas_bd.php on line 17

[carlos_belisario]

esto tienes que estar es en un atributo para que el ambito abarque toda la clase, ya que como lo tienes la variable $class_mysql esta dentro del ambito es del constructor de tu clase que tienes que hacer cambiarlo a un atributo

Código PHP:

$this->class_mysql = class_mysql::getInstance();
 

de tal manera que en este atributo tienes el objeto que va a hacer la conexion con la base de datos

[fcmonier85]

Gracias carlos_belisario: me quedo de la siguiente forma y trabaja de lo mejor grasias:

Código PHP:

function __construct()
        {
            $this->class_mysql = class_mysql::getInstance(); 
        }
        //se ejecuta una consulta
        public function ejecutar_consulta($consulta)
        {
            //me conecto a la BD
            $this->class_mysql->conectarme_mysql();
            //ejecuto consulta
            $resultado = @mysql_query($consulta);
            if(!$resultado)
            {
                echo 'MySQL Error: ' . mysql_error();
                exit;
            }
            //me desconecto de la BD
            $this->class_mysql->cerrar_conexion();
            //mand resultado
            return $resultado; 
        }
 

[carlos_belisario]

que bien que hallas acomodado tu clase, es bueno que refuerces las bases de la OOP ya que cuando uno esta comenzando con eso es medio complicado pero reforzando las bases de estas te acostumbras y vas mejorando tus clases por cierto seria bueno que trabajaras un poco con PDO que es una clase nativa de PHP que te permite trabajar con base de datos y puedas heredar tus clase de esta como tal, saludos

[pateketrueke]

aun que yo no definiria la conexión en una propiedad, tan solo usaría esto:

Código PHP:

  class_mysql::getInstance()->cerrar_conexion(); 
   

[fcmonier85]

Gracias a todos y carlos_belisario vere la clase PDO ya que seria mejor... aunque si puedes decirme donde podría encontrar documentación o ejemplo de su uso te lo agradecería.

[carlos_belisario]

Pues creo que lo primero es ver el manual , tambien el amigo arley da una buena explicacion sobre PDO tambien el amigo Christopher tiene informacion sobre esta http://web2development.blogspot.com/...s-con-pdo.html pero es cuestion de googlear un poco y de seguro encontraras bastante informacion, saludos

[JaimeSavines]

Iniciado por fcmonier85

Gracias a todos y carlos_belisario vere la clase PDO ya que seria mejor... aunque si puedes decirme donde podría encontrar documentación o ejemplo de su uso te lo agradecería.

Usa PDO es una buena decisión, te dejo la liga al manual:
http://www.php.net/manual/es/book.pdo.php

[fcmonier85]

Hola nuevamente nesesitos de su ayuda denuevo:

clase consultas:

Código PHP:

<?php
    require_once "class_conexion_bd.php";
    
    class class_mysql_consultas
    {   
        function __construct()
        {
            $this->class_mysql = class_mysql::getInstance(); 
        }
        //se ejecuta una consulta
        public function dbquery($consulta)
        {
            //me conecto a la BD
            $this->class_mysql->conectarme_mysql();
            //ejecuto consulta
            $resultado = @mysql_query($consulta);
            if(!$resultado)
            {
                echo 'MySQL Error: ' . @mysql_error();
                exit;
            }
            //me desconecto de la BD
            $this->class_mysql->cerrar_conexion();
            //mand resultado
            return $resultado; 
        }
        //debuelbe un array con los resultados de la consulta donde la cual se ve los resultados por el nombre del campo de la BD
        public function dbarray($consulta)
        { 
            //me conecto a la BD
            //$this->class_mysql->conectarme_mysql();
            //$this->coneccion();
            $resultado = @mysql_fetch_assoc($consulta);
            //me desconecto de la BD
           // $this->class_mysql->cerrar_conexion();
            return $resultado;
            
        }
        //me da la cantidad de duplas de un una consulta a la BD
        public function dbrows($consulta)
        {   
            //me conecto a la BD
            //$this->class_mysql->conectarme_mysql();
            //$this->coneccion();
            $resultado = mysql_num_rows($consulta);
            //me desconecto de la BD
            //$this->class_mysql->cerrar_conexion();
            return $resultado;
        }
    }
    
?>
 

clase conexion:

Código PHP:

<?php
    class class_mysql
    {   
        private $conexion;      
        private static $_instance = null;
        //singelton
        public static function getInstance() {
            if( self::$_instance == null ) 
            {
                self::$_instance = new self();
            }
            return self::$_instance;
         } 
        //metodo para conectarme a la BD
        public function conectarme_mysql()
        {
            if(!isset($this->conexion))
            {
                $this->conexion = (mysql_connect("localhost","root","xxx")) or die(mysql_error());
                mysql_select_db("ext_aprendiando",$this->conexion) or die(mysql_error());
            }
        }
        //cerrar conexion
        public function cerrar_conexion() 
        {
            return mysql_close($this->conexion); 
        }   
    }
?>
 

pagina php

Código PHP:

<?php   
    require_once "class/class_consultas_bd.php";
 
    $class_mysql_consultas = new class_mysql_consultas();
      
    $consulta5 = $class_mysql_consultas->dbquery("SELECT nombre_usuario FROM ext_login WHERE nombre_usuario = 'admin'");
    $cantidad = $class_mysql_consultas->dbrows($consulta5) ;
    echo $cantidad."<br>" ;
  
    $consulta = $class_mysql_consultas->dbquery("SELECT nombre_usuario FROM ext_login");
    
    if($class_mysql_consultas->dbrows($consulta)>0)
    {
        while($resultados = $class_mysql_consultas->dbarray($consulta))
        {
            echo "ID: ".$resultados['nombre_usuario']."<br />";
        }
    }
?>
 

la cuestion es la siguiente cuando ejecuto el llamado a la funcion dbquery que la llamo en la pagina php dos veses me da bateo, es decir me pone este error:

4
MySQL Error: Access denied for user 'ODBC'@'localhost' (using password: NO)

y en el primero si se conecta pero ya para el segundo llamado no se conecta...

la cosa es que no se pq pasa pq me conecto y me desconecto cuando llamo ala funcion...

[pateketrueke]

en primer lugar no es necesario que conectes y desconectes en cada consulta, si conectas tan solo una ves en el constructor y -de hecho- jamás cierras la conexión debería ir todo bien, ya que php cierra las conexiones automáticamente al finalizar el script...

o bien, al momento de desconectar procura eliminar el valor de $this->conexion ya que al no hacerlo seguramente la conexión intenta usar un apuntador que ya has destruido...

[fcmonier85]

ahh gracias pateketrueke puse en el costructor de la clase class_mysql_consultas

Código PHP:

function __construct()
        {
            $this->class_mysql = class_mysql::getInstance(); 
            $this->class_mysql->conectarme_mysql();
        }
 

y me trabajo bien asi es como me decias ???

Saludos

[pateketrueke]

¡excelente!