Problema con sistema de usuarios activos

[layker]

Hola, he diseñado un sistema para mostrar los usuarios activos, pero nose xk razon cada usuario se inserta varias veces en la bd... no ntiendo xk yo creo k script sta bien...aki lo dejo... aver si alguien ve el error. un saludo gracias

Código PHP:

  if(isset($_SESSION["usuario"])) 
{ 
include("conexion.php"); 
 
// Tiempo que dura inactivamente 
    $tiempo = time()+(5*60); 
    $tiempo_actual = time(); 
     
// Comprueba si esta el usuario 
    $query = "SELECT * FROM `online` WHERE `usuario`='$_SESSION[usuario]' ;"; 
    $con = mysql_query($query); 
    $num = mysql_num_rows($con); 
 
     
// Sino esta el usuario lo inserta y si esta lo actualiza 
    if($num==0) 
    { 
     
        $query = "INSERT INTO `online` ( `usuario` , `tiempo` ) 
VALUES ('$_SESSION[usuario]', '$tiempo' )"; 
        mysql_query($query); 
    } 
    else 
    { 
        $query = "UPDATE `online` SET `usuario` = '$_SESSION[usuario]', `tiempo` = '$tiempo' LIMIT 1 ;"; 
        mysql_query($query); 
    } 
} 
 
    // Borrar los usuarios inactivos 
    $query = "DELETE FROM `online` WHERE `tiempo`<'$tiempo_actual' ;"; 
    $con = mysql_query($query);  
   

[cybersersupremo]

imprime el valro de $num aver que valor te esta cogiendo, yo creo que no se hace directamente asi.

[layker]

he probado a imprimirlo :S, (este script lo tngo incluido al principio de cada pagina de mi web), me acabo de dar cuenta que al ingresar, cuando lleva a cabo las comprobaciones de user y pass, tengo un header("location: ...."); que me redirecciona a un panel de usuario. Una vez alli el valor de $num me muestra 0 pero sinembargo se ha regsitrado en la bd.. :S creo k ay sta el problema como puedo solucionarlo... gracias

[manchilp]

Si te fijas , falta cerrar el parentesis en esta query :

Código PHP:

  $query = "INSERT INTO `online` ( `usuario` , `tiempo` )
VALUES ('$_SESSION[usuario]', '$tiempo' ";  
   

Dudo que sea ese el causante del problema , pero puede que si , asique debes modificarlo por algo asi:

Código PHP:

  $query = "INSERT INTO `online` ( `usuario` , `tiempo` )
VALUES ('$_SESSION[usuario]', '$tiempo') ";  
   

[layker]

Buena observacion, pero me he equivocado al copiarlo.. lo abre borrado sin querer... asias de todos modos..

[jaronu]

hazlo de esta otra manera:

Código PHP:

   
if($num>0)  
    {  
      $query = "UPDATE `online` SET `usuario` = '$_SESSION[usuario]', `tiempo` = '$tiempo' LIMIT 1 ;";  
        mysql_query($query);  
 
           }  
    else  
    { $query = "INSERT INTO `online` ( `usuario` , `tiempo` )  
VALUES ('$_SESSION[usuario]', '$tiempo' ";  
        mysql_query($query);  
 
           }  
   

Un saludo

[layker]

Iniciado por jaronu

hazlo de esta otra manera:

Código PHP:

   
if($num>0)  
    {  
      $query = "UPDATE `online` SET `usuario` = '$_SESSION[usuario]', `tiempo` = '$tiempo' LIMIT 1 ;";  
        mysql_query($query);  
 
           }  
    else  
    { $query = "INSERT INTO `online` ( `usuario` , `tiempo` )  
VALUES ('$_SESSION[usuario]', '$tiempo' ";  
        mysql_query($query);  
 
           }  
   

Un saludo

Parece ser que asi funciona correctamente, muxas gracias x la ayuda y x la rapidez. sois uns fieras. un saludo

[layker]

Hola, toy aki de nuevo con este post... resulta k parecia k se abia arreglado pero sigue pasando lo mismo.. alguien ve el fallo?? tiene k ser k no sta bien estructurao el algoritmo o algo... un saludo

[jaronu]

has provado ha cambiar esas comillas raras que tienes en los SELECT

$query = "SELECT * FROM `online` WHERE `usuario`='$_SESSION[usuario]' ;"

por

$query = "SELECT * FROM 'online' WHERE 'usuario'='$_SESSION[usuario]' ;"

[layker]

Eso no es, m van bien lasconsultas con esas comillaas, son las que salen en phpmyadmin... pero no se xk puede pasar

[jaronu]

no se que sera pero veo que en el query le faltan unas comillas al $_SESSION

deberia de ser, vamos digo yo, a si

$_SESSION ['usuario']

[argy]

bueno,siempre utiliza en tus consultas el mysql_real_escape_string y el sprintf, aunque este último es un poco lenta, pero vale la pena que tengas seguridad en tus scripts