[PHP] pobierz, plaintext 
<?php
/********************************
System obsługi sesji						*
	Autor: Damned Land Team				*
	Stworzono: 2007-03-13 19.05		*
********************************/
 
#Interfejs httpRequest
interface _httpRequest
{
	/*Konstruktor przpisze wartości*/
	public function __construct();
	/*Zwróć ip usera*/
	public function turnIp();
	/*Zwróć dane o systemach usera */
	public function turnBrowser();
	/*Test na bota*/
	public function checkBot();
	/*Destruktor klasy*/
	public function __destruct();
}
 
#Interfejs Session
interface _Session
{
	/*Konstruktor klasy
	[(bool)$status]-TRUE(nietwozyc sesji anonimowego usera)*/
	public function __construct($status=false);
	/*Funkcja tworzy sesje*/
	public function create();
	/*Funkcja kończy sesje*/
	public function finalSession();
	/*Funkcja zwraca ID użytkownika sesji*/
	public function turnUserId();
	/*Sprawdza czy sesja jest aktywna i zwraca
	(bool)true lub (bool)false*/
	public function checkActiv();
	/*Zwraca id sesji*/
	public function turnSessId();
	/*Funkcja zrzucająca sesje do pliku 
	Zapisuje w sposób
	**|**|**|**|**
	[(string)$dir]-katalog dostępu*/
	public function dropToFile($toDir='./');
	/*Uaktualnij ostatnią wizyte*/
	public function sessionUpdate();
}
 
#Klasa załaduje dane o użytkowniku
Class httpRequest implements _httpReuest
{
	#Wymagane zmienne
	#IP użytkownika
	private $ip;
	#Dane o przegląadarce i systemie
	private $browser;
	#test na bota
	private $scout;
 
 
	public function __construct()
	{
		$this -> ip = $_SERVER['REMOTE_ADDR'];
		$this -> browser = $_SERVER['HTTP_USER_AGENT'];
		$this -> scout = '';
	}
 
	final public function turnIp()
	{
		return $this -> ip;
	}
 
	final public function turnBrowser()
	{
		return $this -> browser;
	}
 
	final public function checkBot()
	{
		$this -> scout = '/(googlebot|slurp@inktomi|ask jeeves|lycos|whatuseek|ia_archiver)/i';
		return ((preg_match($this->scout, $this->browser, $match)) ? true : false);
	}
 
	public function __destruct()
	{
		unset($this->ip);
		unset($this->browser);
		unset($this->scout);
	}
}
 
