Frage zum Factory Design Pattern

axelf · 01.06.2011, 12:38

Hi,

für verschiedene Produkte gibt es bei mir je eine Klasse für ein Produkt.
In der Factory hole ich mir je nach Produkttyp die entsprechende Klasse

PHP-Code:

  $navi = ProductFactory::get('navi') // Navigationsgeräte

oder

$handy = ProductFactory::get('handy') // Handys

Auf diesen Seiten weiß ich auch, welchen Typ ich benötige.

Auf der Suchergebnisseite sollen aber Produkte beider Typen angezeigt werden und ich weiß ja vorher nicht ob es ein Handy oder Navi ist.
Es sollen als kleine Beschreibung dann jeweils spezifische Daten zu jedem Produkttyp ausgegeben werden. Wie könnte ich hier am besten vorgehen?

01.06.2011, 12:45

Wenn du vorher prüfst welche Geräte alles auf der Suchergebnisse Seite angezeigt werden sollen -> Danach die entsprechenden Klassen nachladen!?

Ansonsten bist du dir sicher das du das korrekte Design Konzept rausgesucht hast?

axelf · 01.06.2011, 12:57

Also z.B. über eine DB-Abfrage aus den Suchergebnissen den Typ ermitteln und dann jeweils die Klasse aufrufen?

Was wäre denn ein bessere Design Konzept? Ich habe noch nicht so viel mit Design Pattern gemacht, daher bin ich mir nicht so sicher. Aber dachte das es für diesen Anwendungsfall ok ist.

Chriz · 01.06.2011, 13:07

Hallo,

was bekommst du denn an Daten für das Suchergebnis? Pro Treffer musst du doch irgendwie feststellen, welchen Typ der Treffer hat.

PHP-Code:

  <?php
$select = "
SELECT *, 'Handy' AS deviceType FROM handys WHERE description LIKE '%$suche%'
UNION
SELECT *, 'Navi' AS deviceType FROM navis WHERE title LIKE '%$suche%'";
// ..
$products = array();
while ($row = mysql_fetch_assoc($result)) {
  $deviceType = $row["deviceType"];
  $product = ProductFactory::get($deviceType);
  $product->fromArray($row);
  $products[] = $product;
}
?>

Wenn du z.B. eine Handy- und eine Navi-Klasse hast, könnten beide von einer gemeinsamen Product-Klasse erben, die die Factory zur Verfügung stellt.

PHP-Code:

  <?php
abstract class Product {
  public static function factory($deviceType) {
    $class = __CLASS__ . "_" . $deviceType;
    if (class_exists($class)) {
      return new $class();
    }
    throw new InvalidArgumentException("invalid device type");
  }
  public function fromArray(array $data) {
    foreach ($data as $key => $value) {
      if (property_exists($this, $key)) {
        $this->{$key} = $value;
      }
    }
  }
}
class Product_Handy extends Product {
  public $description;
}
class Product_Navi extends Product {
  public $title;
}
?>

Der Einfachheit halber ohne Namensmapping (SQL -> PHP-Property) und ohne eigene Setter/Getter.

Wäre zumindest mein Ansatz.

dr.e. · 01.06.2011, 18:25

Zitat:

Wie könnte ich hier am besten vorgehen?

Warum unterscheiden?

nikosch · 01.06.2011, 18:42

Ich benutze dafür ein Verfahren dieser Art:

PHP-Code:

  $oProductFactory = new ProductFactory;

$oProductFactory->registerFactory (new ProductFactory_Navi);
$oProductFactory->registerFactory (new ProductFactory_Handy);

$aProducts = $oProductFactory->create ($oDatabase , $whereClause , $orWhatever);



class ProductFactory
  {
  ...
  registerFactory (ConcreteProductFactory $oProductFactory)
    {
    $keys = $oProductFactory->canHandle ();
    foreach ((array) $keys as $key)
      {
      $this->aSubFactories[$key] = $oProductFactory;
      }
    }
  }

key repräsentiert dann bspw. ein Feld aus der Datenbank-Result-Menge, über das eine spezifische Fabrik angefordert werden kann.

