please help with code

jasondavis

function check_online_status($user_id,$localip){
 //define ('TIMEOUT', 20);
 $inactive = time()-(60*60*TIMEOUT);
 $sql_online="select * from friend_online where userid='$user_id' and ipaddress='$localip' and lastactive > $inactive";  

 $result_online=executeQuery($sql_online);
 if($line_online=mysql_fetch_array($result_online))
    return true;
  else
    return false;
}

This requires
$user_id
$localip

to run but instead I need to make a script that will take every row in this table friend_online and check if there are any "lastactive" colum times greater then x amount of minutes, if there is it should delete them so a user appears offline, I wanan run this script from a cron job

jasondavis

so I guess I could do it like this

function check_online_status(){
//define ('TIMEOUT', 20);
$inactive = time()-(60*60*TIMEOUT);
$sql_online="select * from friend_online where lastactive > $inactive";  

$result_online=executeQuery($sql_online);
if($line_online=mysql_fetch_array($result_online))
    return true;
  else
    return false;
}

then something similar to

$sql_online="delete from friend_online where userid=";

but I dont know how to combine the 2 have it work

rulian

I had a similar user login script, the way I kept the table small is with every user trigger, I deleted all rows where timestamp was smaller then the current timestamp minus my range
it was quite a while back ago, and I dont think it was very optimal but it worked
I've never seen executeQuery, only mysql_query to execute Mysql commands, but if it works then I just learned something..

I dont think you need to delete the rows if you dont want to , let the db handle the rows produce a different output on the row information rather then requiring the query to come back false to determine if a user is offline.

Meaning, rather then deleting the user from the log book, let the log expire. The next time the user comes on, update their record and they will look online again. That way you dont have to keep writing/erasing the same user