#Zdefiniuj stałe wartości
#Nazwa SESJI
define('COOKIENAME', 'PT_SESSION');
 
 
#Klasa obsługi sesji
Class Session implements _Session
{
	public $request;
	private $test;
 
	public function __construct($status=true)
	{
		#Odwołaj sie do klasy httpRequest;
		$this->request = new httpRequest;
		$this->test = @mysql_fetch_assoc(mysql_query("SELECT sess_id, sess_time, sessuser_id FROM session WHERE sess_id='".$_SESSION[COOKIENAME]."'"));
		#Sprawdz czy ma towyrzyć sesje
		if ($status==true && !isset($this->test['sess_time']))
		{
			$this->create();
		}
	}
 
	public function create($user=0)
	{		
		#AND jest wymagany bo gdy jest brak wpisu w bazie to $this->test zwraca 0=FALSE
		if	(isset($this->test['sessuser_id']) && $this->test['sessuser_id'] == $user)
		{
			return false;
			exit;
		}		
		#Połącz czas oraz ip(można losowa liczba ale wtedy bedzie wieksza szansa na bug)
		$data = COOKIENAME.$this->request->turnIp();
		#Utwórz i zakoduj unikalne id z danych powyżej 
		$data = md5(uniqid($data));
		#Utwórz sesje z uniqid
		$_SESSION[COOKIENAME]=$data;
		#Stwórz indeks sesji w bazie
		if (isset($_SESSION[COOKIENAME]))
		{
		mysql_query("INSERT INTO session (sess_id, sessuser_id, sessuser_info, sessuser_ip, sess_time) 
		VALUES ('".$data."', 
						".$user.", 
						'".$this->request->turnBrowser()."',
						'".$this->request->turnIp()."',
						".time().")");
		}
	}
 
	public function finalSession()
	{
		if (!isset($_SESSION[COOKIENAME]))
		{
			return false;
		}
		else
		{
			unset($_SESSION[COOKIENAME]);
			unset($this->test);
			unset($this->request);
		}
	}
 
	public function turnUserId()
	{
		echo $this->test['sessuser_id'];
	}
 
	public function turnSessId()
	{
		if ($_SESSION[COOKIENAME] != $this->test['sess_id'] || strlen($_SESSION[COOKIENAME]) != 32)
		{
			return false;
		}
		else
		{
			return $this->test['sess_id'];
		}
	}
 
	public function checkActiv()
	{
		return (isset($_SESSION[COOKIENAME]) && isset($this->test['sess_time'])) ? true : false;
	}
 
	public function dropToFile($toDir='./')
	{
		$fileName = $toDir.date('Y-m-d').'.txt';
		$file = @fopen($fileName, 'w');
		echo (!$file) ? false : true;
		$_query = mysql_query("SELECT * FROM session");
		fwrite($file, 'Zrzut sesji z dnia: '.date('Y-m-d H:i:s').', autor klasy do zrzutu daniel1302
		ID sesji|ID użytkownika|Dane o przeglądarce i systemie|IP użytkownika|Data utwor
zenia|Ostatnia wizyta
		');
		while($query = mysql_fetch_assoc($_query))
		{
			fwrite($file, ''.$query['sess_id'].'|'.$query['sessuser_id'].'|'.$query['sessuser_info'].'|'.$query['sessuser_ip'].'|'.$query['sess_time'].'|'.$query['sess_lastvisit'].'
		');
		}
		fclose($file);
	}
 
	public function sessionUpdate() 
	{
		mysql_query("UPDATE session SET sess_lastactiv=".time()." WHERE sess_id='".$this->test['sess_id']."'")
	}
}
?>
[PHP] pobierz, plaintext 

OOP zajmuje się jakieś 3-4 tygodnie(Napisałem już trochę). Proszę o ocenę i pomoc co zmienić
Nie piszcie, że lepsze są ciasteczka, ponieważ to jest dla mnie lepsze.

Opisy funkcji macie w interfejsach oraz na stronie http://pastime.tivi.net.pl/team/php-files/...ead_id=246#2571

Osobiście zrobił bym httpRequest jako klasę abstrakcyjną i Session powinno dziedziczyć.

Ps. Okomentowanie jest okropne...

Też bym tak zrobił, jak by mi ta klasa niebyła potrzebna do innych rzeczy

Moje uwagi:
1. Szalejesz z interfejsami, a (moim zdaniem) nie do konca rozumiesz ich zastosowanie (kontruktor? destruktor? po co to...).
2. turnIp? ObróćIp? Chyba returnIp lub getIp.
3. Użycie $_SERVER w klasie (niby to jeszcze obleci ale dla mnie to to samo co używanie $_POST/$_GET).
4. define - użyj stałych klasowych (const).
5. Zapytania SQL bez żadnych nakładek.

Czyli jak mam zastosować te interfjsy,
Zapytania mam zabezpieczone,
a co uzyc za miast $_SERVER

httpRequest jeżeli dajesz do oceny, to nie jako cześć jakiegoś frameworka, tylko na potrzeby danej klasy, a na potrzeby tej klasy spokojnie jakby był klasą abstrakcyjną.

Może i lepiej, ale to jest wycięty fragment z mojego frameworka

Nie wspominając już o tym jaka jest ta klasa dziurawa... Session Fixation i Hij to nie wszystko... oO nawet SQL Incjetion

$_SESSION[COOKIENAME] wypada przefiltrować, bo można podmienić wartość jej i dopisać coś w stylu DROP TABLE...