[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 

[Crozin]

1) Twój kod nie ma wiele wspólnego z MVC - brak w nim w ogóle warstwy widoku.
2) Nie za bardzo wiem co Pawel_W miał na myśli przez dziedziczenie. Tutaj jego wykorzystanie byłoby błędem - przecież ShowCharacter::DoAction() jedynie wykorzystuje CharacterModel.
3) Twoja klasa GameController bardziej przypomina Dispatcher niż kontroler.

btw: return null; jest zbędne.