[SOLUCIONADO] Update con suma

[xPHPerox]

Buenas noches, estoy intentado desarrollar un pequeño panel de actualización para la web de un amigo.

Me pidió que en un formulario, rellenando una cantidad de campos con las actualizaciones, estos se sumaran a los puntos que ya tenía el usuario, hasta aquí todo bien.

El problema está en que el Update se ejecuta siempre, por lo tanto siempre está sumando los puntos que aparecen en el campo input del form.

El código que tengo es el siguiente:

Código PHP:

$qy = "SELECT * FROM clasificaciones";
$qry = mysqli_query($con, $qy);
while($rows = mysqli_fetch_assoc($qry)){
    $pts = $rows['puntos'];
    $pj = $rows['partidos_jugados'];
    $pg = $rows['partidos_ganados'];
    $pp = $rows['partidos_perdidos'];
    $um = $rows['unidades_matadas'];
    $up = $rows['unidades_perdidas'];
    $ea = $rows['edifios_arrasados'];
    $ep = $rows['edificios_perdidos'];
    $ct = $rows['catillos_totales'];
    $rc = $rows['reliquias_capturadas'];
    $pa = $rows['poblacion_aldeanos'];
    $pt = $rows['puntuacion_total'];
    echo $pts;
}
 
if(isset($_POST['guardar'])){
    $id = $_POST['id'];
 
    for($i=0; $i<count($id); $i++){
        $ptos = $_POST['puntos'][$i];
        $p_j = $_POST['pj'][$i];
        $p_g = $_POST['pg'][$i];
        $p_p = $_POST['pp'][$i];
        $u_m = $_POST['um'][$i];
        $u_p = $_POST['up'][$i];
        $e_a = $_POST['ea'][$i];
        $e_p = $_POST['ep'][$i];
        $c_t = $_POST['ct'][$i];
        $r_c = $_POST['rc'][$i];
        $p_a = $_POST['pa'][$i];
        $p_t = $_POST['pt'][$i];
 
        echo "<br>";
 
$update = "UPDATE clasificaciones SET 
puntos=$pts+$ptos, 
partidos_jugados=$pj+$p_j, 
partidos_ganados=$pg+$p_g, 
partidos_perdidos=$pp+$p_p, 
unidades_matadas=$um+$u_m, 
unidades_perdidas=$up+$u_p, 
edifios_arrasados=$ea+$e_a, 
edificios_perdidos=$ep+$e_p, 
catillos_totales=$ct+$c_t, 
reliquias_capturadas=$rc+$r_c, 
poblacion_aldeanos=$pa+$p_a, 
puntuacion_total=$pt+$p_t 
WHERE id=$id[$i]";
        
$updt = mysqli_query($con, $update);
 
echo $update;
    }
}
 

Para realizar la suma lo que he hecho ha sido realizar una consulta a la BD, metiendo los resultados en variables que luego utilizo en el Update junto a las que vienen del formulario.

Más tarde me di cuenta de que en realidad la consulta da lo mismo, basta con poner en el Update SET puntos = puntos + $pts, es algo a modificar también, pero ahora me urge mucho más el tema de la suma.

Le he dado mil vueltas a la cabeza, y no encuentro la forma de solucionar esto.

Un saludo y gracias

[xerifandtomas]

Es que sumar puede ser un error de concepto
Como tienes el formulario?
En tu formulario, deberías pasar el valor de puntos actuales en la bd como value por defecto, para si se quiere modificar lo o no.

[xPHPerox]

El formulario es el siguiente:

Código PHP:

<tr>
                <input type="hidden" name="id[<? echo $cont; ?>]" value="<? echo $id; ?>"/>
                <td><input type="text" name="usuario[<? echo $cont; ?>]" value="<? echo $usuario; ?>" readonly /></td>
                <td><input type="number" name="puntos[<? echo $cont; ?>]" value="<? echo $puntos; ?>"/></td>
                <td><input type="number" name="pj[<? echo $cont; ?>]" value="<? echo $pj; ?>"/></td>
                <td><input type="number" name="pg[<? echo $cont; ?>]" value="<? echo $pg; ?>"/></td>
                <td><input type="number" name="pp[<? echo $cont; ?>]" value="<? echo $pp; ?>"/></td>
                <td><input type="number" name="um[<? echo $cont; ?>]" value="<? echo $um; ?>"/></td>
                <td><input type="number" name="up[<? echo $cont; ?>]" value="<? echo $up; ?>"/></td>
                <td><input type="number" name="ea[<? echo $cont; ?>]" value="<? echo $ea; ?>"/></td>
                <td><input type="number" name="ep[<? echo $cont; ?>]" value="<? echo $ep; ?>"/></td>
                <td><input type="number" name="ct[<? echo $cont; ?>]" value="<? echo $ct; ?>"/></td>
                <td><input type="number" name="rc[<? echo $cont; ?>]" value="<? echo $rc; ?>"/></td>
                <td><input type="number" name="pa[<? echo $cont; ?>]" value="<? echo $pa; ?>"/></td>
                <td><input type="number" name="pt[<? echo $cont; ?>]" value="<? echo $pt; ?>"/></td>
            </tr>
 

