[SOLUCIONADO] problema al realizar consulta mysql

rodrypaladin · #1 (**permalink**) 14/02/2014, 16:08

Buenas chicos. entre que soy novato y que llevo muchas horas con ello, no consigo solucionarlo y estoy algo bloqueado.

No sabía si postear el hilo en el foro de Mysql o en el de PHP, pero creo que el error es de la consulta y por ello lo publico aquí.

Tengo la siguiente tabla llamada extension

Cita:

id / ext / definicion / software / caracteristicas / historia
1 / mp3 / blabla... / blabla... / blabla..... / blabla..

Lo que intento realizar es que muestre el contenido del campo definición de la fila que la extensión = "$extension" ( Esta variable me viene desde un formulario, en este ejemplo...mp3)

Código PHP:

Ver original<html>
    <head>
        <meta charset="UTF-8">
        <?php 
        $extension = $_GET["extension"]; 
        $extensionMAY = strtoupper($extension); 
        echo "<title>.$extensionMAY Información sobre la extension .$extensionMAY</title>";
        ?>
    </head>
    <body>
    <?php
      if (!empty($_GET["extension"])) {
    echo "<h1>Información sobre la extensión .$extensionMAY</h1><br />";
    echo "<h2>¿ Que es un archivo .$extensionMAY ?</h2><br />"; 
    // realizamos consulta
    $consulta = mysqli_query($conexion, "SELECT definicion FROM extension WHERE ext='$extension'");
      if($consulta == FALSE) {
                  echo "Error en la consulta SQL";
                  exit(); }
    
   //mostramos los datos
        while ($fila = mysqli_fetch_row($consulta)) {
        foreach($fila as $datoFila) {
            echo $datoFila; }
        }
  
  // finalizamos
        mysqli_free_result($consulta);
        mysqli_close($conexion);
              
        } else {
        ?>
        <form action="busqueda.php" target="_top" method="GET"/>archivo.
        <input type="text" name="extension" size="5" value=""/><input type="submit" value="Buscar" /><br/>
        </form>
        <?php
        echo "No ha buscado nada, escriba la extension"; }
?>
 
    </body>
</html>

Los errores al ejecutarlo son los siguientes:

Notice: Undefined variable: conexion in C:\wamp\www\infoextensiones\busqueda.php on line 21
Warning: mysqli_query() expects parameter 1 to be mysqli, null given in C:\wamp\www\infoextensiones\busqueda.php on line 21

Y recalco la línea 21:

Código MySQL:

Ver original$consulta = mysqli_query($conexion, "SELECT definicion FROM extension WHERE ext='$extension'");

gnzsoloyo · #2 (**permalink**) 14/02/2014, 16:52

Cita:

Notice: Undefined variable: conexion in C:\wamp\www\infoextensiones\busqueda.php on line 21
Warning: mysqli_query() expects parameter 1 to be mysqli, null given in C:\wamp\www\infoextensiones\busqueda.php on line 21

Eso es un error de PHP, no de MySQL. Está diciendo que la variable $conexion no existe en el punto de ejecución.
Por su lado, la query está perfectamente escrita, en consecuencia si hay un problema, no está allí.

OFF-TOPIC.

Movido a PHP.

Lo que yo no veo es dónde se crea esa variable $conexion, pero obviamente no se ha creado al momento de ejecutar la consulta...

rodrypaladin · #3 (**permalink**) 14/02/2014, 18:13

Si.. es lo que tiene no meter include 'funciones.php';

Ahora el error es en funciones.php linea 8
Fatal error: Cannot redeclare conectar_a_bd() (previously declared in C:\wamp\www\infoextensiones\funciones.php:2) in C:\wamp\www\infoextensiones\funciones.php on line 8

Código PHP:

Ver original<?php
function conectar_a_bd() { 
$conexion = mysqli_connect('localhost','root','','extensiones');
if ($conexion == FALSE) {
    echo "Error en la conexion a la BD: " . mysqli_error($conexion);
    exit();
    } else { return $conexion; }
}
?>

gnzsoloyo · #4 (**permalink**) 14/02/2014, 19:14

Básicamente eso sucede en situaciones como:

- Tienes la misma función declarada en dos scripts distintos.
- Tienes la misma función definida en dos partes del mismo script.
- El script en el que está la definición está siendo incluido dos veces en diferentes momentos de la ejecución, y por tanto se toma como redefinición.

Yo me inclino por esta última posibilidad. Es bastante habitual que accidentalmente se invoque en forma transitiva un mismo script sin verlo a simple vista.

