Decorator-Pattern

grizu · 09.08.2009, 11:54

Hallo!

Ich versuche mich gerade am Decorator-Pattern, hab mir als kleine Aufgabe gestellt, das Java-IO-System etwas nachzubasteln, natürlich minimalst.

Ich hab jetzt bisher drei Klassen: StringReader, FileReader und XMLReader. XMLReader ist der eigentliche Decorator.

Ich wollte eigentlich nur fragen, ob ihr da etwas drüberschauen könntet? Was könnte man besser machen, und wie könnte man das ganze noch erweitern? Es wäre ja echt cool, wenn da ein kleines verwendbares package draus wird.
Ich find das Pattern grad irrsinnig geil

Ich poste mal meine Klassen.

IReader:

PHP-Code:

  interface IReader
{
    public function readLine ();
    public function readChar ();
    public function rewind ();
    public function read ();
}

Reader:

PHP-Code:

  abstract class Reader implements IReader
{
    
    protected $_lineCounter = 0;
    protected $_charCounter = 0;
    
    
    public function read ()
    {
    }
    
    public function readLine ()
    {
    }
    
    public function readChar ()
    {
    }
    
    public function rewind ()
    {
        $this->_lineCounter = 0;
        $this->_charCounter = 0;
    }
}

StringReader:

PHP-Code:

  class StringReader extends Reader
{
    
    private $_string;

    
    public function __construct ($string)
    {
        $this->_string = $string;
    }

    public function read ()
    {
        return $this->_string;
    }
    
    public function readLine ()
    {
        $lines = split ("\n", $this->_string);
        $line = $lines[$this->_lineCounter];
        
        $this->_lineCounter ++;
        
        return $line;
    }
    
    public function readChar ()
    {
        $char = substr($this->_string, $this->_charCounter, 1);
        
        $this->_charCounter ++;
        
        return $char;
    }
}

FileReader:

PHP-Code:

  class FileReader extends Reader
{
    
    private $_file;
    private $_content;

    
    public function __construct ($file)
    {
        $this->_file = $file;
    }

    public function read ()
    {
        return ($this->_content = file_get_contents ($this->_file));
    }
    
    public function readLine ()
    {
        if (is_null($this->_content))
            $this->read ();
        
        $lines = split ("\n", $this->_content);
        $line = $lines[$this->_lineCounter];
        
        $this->_lineCounter ++;
        
        return $line;
    }
    
    public function readChar ()
    {
        if (is_null($this->_content))
            $this->read ();
            
        $char = substr($this->_content, $this->_charCounter, 1);
        
        $this->_charCounter ++;
        
        return $char;
    }
}

XMLReader:

PHP-Code:

  class XMLReader extends Reader
{
    
    private $_reader;

    
    public function __construct (Reader $reader)
    {
        $this->_reader = $reader;
    }

    public function read ()
    {
        return $this->_reader->read();
    }
    
    public function readLine ()
    {
        return $this->_reader->readLine();
    }
    
    public function readChar ()
    {
        return $this->_reader->readChar();
    }
    
    public function rewind ()
    {
        $this->_reader->rewind ();
    }
    
    public function readXML ()
    {
        return new SimpleXMLElement($this->_reader->read());
    }
}

Also ich glaub, das Prinzip hab ich gecheckt. Die XML-Klasse gehört natürlich noch ausgebaut. Ist das Interface und die abstrakte Klasse sinnvoll definiert? Welche Klassen könnte man noch einbauen (ArrayReader?) ?

Danke!

lg
grizu

nikic · 09.08.2009, 14:28

Hm, ich hab nur mal ne Frage zur Funktionsweise (nutze Java nicht):

Wenn ich jetzt einen Text habe:
"Hallo Welt
Moin moin
Ende"
Und ich sage jetzt:

