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

[enigma]

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

ale o to chodzi, dostaję index.php, obcinam .php, dodaję .txt i mam plik index.txt w którym jest liczba odwiedzin

[bim2]

I tu jest problem, bo ja np uzywam index.php?podstrona=index index.php?podstrona=kontakt... rozumiesz?

[Adrian Staniszew...]

Przeczytaj jeszcze raz mojego posta.. podstrona to nie zawsze - a nawet zazwyczaj - samo index.php, news.php itp..

[enigma]

hmm, a cobyście poradzili ?

[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 ($$param - juz pomijajac $_GET i te sprawy). Moze wlasnie tak.

[Adrian Staniszew...]

Ja bym to wrzucil do klasy i w przypadku stron statycznych lub "pojedynczych" normalnie wywolywal klase dodajac parametr "name"

np.

[PHP] pobierz, plaintext 
<?
 
$licznik = new Counter('0', 'news#id');
 
?>
[PHP] pobierz, plaintext 

teraz taki kod dajesz w plikach zmieniajacych sie - np. w news czy artach, a gdzie indziej dajesz po prostu $PHP_SELF

parametr $id mozesz oczywiscie zamienic innym, identyfikujacym twoja strone np. page#news, page#arts, go#download itp..

[enigma]

Ja bym to wrzucil do klasy i w przypadku stron statycznych lub "pojedynczych" normalnie wywolywal klase dodajac parametr "name"

znaczy się, cały licznik do klasy

parametr $id mozesz oczywiscie zamienic innym, identyfikujacym twoja strone np. page#news, page#arts, go#download itp..

czyli jakby strona z newsami miała 5 podstron, numerowanych, to jako id zwracany byłby numer podstrony i taki też plik tworzony np. news#5.txt , tak??

[Adrian Staniszew...]

Tak, ale niekoniecznie uzywaj # w nazwach plikow.. Polecam uzyc np. "_"

[enigma]

tak tyczy się też pierwszego pytanie z postu nr 9
czy może są jeszcze jakieś uwagi, jeśli nie to wielkie głównie dla Adriana

[Adrian Staniszew...]

Tak do obu pytan

Skoro - a raczej jesli - przepiszesz to do klasy to dodaj mozliwosc zablokowania countera ciasteczkiem - bedzie to szybsze i wydajniejsze rozwiazanie.

Osobiscie niezbyt tez widze zastosowanie dla Cach_IP.. niby fajnie bo mam adresy IP odwiedzajacych ale po co to mi ? geolokalizacje moglbym zrobic, ale i tak nie bede wiedzial kiedy mialy wejscia z danych IP, nie wiem tez ile razy mialy miejsce.

Pomysl nad rozbudowaniem systemu to prosciutkich statystyk.

Pozdrawiam

p.s
co wielkie dla adriana:P

[enigma]

dodaj mozliwosc zablokowania countera ciasteczkiem - bedzie to szybsze i wydajniejsze rozwiazanie.

to znaczy, co konkretnie mam blokować, albo raczej w jakim celu

Osobiscie niezbyt tez widze zastosowanie dla Cach_IP.. niby fajnie bo mam adresy IP odwiedzajacych ale po co to mi ? geolokalizacje moglbym zrobic, ale i tak nie bede wiedzial kiedy mialy wejscia z danych IP, nie wiem tez ile razy mialy miejsce.

heh w sumie to nie jest bardzo potrzebne, ale nie trzeba odpalać tej funkcji na p[odstronach a co do tego ile razy miały miejsce odwiedziny z danego ip, przeciez liczy dla każdego osobno

Pomysl nad rozbudowaniem systemu to prosciutkich statystyk.

do tego właśnie chcę zmierzać

co wielkie dla adriana:P

hmm wielkie DZIĘKUJE

[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

jako że nie miałem jeszcze z o obiektami wiele do czynienia zapytam o klasę, czy dobrze myślę

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

[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 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 ) 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 a narazie napisz cos wiecej na temat swojej

to jeszcze nic nie pokazuj chcę sam do tego dojść, oczywiście ze wskazówkami

Pozdrawiam

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

[Adrian Staniszew...]

napewno nie framewroka raczej malego systemu, jednak nie rozdrabniaj sie za bardzo

funkcje definiujemy

Kod

public/private function nazwa() {}

a tak wlasciwie to piszesz w php4 czy 5 ? jak 5 to zamien te var na public/private/protected

caly czas czekam na jakis kod.. nie same "Szablony"

Ten post edytował Adrian Staniszewski 7.11.2006, 09:16:28

[enigma]

napewno nie framewroka raczej malego systemu

o czegos trzeba zacząć

jednak nie rozdrabniaj sie za bardzo

więcej klas nie będzie, a te 2 pierwsze mogą się przydać do innych celów

a tak wlasciwie to piszesz w php4 czy 5 ?

chcę w 5, jestem w trakcie czytania manuala

caly czas czekam na jakis kod.. nie same "Szablony"

spokojnie pierwsze muszę stworzyć dobry szablon, a późnie zabierać się za konkretne pisanie, także proszę jescze o uwagi do szablony
a kod będzie powstawał bardzo powoli, ponieważ muszę dużo czasu poświęcać na lekturę wspaniałej książki

[bim2]

Ja bym rozwiązał to tak jak w rapide. Mam taki system logow i w zmiennej private $oModule zapisuje obiekt systemu zapisu. I moze byc kilka klas. A wywoluje:
$log = new Log;
$log->register('file', new Log_file);
$log->register('base', new Log_base);
$log->setDefault('base');
i uzycie:
$log->make('msq'[, nazwa modulu]); jesli nie ma bierze domyslny