Es un poco más complicado de encontrar porque deberás seguir la lógica de ejecución paso a paso para encontrar donde se produce el doble include.

#5 (**permalink**) 14/02/2014, 19:35

pega funciones.php aquí

rodrypaladin · #6 (**permalink**) 14/02/2014, 19:42

Tenía 2 includes 'funciones'; y ha desaparecido el error, pero me aparecen otros:

Notice: Undefined variable: conexion in C:\wamp\www\infoextensiones\busqueda.php on line 23
Warning: mysqli_query() expects parameter 1 to be mysqli, null given in C:\wamp\www\infoextensiones\busqueda.php on line 23

Código PHP:

Ver original<!DOCTYPE html>
<!--
Página de búsqueda, donde una vez pongan la extensión aparecerá toda
la información.
-->
<html>
    <head>
        <meta charset="UTF-8">
        <?php 
        include 'funciones.php';
        $extension = $_GET["extension"]; 
        $extensionMAY = strtoupper($extension); 
        echo "<title>.$extensionMAY Información sobre la extension .$extensionMAY</title>";
        ?>
    </head>
    <body>
    <?php
    conectar_a_bd();
      if (!empty($_GET["extension"])) {
    echo "<h1>Información sobre la extensión .$extensionMAY</h1><br />";
    echo "<h2>¿ Que es un archivo .$extensionMAY ?</h2><br />"; 
    // realizamos consulta
    $consulta = mysqli_query($conexion, "SELECT definicion FROM extension WHERE ext='$extension'");
      if($consulta == FALSE) {
                  echo "Error en la consulta SQL";
                  exit(); }
    
   //mostramos los datos
        while ($fila = mysqli_fetch_row($consulta)) {
        foreach($fila as $datoFila) {
            echo $datoFila; }
        }
  
  // finalizamos
        mysqli_free_result($consulta);
        mysqli_close($conexion);
              
        } else {
        ?>
        <form action="busqueda.php" target="_top" method="GET"/>archivo.
        <input type="text" name="extension" size="5" value=""/><input type="submit" value="Buscar" /><br/>
        </form>
        <?php
        echo "No ha buscado nada, escriba la extension"; }
?>
 
    </body>
</html>

No entiendo si supuestamente la consulta mysql esta bien construida :S

#7 (**permalink**) 14/02/2014, 19:50

pega el contenido de funciones.php aquí, que lo depuramos

rodrypaladin · #8 (**permalink**) 14/02/2014, 19:57

Funciones.php

Código PHP:

Ver originalfunction conectar_a_bd() { 
$conexion = mysqli_connect('localhost','root','','extensiones');
if ($conexion == FALSE) {
    echo "Error en la conexion a la BD: " . mysqli_error($conexion);
    exit();
    } else { return $conexion; }
}

#9 (**permalink**) 14/02/2014, 20:05

Código PHP:

