[PHP] Czy dobrze zastosowałem wzorzec Metoda Fabrykująca? Opcje

[Bodzimier]

Witajcie,

Postanowiłem, że już pora, aby poznać wzorce projektowe. Poprosiłbym o sprawdzenie czy poprawnie zastosowałem wzorzec Metoda Fabrykująca. Jeżeli potrzeba dodatkowych informacji - pytajcie, bo na chwile obecną nic nie przychodzi mi do głowy (IMG:style_emoticons/default/smile.gif)

Interfejsy:

[PHP] pobierz, plaintext 
//FactoryAbstract.php
abstract class FactoryAbstract
{
     /**
     * @param $name
     *
     * @return object
     * @throws Exception
     */
    public function createProduct( $name )
    {
        if ( file_exists( $this->getProductPath() . $name . '.php' ) ) {
            include_once($this->getProductPath() . $name . ".php");
        } else {
            throw new Exception("Plik '$name.php' nie istnieje.");
        }
 
        if ( class_exists( $name ) ) {
            return new $name();
        } else {
            throw new Exception("Klasa '$name' nie istnieje.");
        }
    }
 
    /**
     * @return string
     */
    protected abstract function getProductPath();
}
 
//ProductAbstract.php
abstract class ProductAbstract
{
    protected $name;
    protected $price;
 
     /**
     * @return string
     */
    public abstract function getName();
 
    /**
     * @return string
     */
    public abstract function getPrice();
}
[PHP] pobierz, plaintext 

Fabryki:

[PHP] pobierz, plaintext 
//FoodFactory.php
include_once(ROOT . "FactoryAbstract.php");
 
class FoodFactory extends FactoryAbstract
{
    /**
     * @return string
     */
    protected function getProductPath()
    {
        return ROOT . 'food/';
    }
} 
 
//GlassFactory.php
include_once(ROOT . "FactoryAbstract.php");
 
class GlassFactory extends FactoryAbstract
{
    /**
     * @return string
     */
    protected function getProductPath()
    {
        return ROOT . 'glass/';
    }
}
[PHP] pobierz, plaintext 

Produkty:

[PHP] pobierz, plaintext 
//glass/Kieliszek.php
include_once(ROOT . "ProductAbstract.php");
 
class Kieliszek extends ProductAbstract
{
    public function __construct()
    {
        $this->name  = "Kieliszek gran party";
        $this->price = 9.91;
    }
 
    public function getName()
    {
        return $this->name;
    }
 
    public function getPrice()
    {
        return $this->price;
    }
}
 
//food/Pierogi.php
include_once(ROOT . "ProductAbstract.php");
 
class Pierogi extends ProductAbstract
{
    public function __construct()
    {
        $this->name  = "Pierogi ruskie delux, 10 sztuk.";
        $this->price = 12.99;
    }
 
    public function getName()
    {
        return $this->name;
    }
 
    public function getPrice()
    {
        return $this->price;
    }
} 
[PHP] pobierz, plaintext 

Kod wysyłający żądanie:

[PHP] pobierz, plaintext 
ini_set( 'display_errors', '1' );
ERROR_REPORTING( E_ALL );
 
define('ROOT', __DIR__ . '/');
include_once(ROOT . "GlassFactory.php");
include_once(ROOT . "FoodFactory.php");
 
class Client
{
    private $glassFactory;
    private $foodFactory;
    private $products;
 
    public function __construct()
    {
        //stawiamy fabryki - ich "produkty" posiadają wspólny interfejs (getName, getPrice)
        $this->glassFactory = new GlassFactory();
        $this->foodFactory  = new FoodFactory();
 
        try {
            //tu dodać nowy produkt
            $this->products = array(
                $this->glassFactory->createProduct( 'Kieliszek' ),
                $this->foodFactory->createProduct( 'Pierogi' )
            );
 
        } catch ( ErrorException $e ) {
            die($e->getMessage());
        }
    }
 
    public function displayProducts()
    {
        foreach ( $this->products as $product ) {
            printf( "Nazwa: %s<br/>Cena: %.2f PLN<br/><br/>", $product->getName(), $product->getPrice() );
        }
    }
}
 
$client = new Client();
$client->displayProducts();
[PHP] pobierz, plaintext