[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

[TomASS]

@Jim: Bardzo trafne i fajne uwagi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Bardzo dobrym pomysłem jest użycie klasy abstrakcyjnej lub interfejsu.

@smajster:
Skoro używasz __construct to znaczy się, że używasz PHP5, więc metody i właściwości nadawaj atrybuty private/protected/public, tak będzie bardziej elegancko.

Z dodawaniem to zrób tak:
1. konstruktor nie może miec domyslny ID taki jak to podał Jim:

[PHP] pobierz, plaintext 
<?php
__construct( $id = null )
?>
[PHP] pobierz, plaintext 

2. tworzysz sobie taki obiekt bez parametru ID
3. wywołujesz metodę dodaj, a jako parametr możesz przekazac tablice dodatkowych parametrów.
zamiast

[PHP] pobierz, plaintext 
<?php
function dodaj($grupa, $podgrupa, $producent, $nazwa, $parametr1, $parametr2, $cena, $cenahurt, $promocja, $ilosc, $opis, $zdjecie)
?>
[PHP] pobierz, plaintext 

zrób tak

[PHP] pobierz, plaintext 
<?php
function dodaj($Array)
?>
[PHP] pobierz, plaintext 

Jak na pierwszą klasę to jest dobrze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Tak jak zaleca kolega Jim - poczytaj o klasach abstrakcyjnych i interfejsach.

Powodzenia