[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

[Sh4dow]

No niestety to raczej optymalne nie jest, zamiast zliczać rekordy tak:

[PHP] pobierz, plaintext 
<?php
//...
 $link=mysql_query("SELECT * FROM dzis");
 $ile=mysql_num_rows($link);
 //...
?>
[PHP] pobierz, plaintext 

zrob tak:

[PHP] pobierz, plaintext 
<?php
$link=mysql_query("SELECT count(*) FROM dzis");
$row = mysql_fetch_array($link);
$ile = $row[0]
?>
[PHP] pobierz, plaintext 

to powinno przyzpieszyc troche i nie bedzie przesylac tyle danych miedzy serverm mysqla a php i ogranicz ilość zapytań dwa powinny w zupełności wystarczyć. Pierwsze kasuje wszystkie stare wpisy i twoj, a drigi dodaje twoj wpis.

[PHP] pobierz, plaintext 
<?php
mysql_query("DELETE FROM dzis WHERE czas<'$staryczas' AND ip='$ip'");
?>
[PHP] pobierz, plaintext 

a drugie praktycznie bez zmian

[PHP] pobierz, plaintext 
<?php
mysql_query("INSERT INTO dzis VALUES(NULL,'$ip','$czas')");
?>
[PHP] pobierz, plaintext 

no i trzecie zapytanie do zlicznia jak wyżej w przykładzie

Ten post edytował Sh4dow 10.11.2008, 15:40:01

[GigaDownload]

Nie kasują się stare rekordy, wgrałem stary plik i wyświetliła się poprawna wersja niższa o 600 rekordów (pokazywało wcześniej 3100)

Ten post edytował GigaDownload 11.11.2008, 09:45:46

[Sh4dow]

chyba jednak był błąd, pisałem wszystko z głowy nie mialem czasu sprawdzić. W zapytaniu kasującym zamiast połączenia warunków słowem AND powinno użyć się słowa OR

[PHP] pobierz, plaintext 
<?php
mysql_query("DELETE FROM dzis WHERE czas<'$staryczas' OR ip='$ip'");
?>
[PHP] pobierz, plaintext 

[GigaDownload]

Przy okazji się spytam...

Mam dwa liczniki online działające na tej samej bazie danych, chciałbym wsumować ich liczbę rekordów (czyli ludzi online) - może ktoś podać gotowy kod?

[mike]

Dwóch liczb dodać nie potrafisz?
A jeśłi chodzi o gotowca to ja mogę podać gotowe (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) Ile płacisz?

[GigaDownload]

Nie zamierzam kształcić się w kierunku specjalisty od zapytań MySQL, ani programisty php... wiem za co jest odpowiedzialny każdy kawałek kodu, tylko nie wiem jak napisać optymalnie własny kod...

[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

[GigaDownload]

Do wyświetlania ilości online dla jednego serwisu mam:

Kod

<?php
$polaczenie=mysql_connect('','','')
    or die('Nieudane poczenie z baz danych...');

mysql_select_db('')
    or die('Nieudane wybranie bazy danych...');
    
$link=mysql_query("SELECT count(*) FROM tabela1");
$row = mysql_fetch_array($link);
$ile = $row[0];

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

Nie możesz przekształcić tego, aby zliczało też tabela2? Nie musi być niezliczanie takich samych IP.


edycja: prosiłbym o odpowiedź

[lukasz_os]

Jezeli nazwa tabeli to tabela2 to zrob tak:

[PHP] pobierz, plaintext 
<?php 
 $polaczenie=mysql_connect('','','')
     or die('Nieudane poczenie z baz danych...');
 
 mysql_select_db('')
     or die('Nieudane wybranie bazy danych...');
     
 $link=mysql_query("SELECT count(*) FROM tabela1");
 $row = mysql_fetch_array($link);
 $ile = $row[0];
$link=mysql_query("SELECT count(*) FROM tabela2");
 $row = mysql_fetch_array($link);
 $ile2 = $row[0];
 $ilosc=ile+ile2;
 echo ' <b>'.$ilosc.'</b>';
 ?>
[PHP] pobierz, plaintext 

Pozdrawiam.

[GigaDownload]

Coś nie zlicza, pojawia się 0...


edycja: poradziłem sobie, wystarczyła mała poprawka.

Ten post edytował GigaDownload 30.11.2008, 19:50:09