[skrypt] Licznik, Licznik odporny na odświzanie... Opcje

[slammer]

[PHP] pobierz, plaintext 
<?php
//tworzenie nowej sesji
session_start();
 
//odczytywanie danych z pliku
$path = 'Status/licznik.txt';
 
//sprawdza czy plik istnieje jesli nie to tlumi blad i tworzy nowy
if (!file_exists($path))
{
	@fopen($path, "w");
	@fputs($fp, "0|0|0|0|0|0");
	@fclose($fp);
}
$plik = file($path);
$data = explode('|',$plik[0]);
$fp = fopen($path, "w");
 
//przypisanie danych z pliku do zmiennych
$dzis = $data[0];
$wczoraj = $data[1];
$miesiac = $data[2];
$calos = $data[3];
$zeszly_miech = $data[4];
$d = $data[5];
$m = $data[6];
 
//sprawdzenie aktualnego dnia i w wypadku zmiany odpowiednie ustawienie licznika
$d_dzis = date('d');
if ($d != $d_dzis) 
{
	$wczoraj = $dzis;
	$dzis = 0;
}
 
//sprawdzenie aktualnego miesiaca i w wypadku zmiany odpowiednie ustawienie liczni
ka
$m_dzis = date('m');
if ($m != $m_dzis)
{
	$zeszly_miech = $miesiac;
	$miesiac = 0;
}
 
//sprawdzenie czy strona zostala odswiezona, jesli tak nie zostana przekazane dane
 do licznika
if(!session_is_registered('sesja')){
$dzis += 1;
$miesiac += 1;
$calos += 1;
}
 
//sprawdza czy plik jest zablokowany
if(!flock($fp, LOCK_EX))
{
   fclose($fp);
   return;
}
 
//zapisanie stnu licznika do pliku
else
{
   $ciag = $dzis.'|'.$wczoraj.'|'.$miesiac.'|'.$calos.'|'.$zeszly_miech.'|'.$d_dzis.'|'.$m_dzis;
   fwrite($fp, $ciag);
   flock($fp, LOCK_UN);
   fclose($fp);
}
 
//rejesracja zmiennej sesji ktora posluzy do sprawdzenia odswierzania strony
session_register('sesja');
 
//wypisanie stanu licznika na ekran
echo 'Odwiedzin Dzis:'.$dzis.'<br>Odwiedzin Wczoraj:'.$wczoraj.'<br>Odwiedzin w tym m-cu:'.$miesiac.'<br>Odwiedzin w zeszlym m-cu:'.$zeszly_miech.'<br>Lacznie:'.$calos;
 
?>
[PHP] pobierz, plaintext 

[strife]

Witam,

Proponuje zamknąć skrypt w funkcji/klasie, a kod uczynić bardziej przejrzystym, więcej odstępów. Jeśli zawierało by się to w klasie dodał bym kilka metod które odpowiadały by za tworzenie nowego pliku licznika, zerowania, wyświetlania jego zawartości etc. Generalnie nic nadzwyczajnego, skrypt do napisania w kilka minut...

Pozdrawiam!

[Zajec]

Odporny na odświeżanie? A co z przeglądarkami z wyłączonymi cookies?

[slammer]

ale to dziala na sesje nie na cookies

[wojto]

Troche nieoptymalnie napisany, do zmiany:

[PHP] pobierz, plaintext 
<?php
$dzis = $data[0];
$wczoraj = $data[1];
$miesiac = $data[2];
$calos = $data[3];
$zeszly_miech = $data[4];
$d = $data[5];
$m = $data[6];
?>
[PHP] pobierz, plaintext 

niepotrzebne przypisania, mozesz korzystac caly czas z elementow tablicy $data, czyli $data[0], $data[1] i na nich pracowac, wtedy moglbys na koncu zamiast:

[PHP] pobierz, plaintext 
<?php
$ciag = $dzis.'|'.$wczoraj.'|'.$miesiac.'|'.$calos.'|'.$zeszly_miech.'|'.$d_dzis.'|'.$m_dzis;
?>
[PHP] pobierz, plaintext 

dac implode dla tej tablicy

[PHP] pobierz, plaintext 
<?php
$dzis += 1;
?>
[PHP] pobierz, plaintext 

szybciej bedzie: $dzis++;

[PHP] pobierz, plaintext 
<?php
session_register('sesja');
?>
[PHP] pobierz, plaintext 

skrypt bedzie dzialal tylko na serwerach z wylaczonym register_globals, aby dzialal na kazdym serwerze powinienes uzywac $_SESSION['sesja'];

[slammer]

Dzieki za wskazówki! Jestem początkujący więc ma to dla mnie wielkie znaczenie.

[PHP] pobierz, plaintext 
<?php
$dzis = $data[0];
$wczoraj = $data[1];
$miesiac = $data[2];
$calos = $data[3];
$zeszly_miech = $data[4];
$d = $data[5];
$m = $data[6];
?>
[PHP] pobierz, plaintext 

