[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

[smajster]

Witam ponownie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Postarałem się wprowadzić w życie zmiany jakie mi zaproponowaliście i wyszło mi takie cuś:

[PHP] pobierz, plaintext 
<?php
////////////////////////////////
 ///===== KLASA DATA
 //
 //
 
abstract class data 
{
	protected $data;
	protected $tabela;
 
	abstract function dodaj($Array);
	abstract function popraw($Array);	 
	abstract function usun($id);
   
   protected function pobierz_dane($id)
   {
	   $pyt = "SELECT * FROM ".$tabela." WHERE id = ".$id;
	   $rezult = mysql_query($pyt);
	   if(mysql_num_rows($rezult)>0)
	   $data = mysql_fetch_array($rezult);
	   else echo"nie ma takiego id";
   } 
	public function podaj($id,$kolumna)
	{
		$this->pobierz_dane($id);
		if(isset($this->data[$kolumna]))
		return $this->data[$kolumna];
		else
		echo"nie ma takiej kolumny!";
	}
	protected function id()
	{
		$pyt="SELECT MAX(id) as ilgrup FROM " .$tabela;
		$rezult = mysql_query($pyt);
		if($rezult != NULL)
		{
		  $rekord = mysql_fetch_array($rezult);
		  var $id = $rekord['ilgrup']+1;
		  return $id;
		  }
		  else
		  return 1;
	}
}
 
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
////////////////////////////////
 ///===== KLASA GRUPA
 //
 //
class grupa extends data
{
	protected $tabela = "_dbProducts._dbtGroup";
	public function dodaj($Array)
	{
		if($Array["kod"] =="")
		{
			$pyt = "INSERT INTO " .$tabela."(id, kod, nazwa, iloscpodg)																		   VALUES(".$this->id().", " .$this->kod_grupy().", " .$Array["nazwa"].", "0");";
		}
		else
		{
			$pyt = "INSERT INTO " .$tabela."(id, kod, nazwa, iloscpodg)																		   VALUES(".$this->id().", " .$Array["kod"].", " .$Array["nazwa"].", "0");";
		}
	   $rezult = mysql_query($pyt);
	}
	private function kod_grupy()
	{
		$pyt="SELECT MAX(id) as ilgrup FROM " .$tabela;
		$rezult = mysql_query($pyt);
		if($rezult != NULL)
		{
		  $rekord = mysql_fetch_array($rezult);
		  var $kod = $rekord['ilgrup']+1;
		  return $kod;
		  }
		  else
		  return 1;
	}
	public function popraw($Array)
	{
		$pyt = "UPDATE " .$tabela." SET nazwa = '" .$Array["nazwa"] ."'																	 WHERE grupa = " .$Array["id"];			 
		 $rezult = mysql_query($pyt);
		 if($rezult != NULL)
		 return 1;
		 else return 0;
	}
	public function usun($id)
	{
		// POBRANIE Z BAZY NAJWIĘKSZEGO ID
		$maxGrp=($this->id - 1);
		// POBRANIE GRUPY O NAJWIĘKSZYM ID
		$pyt  = "SELECT * FROM " .$tabela." WHERE id = " .$maxGrp;
	   $rezult = mysql_query($pyt);
	   $ostatni = mysql_fetch_array($rezult);
	   // ZAMIANA GRUPY O NAJWIĘKSZYM ID NA ID KTÓRE MA BYĆ USUNIĘTE
	   $pyt = "UPDATE " .$tabela." SET grupa = " .$ostatni["grupa"] .", nazwa = '" .$ostatni["nazwa"] ."',				  iloscpgp = " .$ostatni["iloscpgp"] ." WHERE id = " .$id;
	   $rezult = mysql_query($pyt);
	   // USUWANIE OSTATNIEJ (O NAJWIĘKSZYM ID) GRUPY
		$pyt = "DELETE FROM " .$tabela." WHERE id = " .$id;
		$rezult = mysql_query($pyt);
		if(!(mysql_num_rows($rezult)>0))
		echo"nie ma takiego id";
	} 
}
?>
[PHP] pobierz, plaintext 

Klasa grupa jest mniej wiecej analogiczna do klasy produkt tylko prostrza i krotsza dlatego ją zamieściłem, obie dziedziczą od klasy data. Jak myślicie tak to powinno wyglądać? Obie te klasy mają znajdować się niby w klasie modelu. Są to moje pierwsze skończone klasy także proszę o wyrozumiałość (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Jak zauważacie jakieś błędy bardziej merytoryczne niż składniowe to skrobnijcie, bo chciałbym się jak najszybciej nauczyć poprawnie pisać w OOP.

Pozdrawiam