[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

[phpion]

Moze jakis parametr w kodzie, ktory bedzie okreslal czy linki sa jako index.php?strona=jakas czy jako jakas.php. Parametrem tym moze byc takie cos:

[PHP] pobierz, plaintext 
<?php
$param = 'strona';
// lub
$param = NULL;
?>
[PHP] pobierz, plaintext 

i potem w kodzie sprawdzasz czy $param is_null i jesli tak to przyjmujemy ze linki sa jak do tej pory czyli jakas.php, natomiast jesli ma jakas wartosci to jako nazwe strony podajesz wartosc zmiennej, ktorej nazwa jest wartoscia parametru (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ($$param - juz pomijajac $_GET i te sprawy). Moze wlasnie tak.