Problem z licznikiem Opcje

[Morek]

Wiem, że tamatów o licznikach było sporo.
Ale mam taki skrypt licznika:

[PHP] pobierz, plaintext 
<?
setcookie("odwiedziny", "1", time()+3600*2); 
?>
 
 
<?
 
if(file_exists("licznik.txt"))			 /* sprawdza czy plik istnieje */
{ 
$file=fopen("licznik.txt", "r");		 /* otwiera plik */
flock($file, 1);				 /* blokuje plik */
 
$ile=fgets($file, 100000);			 /* odczytuje wartość */
 
flock($file, 3);				/* odblokowuje plik */
fclose($file);					 /* zamyka plik */
 
if($odwiedziny!="1")				 /* sprawdza plik cookie */
{
$ile++;						/* zw. wartość o 1 tylko po pierwszym wejściu */
}
}
else
{
$ile=1;					 /* jeśli plik nie istnieje wyświetla wartość 1 i tworzy plik */
}
 
$file=fopen("licznik.txt", "w");		 /* otwiera plik do zapisu */
flock($file, 2);				 /* blokuje plik */
 
fwrite($file, $ile);				 /* zapisuje wartość */
 
flock($file, 3);				 /* odblokwuje plik */
fclose($file);					 /* zamyka plik */
 
?>
[PHP] pobierz, plaintext 

i czasami zdarza się, że gdy stron się wgrywa i ktoś da "odśwież" licznik sie zeruje

W poście Mifczu znalazłem podobny problem i mike_mech poradził by

[PHP] pobierz, plaintext 
<?php
if( $visited != "1" ) /* sprawdza, czy użytkownik już odwiedzał stronę */
	{
		$ile++; /* zwiększa wartość o 1 tylko po pierwszym wejściu */
	}
?>
[PHP] pobierz, plaintext 

zmienić na

[PHP] pobierz, plaintext 
<?php
if( $_COOKIE[ 'visited' ] != "1" ) /* sprawdza, czy użytkownik już odwiedzał stronę */
	{
		$ile++; /* zwiększa wartość o 1 tylko po pierwszym wejściu */
	}
?>
[PHP] pobierz, plaintext 

ale gdy zamieniam to licznik przestał być odporny na odświeżanie i nie bardzo wiem, gdzie robie błąd?

Jeśli ktoś wie, jak sprawić by licznik był odporny na odswieżanie, nie zerował się tak jak to opisałem i nie potrzebował bazy danych,

lub chociaż gdzie jest błąd w kodzie to PLEASE pomocy!
Będe wdzięczny.

Ten post edytował Morek 11.08.2005, 14:24:49

[yavaho]

Tez mialem kiedys problem z licznikiem. I zauwazylem ze w tego typu skryptach jak w Twoim gdzie plik zostaje dwa razy otwarty podczas jednego uzycia skryptu istnieje takie niebezpieczenstwo ze cos sie pokreci.
Moim zdaniem rozwiazanie jest takie ze: nalezy otworzyc plik tylko jeden raz, zrobic obiczenia i zapisac.

Teraz uzywam juz tylko takich licznikow - dzialajacych na sesjach, bo na ciasteczkach nie mozna polegac w 100% a zwlaszcza gdy na strone wpadnie robot generujacy duzy ruch w krotkim czasie, ktory z pewnoscia nie przyjmuje ciasteczek.

[PHP] pobierz, plaintext 
<?php
if(!isset($_SESSION['licznik'])){
 $nazwa = "licznik.txt";
 $licz='';
 if($plik=fopen($nazwa, 'r+b')){
   flock($plik, LOCK_EX);
   $licz = fgets($plik,100);
   $licz=$licz+1;
   fseek($plik, 0, SEEK_SET);
   fputs($plik, $licz);
   flock($plik, LOCK_UN);
   fclose($plik);
 }
 $_SESSION['licznik'] = $licz;
}
echo $_SESSION['licznik'];
?>
[PHP] pobierz, plaintext 

Podczas pierwszego uruchomienia skrypt moze nie utworzyc pliku (nie jestem pewien) wiec utworz sobie plik na serwerze oczywiscie z odpowiednim chmod'em.