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

[eVolution]

Skrobnąłem na szybko coś takiego:

[PHP] pobierz, plaintext 
<?php
class Kalkulator
{
	function dodaj($liczba, $liczba2)
	{
		$this->wynik = $liczba + $liczba2;
		return $this;
	}
 
	function odejmij($liczba, $liczba2)
	{
		$this->wynik = $liczba - $liczba2;
		return $this;
	}
 
	function mnozenie($liczba, $liczba2)
	{
		$this->wynik = $liczba * $liczba2;
		return $this;
	}
 
	function wynik()
	{
		return $this->wynik;
	}
}
 
$kalkulator = new Kalkulator;
echo $kalkulator->odejmij(2,5)->wynik();
[PHP] pobierz, plaintext 

Czy tu jest wszystko ok? I mi nie chodzi o to, żeby ktoś mnie uczył OOP, tylko o to, że mam problem z tym, co np w danej klasie powinno się znaleźć. Np co powinno być w klasie odpowiedzialnej z 1 ucznia itp tak jak np w tej podanej powyżej to wiem co powinno być, ale jak już np chodzi o zrobienie bardziej sensownej klasy to już jest problem.

Poprawcie mnie jeśli się mylę:
Mam np obiekt samochód, czyli jego metody to skręcanie, hamowanie, przyśpieszenie, zwalnianie?
Kolejne mam obiekt użytkownik, jego metody to dodaj, usuń, aktualizuj, i UserManager czyli pokaz wybranego, pokaż wszystkich etc...? Wiem głupie pytania, ale żadna książka akurat tego nie opisuje

Ten post edytował eVolution 21.03.2012, 02:12:59