Un simple IF: Quien podra resolver esto??

[horape]

Al cargar una página, magicamente el contenido del siguiente 'if' es ejecutado 2 veces, cuando solo deberia ocurrir 1 vez.

Código php:

if($_SESSION["{$image_id}_viewed"] != 1) {
    $connection = mysql_connect($db_host,$db_username,$db_password) or die('Error al conectar con mysql');
    mysql_select_db($db_database, $connection) or die('Error al seleccionar base de datos');
   
    $query = "UPDATE wallpapers SET image_views = image_views+1 WHERE image_id = {$image_id}";
    mysql_query($query,$connection) or die(mysql_error());
    
    $_SESSION["{$image_id}_viewed"] = 1;
  }
 

Si se vuelve a cargar una segunda vez la página, el contenido del if ya no se ejecuta, lo cual es correcto.

Otros datos: Si se borran las cookies y se reinicia el navegador, al volver a cargar una de estas paginas, otra vez se ejecuta 2 veces el if. Alguna idea?? Quiero que se ejecute 1 vez por sesión nada mas. Parece como si estuviera cargando la pagina 2 veces simultaneamente, para guardar en cache o algo similar.

[Kamae]

Buenas!

Eso de {$image_id}_viewed intuyo que por ejemplo hace 123_viewed y que por lo tanto es una variable variable (valga la redundancia)? Vaya, sea así o no, el primer consejo es que hagas un echo $_SESSION["{$image_id}_viewed"]; antes del IF y así vas a tener más pistas sobre cómo solucionar el error.

[GatorV]

Verifica que no tengas instalado en tu browser algún acelerador, ya que muchos usan algo que se conoce como PreFetch y eso causa que se ejecute dos veces tu script.

Saludos.

[horape]

Iniciado por Kamae

Buenas!

Eso de {$image_id}_viewed intuyo que por ejemplo hace 123_viewed y que por lo tanto es una variable variable (valga la redundancia)? Vaya, sea así o no, el primer consejo es que hagas un echo $_SESSION["{$image_id}_viewed"]; antes del IF y así vas a tener más pistas sobre cómo solucionar el error.

si ya probe: antes del if no imprime nada, y al final del if imprime 1. Eso poniendo un exit; al final del if. Debe ser lo que dice gatorV, pero no le veo la logica a algo que ejecuta el mismo programa 2 veces al mismo tiempo. O sea está volviendo a ejecutarlo antes de q surta efecto la asignación a la variable de sesión.

Iniciado por GatorV

Verifica que no tengas instalado en tu browser algún acelerador, ya que muchos usan algo que se conoce como PreFetch y eso causa que se ejecute dos veces tu script.

Saludos.

y no se podrá solucionar desde el codigo??

[Marvin]

Una de las soluciones es que la variable la uses sin sesiones y una vez tengas el valor definitivo lo pases a la sesion.

Pasa tal cual sucede cuando apretas cerrar sesion y no pasas por una pagina intermedia que haga el session_destroy().

Trata de cargar una variable normal desde la de session al comienzo de la pagina y trabaja con ella hasta que debas cambiar el valor que se envia en la variable de sesion.

Suerte!

[GatorV]

Puedes usar este código para prevenir el Prefetch:

Código php:

if ((isset($_SERVER['HTTP_X_MOZ'])) && ($_SERVER['HTTP_X_MOZ'] == 'prefetch')) {
       // This is a prefetch request. Block it.
       header('HTTP/1.0 403 Forbidden');
       echo '403: Forbidden<br><br>Prefetching not allowed here.';
       die();
}
 

Saludos.

[horape]

Iniciado por Marvin

Una de las soluciones es que la variable la uses sin sesiones y una vez tengas el valor definitivo lo pases a la sesion.

Pasa tal cual sucede cuando apretas cerrar sesion y no pasas por una pagina intermedia que haga el session_destroy().

Trata de cargar una variable normal desde la de session al comienzo de la pagina y trabaja con ella hasta que debas cambiar el valor que se envia en la variable de sesion.

Suerte!

gracias

igual no entendí como trabajar con variables normales, si necesito que permanezcan luego de cerrar la página.

O sea estoy usando una variable para cada imagen, para saber si ya fue visualizada, y así dejar de sumarle visualizaciones.

Si uso una variable normal, al cerrar la pestaña y volver a abrirla (sin cerrar el navegador), volveria a contar una visualización.

Iniciado por GatorV

Puedes usar este código para prevenir el Prefetch:

Código php:

if ((isset($_SERVER['HTTP_X_MOZ'])) && ($_SERVER['HTTP_X_MOZ'] == 'prefetch')) {
       // This is a prefetch request. Block it.
       header('HTTP/1.0 403 Forbidden');
       echo '403: Forbidden<br><br>Prefetching not allowed here.';
       die();
}
 

Saludos.

faa, gracias ahora lo pruebo

[horape]

Iniciado por GatorV

Puedes usar este código para prevenir el Prefetch:

Código php:

if ((isset($_SERVER['HTTP_X_MOZ'])) && ($_SERVER['HTTP_X_MOZ'] == 'prefetch')) {
       // This is a prefetch request. Block it.
       header('HTTP/1.0 403 Forbidden');
       echo '403: Forbidden<br><br>Prefetching not allowed here.';
       die();
}
 

Saludos.

ya probe, no me detiene el programa.

me voy a volver loco, aun con prefetch, supongamos q anda mal y cuenta 3 visualizaciones la primera vez.. pero por qué sigue contando mas visualizaciones si ahi ya definitivamente le asignó 1 a su variable de sesion?!?!

puede estar mal el IF?? ya probé mostrar la variable y está bien, y vale 1. Probe cambiar la condición del IF por un !isset, pero sigue ignorándola.

[GatorV]

Pues muy raro, si no se esta ejecutando el script más de una vez, revisa que no estes ejecutando ese código dentro de un bucle for/while o algo por el estilo, ya que eso también puede ocasionar el problema.

Te recomiendo instalar el plugin firebug para Firefox con el puedes ver como se comporta la red y que peticiones esta haciendo.

Saludos.