axelf · 01.06.2011, 20:51

Zitat:

Zitat von dr.e.

Warum unterscheiden?

Öhm, warum sollte ich nicht unterscheiden? Ich möchte ja von den beiden Klassen spezielle Daten zurückgeliefert bekommen.

axelf · 01.06.2011, 21:03

Hi Chriz,

Zitat:

Zitat von Chriz

Hallo,

was bekommst du denn an Daten für das Suchergebnis? Pro Treffer musst du doch irgendwie feststellen, welchen Typ der Treffer hat.

Die Daten zum Suchergebnis bekomme ich über eine Abfrage. Hatte es nicht so gut gelöst wie in deinem Beispiel, aber danke für den hilfreichen Beitrag. Das hat mir sehr weitergeholfen. Die abstrakte Klasse Produkt hatte ich auch angelegt und Handy bzw. Navi werden dann abgeleitet.

Ich hatte noch eine Factory-Klasse angelegt, aber finde deine Möglichkeit auch gut. Nochmals viele Dank. Wieder was dazugelernt

Chriz · 01.06.2011, 21:16

Wobei das mit dem UNION so nicht ganz klappt, die Spaltentypen dürfen sich nicht unterscheiden und auch die Anzahl muss dieselbe sein.

axelf · 01.06.2011, 21:27

Ich habe mir eine Tabelle gemacht, wo so allgemeine Daten drinstehen und dann speziell eine Tabelle für die einzelnen Produkte (Fremdschlüssel). In der allgemeinen Tabelle gibt es die Spalte typ, und diese wird ausgelesen

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Eine Frage zum guten Code-Layout	SilentSight	JavaScript, Ajax und mehr	3	14.07.2010 09:18
[Erledigt] Design von Klassen für Datenbankabfragen	Sirke	Software-Design	10	17.06.2010 12:40
[Erledigt] Frage zu JOIN Befehl und dem Auslesen desselben	apo	PHP Tipps 2010	11	13.06.2010 19:32
Datenbank Design (Idee / Frage)	suche007	Datenbanken	1	30.07.2009 13:38
Design Frage - Buchungskalender	-Dirk-	PHP-Fortgeschrittene	4	28.03.2009 23:20
URL Frage	Eclipse16V	PHP Tipps 2009	8	23.02.2009 17:02
Hallo eine Frage in sachen Forensysteme	spikytux	HTML, Usability und Barrierefreiheit	11	03.01.2009 15:06
Dringende Frage	odde	PHP Tipps 2008	5	28.10.2008 19:03
Frage zu LDAP Frage (Active Directory)	FRAD	PHP Tipps 2008	0	01.10.2008 16:11
Design und Code Trennen	TeazY	PHP Tipps 2008	29	21.05.2008 12:08
Frage zu meinem DB Design	Schandi	Datenbanken	7	25.07.2007 11:26
Mit Sessionen das Design wählen lassen.		PHP Tipps 2004-2	8	28.12.2004 08:30
[Erledigt] Frage zur Funkrionen?		PHP Tipps 2004-2	10	01.12.2004 09:42
[Erledigt] Frage!		PHP Tipps 2004	4	27.07.2004 11:25

01.06.2011, 12:38
axelf Benutzer Registriert seit: 29.05.2011 Beiträge: 58 PHP-Kenntnisse: Anfänger	Frage zum Factory Design Pattern Hi, für verschiedene Produkte gibt es bei mir je eine Klasse für ein Produkt. In der Factory hole ich mir je nach Produkttyp die entsprechende Klasse PHP-Code: `$navi = ProductFactory::get('navi') // Navigationsgeräte oder $handy = ProductFactory::get('handy') // Handys` Auf diesen Seiten weiß ich auch, welchen Typ ich benötige. Auf der Suchergebnisseite sollen aber Produkte beider Typen angezeigt werden und ich weiß ja vorher nicht ob es ein Handy oder Navi ist. Es sollen als kleine Beschreibung dann jeweils spezifische Daten zu jedem Produkttyp ausgegeben werden. Wie könnte ich hier am besten vorgehen?


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

