[php OOP] Prosba o opinie nt. pierwszej klasy Opcje

[phpion]

Chcialbym was prosic o opinie i sugestie nt. mojej pierwszej klasy podczsa nauki OOP. Co mozna w niej poprawic, co nalezaloby napisac inaczej. Poczatek czyli laczenie z baza i ustawianie sesji jest tylko tymczasowo. Prosze o informacje co i jak. Do czego moglby sluzyc destruktor w tej klasie? Do zerowania atrybutow? Wogole nie widze wiekszego sensu destruktora w tym przypadku - moze sie myle, prosze mnie wiec oswiecic.

[PHP] pobierz, plaintext 
<?php
$sql = mysql_connect("localhost", "root", "");
mysql_select_db("logowanie");
 
session_start();
 
$_SESSION['login'] = "phpion";
$_SESSION['haslo'] = "haslo";
 
class Uzytkownik
{
	private $login;
	private $haslo;
	private $grupa;
	private $zalogowany;
	private $dostep;
 
	function __construct()
	{
		$this->login = (isset($_SESSION['login'])) ? $_SESSION['login'] : NULL;
		$this->haslo = (isset($_SESSION['haslo'])) ? $_SESSION['haslo'] : NULL;
		$this->grupa = 0;
		$this->zalogowany = $this->dostep = FALSE;
	}
 
	function sprawdzZalogowanie()
	{
		if ($this->login != NULL && $this->haslo != NULL)
		{
			$q = "SELECT grupa FROM uzytkownicy WHERE BINARY login='".$this->login."' AND BINARY haslo='".$this->haslo."' AND stan='1'";
			$q = mysql_query($q);
 
			if (mysql_num_rows($q) > 0)
			{
				$r = mysql_fetch_row($q);
 
				$this->grupa = $r[0];
				$this->zalogowany = TRUE;
			}
		}
	}
 
	function sprawdzDostep($typ, $lista)
	{
		$this->sprawdzZalogowanie();
 
		if ($this->zalogowany == TRUE && $this->grupa != 0)
		{
			$tablica = explode(" ", $lista);
 
			if (($typ == 1 && in_array($this->grupa, $tablica) || ($typ == 0 && !in_array($this->grupa, $tablica))))
				$this->dostep = TRUE;
		}
 
		if ($this->dostep != TRUE)
		{
			header("Location: accessDenied.php");
			exit;
		}
	}
}
 
$uzytkownik = new Uzytkownik();
$uzytkownik->sprawdzDostep(0, "1 2");
?>
Tajna tresc
[PHP] pobierz, plaintext 

PS: w metodzie sprawdzDostep $typ moze byc 0 lub 1 (0 dla wylaczania dostepu dla grup, 1 dla udostepnianiu dla wskazanych grup) i liste grup podajemy jako ciag liczb oddzielonych spacja.

Ten post edytował phpion.com 17.07.2006, 20:18:36

[-Gość-]

Ok, wszystko dokladnie przeczytalem. Wlasnie chodzilo mi o to jak to wszystko zgrabnie napsiac (ten podzial na 3 klasy - wlasnie o takie rady mi chodzilo!). Tak jak pisalem - jest to 1 klasa i w zasadzie wszystko bym do niej wpakowal (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) a wiem ze nie tedy droga.

Czy to tak wiele kosztuje przekazanie $_SESSION["user"] i "pass" do konstruktora

hmmm a to tak nie jest teraz?
Co do tego ze exit nie sa zbyt mile widziale w klasach - nie wiedzialem, dziekuje za info. Filtrowanie danych itd. na razie nie ma bo jak widac to klasa testowa z danymi umieszczonymi przeze mnie w kodzie wiec tym sie na razie zajmowal nie bede. Chodzi mi o sama klase. Natomiast co to samego dzialania skryptu:

[PHP] pobierz, plaintext 
<?php
 
$uzytkownik = new Uzytkownik();
 
if($uzytkownik->sprawdzDostep(0, "1 2")){
 
   echo 'Tajna treść';
 
}
 
else{
 
  echo 'Brak dostępu';
 
}
 
?>
[PHP] pobierz, plaintext 

Chcialem wlasnie to zrobic w jak najkrotszy sposob, zeby nie trzeba bylo wstawiac duzo kodu na kazdej chronionej podstronie. Stad takie rozwiazanie z exitem w srodku...