[PHP][MySQL] Liczba aktywnych użytkowników przez ostatnie 24h Opcje

[GigaDownload]

Mam taki kod dla liczby aktywnych użytkowników przez ostatnie 24h:

Kod

<?php 
$polaczenie=mysql_connect('','','')
    or die('');

mysql_select_db('')
    or die('');
    
$czas=mktime();
$ip=$_SERVER['REMOTE_ADDR'];
$staryczas=mktime()-86400;


$zapytanie=mysql_query("DELETE FROM dzis WHERE czas<'$staryczas'");

$link=mysql_query("SELECT ip FROM dzis WHERE ip='$ip'");
$ile=mysql_num_rows($link);

if($ile==0)
    {
    mysql_query("INSERT INTO dzis VALUES(NULL,'$ip','$czas')");
    }
else
    {
    mysql_query("UPDATE dzis SET czas='$czas' WHERE ip='$ip'");
    }
$link=mysql_query("SELECT * FROM dzis");
$ile=mysql_num_rows($link);

echo '<b>'.$ile.'</b>';
?>

Czy ten kod jest optymalny? Gromadzi mi się w tabeli od 2300 do 2700 rekordów, więc obrót danych jest trochę duży.
Czy da się jakoś jeszcze zoptymalizować ten kod, by zżerał mniej zasobów konta na hostingu współdzielonym?

Demo działania jest na http://www.gigadownload.net.pl w panelu Statystyki (po prawej).


Pozdrawiam

Ten post edytował GigaDownload 9.11.2008, 19:48:33

[singollo]

Moja propozycja jest taka:

jeśli uznajesz IP za wystarczająco unikalne, zrób sobie taką tabelę w MySQL

[SQL] pobierz, plaintext 
CREATE TABLE stats (ip VARCHAR(15), lastSeen DATETIME, PRIMARY KEY(ip))
[SQL] pobierz, plaintext 

aktualizację robisz tak:

[PHP] pobierz, plaintext 
<?php
$czas = time();
mysql_query("INSERT INTO stats(ip, lastSeen) 
VALUES 
($_SERVER['REMOTE_ADDR'], FROM_UNIXTIME($czas)) 
ON DUPLICATE KEY UPDATE 
lastSeen = FROM_UNIXTIME($czas)");
?>
[PHP] pobierz, plaintext 

ilość ostatnio widzianych robisz tak:

[PHP] pobierz, plaintext 
<?php
$wczoraj = time() - 24 * 60 * 60; // dokładnie 24h temu
$result = mysql_query("SELECT COUNT(*) AS counter FROM stats WHERE lastSeen > FROM_UNIXTIME($wczoraj)");
$row = mysql_fetch_assoc($result);
echo $row['counter'];
?>
[PHP] pobierz, plaintext 

Od razu mówię - piszę z głowy, bez sprawdzania, więc nie gwarantuję 100% poprawności/braku literówek itp.

Edit:
1. tak sobie pomyślałem, że na kolumnie lastSeen też możesz założyć index. I być może będzie szybciej, jeżeli ta kolumna nie będzie typu DATETIME ale po prostu INT - odpadną kwestie przekształcania i przetwarzania funkcji FROM_UNIXTIME
2. Nie umieszczam tu kasowania: nie musisz kasować na bieżąco, bo filtrujesz wyniki przy liczeniu -> w efekcie masz jedno zapytanie mniej. Kasowanie możesz sobie odpalić raz na jakiś czas...

Ten post edytował singollo 27.11.2008, 00:34:35