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

twoja metoda obslugi podstron jest bledna.. PHP_SELF nie zwraca calego linku..dostajesz postac typu.. index.php, news.html.. itp..

a co gdy moje linki maja postac np. index.php?go=news, news.php5?id=12 ?

musisz inaczej pobieraz linki, a nawet wtedy nie bedziesz mial pewnosci czy jestes na innej podstronie czy tez po prostu zmieniles jakas wartosc lub wyslales jakis parametr zmieniajacy jakas drobna czesc www, ale nie zmieniajacej tresci podstrony..