Múltiples inserciones en MySql

[txoler]

Hola,

Tengo un problema con mis bases de datos Mysql que me está volviendo loco.
Los visitantes de la página pueden ir viendo fotos y dar una puntuación del 1 al 5 de cada foto.
Cada foto tiene su correspondiente registro y lo que hago cuando alguien le da una puntuación es actualizar el registro con la nueva puntuación y con el número de votos.
En resumen, al campo "puntos" le debo sumar la puntuación, y al campo "votos" le debo sumar 1.

Pero lo que obtengo es que se suma correctamente la puntuación, pero en cambio el número de votos a veces incrementa en 1, y a veces (la mayoría) en 2.
Aquí os pongo el código php que uso:

$link = mysql_connect("localhost","usuario","contrasenya")
or die ("no se ha podido conectar");
mysql_select_db("basedatos")
or die("Error al tratar de selecccionar esta base");
$result=mysql_query("select * from fotomontajes where numreg='$n'",$link);
$row = mysql_fetch_array($result);

$votosnou=$row["votos"]+1;
$puntosnou=$row["puntos"]+$puntuacion;

$sSQL="UPDATE fotomontajes SET puntos='$puntosnou', votos='$votosnou' WHERE email='$email2' and titulo='$titulo2'";
mysql_query($sSQL);
mysql_free_result($result);
mysql_close($link);

Los parámetros $puntuación y $n vienen de la página anterior mediante un post.

Espero que me podaís ayudar porque realmente no entiendo qué está pasando. Muchas gracias!!!

[BrujoNic]

Movido desde BD a PHP.

Por favor lee la Función de la sección de Base de Datos para que sepas el motivo.

[txoler]

Gracias Brujonic,

Siento haberme equivocado de sección.

Alex

[Nefertiter]

2 recomendaciones

1. para actualizar datos:
no hay necesidad de rescatar el valor anterior para hacer el update, simplemente

Código PHP:

  $sSQL="UPDATE fotomontajes SET  
puntos= puntos + $puntuacion, 
votos= votos + 1  
WHERE email='$email2'  
AND titulo='$titulo2'";  
   

2.si los votos solo pueden ser unicos te recomiendo crees una tabla separada
con campos como

voto_id
foto_id
voto_valor
voto_ip // o usuario_id, no se si se tiene q estar registrado o como manejas la cosa

y entonces a la hora de votar tenes lo siguientes pasos:

1. tomas los datos del formulario
2. verificas que no se haya votado previamente: "SELECT voto_id FROM votos WHERE foto_id=$id_foto AND voto_ip='$ip'"
3. si ya voto, mostras error, sino insertas el nuevo voto: INSERT INTO votos SET foto_id = $foto_id, voto_valor = $puntuacion, voto_ip = '$ip'
4. para mostrar la puntuacion que posee una foto podes hacer la consulta SELECT AVG(voto_valor) as promedio FROM votos WHERE foto_id = $foto_id
   

Saludos

[txoler]

Hola Nefertiter!

Muchísimas gracias por tu consejo. Ahorafunciona perfectamente y ya no he detectado más el error del que hablaba.

Y en cuanto a la solución que me das para el voto único, lo pienso aplicar.

De nuevo, muchísimas gracias!!