PHP OO Insertar varios registros dinamicos generados a partir de una consulta

[tony_la09]

Este dia les traigo una duda que me ha tenido con insomnio por un buen rato, lo que pasa es que deseo hacer un insert a varios registros traidos desde una consulta, tales insert son dinamicos, a continuacion les pongo el codigo fuente k manejo para que me entiendan mejor.

Código PHP:

<?php
include ("conexion.php");
$grupo = $_POST['grupo'];
$consultar = "SELECT matricula,nombre_alumno,nombre_grupo FROM grupos WHERE nombre_grupo = '$grupo'";
$resultado = mysql_query($consultar,$conex);
$num_rows = mysql_num_rows($resultado);
if ($num_rows <= 0){
echo "El grupo ".$grupo." esta vacio<br><br>";
echo "Pulse <a href='frmRegistrar_Calificaciones.php'>aqui</a> si deseas buscar otro Grupo.<br><br>";
echo "Pulse <a href='Inicio.php'>aqui</a> si desea ir a la pagina principal.";
echo "<br><br><br>";
include ("pie.php");
mysql_close($conex);
exit;
}elseif ($num_rows > 0){
echo "<form action='Guardar_Calificaciones.php' method='POST'>";
echo "\n<table align='center'>";
echo "\n<tr><th>NOMBRE</th><th>L</th><th>W</th><th>G</th><th>S</th><th>A</th><th>R</th><th>PROM</th><th>NIV</th><th>MOD</th><th>PER</th><th>HRS</th><th>GRUPO</th></tr>";
while ($fila = mysql_fetch_array($resultado))
{
echo "\n<input type='hidden' name='matricula[]' value='" . $fila["matricula"] . "'>";
echo "<tr>";
echo "<td><input type='text' name='nombre_alumno[]' value='" . $fila["nombre_alumno"] ."' readonly></td>";
echo "<td><input type='text' name='L[]' value='' size='3'></td>";
echo "<td><input type='text' name='W[]' value='' size='3'></td>";
echo "<td><input type='text' name='G[]' value='' size='3'></td>";
echo "<td><input type='text' name='S[]' value='' size='3'></td>";
echo "<td><input type='text' name='A[]' value='' size='3'></td>";
echo "<td><input type='text' name='R[]' value='' size='3'></td>";
echo "<td><input type='text' name='promedio[]' value='' size='3'></td>";
echo "<td><input type='text' name='nivel[]' value='' size='3'></td>";
echo "<td><input type='text' name='modulo[]' value='' size='3'></td>";
echo "<td><input type='text' name='periodo[]' value='' size='3'></td>";
echo "<td><input type='text' name='horas[]' value='' size='3'></td>";
echo "<td><input type='text' name='grupo[]' value='" . $fila["nombre_grupo"] . "' size='3'></td>";
echo "</tr>";
}
echo "</table>";
echo "<table align='center'>";
echo "<tr><td><input type='submit' name='btnGuardar' value='GUARDAR'></td></tr>";
echo "</table>";
echo "</form>";
}
?>
 

y el codigo que manejo para realizar el insert de los datos es el siguiente

Código PHP:

<?php
include ("conexion.php");
for($i=0;$i<$_POST['matricula'];$i++){
$matricula = $_POST['matricula'][$i];
$nombre_alumno = $_POST['nombre_alumno'][$i];
$L = $_POST['L'][$i];
$W = $_POST['W'][$i];
$G = $_POST['G'][$i];
$S = $_POST['S'][$i];
$A = $_POST['A'][$i];
$R = $_POST['R'][$i];
$promedio = $_POST['promedio'][$i];
$nivel = $_POST['nivel'][$i];
$modulo = $_POST['modulo'][$i];
$periodo = $_POST['periodo'][$i];
$horas = $_POST['horas'][$i];
$grupo = $_POST['grupo'][$i];
$consulta = "INSERT INTO Calificaciones (matricula,nombre_alumno,listening,writing,grammar,speaking,attitude,reading,promedio,nivel,modulo,periodo,horas,grupo)
                VALUES ($matricula,'$nombre_alumno',$L,$W,$G,$S,$A,$R,$promedio,$nivel,$modulo,'$periodo',$horas,'$grupo')";
$resultado = mysql_query($consulta,$conex);
echo $consulta;
echo "SE GUARDO CON EXITO";
mysql_close($conex);}
?>
 

El codigo de insercion lo tome de un post anterior, solo que no me funciono, por eso acudo a ustedes..

http://www.forosdelweb.com/f18/insertar-multiples-registros-dinamicos-763571/

[Triby]

Los campos listening, writing, etc., son numéricos?

Por otra parte, no estás haciendo validación de los datos, simplemente los tomas y tratas de insertar, lo cual es un problema porque no sabes si el alumno realmente existe o si la información se va a duplicar o si hay algún error.

Luego, como no mencionas cuál es exactamente el problema:

$resultado = mysql_query($consulta, $conex) or die(mysql_error);

Y luego nos cuentas si te da error y lo que obtienes de echo $consulta;

[tony_la09]

ya añadi $resultado = mysql_query($consulta, $conex) or die(mysql_error);

a mi codigo, lo que obtengo de echo $consulta;

es la cadena SQL y los valores que se inserten, solo que ahora que añadi or die a mi codigo al momento de insertar solo me muestra mysql_error

