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)

[mortus]

[PHP] pobierz, plaintext 
class abstractMessage {
 /* ... */
 public static function load($databaseRow) {
  $messageType = $databaseRow->type;
  $className = ucfirst(strtolower(substr($messageType, 6, strpos('_', $messageType, 6)-1))) . 'Message';
  return new $className($databaseRow);
}
[PHP] pobierz, plaintext 

Oczywiście to wtedy, jeśli $databaseRow->type przechowuje nazwy stałych, to jest TYPE_SMS_MESSAGE, czy TYPE_EMAIL_MESSAGE.
Pisane z palca, ale na oko widać, że metodę można "udoskonalić".

Wypada również sprawdzić, czy klasa o nazwie zawartej w zmiennej $className istnieje i jest załadowana, a dopiero później spróbować utworzyć obiekt tej klasy a potem go zwrócić. W przypadku niepowodzenia można rzucić wyjątek.

Ten post edytował mortus 16.11.2011, 15:11:48