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

[crash]

A nie masz przypadkiem:

[PHP] pobierz, plaintext 
<?php
setcookie("odwiedziny", "1", time()+3600*2);
?>
[PHP] pobierz, plaintext 

I odwołanie przez:

[PHP] pobierz, plaintext 
<?php
$_COOKIE[ 'visited' ]
?>
[PHP] pobierz, plaintext 

[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.

[Morek]

crash - no właśnie nie, zmieniałem nazwe, i dla tego byłem zdziwiony... Ale i tak dzięki za pomoc.


yavaho - Wielkie dzięki za informacje! Nie wiedziałem o tym, i faktycznie licznik się nie zeruje ale mam jeszcze pytanie... Wiem, że może dziwnie to zabrzmi ale czy on jest odporny na odswieżanie? Bo jak odświeżam stronę to dodaje mi kolejne oczko (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Nie bardzo jestem w stanie sam go rozgryźć bo jeszcze niegdy nic na sesjach nie robiłem (IMG:http://forum.php.pl/style_emoticons/default/worriedsmiley.gif)

Ten post edytował Morek 11.08.2005, 21:24:13

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

[NuLL]

Skrypty php -> php

[Morek]

mike_mech Wszystko działa bez problemu, nie zlicza odświeżeń i się nie zeruje! Wielkie dzięki.
(IMG:http://forum.php.pl/style_emoticons/default/smile.gif)