[PHP]OOP'owe początki Opcje

[Fanatyko]

Witam, zaczynam zabawę z OOP'em przy wykorzystaniu wzorca MVC. Czy kod poniżej jest poprawny, czy też nie ? Co zmienić, co robić, czego nie robić (ogólnie tak jakos (IMG:style_emoticons/default/winksmiley.jpg) )
Z czasem będę dodawał nowe pytania, bo zapewne na jednym się nie skończy

[PHP] pobierz, plaintext 
<?php
 
$Controler = new GameController;
 
 
class GameController
{
    public function __construct()
    {
      if(isset($_GET['action']))
      {
        $strAction = $_GET['action'];
 
      }
      else
      {
        $strAction = "statystyki";
 
      }
      $objAction = new $strAction();           
      $objAction->DoAction();  
 
 
 
    }
 
}
 
class ShowCharacter 
{
    public function DoAction()
    {
      $objModel = new CharacterModel();
      $arrCharacter = $objModel->GetCharacter();
 
      foreach ($arrCharacter as $arrStatisctics)
      {
        echo "Sila: ".$arrStatisctics['strenght']." Zrecznosc: ".$arrStatisctics['agility']."";
      }
 
      return null;
 
    }
 
}
 
class CharacterModel
{
    public function GetCharacter()
    {
       return array(array( 'strenght' => 15, 'agility' => 20)) ;    
    }
 
 
}
 
 
 
?>
[PHP] pobierz, plaintext 

[Pawel_W]

po 1:

[PHP] pobierz, plaintext 
      if(isset($_GET['action']))
      {
        $strAction = $_GET['action'];
 
      }
      else
      {
        $strAction = "statystyki";
 
      }
[PHP] pobierz, plaintext 

żadne takie ify z $_GET, to powinno być przekazane w konstruktorzem czyli np.

[PHP] pobierz, plaintext 
$Controler = new GameController($_GET['action']);;
[PHP] pobierz, plaintext 

(IMG:style_emoticons/default/smile.gif)

2.

[PHP] pobierz, plaintext 
      $objAction = new $strAction();           
      $objAction->DoAction();  
[PHP] pobierz, plaintext 

to się aż prosi o zastosowanie wzorca projektowego "factory" (IMG:style_emoticons/default/smile.gif)

po 3: DZIEDZICZENIE! - naprawdę warto byłoby z niego tu skorzystać (IMG:style_emoticons/default/winksmiley.jpg)