[php 5.1] Pisać osobne klasy czy dziedziczenie? Opcje

[szczurek]

Witam,

Mam pytanie. Na stronie będę musiał zarządzać takimi częściami jak Uzytkownik, Autor, Utwor, Komentarz, News

No i dla każdego zamierzam napisać klasę z metodami tylko, że poza innimi metodami wszystkie zawierają wspólne:

• doaj
• pobierz
• aktualizuj
• usun

Jeszcze sprawdz dane, ale to będzie na pewno metoda prywatna.

Zastanawiam się czy mam kombinować z napisaniem jakiejś klasy zawierającej te 4 metody i później klasy uzytkownik, autor itd. Mają dziedziczyć z tej klasy modyfikując właściwie pola które dodają czy pobierają czy każda klasa ma zawierać pisane od początku metody i tylko mają implementować interface który zawiera abstrakcyjne metody dodaj, pobierz, aktualizuj i usun.

[cadavre]

Sądzę, że te metody będą zajmować się zupełnie czym innym (dodanie usera, dodanie newsa) więc myślę, że nie wiele wspólnego będą mieć. Ty powinieneś znać logikę swojej aplikacji. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[batman]

Odpowiedź na swoje pytanie znajdziesz w moim projekcie konkursowym (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
A tak na poważnie. Tu masz przykład jak może wyglądać klasa, po której mogą dziedziczyć Twoje klasy (wyjęte na żywca z projektu):

[PHP] pobierz, plaintext 
<?php
/**
 * Klasa Fieldset
 *
 * Klasa udostepniająca jednolity interface do pobierania i modyfikacji danych. 
Nie można bezpośrednio utworzyć obiektu klasy. Jedynie przez klasę potomną.
 * Klasy nie się zastosować w przypadu tabel - relacji, w których nie ma jasno określonego klucza podstawowego. Rozwiązaniem t
ego problemu może być
 * utworzenie widoku.  
 *
 * @author Maciej "Batman" Wilgucki
 * @package DataList 
 *
 */
abstract class Fieldset
{
	/**
	 * Nazwa tabeli, którą wykorzystuje klasa
	 *
	 * @var string
	 */
	protected $stable;
 
	/**
	 * Nazwa pola będącego kluczem podstawowym
	 *
	 * @var string
	 */
	protected $idfield;
 
	/**
	 * Wartość pola będącego kluczem podstawowym
	 *
	 * @var mixed
	 */
	protected $idvalue;
 
	/**
	 * Flaga określająca, czy dane zostały już załadowane
	 *
	 * @var bool
	 */
	protected $bloaded = false;
 
	/**
	 * Tablica asocjacyjna zawierająca wiersz o identyfikatorze określonym przez $id
value
	 *
	 * @var array
	 */
	protected $arow = array();
 
	/**
	 * Flaga określająca, czy dokonano modyfikacji danych
	 *
	 * @var bool
	 */
	protected $bmodified = false;
 
	/**
	 * Tablica asocjacyjna zawierająca zmodyfikowane pola
	 *
	 * @var array
	 */
	protected $amodified_fields = array();
 
	/**
	 * Metoda, która powinna zwracać kod SQL odpytujący bazę pod kątem ilości wiersz
y w tabeli. Musi zostać zadeklarowana w klasie potomnej
	 */
	abstract public function getTotal();
 
	/**
	 * Metoda, która powinna zwracać kod SQL zwracający dane z bazy danych. Musi zos
tać zadeklarowana w klasie potomnej
	 */
	abstract public function getList();
 
	/**
	 * Inicjalizacja obiektu (nie mylić z konstruktorem). Metoda ta musi być wywołana, nim jakiekolwiek inne operacje na danych będ
ą wykonywane.
	 * W metodzie tej ustawiane są nazwa tabeli, nazwa pola będącego kluczem podstaw
owym oraz opcjonalnie wartość tego pola
	 *
	 * @param string nazwa tabeli
	 * @param string nazwa pola będącego kluczem podstawowym
	 * @param string wartość pola będącego kluczem podstawowym	 
	 * @return void
	 */
	public function init($stable,$idfield,$idvalue=0) {
		$this->stable = $stable;
		$this->idfield = $idfield;
		$this->idvalue = (int)$idvalue;
	}
 
	/**
	 * Zapisanie wiersza, jeśli jeszcze to nie miało miejsca, do tabeli asocjacyjnej
. Jeśli wiersz został pobrany, 
	 * wówczas wszystkie zapytania o wartości poszczególnych pól, będą kierowane do 
tablicy asocjacyjen $arow.
	 *
	 * @return void
	 */
	public function load() {
		if(!$this->bloaded) $this->forceLoad();
	}
 
	/**
	 * Wymuszenie pobrania danych z bazy danych. Zawartość tablicy asocjacyjnej $aro
w zostaje zastąpiona wartościami pobranymi z bazy.
	 * Nie powinno się używać tej metody do ładowania danych, ponieważ każde jej wyw
ołanie powoduje wykonanie zapytania do bazy danych,
	 * co może spowodować spadek wydajności.
	 *
	 * @return void
	 */
	public function forceLoad() {
		if(empty($this->stable) || empty($this->idfield) || empty($this->idvalue))
			throw new FieldSetException("You can't load data with empty params. Use init first!");
 
		$sql = "select * from ".$this->stable." where ".$this->idfield." = ".$this->idvalue;
		$Connection = Connection::Main();
		$tab = $Connection->getRows($sql);
		$this->arow = $tab[0];
		$this->bloaded = true;
	}
 
	/**
	 * Pobranie wartości pola bazy danych
	 *
	 * @param string nazwa pola z tabeli, którego wartość ma zostać zwrócona
	 * 
	 * @return string
	 */
	public function getField($sname) {
		if(!$this->bloaded) $this->load();
		if(!array_key_exists($sname, $this->arow))
			throw new FieldSetException("There is no <b>$sname</b> field in ".$this->stable);
		return $this->arow[$sname];
	}
 
	/**
	 * Zapisanie nowej wartosci pola. By dane zostały zapisane do bazy należy wywoła
ć metodę save
	 * @see save	 
	 *
	 * @param string nazwa pola z tabeli, którego wartość ma zostać zapisana
	 * @param string wartość, która ma zostać zapisana
	 * 
	 * @return void
	 */
	public function setField($sname, $svalue) {
		$this->bmodified = true;
		$this->amodified_fields[$sname] = $svalue;
	}
 
	/**
	 * Zapisuje do bazy danych zmiany dokonane w wiersdzu. Jeśli dane nie zostały wc
zesniej załadowane lub nie jest okreslona wartość
	 * pola bedącego kluczem podstawowym, wówczas dane dodawane są do tabeli. W prze
ciwnym razie - modyfikowane.	 
	 *
	 * @return void
	 */
	public function save() {
		if($this->bmodified) {
			$Connection = Connection::Main();
			if($this->bloaded && $this->idvalue > 0)
				$Connection->update($this->stable,$this->amodified_fields,"where ".$this->idfield." = ".$this->idvalue);
			else
				$Connection->insert($this->stable,$this->amodified_fields);
		}
	}
}
?>
[PHP] pobierz, plaintext 

[Cysiaczek]

@szczurek

Moim zdaniem najlepsza jest kompozycja w tym wypadku, zatem lepiej jak klasy będą implementowały interfejs.

Pozdrawiam.

[ActivePlayer]

@batman opisywalem tą metodę w phpsolutions 04/2006, podałem tam gotowy kod klasy, razem z rozwiązaniem lazy init'a, finnderów danych itp

[batman]

@ActivePlayer
Tak się składa, że nie czytuję phpsolutions. A klasę sam napisałem, nie wzorując się na niczyich przykładach.

[athabus]

Ja zgodzę się z Cysiaczkiem - najlepiej w takich przypadkach zazwyczaj użyć Interfejsu - wtedy możesz w razie konieczności sprawdzać czy dany obiekt implementuje dany interfejs (np. gdy gdzieś w innym miejscu aplikacji będzie metoda mogąca brać dowolne obiekty, które implementują te metody).

Z Twojego opisu sytuacji wnioskuje jednak, że poza takimi samymi nazwami metod nie idzie za nimi większa abstrakcja, więc pytanie czy w ogóle jest sens tutaj wprowadzać jakiekolwiek zależności.

Pamiętaj, że dziedziczenie nie powstało po to aby "zaoszczędzić pisania", a żeby pozwalać na różne poziomy abstrakcji i ułatwiać pisanie/rozbudowywanie kodu. Dziedziczenie na siłę klas, które nie są logicznie powiązane powoduje zazwyczaj tylko bałagan w kodzie i problemy z jego utrzymaniem.

//edit

No może tutaj można zrobić specjalny obiekt, który będzie operował na bazie i będzie pozwalał na dodawanie / usuwanie /modyfikowanie/odczytywanie poszczególnych danych - coś jak Active Record Pattern

Ten post edytował athabus 4.01.2007, 17:32:17

[dr_bonzo]

Przy ActiveRecord dziedziczenie ulatwia sprawe -- definiujesz pola i ich typy w podklasach a nadrzedna klasa zajmuje sie ich zapisem i odczytem.

[szczurek]

Dzięki wielkie za odpowiedzi, zdecydowałem się na użycie interface'u.

Tylko oczywiście "znowu w życiu mi nie wyszło.

Jak już ktoś zauważył metody różnych klas będą miały różne pola w bazie, no więc i różne parametry(mógłbym przyjmować tablice jako jeden parametr, ale nie widzę w tym większego sensu).

Napisałem więc interface

[PHP] pobierz, plaintext 
<?php
interface nazwa{
function Dodaj();
 
(Pozosta&#322;e metody tak samo)
}
?>
[PHP] pobierz, plaintext 

W klasie jest np.

[PHP] pobierz, plaintext 
<?php
class Uzytkownik implements nazwa{
public function Dodaj($param, $param2)
return true;
}
?>
[PHP] pobierz, plaintext 

No i niestety wywala mi kominukat że deklaracja musi być zgodna z interface'm

Jeśli w interface będzie

[PHP] pobierz, plaintext 
<?php
interface nazwa{
public function Dodaj($param1, param2);
}
?>
[PHP] pobierz, plaintext 

To oczywiście nie ma problemu. Ale ja nie mogę jasno orkęslić jakie to będą paramerty(chyba, że jak pisałem tablica).


Mógłbym napisać abstrakcyjną klasę, no ale jak mówiłem zdecydowałem się na interface, po waszych wypowiedziach doszedłem do wniosku, że w moim przypadku to będzie lepsze.

Ten post edytował szczurek 8.01.2007, 10:39:47

[dr_bonzo]

W twoim przypadku wspolny interfejs Dodaj(...) jest nieprzydatny bo nie jest wspolny (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) kazda klasa musi u ciebie miec inna liste parametrow, a zadeklarowanie tych metod jako interfejs nic nowego nie wnosi.

Mozesz np zrobic tak (jak jest np w RoR), wzorzec ActiveRecord:

[PHP] pobierz, plaintext 
<?php
$user = new User();
$user->setName( 'Bob' );
$user->setAge( 23 );
...
$user->save();
?>
[PHP] pobierz, plaintext 

i dopisz do tego implementacje

[szczurek]

Dzięki wielkie, pozdrawiam.

[athabus]

Musisz zadać sobie pytanie: "po co ja chcę to zrobić". Użycie interfejsu ma sens jeśli chcesz ujednolicić interfejs - jeśli natomiast to nie jest Twoim celem to jest to używanie mechanizmów obiektowych "na siłę".

Wspólny interfejs jest po to aby potem np. pisać metody, które jako parametr mogą brać obiekty implementujące dany interfejs

np.

[PHP] pobierz, plaintext 
<?php
class a
{
 
public static function (ICRUD $obj)
{
//ta metoda moze wykonac operacje na dowolnym obiekcie implemetujacym interfejs IC
RUD
}
}
?>
[PHP] pobierz, plaintext 

W twoim wypadku, jeśli każda metoda bierze inne parametry trudno będzie pisać taki kod, bo w zasadzie nic poza nazwą nie jest wspólne. Tak jak pisałem wcześniej i to co pisze dr_bonzo - możesz zastosować wzorzec ActiveRecord - ale napisanie dobrej klasy tego typu jest dość trudne. Może popatrz jak robili to inni.