[skrypt]licznik odwiedzin, proszę o ostre oceny + wskazówki Opcje

[enigma]

Witam, php jak narazie tylko liznąłem i będzie to widać po skrypcie (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
Ma on służyć do robienia statystyk dla każdej podstrony, oraz zliczać ilość odwiedzin z każdej strony
skrypt musi operować na plikach (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
prosze o krytykę, tylko błagam nie śmiać się, wiem że tu wszystko będzie do krytykowania, ale jednoczesnie proszę o uwagi, jak coś można zrobić szybciej lepiej (w sumie już mam pomysł na dużo krótszy i szybszy skrypt), bo napewno pomoze mi to w nauce

[PHP] pobierz, plaintext 
<?php
 
 function FunMeter ()
  {
   if (!(file_exists('MetersFile/visits/')))//tworzy katalog na pliki ze statystykami
	mkdir ('MetersFile/visits/', 0700);
   $file=($_SERVER['PHP_SELF']);//pobieram  nazwe strony (pliku php) - plik z liczba odwiedzin nazywa się tak samo
   $begin=strpos($file,'/') + 1;
   $end=strpos($file,'.php');
   $name=substr($file,$begin,$end-$begin);
   if (!(file_exists('MetersFile/visits/'.$name.'.txt')))//tworzenie plików na statytyki
	 {
	 $file = fopen ('MetersFile/visits/'.$name.'.txt', 'w+');
	 fputs ($file,'0');
	 fclose ($file);
	 }
   $file = fopen ('MetersFile/visits/'.$name.'.txt', 'r+'); 
   if ($file)//odczytywanie i uaktualnianie statystyk
	 {
	 flock ($file, 2);
	 $visits = fgets($file, 20);
	 $visits++;
	 fseek ($file, 0);
	 fputs ($file, $visits);
	 flock ($file,3);
	 fclose ($file);
	 }
   return $visits;//zwraca tablicę w celu późniejszego odczytania na stronie
  }
 
 
  function CatchIP ()
  {
   if (!(file_exists('MetersFile/ip/')))//tworzy katalog na pliki ze statystykami
	mkdir ('MetersFile/ip/', 0700);
   $ip = $_SERVER['REMOTE_ADDR']; 
   $file=($_SERVER['PHP_SELF']);//pobieram  nazwe strony (pliku php) - plik z liczba odwiedzin nazywa się tak samo
   $begin=strpos($file,'/') + 1;
   $end=strpos($file,'.php');
   $name=substr($file,$begin,$end-$begin);
   if (!(file_exists('MetersFile/ip/'.$name.'IP.txt')))//tworzenie plików na statytyki
	 {
	 $file = fopen ('MetersFile/ip/'.$name.'IP.txt', 'w+');
	 fputs ($file,'');
	 fclose ($file);
	 }
   $file = fopen ('MetersFile/ip/'.$name.'IP.txt', 'r'); 
   $i = 0;
   if ($file)//odczyt i aktualizacja pliku
	 {  
	 flock ($file, 2);
	 while(!feof($file))//zapis z pliku do tablicy, i zwiększenie liczby odwidzin dla konkretnej strony
	 {
	 $ip_count = fgets($file, 20);
	 $data = explode(':',$ip_count);
	 $ip_addr[$data[0]] = trim($data[1]);
	 }
	$ip_addr[$ip]++;
	fclose ($file);
	$klucze = array_keys($ip_addr);
	if(in_array($ip, $klucze))//nadpisanie zaktualizowanych informacji
	  {
	  $file = fopen ('MetersFile/ip/'.$name.'IP.txt', 'w+'); 
	  for($x=0,$n=count($klucze);$x<$n;$x++)
		if($klucze[$x])
		  fputs($file, $klucze[$x].':'.$ip_addr[$klucze[$x]]."rn");
	  }  
	else
	  {$file = fopen ('MetersFile/ip/'.$name.'IP.txt', 'a');
		fputs($file, $ip.':1'."rn");}
	 fclose ($file);
	 }
   return $ip_addr;//zwraca tablicę w celu późniejszego odczytania na stronie
  }
?>
[PHP] pobierz, plaintext 

całość razem z funkcjami odczytującymi można pobrać http://www.pawel-ryznar.yoyo.pl/Projekty/php/Visionsave.zip

Pozdrawiam

[Adrian Staniszew...]

dodaj mozliwosc dodania ciastka w zliczaniu odslon oraz moze okreslenia czasu dla ciastka

sprawdzasz wtedy czy jest ciacho jak jest to nie zliczasz a jak nie ma to zliczasz

pozwoli to np. na stworzenie statystyk odslon i odwiedzin(teoretycznie uniq)

[enigma]

pozwoli to np. na stworzenie statystyk odslon i odwiedzin(teoretycznie uniq)

- odwiedziny - czyli co, zliczanie wejścia danego gościa raz w ciągu dnia np??
- a odsłona (to znaczy ile razy odświeżono stronę + ilość wejść w ciągu dnia)
dobrze te terminy rozumiem (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

jako że nie miałem jeszcze z o obiektami wiele do czynienia zapytam o klasę, czy dobrze myślę (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[PHP] pobierz, plaintext 
<?php
class licznik
{
var $tryb_wyswietlania   //0-tekstowy, 1-graficzny
var $plik_statystyk   //$PHP_SELF dla zwykłych stron, nazwa_id dla kombinowanych (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) 
var $tryb_zapisu   //0 -baza danych, 1 -pliki
 
function licznik
 {
	//inicjalizacja trybów, rodzaju plików
	//i inne ...
 }
 
function odczyt_statystyk {;}
function zapis_statystyk {;}
.
.	//itd itp :)
.
}
?>
[PHP] pobierz, plaintext