[SOLUCIONADO] Problemas con un formulario que no ingresa valores en mi base de datos

sven_uri · #1 (**permalink**) 24/08/2013, 14:12

Buenas noches a todos,

Llevo todo el día haciendo unos formularios para practicar y no he tenido ningún problema hasta ahora. Resulta que después de rellenar el formulario y darle a enviar no solo no se guardan los valores sino que tampoco me da ningún mensaje de error.

¿Alguna idea? Os muestro el código por si veis algo que a mí se me haya escapado.

el formulario es: form_set.php

Código PHP:

Ver original<?php
    include("conexio.php");
    $rst_categories=mysql_query("SELECT * FROM pua;",$con);
    $rst_categories8=mysql_query("SELECT * FROM report;",$con);
                
?>
<html>
    <head>
        <title>Informe</title>
    </head>
<body>
    <form action="set_registrat.php" method=post name="formulari">
        Nombre: <select name="categoria_pua">
        <option value=""></option">
        <?php
        
        while ($fila=mysql_fetch_array($rst_categories))
        {
            echo "<option value='". $fila["idpua"] ."'>".$fila["pua_nom"] ."</option>";
        }
        mysql_close($con)
        ?>
        </select><br>
        Nom del Report: <select name="categoria_report">
        <option value=""></option">
        <?php
        
        while ($fila8=mysql_fetch_array($rst_categories8))
        {
            echo "<option value='". $fila8["idreport"] ."'>".$fila8["report_nomreport"] ."</option>";
        }
        mysql_close($con)
        ?>
        </select><br>
        Sede: <input type=text name="nombre_7"></input><br>
        HQ: <input type=text name="hq_7"></input><br>
        Edad:  <input type=number name="edat_7"></input><br>
        Tiempo Obertura: <input type=number name="obertura_7"></input><br>
        Tiempo Anclatge:<input type=number name="anclatge_7"></input><br>
        Tiempo Traccion:<input type=number name="traccio_7"></input><br>
        Tiempo Aislar:<input type=number name="aillar_7"></input><br>
        Tiempo Comfort Zone:<input type=number name="comfort_7"></input><br>
        Tiempo Cierre:<input type=number name="close_7"></input><br>
        Observacions:<input type=text name="observ_7"></input><br>
        <input type="submit" name="boto_guardar" value="Guardar">
    </form>
 
</body>
<footer>
</footer>
 
</html>

Y aqui el otro archivo:

Código PHP:

Ver original<?php
    include("conexio.php");
    
?>
<?php
    
    if (isset($_POST['categoria_pua']) && !empty($_POST['categoria_pua']))
    
    {
        
        $name_prota = $_POST['categoria_pua'];
            
        $name_informe = $_POST['categoria_report'];
                
        $n_7 = $_POST['nombre_7'];
                
        $hq_7 = $_POST['hq_7'];
                
        $edat_7 = $_POST['edat_7'];
                
        $open = $_POST['obertura_7'];
        
        $hook = $_POST['anclatge_7'];
                
        $traction = $_POST['traccio_7'];
                
        $isolate = $_POST['aillar_7'];
                
        $comfort = $_POST['comfort_7'];
                
        $close = $_POST['close_7'];
                
        $total_sede = $open+$hook+$traction+$isolate+$comfort+$close;
        
        $obs_7 = $_POST['observ_7'];
                
        mysql_query("INSERT INTO set(set_tipus,set_hb,set_edat,set_obertura,set_anclatge,set_traccio,set_aillar,set_comfort,set_close,set_scoregame,pua_idpua,report_idreport,set_observacio) values ('$n_7','$hb_7','$edat_7','$open','$hook','$traction','$isolate','$comfort','$close','$total_sede','$name_prota','$name_informe','$obs_7')");
        
        echo "Info empresa guardada. <br>";
        echo "<a href='form_set.php'> Ingressar mas info </a><br>";
        echo "<a href='menu.php'> Volver al menu<br> </a>";
    }
    else
    {
        echo "Tienes que escribir alguo.";
    }
?>

Por si sirve de ayuda, este es el único formulario donde hago una operación matemática.

Muchas gracias por vuestro tiempo.

gnzsoloyo · #2 (**permalink**) 24/08/2013, 14:32

Cita:

INSERT INTO set(

¿Y has advertido que "SET" es una palabra reservada en todos los DBMS?

Todo lenguaje, incluyendo un de consultas como es el SQL, tiene palabras reservadas, que no deben ser usadas jamás como nombres de objetos creados en su ambiente.
Nunca.

En tu caso lo estás usando una cláusula de dos sentencias distintas, como nombre de una tabla...

Por otro lado, las buenas prácticas en programación indican que toda acción que pueda disparar un error, debe tener su control de errores, donde se capturen los mensajes del sistema correspondientes (en este caso, los devueltos por MySQL).
DE lo contrario, es imposible saber dónde y por qué se produjo el problema...

¿Pusiste una captura de errores?

No. No pusiste ninguna.

sven_uri · #3 (**permalink**) 25/08/2013, 03:22

Primero de todo permíteme disculparme, estaba tan obsesionado en posibles errores que me olvide de lo básico. Efectivamente, estaba utilizando SET y no me di cuenta. Pero muchísimas gracias por darme un toque jejeje.

¿Lo que me comentas de controlar los errores, bastaría con un or die() ? Por ejemplo:

Código PHP:

Ver originalmysql_query("INSERT INTO grupo(set_tipus,set_hb,set_edat,set_obertura,set_anclatge,set_traccio,set_aillar,set_comfort,set_close,set_scoregame,pua_idpua,report_idreport,set_observacio) values ('$n_7','$hb_7','$edat_7','$open','$hook','$traction','$isolate','$comfort','$close','$total_sede','$name_prota','$name_informe','$obs_7')") or die("No se han añadido registros");

¿O sería mejor utilizar otro recurso?

¡Otra vez, muchas gracias por tu comentario! :D

gnzsoloyo · #4 (**permalink**) 25/08/2013, 05:58

En realidad, como método de captura de errores lo que estás haciendo es simple (demasiado para mi gusto), pero al menos es efectivo.

Pero lo que no es efectivo es que muestres ese mensaje, porque eso sólo nos dice que ha ocurrido un error, pero no dice qué error es el que se produjo.

Todos los lenguajes, y todos los sistemas, incluidos los de BBDD, tienen clasificados y codificados los errores que pueden suceder (hasta los errores "inclasificables"), y cuando un DBMS detecta algún error devuelve dos cosas: 1) Un numero de error, 2) un mensaje de error.
Aprovecha eso, y haz que el "or die()", te muestre ambas cosas.
Eso es lo que se hace.

http://www.php.net/manual/es/function.mysql-errno.php
http://www.php.net/manual/es/function.mysql-error.php

PD: La librería MYSQL de PHP es obsoleta y pronto se discontinuará. Por lo menos deberías migrar a MYSQLI, si no quieres usar PDO.

http://www.php.net/manual/es/mysqli.errno.php
http://www.php.net/manual/es/mysqli.error.php
http://www.php.net/manual/es/mysqli.errno.php

sven_uri · #5 (**permalink**) 25/08/2013, 06:03

Perfecto, muchas gracias gnzsoloyo!

Todo me ha resultado muy útil. Ahora toca estudiar más jeje :).