OOP i projektowanie Opcje

[eVolution]

Witam,
Jako, że dopiero zaczynam naukę OOP, chciałbym się Was zapytań, czy jest to dobrze napisana klasa.

Klasa do obsługi bazy danych

[PHP] pobierz, plaintext 
<?php
 
class DB
{
	public function __construct()
	{
		try
	   	{
	       $this->baza = new PDO('mysql: host=localhost; dbname=db', 'login','pass');
	   	}
	   	catch(PDOException $e)
	   	{
	      throw new Exception($e->getMessage());
	   	}
	}
 
	public function prepare($query)
	{
		return $result=$this->baza->prepare($query);
	}
}
 
?>
[PHP] pobierz, plaintext 

Klasa do obsługi użytkownika czyli dodanie, kasowanie

[PHP] pobierz, plaintext 
<?php
class User
{
	protected $_db;
 
	public function __construct(DB $db)
	{
		$this->_db = $db;
	}
 
	public function add($login, $pass)
	{
		$result = $this->_db->prepare("INSERT INTO user VALUES(:login, :pass)");
		$result->bindParam(':login', $login, PDO::PARAM_STR);
		$result->bindParam('pass', $pass, PDO::PARAM_STR);
 
		if(!$result->execute()){
			throw new Exception("Wystąpił błąd podczas dodawania usera.");
		}
	}
 
	public function delete($id)
	{
		$result = $this->_db->prepare("DELETE FROM user WHERE id=:id");
		$result->bindParam(':id', $id, PDO::PARAM_INT);
 
		if(!$result->execute()){
			throw new Exception("Wystąpił błąd podczas usuwania usera.");
		}
	}
}
[PHP] pobierz, plaintext 

I oczywiście osobno UserManger czyli odpowiedzialna za pobranie danych typu login, logowanie etc...

Czy w tym kodzie jest wszystko ok? Kolejna sprawa, blok try - catch dawać w metodach w klasie, czy w miejscu gdzie dana metoda jest wywołana?

Ten post edytował eVolution 20.03.2012, 18:36:29

[by_ikar]

Czyli rozumiem, że najlepiej zawartość konstruktora z klasy DB dać do klasy User w konstruktor, i jest to lepszym rozwiązaniem?

Nie. @wookieb'owi chodziło o to że twoja klasa DB nie robi nic innego prócz połączenia na sztywno, bo zarówno nie można przekazać żadnych ustawień, jak i ta klasa poza łączeniem się z bazą nic nie robi. I coś takiego większego sensu nie ma, bo równie dobrze gdzieś wcześniej możesz utworzyć połączenie w PDO i instancje PDO przekazać do klasy użytkownika. Pomijam już kwestię że praktycznie żadnych danych nie sprawdzasz..

Czy w tym kodzie jest wszystko ok? Kolejna sprawa, blok try - catch dawać w metodach w klasie, czy w miejscu gdzie dana metoda jest wywołana?

A to już zależy od tego w jaki sposób chcesz obsłużyć wyjątek. Może masz tam jakiegoś handlera, wówczas nie musisz stawiać wszędzie try/catch, a jeżeli musisz wykonać inne działanie niż zatrzymanie aplikacji i komunikat o błędzie, to używaj try/catch..