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

[matiit]

Od początku - nazewnictwo jak dla mnie jest trochę "nie tego".
creatPlanet - można już było dać PlanetCreator albo coś innego, bo teraz to wygląda trochę jak createPlanet (a czasownikowa nazwa dla klasy - średnio).

Dalej funcPlanet (może BasePlanet)?

Poza tym w Twoim przypadku wystarczyłoby wszystko wrzucić do klasy Planet.

[q3trm]

Nie myślałem nad nazewnictwem podczas pisania klas, głównie skupiłem się na koncepcji stworzenia skryptu umożliwiającego łatwe rozwijanie w przyszłości.

Rozumiem, że aktualna struktura = "przerost formy nad treścią". Co w przypadku indywidualnego rozszerzania funkcjonalności planet. Nie lepiej by było je mieć w osobnych klasach, tak jak to jest teraz?. Wydaje mi się to bardziej czytelne. Cały czas w OOP mówi się o zasadzie "pojedynczej odpowiedzialności", czyli klasa pobiera parametry, ale nie ingeruje w nie, tak rozumiem zasadę pojedynczej odpowiedzialności. Mylę się?.