Licznik odwiedzin - co jakiś czas sam się zeruje! Opcje

[przemek76]

Mam problem ze skryptem licznika odwiedzin opartym o sesję. Co jakiś czas zeruje się i nie mogę dojść dlaczego? Czy to problem z jednoczesnym dostępem do pliku przez wielu użytkowników? Może wtedy zamiast do pliku lepiej byłoby zapisywać do tabeli MySQL? Może pewniejszy byłby na ciasteczku, a nie na sesji? (przy okazji: zauważyłem, że w IE zlicza wejście za pierwszym razem, a potem nawet jak zamknę przeglądarkę i otwieram stronę ponownie to nie dodaje odwiedzin. Muszę dopiero celowo zakończyć sesję poleceniem session_destroy. Natomiast w Firefoxie jest prawidłowo).

A oto ten skrypt:

[PHP] pobierz, plaintext 
<?php session_start();
if (empty($_SESSION['sesja'])) { //JEŚLI ZMIENNA SESYJNA JEST PUSTA:
   $plik = fopen("licznik.txt", "r"); //otwieramy plik do odczytu
   $odczyt = explode(";", fread($plik, filesize("licznik.txt"))); //czytamy z pliku
   $stan_licznika = $odczyt[0] + 1; //ustalamy stan licznika
   fclose($plik); //zamykamy plik, a następnie...
	  $plik=fopen("licznik.txt","w"); //otwieramy plik do zapisu
	  flock($plik, 2); //blokujemy plik do zapisu
	  fwrite($plik, "$stan_licznika;", 15); //zapisujemy do pliku
	  flock($plik, 3); //odblokowujemy plik
	  fclose($plik); //zamykamy plik
   $_SESSION['sesja'] = 1; //i deklarujemy zmienną sesyjną.
} else { //W PRZECIWNYM RAZIE:
   $plik = fopen("licznik.txt", "r"); //otwieramy plik do odczytu
   $odczyt = explode(";", fread($plik, filesize("licznik.txt"))); //czytamy z pliku
   $stan_licznika = $odczyt[0]; //ustalamy stan licznika
   fclose($plik); //zamykamy plik.
} ?>
[PHP] pobierz, plaintext 

No i potem gdzieś w pliku jest: echo $stan_licznika;

Ten post edytował przemek76 27.08.2007, 23:52:49

[kicaj]

Wyedytuj post i zastosuj kod PHP

[Kostek.88]

Ja miałem coś takiego:

[PHP] pobierz, plaintext 
<?php
		 setcookie('zapamietaj', '1', time()+60*60*24);
		 if(!(file_exists("dane/licznik.txt"))) {
			$plik = fopen("dane/licznik.txt", "w+");
			fputs($plik, "0");
			fclose($plik);
		 }
 
		 $plik = fopen("dane/licznik.txt", "r+");
 
		 if(!($plik)) {
			echo 'BŁĄD: Nie da się otworzyć pliku';
		 } else {
			flock($plik, 2);
			$ile = fgets($plik, 255);
 
			echo 'Licznik wskazuje <b>'.$ile.'</b>';
 
 
/*
 
 sorki, tu niżej była pomyłka (jeśli cookie nie istnieje to zwiększamy stan licznika), zmień też warunek if
 
*/
			if($_COOKIE['zapamietaj']!=1) {
			   fseek($plik, 0);
			   fputs($plik, $ile+1);
			   flock($plik, 3);
			   fclose($plik);
			}
		 }
	  ?>
[PHP] pobierz, plaintext 

Ten post edytował Kaloryfer 28.08.2007, 00:04:53

[przemek76]

Dzięki za odpowiedź - spróbuję tego licznika. (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

Zauważyłem, że w twojej propozycji rozpoznawany jest błąd otwarcia pliku. Czy jest możliwe, że u mnie właśnie to jest problemem?
Tak sobie pomyślałem, że jeśli ktoś czyta z pliku i ktoś inny też spróbuje się do niego dostać (w tej samej chwili wszedł na stronę), to nic nie odczyta, a więc zmienna $odczyt będzie pusta i $stan_licznika ustali się na 1 (pusta zmienna + 1) i taki stan następnie zostanie zapisany do pliku. Czy możliwy jest taki scenariusz?

Ten post edytował przemek76 28.08.2007, 11:44:07

[malvea]

$plik=fopen("licznik.txt","w"); - tutaj licznik się zeruje, a następnie fwrite($plik, "$stan_licznika;", 15); zwraca błąd, czyli zero i się nic nie zapisuje i w tym tkwi problem.