Error en script para hacer consultas SQL

[jruiz_fdez]

Hola a tod@s,

he programado una clase que realiza consultas SQL. El código de la clase es éste:

Código:

<?php

class Consulta extends ConexionDB{
  private $result;  // Declaración de atributos
  private $sql;

  function __construct($sql){
    $this->sql=@mysql_real_escape_string($sql, $this->dbLink) OR die("La consulta no es 
                        válida<br/>");
    $this->result=@mysql_query($this->sql, $this->dbLink) OR die("La consulta no ha podido 
                             llevarse a cabo<br/>");
  }

}

?>

Y el código de su super clase (ConexionDB) es éste:

Código:

<?php
class ConexionDB {
  private $dbHost;
  private $dbName;
  private $dbUser;
  private $dbPass;
  protected $dbLink;

  /**
   * Constructor de la Clase ConexionDB
  **/
  function __construct($dbName = 'votacion'){
    $this->dbHost='localhost';
    $this->dbUser='usuariodb';
    $this->dbPass='passworddb';
    $this->dbName=$dbName;
    $this->dbLink=@mysql_pconnect($this->dbHost, $this->dbUser, $this->dbPass);
    if(!$this->dbLink)
      throw new Exception('No se pudo conectar a la Base de Datos');
    if(!mysql_select_db($this->dbName,$this->dbLink))
      throw new Exception('No se pudo conectar a la Base de Datos: '.
                          $this->dbName);
  }

}

?>

He probado la clase ConexionDB de forma independiente y funciona perfectamente así que el problema está en la sub clase, en la clase Consulta
El problema es que si hago

Código:

<?php

require_once './ConexionDB.class.php';
require_once './Consulta.class.php';

$conexion=new ConexionDB();
$consulta=new Consulta('Select * from votacion');

?>

Me dice que La consulta no es válida y si, en lugar de tratar el error, dejo que el script falle, el error que me dice es que le estoy pasando un argumento NULL a la función mysql_real_escape_string como si la variable dbLink fuera igual a NULL pero en la línea

Código:

if(!$this->dbLink)

ya hago esa comprobación y no me salta la excepción.

¿Puede alguien ayudarme con ésto?

Muchas gracias de antemano por la ayuda.

Saludos,
Jose

[vgonga1986]

Buenas,

No sé exactamente cómo funciona el tema herencia en PHP, porque estoy algo despistado en este punto, pero analizando el código que pasas, tiene toda la pinta de que tu problema es porque no tienes bien enlazado el constructor de la clase hija con el de la clase padre. Del error que te da se puede deducir que $this->dbLink no está inicializado, en cuyo caso, mysql_real_escape_string() lanzará un warning y devolverá false. Para ver el mensaje de warning y asegurarte, quita el @ del principio de la función y a ver qué te imprime, igual te da más información.

Pero vamos, que tiene pinta de ser por eso, que el constructor de la clase hija no llama al de la clase padre, busca algo por ahí y nos comentas. Igual alguien te puede afinar más el tiro, ya te digo que no tengo mucha idea de este tema.

Un saludo.

Edito: es posible que tengas que utilizar parent.

[jotaincubus]

Amigo cuando tu haces herencias recuerda que en la clase hijo debes colocar el constructor del padre, sino esta no funcionara correctamente.

Código PHP:

class Hija extends Padre{
private $c, $d;
public function __construct($a, $b, $c, $d){
parent::__construct($a, $b);
$this->c = 'algo';
$this->d = 'otra cosa';
}
}
 

Espero te sirva.

[Eleazan]

Apredeadme, freídme a negativos, o lo q queráis... pero voy a intentar contestar (sin saber demasiado de POO)


Tu tienes dos clases, Conexion y Consulta. En conexion, tienes los métodos / atributos para conectarte a la bbdd, y en consulta (q extiendes esta ultima) tienes los de la consulta.

A mi forma de entender, si fuera POO solo deberias "iniciar" consulta, ya que esta heredaría todos los métodos / atributos de la primera.

Así puedes, para solucionarlo habria dos formas: el método _contruct de Conexion lo cambiar por otro (inicializa, por ejemplo), y lo llamas desde consulta al hacer el construct.....

O desde el construct de Consulta, llamas a parent::_construct (q seria el de Conexion) para inicializarla.

No sé si me he explicado, pero creo q por ahí van los tiros :)

Edit: jotaincubus se me ha adelantado :)

[jruiz_fdez]

Problema resuelto! Muchas gracias a los 3 por la ayuda.

Un saludo,
Jose