Ver original<!DOCTYPE html>
<!--
Página de búsqueda, donde una vez pongan la extensión aparecerá toda
la información.
-->
<html>
    <head>
        <meta charset="UTF-8">
        <?php 
        include 'funciones.php';
        $extension = $_GET["extension"]; 
        $extensionMAY = strtoupper($extension); 
        echo "<title>.$extensionMAY Información sobre la extension .$extensionMAY</title>";
        ?>
    </head>
    <body>
    <?php
    conectar_a_bd();
      if (!empty($_GET["extension"])) {
    echo "<h1>Información sobre la extensión .$extensionMAY</h1><br />";
    echo "<h2>¿ Que es un archivo .$extensionMAY ?</h2><br />"; 
    // realizamos consulta
    $consulta = mysqli_query($conexion, "SELECT definicion FROM extension WHERE ext='$extension'");

Código PHP:

Ver original$conexion = conectar_a_bd();

rodrypaladin · #10 (**permalink**) 14/02/2014, 20:18

Mil gracias lolainas

#11 (**permalink**) 14/02/2014, 21:54

Comprueba esta solución y dime si no es más difícil cometer errores.

extension.php (encapsulamiento de la extensión, __get proporciona acceso a los campos)

Código PHP:

  class Extension {

    private $db;

    private $nombre;

 
    function __construct(mysqli $db, $nombre) {

        $this->db = $db;

        $this->nombre = $nombre;

    }

 
    function __get($field) {

        return $this->db->query("SELECT $field FROM extension WHERE nombre = '$this->nombre';")->fetch_assoc()[$field];

    }

}

index.php (controlador, el que manda)

Código PHP:

  // config

$host = 'localhost';

$user = 'root';

$pwd = null;

$db = 'extensiones';

 
// comienzo

$extension = filter_input(INPUT_GET, 'extension');

 
if ($extension) {

    include 'extension.php';

    $ext = new Extension(new mysqli($host, $user, $pwd, $db), $extension);

    include 'plantilla.php';

}

else

    echo 'Extensi&oacute;n no recibida.<pre>Ejemplo: index.php?extension=nombre</pre>';

plantilla.php (lo que viene siendo una plantilla)

Código PHP:

 
<!DOCTYPE html>

<html lang="es">

    <head>

        <meta charset="latin1">

        <title>Extensi&oacute;n:&nbsp;<?= $ext->nombre ?></title>

    </head>

    <body>

        <h1>Extensi&oacute;n:&nbsp;<?= $ext->nombre ?>&nbsp;|&nbsp;Software:&nbsp;<?= $ext->software ?></h1>

        <ul>

            <li>

                <h2>Definici&oacute;n</h2>

                <p><?= $ext->definicion ?></p>

            </li>

            <li>

                <h2>Caracter&iacute;sticas</h2>

                <p><?= $ext->caracteristicas ?></p>

            </li>

            <li>

                <h2>Historia</h2>

                <p><?= $ext->historia ?></p>

            </li>

        </ul>

    </body>

</html>

Espero te sirva. Saludos.

rodrypaladin · #12 (**permalink**) 15/02/2014, 07:43

Hola @lolainas, estoy completamente seguro que tu código está mejor y es más correcto, pero llevo poco tiempo con php y si me cuesta programar en una forma referencial imaginate ahora orientado a objetos, creo que es mejor ir usando códigos que pueda entender, ¿no?

#13 (**permalink**) 15/02/2014, 15:31

Lógicamente, debes entender lo que haces sino mal vamos.
En un post que acabo de intentar ayudar a un chico se me han tirado al cuello por escribir POO y no adaptarme a PHP4 en la construcción de arrays, cuando la última versión estable es la 5.4, creo que es la filosofía del foro, que no permite ir mas allá de la programación simple y lineal y todo lo que no sea así está mal o son malas prácticas. Son muchos los usuarios que siguen la estela de otros con más mensajes que se dedican a echar broncas como si de un aula de colegio se tratara.
Sólo te puedo aconsejar que no tengas miedo a la POO, tu satisfacción será más grande y tus códigos más legibles y más entendibles incluso para terceros. Adéntrate, te gustará te lo aseguro, claro que siempre poco a poco y paso a paso, sin prisa, pero sin pausa.

rodrypaladin · #14 (**permalink**) 15/02/2014, 16:20

Por supuesto que cuando cuando me sienta capaz, me metere en POO, pero prefiero controlar bien las cosas...

Ya tengo las consultas realizadas y mostradas correctamente pero mi código queda muy repetitivo y seguro que no es una buena práctica.

este es el código:

Código PHP:

Ver original<!DOCTYPE html>
<html>
    <head>
        <meta charset="UTF-8">
        <?php
        header("Content-Type: text/html;charset=utf-8");
        include 'funciones.php';
        $extension = $_GET["extension"]; 
        $extensionMAY = strtoupper($extension); 
        echo "<title>.$extensionMAY Información sobre la extension .$extensionMAY</title>";
        ?>
    </head>
    <body>
    <?php
    $conexion = conectar_a_bd();
      if (!empty($_GET["extension"])) {
    echo "<h1>Información sobre la extensión .$extensionMAY</h1><br />";
    echo "<ul>";
    echo    "<li>"; 
    echo "<h2>¿ Que es un archivo .$extensionMAY ?</h2><br />"; 
    // realizamos consulta
    $consulta = mysqli_query($conexion, "SELECT definicion FROM extension WHERE ext='$extension'");
      if($consulta == FALSE) {
                  echo "Error en la consulta SQL";
                  exit(); }
    
   //mostramos los datos
        while ($fila = mysqli_fetch_row($consulta)) {
        foreach($fila as $datoFila) {
            echo $datoFila; }
        }
    echo    "<li>"; 
    echo "<h2>Con que abrir los archivos .$extensionMAY ?</h2><br />"; 
    // realizamos consulta
    $consulta = mysqli_query($conexion, "SELECT definicion FROM extension WHERE ext='$extension'");
      if($consulta == FALSE) {
                  echo "Error en la consulta SQL";
                  exit(); }
    
   //mostramos los datos
        while ($fila = mysqli_fetch_row($consulta)) {
        foreach($fila as $datoFila) {
            echo $datoFila; }
        }
    echo    "<li>"; 
    echo "<h2>Propiedades de .$extensionMAY ?</h2><br />"; 
    // realizamos consulta
    $consulta = mysqli_query($conexion, "SELECT definicion FROM extension WHERE ext='$extension'");
      if($consulta == FALSE) {
                  echo "Error en la consulta SQL";
                  exit(); }
    
   //mostramos los datos
        while ($fila = mysqli_fetch_row($consulta)) {
        foreach($fila as $datoFila) {
            echo $datoFila; }
        }
  
    echo    "<li>"; 
    echo "<h2>Historia de  .$extensionMAY ?</h2><br />"; 
    // realizamos consulta
    $consulta = mysqli_query($conexion, "SELECT historia FROM extension WHERE ext='$extension'");
      if($consulta == FALSE) {
                  echo "Error en la consulta SQL";
                  exit(); }
    
   //mostramos los datos
        while ($fila = mysqli_fetch_row($consulta)) {
        foreach($fila as $datoFila) {
            echo $datoFila; }
        }
   echo "</li>"; 
   echo "</ul>";
    
        
  // finalizamos
        mysqli_free_result($consulta);
        mysqli_close($conexion);
              
        } else {
        ?>
        <form action="busqueda.php" target="_top" method="GET"/>archivo.
        <input type="text" name="extension" size="5" value=""/><input type="submit" value="Buscar" /><br/>
        </form>
        <?php
        echo "No ha buscado nada, escriba la extension"; }
?>
 
    </body>
</html>

Ahora lo que intengo es hacer las consultas llamando a la función, y he creado la siguiente función:

Código PHP:

Ver originalfunction consulta_a_bd($campo) {
        $conexion = connectar_a_bd(); 
        $consulta = mysqli_query($conexion, "SELECT $camnpo FROM extension WHERE ext='$extension'");
      if($consulta == FALSE) {
                  echo "Error en la consulta SQL";
                  exit(); }
    
   //mostramos los datos
        while ($fila = mysqli_fetch_row($consulta)) {
        foreach($fila as $datoFila) {
            echo $datoFila; } }
            return $consulta;
}

Pero como sorpresa...NO funcionaa jajaja, mostrando los siguientes errores:

Notice: Undefined variable: camnpo in C:\wamp\www\infoextensiones\funciones.php on line 11
Notice: Undefined variable: extension in C:\wamp\www\infoextensiones\funciones.php on line 11
Warning: mysqli_query() expects parameter 1 to be mysqli, null given in C:\wamp\www\infoextensiones\funciones.php on line 11

arcanisgk122 · #15 (**permalink**) 15/02/2014, 16:37

esta es una muy mala practica:

Código PHP:

Ver original<meta charset="UTF-8">
        <?php
        header("Content-Type: text/html;charset=utf-8");
        include 'funciones.php';
        $extension = $_GET["extension"]; 
        $extensionMAY = strtoupper($extension); 
        echo "<title>.$extensionMAY Información sobre la extension .$extensionMAY</title>";
        ?>
    </head>

te imagina que pasa si esto se lo pones a todos los archivos??? y que pasara con el consumo de tu ancho de banda???

#16 (**permalink**) 15/02/2014, 16:57

Te adjunto un ejemplo procedural de tu problema (depuralo un poco porque al no tener tu misma base de datos creada y estar en otro ordenador no puedo comprobar resultados exactos)

funciones.php

Código PHP:

Ver original// obtiene conexion a bd
function conexion() {
    return mysqli_connect('localhost', 'root', null, 'base_de_datos');
}
 
// seleccionar campo de la bd
function seleccionar_campo(mysqli $db, $ext, $campo) {
    return mysqli_query($db, "SELECT $campo FROM extension WHERE ext='$ext'")->fetch_assoc()[$campo];
}
 
// obtiene un valor del array $_GET
function get($var) {
    return filter_input(INPUT_GET, $var);
}

index.php

Código PHP:

Ver originalinclude 'funciones.php';
 
$conexion = conexion();
$extension = get('extension');
?>
<html>
    <head>
        <meta charset="UTF-8">
        <?php echo "<title>.$extension Información sobre la extension .$extension</title>"; ?>
    </head>
    <body>
        <?php
        if ($ext = strtoupper($extension)) {
            $descripcion = seleccionar_campo($conexion, $ext, 'descripcion');
            echo "<h1>Información sobre la extensión .$ext</h1><br />";
            echo "<h2>¿ Que es un archivo .$ext ?</h2><br />";
            echo $descripcion;
        }
        else {
            ?>
            <form action = "busqueda.php" target = "_top" method = "GET">archivo.
                <input type = "text" name = "extension" size = "5" value = "" />
                <input type = "submit" value = "Buscar" /><br/>
            </form>
            <?php
            echo "No ha buscado nada, escriba la extension";
        }
        ?>
    </body>
</html>

Respeto bastante tu diseño inicial pero trabajando con funciones para reducir código repetitivo.

#17 (**permalink**) 15/02/2014, 17:04

He respetado tu diseño inicial para que notes la diferencia, así que ahora que todo se ve un poco más claro y más limpio, puedes implementar algo para comprobar antes de mostrar nada, que realmente la extensión existe en la base de datos, para en tal caso, informar al usuario que está consultando una extensión aún no definida.

Código PHP:

Ver original<?php echo "<title>.$extension Información sobre la extension .$extension</title>"; ?>

Ten cuenta que ese código se ejecutará siempre, haya buscado o no el usuario. Lo sabes desde el momento que ese código no está dentro de la condición del IF.

rodrypaladin · #18 (**permalink**) 15/02/2014, 17:25

Entiendo que has metido la consulta en el archivo funciones.php y que la ejecutas con $descripcion = seleccionar_campo($conexion, $ext, 'descripcion'); , pero por que no la imprime si los parametros que habias puesto son los correctos ?. Da el eror: Notice: Undefined variable: extension in C:\wamp\www\infoextensiones\busqueda.php on line 6 y muestra el formulario de nuevo como si no se hubiera buscado nada

#19 (**permalink**) 15/02/2014, 17:37

Pues no se qué hay en búsqueda.php

rodrypaladin · #20 (**permalink**) 15/02/2014, 18:01

busqueda es lo que en tu ejemplo es el index.php, pero mi index.php es solo el formulario que tiene el action="busqueda.php" y ahí el código que me has puesto

#21 (**permalink**) 16/02/2014, 05:21

Vamos paso a paso:

funciones.php

Código PHP:

Ver original// obtiene un valor del array $_GET
function get($var) {
    return filter_input(INPUT_GET, $var);
}
 
// obtiene conexion a bd
function conexion() {
    return mysqli_connect('localhost', 'root', null, 'extensiones');
}
 
// obtiene campo de la bd
function obtener_campo(mysqli $db, $ext, $campo) {
    return mysqli_query($db, "SELECT $campo FROM extension WHERE ext='$ext'")->fetch_assoc()[$campo];
}

Todo correcto, seguimos con extension.php, que hará de plantilla para mostrar única y exclusivamente información de una sola extensión

Código PHP:

Ver original<html>
    <head>
        <meta charset="utf-8">
        <title>Extensi&oacute;n '.<?= $extension ?>'</title>
    </head>
    <body>
        <h1>Informaci&oacute;n sobre la extensi&oacute;n '.<?= $extension ?>'</h1>
        <h2>&iquest;Qu&eacute; es un archivo '.<?= $extension ?>?'</h2>
        <p><?= $descripcion ?></p>
    </body>
</html>

busqueda.php (sólo formulario)

Código PHP:

Ver original<p>No ha buscado nada</p>
<form method="get" action="index.php">
    <input type="text" name="extension" />
    <button type="submit">Buscar</button>
</form>

Y por último, el index.php de forma más expresiva

Código PHP:

Ver originalinclude 'funciones.php';
 
// conexión a la base de datos
$conexion = conexion();
 
// extensión desde $_GET
$extension = get('extension');
 
// si existe la extensión se muestra la extensión
if ($extension !== null) {
 
    // podrías obtener más campos, por ejemplo:
    // $nombre_campo = obtener_campo($conexion, $extension, 'nombre_campo');
    $descripcion = obtener_campo($conexion, $extension, 'definicion');
 
    // al incluír el fichero 'extension.php' usará todas las variables
    // previamente definidas, en este caso sólo $descripcion,
    // ya que es la única que se ha definido
    include 'extension.php';
 
}
// si no existe la extension se muestra el formulario de búsqueda
else {
 
    include 'busqueda.php';
 
}

Espero así lo entiendas mejor :).