01.06.2011, 12:45
ChrisA Gast Beiträge: n/a	Wenn du vorher prüfst welche Geräte alles auf der Suchergebnisse Seite angezeigt werden sollen -> Danach die entsprechenden Klassen nachladen!? Ansonsten bist du dir sicher das du das korrekte Design Konzept rausgesucht hast?

01.06.2011, 12:57
axelf Benutzer Registriert seit: 29.05.2011 Beiträge: 58 PHP-Kenntnisse: Anfänger	Also z.B. über eine DB-Abfrage aus den Suchergebnissen den Typ ermitteln und dann jeweils die Klasse aufrufen? Was wäre denn ein bessere Design Konzept? Ich habe noch nicht so viel mit Design Pattern gemacht, daher bin ich mir nicht so sicher. Aber dachte das es für diesen Anwendungsfall ok ist.

01.06.2011, 13:07
Chriz Moderator Registriert seit: 11.05.2008 Beiträge: 6.269	Hallo, was bekommst du denn an Daten für das Suchergebnis? Pro Treffer musst du doch irgendwie feststellen, welchen Typ der Treffer hat. PHP-Code: `<?php $select = " SELECT , 'Handy' AS deviceType FROM handys WHERE description LIKE '%$suche%' UNION SELECT , 'Navi' AS deviceType FROM navis WHERE title LIKE '%$suche%'"; // .. $products = array(); while ($row = mysql_fetch_assoc($result)) { $deviceType = $row["deviceType"]; $product = ProductFactory::get($deviceType); $product->fromArray($row); $products[] = $product; } ?>` Wenn du z.B. eine Handy- und eine Navi-Klasse hast, könnten beide von einer gemeinsamen Product-Klasse erben, die die Factory zur Verfügung stellt. PHP-Code: <?php abstract class Product { public static function factory($deviceType) { $class = __CLASS__ . "_" . $deviceType; if (class_exists($class)) { return new $class(); } throw new InvalidArgumentException("invalid device type"); } public function fromArray(array $data) { foreach ($data as $key => $value) { if (property_exists($this, $key)) { $this->{$key} = $value; } } } } class Product_Handy extends Product { public $description; } class Product_Navi extends Product { public $title; } ?> Der Einfachheit halber ohne Namensmapping (SQL -> PHP-Property) und ohne eigene Setter/Getter. Wäre zumindest mein Ansatz. __________________ "Nuschel ich?" - "Was?"

01.06.2011, 18:42
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.994 PHP-Kenntnisse: Fortgeschritten	Ich benutze dafür ein Verfahren dieser Art: PHP-Code: $oProductFactory = new ProductFactory; $oProductFactory->registerFactory (new ProductFactory_Navi); $oProductFactory->registerFactory (new ProductFactory_Handy); $aProducts = $oProductFactory->create ($oDatabase , $whereClause , $orWhatever); class ProductFactory { ... registerFactory (ConcreteProductFactory $oProductFactory) { $keys = $oProductFactory->canHandle (); foreach ((array) $keys as $key) { $this->aSubFactories[$key] = $oProductFactory; } } } key repräsentiert dann bspw. ein Feld aus der Datenbank-Result-Menge, über das eine spezifische Fabrik angefordert werden kann. __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

01.06.2011, 21:16
Chriz Moderator Registriert seit: 11.05.2008 Beiträge: 6.269	Wobei das mit dem UNION so nicht ganz klappt, die Spaltentypen dürfen sich nicht unterscheiden und auch die Anzahl muss dieselbe sein. __________________ "Nuschel ich?" - "Was?"

01.06.2011, 21:27
axelf Benutzer Registriert seit: 29.05.2011 Beiträge: 58 PHP-Kenntnisse: Anfänger	Ich habe mir eine Tabelle gemacht, wo so allgemeine Daten drinstehen und dann speziell eine Tabelle für die einzelnen Produkte (Fremdschlüssel). In der allgemeinen Tabelle gibt es die Spalte typ, und diese wird ausgelesen