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 

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

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.

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 )

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