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

no mozna powiedziec ze dobrze.. ale napisz cos wiecej - ja bym zrobil to nieco inaczej, moze w wolnej chwili napisze taka klase w PHP5 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) a narazie napisz cos wiecej na temat swojej

nadaj jakies wartosci zmiennym w var - wartosci domyslne i dodaj na koncu ";"

[enigma]

no mozna powiedziec ze dobrze.. ale napisz cos wiece

trochę to przemyślałem, i myślę że można zrobić kilka klas (żeby nie wszystko było pod licznik, można powiedzieć: początek frameworka (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) ) Przypominam że nie miałem do czynienia z pisaniem programów obiektowych no i narazie nie znam zbyt dokładnie obiektówki w php, znam c++, więc może być trochę błędów składniowych
dobra, a teraz przechodzę do licznika

=================================

klasa do pobierania informacji o pliku php, jego tytuł, nazwę, może się przydać do innych rzeczy, nie tylko do licznika
file_info.php

[PHP] pobierz, plaintext 
<?php
class file_info
 {
  var $file_name;
  var $site_title; // pobierane z <title> </title>
  var $name_type; // 0 - $PHP_SELF, 1 - rozszerzona
 
  public:
  function file_info($param = 0) // konstruktor
   {
	 $this -> $name_type = $param;
	}
  function get_title() {;} // zapis do zmiennej $site_title
  function get_file_name() {;} // zapis do zmiennej $file_name
 }
?>
[PHP] pobierz, plaintext 

klasa tworząca "pojemnik" na dane
data_container.php

[PHP] pobierz, plaintext 
<?php
class data_container
 {
   // zmienne
   function data_container($param = 0) // konstruktor
	{
	  // 0 - pliki,  1 - baza danych
	}
   // inne funkcje odpowiedzialne za łączenie z bazą danych
   // lub tworzenie plików
  }
?>
[PHP] pobierz, plaintext 

główna klasa - licznik
counter.php

[PHP] pobierz, plaintext 
<?php
class counter
 {
  include("file_info.php");
  include("data_container.php");
  var $name_type; // 0 - $PHP_SELF, 1 - rozszerzona
  var $file = new file_info($name_type); // tworzy nowy obiekt, informacji o pliku
  var $site_title; // pobierane z <title> </title>
  var $file_name = $file -> get_file_name(); // do zmiennej zapisuje nazwę bieżącego pliku php
 
  public:
  function record_stat() // jeśli nie chce wypisywać statystyki na stronie to używamy tej funkcji w pliku
  {
   if(isset($refreshes))
	 //jeśli ciacho istnieje no to tylko zwiększamy ilość ODSŁON
  else
	//zwiększenie liczby ODWIEDZIN, i aktualizacja pliku lub bazy
  }
 
  public:
  function show_stat() // jeśli chce wypisywać statystyki na stronie to używamy tej funkcji w pliku
  {
   $this -> record_stat();
   // wypisywanie statystyki na stronę
  }
}
?>
[PHP] pobierz, plaintext 

przykładowa strona index.php

[PHP] pobierz, plaintext 
<html>
<?php
include("counter.php");
licznik = new counter() // zostawiam parametr domyślny, bo plik nazywa się normalnie
?>
<body>
<?php
echo $licznik -> show_stat();
?>
</body>
</html>
[PHP] pobierz, plaintext 

ja bym zrobil to nieco inaczej, moze w wolnej chwili napisze taka klase w PHP5 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) a narazie napisz cos wiecej na temat swojej

to jeszcze nic nie pokazuj (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) chcę sam do tego dojść, oczywiście ze wskazówkami

Pozdrawiam

Ten post edytował enigma 6.11.2006, 19:35:23