[php] mega prosty licznik odwiedzin, a jednak zawodny Opcje

[marcinek37]

[PHP] pobierz, plaintext 
<?
$file = fopen('counter.txt', 'r'); $counter = (int)fgets($file); fclose($file);
if($_SESSION['opened'] != '1'){ $counter++; $file = fopen('counter.txt', 'w'); fwrite($file, $counter); fclose($file); $_SESSION['opened'] = '1'; }
?>
[PHP] pobierz, plaintext 

raz na jakiś czas nie zapisuje wartości i liczy od nowa... ale dlaczego? jak można temu zaradzić?

[Sephirus]

Możliwe, że są równoległe odwołania do tego pliku. Proponuje zapoznać się z flock (IMG:style_emoticons/default/wink.gif)

[cudny]

Znając życie wkrada ci się biały znak na początku i dostajesz 0 - czyli to co jest w pierwszej linii.
Daj zamiast fopen i tak dalej file_get_contents('counter.txt');

przykład;

[PHP] pobierz, plaintext 
 
$path = 'counter.txt';
$counter = trim( file_get_contents($path) );
 
file_put_contents( $path,++$counter );
 
echo $counter;
 
[PHP] pobierz, plaintext 

[stealz]

Inny sposób: zamiast odczytywać wartość, dopisuj do pliku pojedynczy bajt (np. jakikolwiek znak) za pomocą file_put_contents z flagą FILE_APPEND.
Stan licznika zwróci Ci funkcja filesize().
To rozwiązanie jest szybsze i bardziej niezawodne, natomiast bardziej obciążające dysk. Przy milionowej odsłonie plik będzie ważył już 1 MB. Up to you.
Pozdrawiam.

[marcinek37]

[PHP] pobierz, plaintext 
<?
$path = 'counter.txt';
$counter = trim( file_get_contents($path) );
 
file_put_contents( $path,++$counter );
?>
[PHP] pobierz, plaintext 

czy ten kod jest bezpieczny, jeśli naraz wejdzie kilka osób na stronę? może ten flock to nie taki głupi pomysł? co sądzicie?

[cudny]

Jeśli się o to boisz to daj flagę LOCK_EX

[PHP] pobierz, plaintext 
 file_put_contents($file,$content,LOCK_EX); 
[PHP] pobierz, plaintext 

Bez obrazy ale bez sensu jest pytać na forum o rzeczy, które są w manualu (IMG:style_emoticons/default/smile.gif)

@stealz - no co ty ? Szybsze ? Bardziej niezawodne ? Otwieraj potem plik aby zapisać dane... Rób tą inkrementację i się nie zastanawiaj - to tylko licznik odwiedzin, a jeśli chodzi o file lock to masz napisane wyżej co robić

Ten post edytował cudny 28.11.2012, 23:19:49

[stealz]

Ok, zgadzam się. Ale tylko po części : )
Musiałem sprawdzić. Nie myślałem, że dopisywanie danych do dużego pliku przez file_put_contents jest tak wolne.
Z mojego testu wynika, że odczytywanie stanu licznika przez filesize() jest prawie 5x szybsze. Natomiast zwiększanie jego stanu w przypadku zasugerowanego przeze mnie algorytmu - od 2 do 4 razy wolniejsze.
Ponieważ kolega chce, aby zwiększenie licznika nastąpiło tylko raz w ciągu sesji, to z obliczeń, które poczyniłem wynika, że jeżeli użytkownik odwiedzi średnio 15 i więcej podstron, wówczas opcja z filesize() jest szybsza.

I jeszcze jedno: LOCK_EX. Jeżeli jeden użytkownik zablokuje dostęp do pliku przy podanym przez Ciebie file_put_contents($file,$content,LOCK_EX), wówczas drugi użytkownik w ciągu tych kilku milisekund nie ma dostępu do odczytu jego wartości linijkę wyżej. Odczytana wartość będzie równa '' i licznik zostanie wyzerowany. LOCK powinien nastąpić przed odczytaniem danych.

Ten post edytował stealz 29.11.2012, 01:18:03

[marcinek37]

czyli w ten sposób?

[PHP] pobierz, plaintext 
    <?
    $path = 'counter.txt';
    $counter = trim( file_get_contents($path, LOCK_EX) );
 
    file_put_contents( $path,++$counter );
    ?>
[PHP] pobierz, plaintext 

chyba jednak nie... więc możesz mi powiedzieć, gdzie mam LOCK_EX wpisać?

[cudny]

[PHP] pobierz, plaintext 
 file_put_contents($file_name,$content,LOCK_EX); 
[PHP] pobierz, plaintext 

[marcinek37]

czyli generalnie chodzi o coś takiego:

[PHP] pobierz, plaintext 
    <?
    $path = 'counter.txt';
    $counter = trim( file_get_contents($path) );
 
    file_put_contents( $path, ++$counter, LOCK_EX);
    ?>
[PHP] pobierz, plaintext 

dziękuję!

[Crozin]

Odnośnie blokady: musisz założyć blokadę (zarówno odczytu jak i zapisu) na plik przed jego odczytaniem oraz zdjąć ją dopiero po zapisaniu danych do pliku - inaczej może dojść do przekłamań w liczniku. Czyli:

[PHP] pobierz, plaintext 
$fh = fopen('plik.txt', 'r+');
flock($fh, LOCK_EX);
 
$cnt = (int) trim(fread($fh, 1024));
$cnt++;
 
fseek($fh, 0);
 
fwrite($fh, $cnt);
flock($fh, LOCK_UN);
fclose($fh);
 
// Pominąłem siedem(!) IF-ów związanych z obsługą błędów...
// Skorzystanie z SplFileObject niestety również będzie ich wymagało
[PHP] pobierz, plaintext 

Blokada przy samym zapisie jest niewystarczająca, jeżeli chcesz mieć solidny kod.

Odnośnie prędkości działania: jeżeli powyższy fragment będzie wąskim gardłem najwyższy czas pomyśleć nad zmianą dysku bądź platformy.

EDIT: Był drobny błąd w pow. przykładzie (typ otwarcia pliku, brak fseek)

Ten post edytował Crozin 4.12.2012, 22:13:44

[marcinek37]

Crozin, czyli jeśli użyję tego kodu, który zaproponowałeś powyżej, mam większą szansę, że licznik nie zepsuje się?
aktualnie licznik jest na malutkiej stronce, ale pechowo co jakiś czas się zerował...

zależy mi jedynie na tym, aby, kiedy już osiągnie jakiś wynik, np. 10000, żeby nagle nie wyskoczyło mi 0 :/

[Crozin]

Żeby licznik nagle nie zwariował musisz zadbać o to by w jednym momencie był przetwarzany (odczyt/zapis) przez jeden proces.
Żeby sprawdzić czy wszystko jest OK, możesz przy pomocy Apache Benchmarka zrobić prosty test: 10.000 wywołań skryptu/30 wątków jednocześnie. Po zakończeniu testu w pliku musi znajdować się liczba 10000.

[marcinek37]

chyba można stwierdzić, że skrypt, który zaproponowałeś, właśnie taki powinien być - blokuje plik dla jednego wywołania, więc powinno być w porządku

przy okazji zapytam, czy te kody mają w ogóle sens, jeśli system ma jedynie odczytać wynik, bez zwiększania wielkości

[PHP] pobierz, plaintext 
flock($fh, LOCK_EX);
flock($fh, LOCK_UN);
[PHP] pobierz, plaintext 

[Crozin]

Jeżeli potrzebujesz jedynie odczytać zawartość licznika, nie ma potrzeby korzystania z blokad.