Wzorzec fabryki Opcje

[Orzeszekk]

Pytanie do mądrzejszych ode mnie forumowiczów.

Mam klase np. abstractMessage która jest szkieletem zapisujacym message do bazy danych.

następne z klasy abstractMessage tworze klasy pochodne np SMSMessage, EmailMessage.
Rodzaj wiadomosci przechowuje w polu $type obiektu abstractMessage, dzieki czemu wiadomo czy to SMSMessage, i EmailMessage.

Potrzebuje wzorca fabryki (metody wytwórczej?), tylko poprawnego. Teraz bym to zrobił tak:

[PHP] pobierz, plaintext 
class abstractMessage{
  public static function load($databaseRow)
  {
      [....]
      switch ($databaseRow->type)
      {
         case TYPE_SMS_MESSAGE: return new SmsMessage($databaseRow); break;
         case TYPE_EMAIL_MESSAGE: return new EmailMessage($databaseRow); break;
      }
  }
}
[PHP] pobierz, plaintext 

jednak takie rozwiązanie jest podobno do kitu ponieważ w przypadku jeżeli powstaną nowe typy dziedziczące po abstractMessage to będe musiał zmodyfikować ten switch tworzący obiekty, a w ten sposob zlamie zasade otwarte-zamknięte - zamiast rozszerzać klasę dodajac metody modyfikuję istniejący kod w celu rozszerzenia funkcjonalności.

No a jeżeli nie utworze odpowiedniego typu (SMSMessage, EmailMessage) to nie będę mógł użyć ich specyficznych metod.

Więc jak to poprawnie rozwiązać? Chcę rozwiązania na maksa poprawnego z dobrymi praktykami programistycznymi, bo już dostałem niedawno durszlakiem pełnym kodu spagetti mocno w głowę i chcę tego uniknąć (IMG:style_emoticons/default/smile.gif)

[Sephirus]

Nie do końca Cię rozumiem ale mogę podpowiedzieć:

Pamiętaj że zapis:

[PHP] pobierz, plaintext 
new SmsMessage($databaseRow); 
[PHP] pobierz, plaintext 

Jest równoważny z:

[PHP] pobierz, plaintext 
$className = 'SmsMessage';
 
new $className($databaseRow); 
[PHP] pobierz, plaintext 

(IMG:style_emoticons/default/wink.gif)

I cytat z jakiegoś kodu dotyczącego fabryki:

[PHP] pobierz, plaintext 
    <?php
    public function make( $className )
    {
        return new $className();
    }
    ?>
[PHP] pobierz, plaintext 

(IMG:style_emoticons/default/tongue.gif)

Ten post edytował Sephirus 16.11.2011, 14:22:51