A la hora de sumar lo hago sin problema, luego el erro está en que el Update toma siempre el valor que hay en el campo del formulario y lo suma una y otra vez :(

Gracias por la respuesta.

[rodocoyote15]

Es porque la update lo tienes dentro del for, y actualiza tantas veces cuenta el id, prueba sacandolo de ahí.

Edito. Una duda, para qué es el for?

[xPHPerox]

Si saco el update del for no me actualizará todos los registros que se modifiquen.

A ver si soy capaz de explicarlo bien, el for lo utilizo dándole un índice a cada registro, para si modifico varios registros en el mismo formulario, dándole únicamente a un submit, se registren todas las actualizaciones.

Es decir, que si modifico 3 filas, se hagan los cambios en las 3 filas, sin tener que ir haciéndolo una por una.

No sé si me expliqué bien.

[xerifandtomas]

Es que no deberías de hacer ninguna suma, simplemente actualizar con el valor que recibes del formulario.

[rodocoyote15]

Para lo que necesitas, podrías en ves del for, usar un foreach con las claves primarias. Algo así

Código PHP:

foreach($_POST['id'] as $key => $value)
{
 
 
}
 

Y dentro

Código PHP:

$ptos = $_POST['puntos'][$key]; // esto para cada POST
 
$update = "UPDATE clasificaciones SET 
puntos=$pts+$ptos, 
partidos_jugados=$pj+$p_j, 
partidos_ganados=$pg+$p_g, 
partidos_perdidos=$pp+$p_p, 
unidades_matadas=$um+$u_m, 
unidades_perdidas=$up+$u_p, 
edifios_arrasados=$ea+$e_a, 
edificios_perdidos=$ep+$e_p, 
catillos_totales=$ct+$c_t, 
reliquias_capturadas=$rc+$r_c, 
poblacion_aldeanos=$pa+$p_a, 
puntuacion_total=$pt+$p_t 
WHERE id=$value";
 

[xerifandtomas]

A ver si me explico. En el el input puntos estas pasando como value los puntos actuales en bd. Y luego es la consulta a esos punto le sumas el valor que envíes. Si lo envías tal cual sin modificar estas es como si multiplicará por 2.

Dos soluciones o no sumar, y escribir en el formulario el valor al que quieres actualizar

Código PHP:

$update = "UPDATE clasificaciones SET 
    puntos=$ptos, 
    partidos_jugados=$pj+$p_j, 
    partidos_ganados=$pg+$p_g, 
    partidos_perdidos=$pp+$p_p, 
    unidades_matadas=$um+$u_m, 
    unidades_perdidas=$up+$u_p, 
    edifios_arrasados=$ea+$e_a, 
    edificios_perdidos=$ep+$e_p, 
    catillos_totales=$ct+$c_t, 
    reliquias_capturadas=$rc+$r_c, 
    poblacion_aldeanos=$pa+$p_a, 
    puntuacion_total=$pt+$p_t 
    WHERE id=$id[$i]";
 

O establecer el value del input a 0 para que sume lo que escribas.

Código PHP:

<td><input type="number" name="puntos[<? echo $cont; ?>]" value="0" placeholder="<? echo $puntos; ?>"/></td>
 

[xPHPerox]

Iniciado por rodocoyote15

Para lo que necesitas, podrías en ves del for, usar un foreach con las claves primarias.

Gracias, no sé si entiendo bien..eso realmente es similar a lo que tengo yo con el for ¿no? Pero con un foreach por el tema de array.

Iniciado por xerifandtomas

A ver si me explico. En el el input puntos estas pasando como value los puntos actuales en bd. Y luego es la consulta a esos punto le sumas el valor que envíes. Si lo envías tal cual sin modificar estas es como si multiplicará por 2.

Dos soluciones o no sumar, y escribir en el formulario el valor al que quieres actualizar

O establecer el value del input a 0 para que sume lo que escribas.

Código PHP:

<td><input type="number" name="puntos[<? echo $cont; ?>]" value="0" placeholder="<? echo $puntos; ?>"/></td>
 

Perfecto, ahora sí.. claro, realmente el problema estaba en lo que enviaba todo el rato con value, pensaba que podría existir alguna forma de usar if y else para tratarlo, pudiendo dejar el value como lo tenía.

De la forma que me indicas dio problema también, pero vamos..con quitar el value y dejar únicamente el placeholder funciona perfecto, muestro el valor actual que tiene con él, y luego relleno el campo y lo envío, y ya se hace la suma una única vez, sin repetirse.

Jamás hubiera pensado en eso, jeje.

Código PHP:

<td><input type="number" name="puntos[<? echo $cont; ?>]" placeholder="<? echo $puntos; ?> "/></td>
 

Muchas gracias, me has quitado un gran dolor de cabeza.