PHP-Code:

  $string = new StringRead("Hallo Welt
Moin moin
Ende");
$string->ReadLine();
$string->ReadChar();

Was ist die gewünschte Ausgabe?
So wie sie jetzt ist, also:
"Hallo WeltH"
oder soll sie eher
"Hallo Welt
"
oder gar
"Hallo WeltM"
sein?

grizu · 09.08.2009, 20:23

Da hast natürlich recht, da stimmt was noch nicht ganz

Sollte natürlich "Hallo Welt M" rauskommen.
Exceptions wären auch noch nett, der XMLReader kann auch noch nix, zu tun ist noch genug...
Die Klassen können ja generell nicht viel, mir gings einfach um das Pattern.

yab · 09.08.2009, 21:36

Mit dem Decorator-Pattern ist es möglich, einem Objekt zur Laufzeit dynamisch Funktionalität hinzuzufügen und es setzt auch nicht auf Vererbung. Also bist du mit deiner Implementierung schonmal am Thema vorbei

Unabhängig vom Pattern ist mir an deinem Beispiel aufgefallen, dass du das Interface in deiner abstrakten Klasse bereits mittels leerer Methoden vollständig implementierst. Die Vorzüge, dass der PHP-Parser bei vergessenen Interface-Methoden in den abgeleiteten Klassen "meckert" ist somit auch dahin.

grizu · 10.08.2009, 13:11

Hm...

Am Pattern vorbei? Aber der XMLReader macht ja genau das: er erweitert den Stringreader btw. Filereader um die Funktionalität, aus einem String ein XML-Objekt zurückzugeben. Die Vererbung von Reader soll ja nur sicherstellen, dass jeder Reader, egal ob XML, String, File, Array, etc. die Methoden besitzt. Wärs sinnvoll, für den Decorator eine eigene Superklasse bzw. Interface zu machen?

Das mit dem Interface und der abstrakten Klasse, stimmt. Also einfach nur das Interface bzw. eine rein abstrakte Klasse verwenden?

grizu · 11.08.2009, 10:28

Jetzt hab ich noch eine konkrete Frage zum Thema Interfaces und abstrakte Klassen, passt auch zu dem Beispiel.

Warum kann ich in einer abstrakten Klasse, die ein Interface implementiert, die Methoden aus dem Interface nicht als abstrakt deklarieren? Das würde doch Sinn machen: Ich hab ein Interface, das die Schnittstelle vorgibt. In meiner abstrakten Klasse, die eine Superklasse für mehrere konkrete Klassen sein soll, möchte ich zwar schon einige Implementierungen vornehmen (Konstruktor, einige Variablen), aber die Interface-Methoden trotzdem abstrakt lassen.

-> Fatal Error: Can't inherit abstract function ...

Warum geht das nicht?

rudygotya · 11.08.2009, 11:26

Einfaches Beispiel:
Du hast 3 verschiedene Benutzertypen.
Angenommen, du bist dir nicht ganz sicher, ob du einer Klasse user ein Interface oder einige Methoden mitgeben möchtest.

Falls du für bereits schon gelöste gemeinsame Probleme gemeinsam verwendete Methoden entwickelt hast, die du für die 3 Klassen einsetzen möchtest, dann gibst du diese in der abstrakten Klasse mit und vererbst von dieser.
Habe ich dann noch ähnliche - aber dennoch unterschiedliche - Schnittstellen, gebe ich leere abstracte Methoden mit. Da diese sich wiederum in den Kindklassen unterscheiden werden.

Im Endeffekt wird damit deine abstrakte Klasse zum erweiterten Interface. Daher kannst du in einem "erweiterten Interface" kein Interface implementieren.

grizu · 11.08.2009, 12:13

Hi, danke für deine Antwort.

Zitat:

Im Endeffekt wird damit deine abstrakte Klasse zum erweiterten Interface. Daher kannst du in einem "erweiterten Interface" kein Interface implementieren.

Also erweitertes Interface einfach in dem Sinn, dass bestimmte Methoden bereits implementiert werden können? Wie soll ich das sagen - ein Mittelding zwischen Interface und Klasse?

- Aber es ergibt sich irgendwie schon aus dem Namen: Ich kann keine Interface-Methode abstrakt implementieren - abstrakt implementieren geht halt einfach nicht.
Ich wollte einfach erreichen, dass meine Superklasse bereits sich selbst und das Interface vorgibt. Aber wenn ich will, dass meine erbenden Klassen das Interface implementieren, muss ich das in den Kindklassen selbst machen.

Ich hoffe, das hab ich jetzt richtig verstanden

rudygotya · 11.08.2009, 12:32

Es bleibt ja eine Klasse. Der wichtigste Grundgedanke dahinter ist der, dass du von einer abstrakten Klasse kein Objekt erzeugen kannst.
Um auf das vorherige Beispiel zurückzugreifen:
Sie enthält eben nur gemeinsame Methoden, würde aber zur Laufzeit kein vollwertiges User-Objekt darstellen.

Ich würde entweder ein interface schreiben ODER eine abstrakte Klasse definieren, aber nicht beides mischen. Weiß auch nicht, ob das geht, macht auch keinen Sinn.

Zitat:

Ich wollte einfach erreichen, dass meine Superklasse bereits sich selbst und das Interface vorgibt.

Genau das macht eine abstrakte Klasse mit abstrakten Methoden.

grizu · 11.08.2009, 13:24

Zitat:

Zitat von rudygotya

Ich würde entweder ein interface schreiben ODER eine abstrakte Klasse definieren, aber nicht beides mischen. Weiß auch nicht, ob das geht, macht auch keinen Sinn.

Stimmt ja, beides macht wirklich keinen Sinn. Das hab ich nämlich vorgehabt...
Und in einem anderen Fall, zb. implementiert meine User-Klasse das Interface "Serializable", wäre es ja kein Problem, diese Methode bereits in der abstrakten Klasse zu implementieren. Aber eben, eine abstrakte Klasse kann eine Interface-Methode nicht als abstrakt deklarieren und so quasi das Interface weitervererben, sie muss die Methode implementieren.

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

09.08.2009, 11:54
grizu Neuer Benutzer Registriert seit: 25.08.2008 Beiträge: 19 PHP-Kenntnisse: Anfänger	Decorator-Pattern Hallo! Ich versuche mich gerade am Decorator-Pattern, hab mir als kleine Aufgabe gestellt, das Java-IO-System etwas nachzubasteln, natürlich minimalst. Ich hab jetzt bisher drei Klassen: StringReader, FileReader und XMLReader. XMLReader ist der eigentliche Decorator. Ich wollte eigentlich nur fragen, ob ihr da etwas drüberschauen könntet? Was könnte man besser machen, und wie könnte man das ganze noch erweitern? Es wäre ja echt cool, wenn da ein kleines verwendbares package draus wird. Ich find das Pattern grad irrsinnig geil Ich poste mal meine Klassen. IReader: PHP-Code: `interface IReader { public function readLine (); public function readChar (); public function rewind (); public function read (); }` Reader: PHP-Code: `abstract class Reader implements IReader { protected $_lineCounter = 0; protected $_charCounter = 0; public function read () { } public function readLine () { } public function readChar () { } public function rewind () { $this->_lineCounter = 0; $this->_charCounter = 0; } }` StringReader: PHP-Code: class StringReader extends Reader { private $_string; public function __construct ($string) { $this->_string = $string; } public function read () { return $this->_string; } public function readLine () { $lines = split ("\n", $this->_string); $line = $lines[$this->_lineCounter]; $this->_lineCounter ++; return $line; } public function readChar () { $char = substr($this->_string, $this->_charCounter, 1); $this->_charCounter ++; return $char; } } FileReader: PHP-Code: class FileReader extends Reader { private $_file; private $_content; public function __construct ($file) { $this->_file = $file; } public function read () { return ($this->_content = file_get_contents ($this->_file)); } public function readLine () { if (is_null($this->_content)) $this->read (); $lines = split ("\n", $this->_content); $line = $lines[$this->_lineCounter]; $this->_lineCounter ++; return $line; } public function readChar () { if (is_null($this->_content)) $this->read (); $char = substr($this->_content, $this->_charCounter, 1); $this->_charCounter ++; return $char; } } XMLReader: PHP-Code: class XMLReader extends Reader { private $_reader; public function __construct (Reader $reader) { $this->_reader = $reader; } public function read () { return $this->_reader->read(); } public function readLine () { return $this->_reader->readLine(); } public function readChar () { return $this->_reader->readChar(); } public function rewind () { $this->_reader->rewind (); } public function readXML () { return new SimpleXMLElement($this->_reader->read()); } } Also ich glaub, das Prinzip hab ich gecheckt. Die XML-Klasse gehört natürlich noch ausgebaut. Ist das Interface und die abstrakte Klasse sinnvoll definiert? Welche Klassen könnte man noch einbauen (ArrayReader?) ? Danke! lg grizu


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

09.08.2009, 14:28
nikic Benutzer Registriert seit: 07.08.2009 Beiträge: 31 PHP-Kenntnisse: Fortgeschritten	Hm, ich hab nur mal ne Frage zur Funktionsweise (nutze Java nicht): Wenn ich jetzt einen Text habe: "Hallo Welt Moin moin Ende" Und ich sage jetzt: PHP-Code: `$string = new StringRead("Hallo Welt Moin moin Ende"); $string->ReadLine(); $string->ReadChar();` Was ist die gewünschte Ausgabe? So wie sie jetzt ist, also: "Hallo WeltH" oder soll sie eher "Hallo Welt " oder gar "Hallo WeltM" sein?

09.08.2009, 20:23
grizu Neuer Benutzer Registriert seit: 25.08.2008 Beiträge: 19 PHP-Kenntnisse: Anfänger	Da hast natürlich recht, da stimmt was noch nicht ganz Sollte natürlich "Hallo Welt M" rauskommen. Exceptions wären auch noch nett, der XMLReader kann auch noch nix, zu tun ist noch genug... Die Klassen können ja generell nicht viel, mir gings einfach um das Pattern.

09.08.2009, 21:36
yab Erfahrener Benutzer Registriert seit: 05.04.2009 Beiträge: 291 PHP-Kenntnisse: Anfänger	Mit dem Decorator-Pattern ist es möglich, einem Objekt zur Laufzeit dynamisch Funktionalität hinzuzufügen und es setzt auch nicht auf Vererbung. Also bist du mit deiner Implementierung schonmal am Thema vorbei Unabhängig vom Pattern ist mir an deinem Beispiel aufgefallen, dass du das Interface in deiner abstrakten Klasse bereits mittels leerer Methoden vollständig implementierst. Die Vorzüge, dass der PHP-Parser bei vergessenen Interface-Methoden in den abgeleiteten Klassen "meckert" ist somit auch dahin.

10.08.2009, 13:11
grizu Neuer Benutzer Registriert seit: 25.08.2008 Beiträge: 19 PHP-Kenntnisse: Anfänger	Hm... Am Pattern vorbei? Aber der XMLReader macht ja genau das: er erweitert den Stringreader btw. Filereader um die Funktionalität, aus einem String ein XML-Objekt zurückzugeben. Die Vererbung von Reader soll ja nur sicherstellen, dass jeder Reader, egal ob XML, String, File, Array, etc. die Methoden besitzt. Wärs sinnvoll, für den Decorator eine eigene Superklasse bzw. Interface zu machen? Das mit dem Interface und der abstrakten Klasse, stimmt. Also einfach nur das Interface bzw. eine rein abstrakte Klasse verwenden?

11.08.2009, 10:28
grizu Neuer Benutzer Registriert seit: 25.08.2008 Beiträge: 19 PHP-Kenntnisse: Anfänger	Jetzt hab ich noch eine konkrete Frage zum Thema Interfaces und abstrakte Klassen, passt auch zu dem Beispiel. Warum kann ich in einer abstrakten Klasse, die ein Interface implementiert, die Methoden aus dem Interface nicht als abstrakt deklarieren? Das würde doch Sinn machen: Ich hab ein Interface, das die Schnittstelle vorgibt. In meiner abstrakten Klasse, die eine Superklasse für mehrere konkrete Klassen sein soll, möchte ich zwar schon einige Implementierungen vornehmen (Konstruktor, einige Variablen), aber die Interface-Methoden trotzdem abstrakt lassen. -> Fatal Error: Can't inherit abstract function ... Warum geht das nicht?

11.08.2009, 11:26
rudygotya Erfahrener Benutzer Registriert seit: 28.05.2008 Beiträge: 2.094 PHP-Kenntnisse: Fortgeschritten	Einfaches Beispiel: Du hast 3 verschiedene Benutzertypen. Angenommen, du bist dir nicht ganz sicher, ob du einer Klasse user ein Interface oder einige Methoden mitgeben möchtest. Falls du für bereits schon gelöste gemeinsame Probleme gemeinsam verwendete Methoden entwickelt hast, die du für die 3 Klassen einsetzen möchtest, dann gibst du diese in der abstrakten Klasse mit und vererbst von dieser. Habe ich dann noch ähnliche - aber dennoch unterschiedliche - Schnittstellen, gebe ich leere abstracte Methoden mit. Da diese sich wiederum in den Kindklassen unterscheiden werden. Im Endeffekt wird damit deine abstrakte Klasse zum erweiterten Interface. Daher kannst du in einem "erweiterten Interface" kein Interface implementieren. __________________ ++++ Wieder einer ins Netz gegangen: Phishers Fritz zufrieden ++++ Blog