[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

[Ludvik]

Oczywiście, że można. Tak nawet lepiej będzie. Ale to tylko koncepcja autora. Jeżeli chodzi o głębszą interpretację tej klasy to bym podzielił ją na dwie, albo nawet trzy, bo teraz za dużo ma "na głowie". Zajmuje się przenoszeniem danych użytkownika, pobieraniem ich oraz autoryzacją. Nie tędy droga. Jedna klasa musi przechowywać same dane i udostępniać interfejs, dzięki któremu można na nich operować. Druga klasa odpowiada za pobranie danych użytkownika z bazy i stworzenie obiektu. Trzecia klasa operuje na obiekcie użytkownik i porównuje jego prawa dostępu do wymagań. Tak wszystko będzie bardziej elastyczne. Ale nie można oczekiwać elastyczności po pierwszych rozwiązaniach...

Do autora wątku: Atrybuty klasy wypada izolować, aby zachować nad nimi kotrolę. Dostęp najlepiej zapewnić przez publiczne metody. Radzę zapoznać się z lekturą związaną z OOP. Np. Headfirst Design Patterns - dziwnie napisana książka, ale łatwo przyswajalna. Nie jest zła...