if( isset($_POST['asistotales']) && !empty($_POST['numsesiones']) ) {
$asistotales = $_POST['asistotales']; //valores de arreglo recibido
$numsesiones = $_POST['numsesiones']; //valor recibido de un input
$n4 = count($asistotales); //cuenta elementos del array
$num4_alumno = 1; //contador de registros de alumno en gpo
for ($i=0; $i < $n4; $i += 1, ++$num4_alumno) {
//regla de 3
$a = $asistotales[$i];
$regla = $a * 100;
$asis = $regla/$numsesiones;
//no funca
$gpo = $grupo[$num4_alumno-1];
$al = $alumno[$num4_alumno-1];
$sql = mysqli_query($connect, "UPDATE alumno_grupo SET asistencias = '".$a."'
WHERE idAlumno = $al AND idGrupo = $gpo") or die (mysqli_error($connect));
-
¿Podrías ser un poco más claro?¿Que error te da? ¿Que tendría que dar? ¿Qué esta dando actualmente?– Francisco Romeroel 4 nov. 2016 a las 6:23
-
al actualizar, los registros en la tabla, nadamas lo hace con el primero, los valores para los demas registros no los actualiza. Imprimiendo con echo $gpo, $al, si salen los que deberia tomar el WHERE, pero nadamas actualiza la primer fila– Armando Arellanoel 4 nov. 2016 a las 6:24
-
El bucle for no lo cierras. ¿Te lo has olvidado en tu pregunta o no lo cierras en tu código?– Francisco Romeroel 4 nov. 2016 a las 6:26
-
lo olvido cerrar en la pregunta, en el codigo si esta– Armando Arellanoel 4 nov. 2016 a las 6:27
-
1muchas, gracias saludos– Armando Arellanoel 4 nov. 2016 a las 16:09
2 respuestas
No se que error tienes, pero yo lo haría diferente:
- En el
for
sustituir$i += 1
pori++
que es la forma habitual de hacerlo y parece más clara. - ¿Haces
++$num4_alumno
a continuación para después usar la variable quitándote 1? Si sólo usas esta variable para obtener los valores de$grupo
y$alumno
, haz el incremento al final delfor
, si tiene alguna otra función antes de acabar elfor
, justo después de obtener esos valores. Luego para preparar la sentencia, si concatenas las variables en un String entre signos {} PHP reconoce que se trata de una variable y es más rápido, por ejemplo:
"UPDATE alumno_grupo SET asistencias = '{$a}' WHERE idAlumno = {$al} AND idGrupo = {$gpo}"
De cualquier forma yo compondría las queries, guardándolas en un array y las ejecutaría después. Puedes hacer una función que sólo realice la ejecución de un grupo de queries, de este modo puedes meterlas todas en una transacción o no, según el caso, pero sobretodo puedes imprimirlas todas en un momento dado y ver si están correctas sintácticamente o si reciben todos los valores esperados.
En tu bucle for
cambia ++$num4_alumno
por $num4_alumno++
. La primera te hace un pre-incremento por lo que la variable se suma para la iteración que se esté dando en ese momento. Para el segundo caso, se suma uno a la variable $num4_alumno
pero para la iteración siguiente.