[tony_la09]

y los campos:

matricula INT
,nombre_alumno VARCHAR
,listening TINYINT
,writing TINYINT
,grammar TINYINT
,speaking TINYINT
attitude TINYINT
,reading TINYINT
,promedio VARCHAR
,nivel TINYINT
,modulo TINYINT
,periodo VARCHAR
,horas TINYINT
,grupo CHAR

[Triby]

Sorry, un pequeño error en la sugerencia!

debe ser or die(mysql_error()) porque es una función y ahí se va a ver el error real.

Qué pasa si dejas en blanco uno de los campos numéricos?, digamos, speaking... a la hora de armar la consulta será una cadena vacía, que MySQL tal vez interprete como NULL, pero si la columna no permite NULL, entonces por ahí viene el error.

[tony_la09]

si dejo un campo en blanco me mandara un error, ya que no permito valores nulos en la BD.

pero mi principal problema es que deseo insertar los valores en los campos generados dinamicamente a partir de la consulta

[tony_la09]

Otro detalle que se me escapaba era que la consulta es generada a partir de este codigo

Código PHP:

<?php
<form action="Registrar_Calificaciones.php" method="POST">
<fieldset align="center">
    <br>
    <table align="center" border="0">
        <td>GRUPO:</td>
        <td id="estilo"><input type="text" name="grupo" value="" onChange="javascript:this.value=this.value.toUpperCase();"></td>
        <td id="estilo"><input type="submit" name="btnBuscar" value="BUSCAR"></td>
    </table>
    <br>
</fieldset>
</form>
?>
 

[Triby]

Bueno, ahora dime si te da error de mysql después de haber agregado or die(mysql_error()) y cuál es ese error.

Realmente no importa si generas el formulario agregando campos con Javascript, en php o directamente con HTML, siempre y cuando lo hagas correctamente.

Puedes ver el contenido de $_POST para saber si todo viene bien desde el formulario, usando al inicio de tu código:

Código PHP:

var_dump($_POST);
die();
 

Si ves que hay algo mal ahí, entonces regresa a ver cómo estás creando los campos y corregir los posibles errores.

Por cierto:

Código HTML:

<td id="estilo"><input type="text" name="grupo" value="" onChange="javascript:this.value=this.value.toUpperCase();"></td>
<td id="estilo"><input type="submit" name="btnBuscar" value="BUSCAR"></td>
 

- Los id deben ser únicos, no puedes tener dos elementos (en este caso dos TD) con igual id
- Los atributos deben ser todos en minúsculas, de forma que onChange debe ser onchange
- Para los nombres de variables, propiedades y métodos de objetos en javascript sí debes respetar mayúsculas y minúsculas

[tony_la09]

te agradesco que estes tomando de tu tiempo para ayudarme con mi problema, y el onChange no creo que se el problema, ya que en ninguno de mi otros formularios me da error, el detalle es que solo me esta insertando un solo registro, los demas no los toma en mi array

[Triby]

Mi estimado tony_la09, HAY QUE LEER LOS MENSAJES COMPLETOS y hacer caso de todas las sugerencias, lo de los id únicos, onChange y esos pequeños detalles son sólo para cumplir con un estándar de codificación, buscando evitar problemas entre los diferentes navegadores y, por supuesto, no van a solucionar el problema en PHP.


Agrego, a propósito de los estándares, si ves el código fuente del navegador encontrarás algo parecido a esto:

Código HTML:

<table>
<input type="hidden" bla bla bla />
<tr>
<td>Aquí otros inputs</td>
<td>Aquí otros inputs</td>
</tr>
 

Y es incorrecto, porque el campo, aunque sea oculto, debe estar dentro de un contenedor válido, en este caso, un td, simplemente lo pones junto con otro campo y listo!

[tony_la09]

Muchas gracias por tu tiempo Triby, ya arregle mi problema, era en el ciclo for, no estaba incluyendo mi include("conexion.php") dentro de dicho ciclo..

dejo el codigo para futuras referencias...

Código PHP:

<?php
for($i=0;$i<=30;$i++){
include ("conexion.php");
$matricula = $_POST['matricula'][$i];
$nombre_alumno = $_POST['nombre_alumno'][$i];
$L = $_POST['L'][$i];
$W = $_POST['W'][$i];
$G = $_POST['G'][$i];
$S = $_POST['S'][$i];
$A = $_POST['A'][$i];
$R = $_POST['R'][$i];
$promedio = $_POST['promedio'][$i];
$nivel = $_POST['nivel'][$i];
$modulo = $_POST['modulo'][$i];
$periodo = $_POST['periodo'][$i];
$horas = $_POST['horas'][$i];
$grupo = $_POST['grupo'][$i];
$consulta = "INSERT INTO Calificaciones (matricula,nombre_alumno,listening,writing,grammar,speaking,attitude,reading,promedio,nivel,modulo,periodo,horas,grupo)";
$consulta = $consulta."VALUES ($matricula,'$nombre_alumno',$L,$W,$G,$S,$A,$R,$promedio,$nivel,$modulo,'$periodo',$horas,'$grupo')";
$resultado = mysql_query($consulta,$conex) or die (mysql_error());
echo $consulta;
echo "SE GUARDO CON EXITO";
mysql_close($conex);}
?>
 

Ahora vere lo de validar los campos, gracias por tu tiempo amigo...