[php] Klasa produkt, dobrze zaprojektowana? Opcje

[smajster]

Do swojego sklepu staram się zaprojektować klasę produkt, ale nie do końca wiem czy dobrze się za to biorę (jest to moja pierwsza klasa), także jakbyście mi mogli powiedzieć czy dobrze kombinuje.

jej wstepny projekt:

[PHP] pobierz, plaintext 
<?php
class produkt
{
	var $id = 0; //jak jest zero ozncza to ze klasa nie odpowiada konkretnemu produktowi
 
	function __construct($id_produktu)
	{
		$this->id = $id_produktu;
	}
	function dodaj($grupa, $podgrupa, $producent, $nazwa, $parametr1, $parametr2, $cena, $cenahurt, $promocja, $ilosc, $opis, $zdjecie)
	{
	//dodaje produkt do bazy	
	}
	function kod_produktu($grupa, $podgrupa)
	{
	//generuje unikalny kod produktu	
	}
	function usun()
	{
	//usuwa produkt z bazy
	}
	function wyswietl($cecha)
	{
	//wyswietla konkretna ceche produktu, np. nazwe lub parametr	
	}
	function edytuj($cecha, $wartosc)
	{
	//zmienia konkretna wartosc jednej z cech produktu
	}
	function wartosc($rabat, $rodzaj)
	{
	//oblicza wartosc produktu zaleznie od rodzaju ceny (hurt, detal, promocja) oraz rabatu
	}
 
 
 
}
?>
[PHP] pobierz, plaintext 

Mam watpliwości co do metody dodaj, bo jest ona trochę niespojna z resztą klasy, bo jako jedyna nie wymaga parametru( id produktu) bo jeszcze takiego poprostu nie posiada, także nie wiem czy powinnem ją umieszczać w tej klasie.
Zastanawiam się też jeszcze czy jeden argument to nie mało, ale zaden inny mi nie przychodzi na mysl.

Będe wdzieczny za wszelkie uwagi które pomogą mi nabrać doświadczenia w tworzeniu klas.

Pozdrawiam

[Jim]

nie jestem specem od obiektowości, ale w ostatnim projekcie użyłem schematu aktywnego rekordu. Czyli w twoim przypadku to by wyglądało tak:

[PHP] pobierz, plaintext 
<?php
class product {
  public $id;
  public $title;
  public $description;
  public $price;
  public $count;
 
 
  /**
   * konstruktor
   * jeżeli podany jest parametr, pobierane sa dane z bazy danych
   */
  public function __construct( $id = null ) { }
 
 
  /**
   * pobieranie danych
   * jeżeli podany jest $id, pobierane sa dane z bazy danych
   */
  public function fetch(  ) { }
 
 
  /**
   * update
   * jeżeli podany jest $id, następuje UPDATE bazy danych, jeśli nie to INSERT
   */
  public function update( ) { }
 
  /**
   * delete
   * jeżeli podany jest $id, rekord jest usuwany z bazy danych
   */
  public function delete(  ) { }
 
}
?>
[PHP] pobierz, plaintext 

to tyle w tej sprawie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)




W tym sposobie najlepsze jest to, że można utworzyć klasę abstakcyjną z metodami fetch(), update(), delete() po której będą dziedziczyły klasy product, user, category itd. Wtedy dopisywanie nowych modułów jest prostrze.

przykład klasy abstrakcyjnej

[PHP] pobierz, plaintext 
<?php
abstract class data {
 
  /**
   * funkcja która w zależności od parametru zwaraca nazwę klucza podstawowego, 
   * lub nazwę tabeli na której dokonywane są operacje.
   * jest to niezbędne dla metod korzystających z bazy danych, przecież muszą wiedzieć
   * do jakiej tabeli wstawić dane.
   */
  abstract function cfg($setting);
 
  function update() { }
  function delete() { }
 
  /**
   * pobiera dane rekordu z bazy danych
   */
  function fetch() { }
 
  /**
   * formatuje dane rekordu tak aby można było je wstawić do zapytania SQL
   */
  function ListDataSql () { }
 
  /**
   * pobiera dane rekordu jako tablicę
   */
  function ListDataArray() { }
}
?>
[PHP] pobierz, plaintext 

wtedy klasa product (user, cokolwiek) wyglądałaby tak:

[PHP] pobierz, plaintext 
<?php
class product extends data{
  public $id;
  public $title;
  public $description;
  public $price;
  public $count;
 
  public function __construct( $id = null ) { }
  public function cfg($setting) { }
 
  /** 
   * można dodać też metodę validate() do walidacji danych np pochodzących z formularza
   */
}
?>
[PHP] pobierz, plaintext 

jednak problem powstał, gdy trzeba było utworzyć bardziej skomlikowane relacje między rekordami (możliwe że jeszcze czegoś nie wiem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) )




zlinczujcie mnie jeśli głupoty gadam. (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

Ten post edytował Jim 28.07.2006, 10:33:05