klasa Artykuly, Czy to dobry tok rozumowania OOP? Czy można to zoptymalizować? Opcje

[l05i3k]

Witam,
jestem początkujący w OOP i mam kilka wątpliwości i pytan w związku z klasami. Napisałem przykładową klasę (właściwie jej część):

[PHP] pobierz, plaintext 
<?php
class Artykuly { // extends Tresci
   private $cBaza; //uchwyt bazy
   private $artId;
   private $artTytul;
   private $artTresc;
   private $artAutor;
   private $artNotatki;
   private $artData;
   private $artCzas;
   private $artOdslony;
   private $artKategoria;
   private $prefix = 'testy'; // prefix do tabel w bazie danych
 
   private function __construct() {
	  $this->cBaza = pg_connect ('user=user password=haslo dbname=baza');
 
	  if (!is_resource($this->cBaza)) {
		 throw new Exception ("Nie można się połączyć z bazą.");
	  }
   }
   
   private function getArt($id, $ile=1, $początek=0) {
	  // umieścić funkcje pobierania w odzielnej fukncji, czy lepiej dodać do konstrukto
ra?
	  $zapytanie = "SELECT * FROM `".$prefix."_artykuly` WHERE `id` = '".$id."' ORDER BY `id` ASC LIMIT '".$poczatek."', '".$ile."';";
	  $res = pg_query($this->cBaza, $zapytanie);
	  if (!is_resource($res)) {
		 throw new Exception("Błąd: Nieprawidłowe zapytanie. Sprawdź skrypt!");
	  }
 
	  if (!pg_num_rows($res)) {
		 throw new Exception("Błąd: Brak artykułu o identyfikatorze ".$id.".");
	  }
 
	  // i tutaj nachodzi mnie pytanie, jak wydobyć kilka artykułów? Czy mogę spokojnie użyć pętli foreach?
	  $dane = pg_fetch_array($res);
	  $this->artID = $id;
	  $this->artTytul = $dane['tytul'];
	  $this->artTresc = $dane['tresc'];
	  $this->artAutor = $dane['autor'];
	  [..] // pozostałe przypisy do właściwości klasy
	  $this->artKategoria = $dane['kategoria'];
   }
   
   function __get($parametr) { // wiadomo co to za funkcja... i tak tylko na tymczasem...
	  if (isset($this->dane[$parametr])) {
		 return $this->dane[$parametr];
	  }
	  return '';
   }
   
   private function usunArt($id) {
	  // czy mogę w tym miejscu utowrzyć egzemplarz tej klasy?
	  // $artDoUsuniecia = new Artykuly();
	  // $artDoUsuniecia->getArt($id);
	  // if (!isset($artDoUsuniecia->artTytul)) {
	  //	echo 'Błąd: Brak artykułu o identyfikatorze '. $id .', który chcesz usunąć!';
	  // }
	  // czy muszę jeszcze raz pisać kod, aby sprawdzić czy dany artykuł istnieje?
	  // czy może istnieje jakiś inny sposób, aby sprawdzić czy ten artykuł został zapis
any w bazie, uzywając np. '$this->getArt($id)'?
 
	  $zapytanie = "DELETE FROM `".$prefix."_artykuly` WHERE `id` = '".$id."';";
	  $res = pg_query($this->cBaza, $zapytanie);
	  if (!is_resource($res)) {
		 throw new Exception("Błąd: Nieprawidłowe zapytanie. Sprawdź skrypt!");
	  }
   }
   
   private function getKategoria($id) {
	  return $this->artKategoria;
   }
   
   private function getNextArtID($id) {
	  [..] // inne fukcje
   }
   
}
?>
[PHP] pobierz, plaintext 

Większość zastanawiających mnie rzeczy zamieściłem w komentarzach w kodzie, byłbym wdzieczny, gdyby ktoś rozwiał moje wątpliwości...
Chciałbym się dowiedzieć, czy mój tk rozumowania OOP jest prawidłowy? Czy można ten fargment klasy jakoś zoptymalizować?

Z góry dziękuję, za wszelkie sugestie...

[jang]

[PHP] pobierz, plaintext 
<?php
class Artykuly { // extends Tresci
 
class Artykuly extends Tresci
?>
[PHP] pobierz, plaintext 

// umieścić funkcje pobierania w odzielnej fukncji, czy lepiej dodać do konstruktora?

nie funkcje tylko metody
ani tu ani tu drogi Winetou tylko w osobnej klasie

[PHP] pobierz, plaintext 
<?php
class DAO_Pg_Artykuly <- dla postgre
class DAO_Pdo_Artykuly <- dla PDO
?>
[PHP] pobierz, plaintext 

W powyższych klasach metody z zapytaniami do bazy

w klasie Artykuly

[PHP] pobierz, plaintext 
<?php
public function __construct()
	$this->db = Db_Pg::getInstance();
	$this->objDaoPg = new DAO_Pg_Artykuly($this->db);
}
 
public function getArt($id, $ile=1, $początek=0)
{
	$this->dane = $this->objDaoPg->getArt($id, $ile=1, $początek=0);
}
 
public function getArtTytul()
{
	return ($this->artTytul = $this->dane['tytul']);
}
?>
[PHP] pobierz, plaintext 

// czy mogę w tym miejscu utowrzyć egzemplarz tej klasy?

Powinieneś raczej poczytać co to private, public ...
no i oczywiście singleton !
Jeśli chcesz to robić (tworzyć obiekt klasy w jej wnętrzu) to zobacz klasę user pod tym adresem - http://artykuly.zyxist.com/czytaj.php/wlasny_mechanizm_sesji