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

[mike]

Na moje oko to powinno być tak:

[PHP] pobierz, plaintext 
<?
 
if( file_exists( 'counter.txt' ) )
{
	$resFile = fopen( 'counter.txt', 'r' );
	flock( $resFile, 1 );
 
	$intCounter = fgets( $resFile, 100000 );
 
	flock( $resFile, 3 );
	fclose( $resFile );
 
	if( $_COOKIE[ 'visited' ] !== '1' )
	{
		$intCounter++;
		setcookie( 'visited', '1', time() + 60 * 60 * 24 ); // ustawiamy "ciacho" na jedną dobę
	}
}
else
{
	$intCounter = 1;
}
 
$resFile = fopen( 'counter.txt', 'w' );
flock( $resFile, 2 );
 
fwrite( $resFile, $intCounter );
 
flock( $resFile, 3 );
fclose( $resFile );
 
echo 'Liczba odwiedzin: ' . $intCounter;
 
?>
[PHP] pobierz, plaintext 

A jak masz php5 to możesz tak:

[PHP] pobierz, plaintext 
<?
 
$intCounter = ( int ) file_get_contents( './counter.txt' );
 
if( $_COOKIE[ 'visited' ] !== '1' )
{
	$intCounter++;
	file_put_contents( './counter.txt', $intCounter );
	setcookie( 'visited', '1', time() + 60 * 60 * 24 ); // ustawiamy "ciacho" na jedną dobę
}
 
echo 'Liczba odwiedzin: ' . $intCounter;
 
?>
[PHP] pobierz, plaintext 

@yavaho Twoje rozwiązanie ma małą wadę. Wyobraź sobie sytuacje kiedy wchodzisz na stronę, licznik pokazuje 100 odwiedzin. Chodzisz po stronce godzinę, dwie, w tym czasie przez strona przewinęło sie kolejne 100 osób. Ale ty widzisz 100 odwiedzin bo masz te dane w sesji.

A poza tym: Ranking.pl : Cookies. 98.2% akceptuje ciasteczka, więc nie ma czym się przejmować.