Moze i nieoptymalne, ale dzieki temu lepiej sie orientuje w skrycie.

[PHP] pobierz, plaintext 
<?php
if($_SESSION['sesja'] != 1){
$dzis++;
$miesiac++;
$calos++;
}
 
if(!flock($fp, LOCK_EX))
{
	fclose($fp);
	return;
}
else
{
	$tab = array($dzis, $wczoraj, $miesiac, $calos, $zeszly_miech, $d_dzis, $m_dzis, $r_dzis);
	$ciag = implode("|", $tab);
 
	   fwrite($fp, $ciag);
	   flock($fp, LOCK_UN);
	   fclose($fp);
}
 
$_SESSION['sesja'] = 1;
?>
[PHP] pobierz, plaintext 

Zmienione wedle zaleceń.

[Pigula]

ale z tego co wiedze to jak ktos wcodzi na strone to go zlicza i dobrze ale jak wylaczy przegladarke i wejdzie jeszcze raz po 10s to znowu go zliczy to zdecydowanie zle powinienes dolozyc cookie i/albo zapis ip do bazy to poprawi miarodajnosc wynikow

[Zajec]

Jak będzie miał odwiedzający cookies wyłączone to niekoniecznie zostanie nawiązana sesja.

Jak zamknie się przeglądarkę, to część sesji jest zrywana.

[slammer]

Z tego co się orientuje to 99% użytkowników ma włączone cookies. Wg mnie wyłączenie tej opcji jest bez sensowne gdyż uniemożliwia kożystanie z większośći stron internetowych.

A co do cookies wyglądałoby to tak:

[PHP] pobierz, plaintext 
<?php
 
//odczytywanie danych z pliku
$path = 'Status/licznik.txt';
 
//sprawdza czy plik istnieje jesli nie to tlumi blad i tworzy nowy
if (!file_exists($path))
{
	@fopen($path, "w");
	@fputs($fp, "0|0|0|0|0|0");
	@fclose($fp);
}
$plik = file($path);
$data = explode('|',$plik[0]);
$fp = fopen($path, "w");
 
//przypisanie danych z pliku do zmiennych
$dzis = $data[0];
$wczoraj = $data[1];
$miesiac = $data[2];
$calos = $data[3];
$zeszly_miech = $data[4];
$d = $data[5];
$m = $data[6];
 
//sprawdzenie aktualnego dnia i w wypadku zmiany odpowiednie ustawienie licznika
$d_dzis = date('d');
if ($d != $d_dzis) 
{
	$wczoraj = $dzis;
	$dzis = 0;
}
 
//sprawdzenie aktualnego miesiaca i w wypadku zmiany odpowiednie ustawienie liczni
ka
$m_dzis = date('m');
if ($m != $m_dzis)
{
	$zeszly_miech = $miesiac;
	$miesiac = 0;
}
 
//sprawdzenie czy strona zostala odswiezona, jesli tak nie zostana przekazane dane
 do licznika
if($_COOKIE['sprawdz'] != 1){
$dzis++;
$miesiac++;
$calos++;
}
 
if(!flock($fp, LOCK_EX))
{
	fclose($fp);
	return;
}
//zapisanie stanu licznika do tablicy
else
{
	$tab = array($dzis, $wczoraj, $miesiac, $calos, $zeszly_miech, $d_dzis, $m_dzis, $r_dzis);
	$ciag = implode("|", $tab);
 
	   fwrite($fp, $ciag);
	   flock($fp, LOCK_UN);
	   fclose($fp);
}
 
//stworzenie pliku cookie ktory posluzy do sprawdzenia odswiezania strony
setcookie("sprawdz", '1', time()+30*86400);
 
//wypisanie stanu licznika na ekran
echo 'Odwiedzin Dzis:'.$dzis.'<br>Odwiedzin Wczoraj:'.$wczoraj.'<br>Odwiedzin w tym m-cu:'.$miesiac.'<br>Odwiedzin w zeszlym m-cu:'.$zeszly_miech.'<br>Lacznie:'.$calos;
 
?>
[PHP] pobierz, plaintext 

No i plik musi byc includowany przez np. index.php z opcja ob_start(); Jak sie go wlaczy bezposrednio to wywala błąd.

[Zajec]

Korzystanie.

Gdyby fałszowana była jedna odsłona strony na 100, to jeszcze bym nie narzekał. Ale jeśli taka osoba odrzucająca cookies wejdzie na 10 podstron, to masz już nieco większe przekłamanie w wartości licznika.

Poza tym ten 1% ludzi nie koniecznie odrzuca cookies z wszystkich stron. W Operze można np. wybrać, aby przed dodaniem cookies zawsze przeglądarka się najpierw mnie zapytała o zgodę. A użytkownik korzystający z tej opcji zgodzi sie na cookie raczej tylko wtedy, gdy się gdzieś loguje.