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

[Helios]

Jak dla mnie kod procedurlany objety w klase z dodatkami wyjatkow, zeby wygladalo bardziej obiektowo.
Klasa artykuly przechowuje zawartosc artykulu, udostepnia ja, pobiera dane z bazy dobrze ze chociaz uzytkownikowi wynikow nie prezentuje. Zobacz kilka postow nizej jest taki sam temat o Produkcie.
Ja to rozumuje tak: klasa podstawowa jest artykul i on zawiera reprezentacje jednego artykulu - i tu funkcja klasy jest latwa do zdefiniowania - klasa przechowuje reprezentacje artykulu. Klasa odrebna, o ktorej artykul nie wie, tylko udostepnia jej interfejs jest klasa zarzadzajaca artykulami, gdzie w jednym miejscu laczysz sie z baza danych lub wybierasz inne zrodlo danych w celu pobrania artykulow, i implementujesz interfejs zarzadzania artykulami. I ta klasa operuje na klasie bazowej.
I to ma sens, jezeli z czasem chcialbys wprowadzic np. drugi typ artykulu operujacy na odrebnej bazie danych, caly obszar decyzyjny masz w jednym miejscu. No i klasa zarzadzajaca moze przekazac gotowy, przyszykowany obiekt artykulu do innego obiektu w celu jego przetworzenia itd.
Zachowanie takiej odrebnosci wydaje mi sie logiczne. Bo nielogiczne dla mnie jest zeby klasa Artykuly (nie Artykul) przechowywala dane jednego artykulu, usuwala, pobierala itd.

Ten post edytował Helios 4.11.2007, 10:47:57