[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ć?

[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