Prosta strona w OOP, klasa albums i proste obiekty.. Opcje

[fannet]

Witam serdecznie wszystkich.. zastanawiam się czy dobrze to wszystko rozumiem. Stworzyłem już standardowo (psełdo)klasę Albums na potrzeby strony z stosunkowo prostą bazą danych z albumami różnych wykonawców. Czy jest to zrobione dobrze w jak dużym stopniu przypomina to programowanie obiektowe. Problem w tym, że moje 3 pozostałe klasy (Artists, Songs, Categories) wyglądałby niemal identycznie tylko inne właściwości i nazwy.

[PHP] pobierz, plaintext 
class Albums {
 
	private $id;		// id Albumu w bazie danych
	private $name;		// nazwa albumu
	private $jear;		// rok wydania
	private $artist;		// artysta
	private $category;	// gatunek
 
	public function __construct() {
		// ustanawiam polaczenie z baza danych
		// wykorzstuje $this->db = new MySQLi..
	}
 
	// funkcje dostepowe 
	public function getId() { return $this->id; }
	public function getName() { return $this->name; }
	public function getJear() { return $this->jear; }
	public function getArtist() { return $this->artist; }
	public function getCategory() { return $this->category; }
 
	// funckje dostepowe - ciag dalszy
	public function setId($id) { $this->id = $id; }
	public function setName($name) { $this->name = $name; }
	public function setJear($jear) { $this->jear = $jear; }
	public function setArtist($artist) { $this->artist = $artist; }
	public function setCategory($category) { $this->category = $category; }
 
	public function create() {
		$query = "INSERT INTO...";
		$this->db->query($query);
 
		while($row = $query->fetch_object()) {
			$this->setName($row->name);
			$this->setJear($row->jear);
			// i tak dalej
		}
	}
 
	public function select() {
		$query = "SELECT album... WHERE id == {$this->getId()}";
		$this->db->query($query);
	}
 
	public function update($field, $value) {
		$query = "UPDATE album SET $field = $value WHERE id == {$this->getId()}";
		$this->db->query($query);
	}
 
	public function delete() {
		// w taki sam sposób kasuje
	}
}
 
// przykładowa strona wyswietlajaca szczegółowe dane na temat albumu
$album = Albums;	// nowy album
$album->setId(4);	// ustawiam id 4
$album->selsect();	// ustawiam album
 
// tworzac album
$album = new Albums;
$album->setName($_POST['name']); 
$album->setJear($_POST['jear']); 
$album->create();
 
// edytując album
$album = new Albums;
$album->setId(4);
$album->selsect();
if($_POST['name'] != $album->getName()) { $album->update('name',$_POST['name']); }
[PHP] pobierz, plaintext 

[drozdii07]

Sprawdz zapytania, masz tam bledy. I to nie szczesne jear..Co ty miales z ang w szkole ?

[Crozin]

1. Tutaj masz klasy Album, Artysta, Piosenka, a nie ich kolekcje więc używaj liczby pojedynczej w ich nazwach.
2. Każdy obiekt zajmuje się wyłącznie jednym zadaniem. Dlatego też powinieneś rozdzielić reprezentowanie danych i ich zapisywanie w bazie danych do dwóch różnych obiektów, Album (getTitle, getCategory, setArtist, ...) oraz AlbumManager (saveAlbum, deleteAlbum). W sumie pobieranie ich z bazy danych to również zadanie dla innego obiektu, powiedzmy AlbumRepository.
3. Ułatw sobie życie i korzystaj z type hintingu.

[Psajkus]

... oraz AlbumManager (saveAlbum, deleteAlbum). W sumie pobieranie ich z bazy danych to również zadanie dla innego obiektu, powiedzmy AlbumRepository.

Myślę, że samo repozytorium wystarczy.Repozytorium zawiera przede wszystkim operacje CRUD (Create, Read, Update, Deleted) oraz wszystkie inne operacje selekcji (np. GetById lub GetByQuery )

Ten post edytował Psajkus 8.07.2011, 07:28:41

[gac3k]

A ja zamiast tworzyć 50 metod typu setId itd. zrobiłbym dwie magiczne metody __set i __get i tak to obsłużył.