Problema con un codigo

[chryo] · #1 (**permalink**) 01/01/2002, 12:33

Saludos

Tengo este codigo:
<BLOCKQUOTE><font size=1 face=arial>Citando:<hr height=1 noshade>if ($Answer=="1") {$tabla="votos1";}
elseif ($Answer=="2") {$tabla="votos2";}
elseif ($Answer=="3") {$tabla="votos3";}
elseif ($Answer=="4") {$tabla="votos4";}

//Conexion a la DB
conectadb();

$query = mysql_query("SELECT * FROM encuestas WHERE id='$pollId'");
$row = mysql_fetch_array($query);
$hits = ++$row[$tabla];

//Enviar informacion a la db
$sql = "UPDATE encuesta SET $tabla='$hits' WHERE id='$pollId'";
$result = mysql_query($sql);
//Fin enviar informacion a la db [/QUOTE]

Pero no me incrementa el campo correspondiente, alguien sabe como debo hacerlo??

Hasta Otra

<STYLE>.ch{text-decoration: none; font: bold 8pt verdana; color: white; background-color: black;}</STYLE><TABLE width=100% cellpadding=0 cellspacing=0><TR><TD class=ch> þøWê®êÐ ß¥ [¢H®¥ø] - Wêß µñÐê®¢øñ§T®µ¢Tîøñ</TD></TR></TABLE>

Ferdy · #2 (**permalink**) 01/01/2002, 14:17

Y porqué no haces una SQL así:<pre>UPDATE $tabla SET votos = votos+1 WHERE id = '$pollid' </pre>

Salu2

- Fernando Pereda ( Ferdy )
<center><a href="http://ferpereda.multimania.com/firmas.php"><img src="http://ferpereda.multimania.com/reg_tira.jpg" border="0"></a></center>

[chryo] · #3 (**permalink**) 01/01/2002, 15:10

He seguido modificando el código sin mejorar el resultado, ahora he optado por usar un switch():

<BLOCKQUOTE><font size=1 face=arial>Citando:<hr height=1 noshade> switch ($Answer) {
case 1:
$query = mysql_query("SELECT * FROM encuestas WHERE id='$pollId'");
$row = mysql_fetch_array($query);
$hit = $row["votos"];
$hits = ++$hit;
$result = mysql_query("UPDATE encuesta SET votos='$hits' WHERE id='$pollId'");
break;
case 2:
$query = mysql_query("SELECT * FROM encuestas WHERE id='$pollId'");
$row = mysql_fetch_array($query);
$hit = $row["votos2"];
$hits = ++$hit;
$result = mysql_query("UPDATE encuesta SET votos2='$hits' WHERE id='$pollId'");
break;
case 3:
$query = mysql_query("SELECT * FROM encuestas WHERE id='$pollId'");
$row = mysql_fetch_array($query);
$hit = $row["votos3"];
$hits = ++$hit;
$result = mysql_query("UPDATE encuesta SET votos3='$hits' WHERE id='$pollId'");
break;
case 4:
$query = mysql_query("SELECT * FROM encuestas WHERE id='$pollId'");
$row = mysql_fetch_array($query);
$hit = $row["votos4"];
$hits = ++$hit;
$result = mysql_query("UPDATE encuesta SET votos4='$hits' WHERE id='$pollId'");
break;
} [/QUOTE]

El resultado es que me incrementa hasta el valor 1 y de alli no se suma más cuando se vuelve a cargar el script.

Hasta Otra

<STYLE>.ch{text-decoration: none; font: bold 8pt verdana; color: white; background-color: black;}</STYLE><TABLE width=100% cellpadding=0 cellspacing=0><TR><TD class=ch> þøWê®êÐ ß¥ [¢H®¥ø] - Wêß µñÐê®¢øñ§T®µ¢Tîøñ</TD></TR></TABLE>

[chryo] · #4 (**permalink**) 01/01/2002, 15:40

Saludos

Na...ya me di cuenta del error que tenia...realmente ***** y es que llamaba la tabla "encuestas" que no existe y grababa en "encuesta" que es la que existe :P

Ahora ya funciona correctamente.

Hasta Otra

<STYLE>.ch{text-decoration: none; font: bold 8pt verdana; color: white; background-color: black;}</STYLE><TABLE width=100% cellpadding=0 cellspacing=0><TR><TD class=ch> þøWê®êÐ ß¥ [¢H®¥ø] - Wêß µñÐê®¢øñ§T®µ¢Tîøñ</TD></TR></TABLE>

Webstudio · #5 (**permalink**) 02/01/2002, 08:42

[chryo] : sin ánimos de ofender, me alegro que hayas corregido el problema que tenías, pero estoy en el deber moral de informarte de que tu código es bastante rebuscado y poco eficiente, cuando podrías estar haciendo lo mismo, gastando menos recursos y sobre todo, escribiendo menos. Aqui va mi sugerencia, pero igual puedes hacer lo que quieres:
<pre>
$nombre = "votos".$Answer;
$result = "UPDATE encuesta SET ".$nombre."='".$nombre."+1' WHERE id='$pollId'";
</pre>

Mira todo el código que te ahorras de escribir (y de parsear con el PHP) y te puedo asegurar que hace exactamente lo mismo.

Saludos.

<hr><font size=2 face="verdana">- Pablo Daniel Rigazzi (Webstudio)
<font size=1 color="#333333">COORDINADOR PROYECTO REGIONALIZACION ARGENTINA
Visita <a href="http://www.web-studio.com.ar" target="_blank">Web Studio</a> - Tutoriales Photoshop</fo

[chryo] · #6 (**permalink**) 02/01/2002, 20:06

Saludos

Gracias x el script k me mandaste pero ya reduci el swith a menos linias y menos codigo y bueno ahora ya no es tan grande :P

Hasta Otra

<STYLE>.ch{text-decoration: none; font: bold 8pt verdana; color: white; background-color: black;}</STYLE><TABLE width=100% cellpadding=0 cellspacing=0><TR><TD class=ch> þøWê®êÐ ß¥ [¢H®¥ø] - Wêß µñÐê®¢øñ§T®µ¢Tîøñ</TD></TR></TABLE>