Ocena - Klasy tworzenia planety. Opcje

[q3trm]

Cześć.

Na początku zaznaczam, że to moje początki OOP, żeby nie było (IMG:style_emoticons/default/facepalmxd.gif) .

[PHP] pobierz, plaintext 
class creatPlanet // klasa konstrukcyjna
 {
  public $planet;
 
	 public function __construct(Planet $class)
	 {
	  return $this ->planet = $class;
	 }
 
 }
 
 
  abstract class FuncPlanet // funkcje planet
  {
	 public function __construct()
	 {
	   $this ->nameplanet = get_class($this);
	   $this ->increase = strlen($this ->nameplanet);
	 }
 
     public function getParamPlanet() //wyświetla wszystkie parametry ustawione przez użytkownika
     {
        foreach ($this ->parameter as $param => $p)
		{
		 echo '<br />'.$param. ' +' .$p;
		}
     } 
	 public function getIncrease() //szybkość rozwoju planety
	 {
	    return $this ->increase;
	 }
	 public function getNamePlanet() //nazwa planety użytkownika
	 {
	     return $this ->nameplanet;
	 }
   }
 
 
  abstract class Planet extends FuncPlanet  // ustawienie początkowych parametrów planety przez użytkownika
  {
   protected $increase;
   protected $nameplanet;
   protected $parameter = array();
 
 
     function __construct($economic = 0, $military = 0, $explorative = 0)
     {  
	   $this ->parameter = array (
	    'economic' => $economic, 
	    'military' => $military,
	    'explorative' => $explorative
	    );
	       parent::__construct();
     }
   }
 
 
  class Mercury extends Planet {}
  class Venus extends Planet {}
  class Earth extends Planet {}
  class Jupiter extends Planet {}
  class Saturn extends Planet {}
  class Uranium extends Planet {}
  class Neptune extends Planet {}
 
[PHP] pobierz, plaintext 

Ten post edytował q3trm 10.04.2013, 21:52:39

[Crozin]

1. Chodzi o to, że takie rzeczy powinny być właściowścią obiektu (każda z osobna), a nie umieszczone w tablicy. Dlaczego? Bo łatwiej wtedy przydzielić im indywidualne cechy, łatwiej udokumentować kod czy ogólnie utrzymywać go.
2. Rozumiem, że te planety nie ograniczają się do naszego Układu Słonecznego? W takim wypadku obiekt typu Planeta powinien mieć jakieś podstawowe właściwości opisujące samą planetę (np. nazwa czy wspomniana szybkość ogólnego rozwoju). Punkty przydzielone przez użytkownika mogą być albo bezpośrednimi właściwościami tego obiektu, mogą też być osobnym obiektem będącym właściwością obiektu Planeta. Które rozwiązanie jest lepsze? To już zależy, m.in. od tego czy obiekt Planeta i MetodykaRozwojuPlanety są rozbudowane czy nie. Jednak prawdopodobnie osobny obiekt, będzie lepszym rozwiązaniem.