Dynamisch Methoden von Klassen erweitern

07/12/2016 22:17 Mr.Tr33#1
Hallo,

ich bin mir sicher das Thema wurde bestimmt schon mal irgendwo ausfühlich erklärt, jedoch finde ich hierzu nichts passendes oder ich habe es bis jetzt einfach noch nicht ganz verstanden.

Als Lernzweck habe ich mich an eine CMS rangesetzt die Modular (ob es das richtige Wort dafür ist weiß ich gerade jedoch nicht) funktionieren soll mit dem Ziel, dass ich theoretisch jede einzelne Stelle mit einem weiterem Modul beliebig erweitern kann.
Nun komme ich langsam an meine Grenzen. Ich weiß nicht ob es einfach nur eine Denkblockade ist oder ich noch nicht den richtigen Ansatz gefunden habe.

Hier ein kleines Beispiel, wie es am Ende funktionieren sollte:
Ich habe eine CMS mit einem Menü und Content, jedoch keine Benutzer etc., alles also ganz simpel.
Nun erstellt eine Person (Person A) eine Erweiterung für die CMS. Er möchte Benutzer, Benutzergruppen und Rechte einfügen. Natürlich auch, dass manche Menüpunkte nur dann sichtbar sind, wenn man die passenden Rechte dafür hat. Also erweitert er die Menu Klasse, indem er z.B. bei $menu->isVisible eine weitere Abfrage einfügt ob man die passenden Rechte hat.
Jetzt aber hat eine noch weitere Person (Person B) ebenfalls eine Erweiterung für die CMS geschrieben. Er möchte eine Sortierung für das Menü einbauen. Also erweitert er die Menu Klasse, indem er vor die $menu->display Methode eine Sortiermethode einbaut.

Nun komme ich jedoch zur Denkblockade.
Theoretisch müsste Person A die Klasse Menu extenden. Person B müsste praktisch die Klasse von Person B extenden. Jedoch weiß Person B gar nicht das es eine andere Erweiterung gibt.
Und soweit ich weiß, kann man bei extend keine Variable am Ende ranhängen.

Würde mich über eure Gedanken sehr freuen. Jedoch bitte mehr als 2 Wörter oder nur ein Satz mit dem man nicht viel anfangen kann. Mir wären simple Codeschnipsel am liebsten :rolleyes:
07/13/2016 00:03 Mikesch01#2
Eine Denkblockade ist das nicht wirklich. Das ist einfach ein falsches Software-Design. Weder Person A noch Person B sollten die Klasse für das Menü überschreiben, sondern nur Elemente zum Menüpunkt hinzufügen können. Die Verwaltung der Menüpunkte sollte aus deinem CMS heraus erfolgen.

Die Personen können dann mittels bestimmter Schnittstellen (Interface) auf Methoden oder Eigenschaften deines Menü's zugreifen. Eine Sortierung des Menü's sollte von außen nicht möglich sein, da sonst jede Person diese Sortierung ändern könnte.
07/13/2016 07:14 paddelx3#3
Ich bin zwar müde, aber Versuchs trotzdem mal x)

Erst einmal weiß ich nicht warum deine Klasse Menu die Methode isVisible besitzt. Nach meinen Verständniss kommt diese Funktion erst durch die Erweiterung des Nutzers A hinzu. In seiner reinen Form hätte damit die dazugehörige Tabelle menu die so aussehen könnte:


id href name
1 google.de Google Deutschland
2 elitepvpers.com Elitepvpers
3 golem.de IT Nachrichten

entscheidend sind nur die Felder href und name, aus den sich der Menüpunkt nachher zusammen setzt. Das kannst du bei belieben natürlich noch erweitern. Wenn man nun in unsere Klasse Menu guckt, in Ihren bisherigen Zustand, so besitzt diese die Funktionen display, die die Daten zu den Menüpunkten formatiert, eine Funktion getData um die Daten aus der Tabelle zu bekommen, sowie eine Funktion zum ändern, hinzufügen und löschen. Dementsprechend würde unsere Menu Klasse wie folgt aussehen:

PHP Code:
class Menu 
{
    // Unser Container für die Daten
    var $data = array();
    
    // unsere funktionen zum editieren, löschen und hinzufügen
    function addMenu()
    {
        //....
    }
    function deleteMenu()
    {
        //....
    }
    function editMenu()
    {
        //....
    }
    
    // unsere Funktion um alle Menü Einträge zu erhalten, am besten die ganze Tabelle ins Array übertragen
    function setData(array $data) 
    {
       $this->data = $databaseTableArray;
    }
    
    /**
     * Unsere Funktion zum Anzeigen
     */
    function display()
    {
        // Formatiert die Elemente in den $data Array basierend auf den Namen und den Link
        foreach ($data as $menuEntry)
        {
            echo '<li><a href="'. $menuEntry['href'].'">'. $menuEntry['name'].'</a></li>';
        }
    }
} 
Anstatt nun die Klasse zu extenden, würde ich eher auf ein Hook System bauen so wie es viele große CMS machen. Als beispiel haben wir nun das Plugin, das einen Menüpunkt versteckt. Die Idee ist das die Daten vor der Ausgabe an das Plugin übermittelt werden, diese die Daten bearbeitet und unsere Hauptdaten in $data mit den gelöschten Einträgen ersetzt. Aber kommen wir erst mal zum Hooksystem an sich, ein Nutzer hat auf Stackoverflow ein verdammt gutes Beispiel gegeben: [Only registered and activated users can see links. Click Here To Register...]. Nach deinen nöten musst du die Klasse anpassen.



Nun können wir einfach ein Event hinzufügen und diese später vor unserer display Methode auslösen, dazu müssen wir das Event in unseren Quellcode setzen. Und beachten das jedes Plugin die Daten manipuliert, also unseren $data Array ersetzt.

PHP Code:
    function display()
    {
        // Event Triggern
        $this->data = event('beforeMenuDisplay', $this->data);
        
        // Formatiert die Elemente in den $data Array basierend auf den Namen und den Link
        foreach ($data as $menuEntry)
        {
            echo '<li><a href="'. $menuEntry['href'].'">'. $menuEntry['name'].'</a></li>';
        }
    } 
Damit haben wir unsere Basis fertig. Nun simulieren wir das Plugin um bspsw. ein Menüpunkt zu verstecken, ich habe leider keine Zeit um irwas zu testen und auch keine IDE zur Hand.

Gehen wir davon aus das unsere Tabelle nun die ID's beinhaltet, die versteckt werden sollen.

id(id des Menüpunktes)
2

PHP Code:
class PluginMenuHide
{
    // Unser Container für die Daten
    var $data = array();
    
    // unsere Daten aus der Tabelle besorgen
    function getData()
    {
        $this->data = $databaseTableArray;
    }
    
    // einträge aus array löschen und array zurückgeben
    function __construct($menuTableArray)
    {
        $this->getData();
        
        for ($i=0;$i<=count($menuTableArray);$i++)
        {
            if(in_array($menuTableArray[$i], $this->data))
            {
                unset($menuTableArray[$i]);
            }
        }
        
        return $menuTableArray;
    }
} 
Prinzipiell haben wir unser Plugin, und unser Event, das war wir noch machen müssen ist unser Plugin an das Event zu assoziieren und das bevor das Event aufgerufen wird und nachdem das Plugin geladen wurde.

PHP Code:
event('beforeMenuDisplay', NULL, new PluginMenuHide(array())); 
(Das so zu assoziieren wird nicht klappen, zumal das ein ungültiger Callback ist, wie gesagt bin extremst müde, hoffe aber das dir mein Post in irgend einer Weise geholfen hat. Das wichtigste an den ganzen Post ist im Endeffekt die Event/Hook Klasse des Nutzers auf Stackoverflow. Wenn ich die Tage aus den Urlaub wieder da bin, kann ich das ganze auch so schreiben das es klappt, hab hier wie gesagt keine Möglichkeit Code auszuführen geschweige den ne IDE)
07/13/2016 17:02 Mr.Tr33#4
Danke, das hat mir sehr geholfen.

Dafür muss aber die Klasse Menu bearbeitet werden oder nicht?
Somist muss jeder der ein Plugin für das Menü schreibt, die Klasse im Endeffekt bearbeiten.
07/13/2016 18:55 paddelx3#5
Nein muss man nicht. Wenn du mehrere Plugins automatisch lädst und die mehrere Events setzen

PHP Code:
$plugin = new event('beforeMenuDisplay', NULL, new PluginMenuHide(array())); 
PHP Code:
$plugin = new event('beforeMenuDisplay', NULL, new PluginMenuOrder(array())); 
die Idee ist, das all diese Plugins mehrere Events für den Punkt beforeMenuDisplay setzen. Dadurch das wir an die Funktion in 2 der Variable den Wert 0 übergeben, werden diese nicht getriggert (ausgelöst) sondern nur zum späteren ausführen gespeichert.

Wenn wir nun in unserer Menu Klasse die Events triggern durch das Übergeben der Datensätze
PHP Code:
$this->data = event('beforeMenuDisplay', $this->data); 
so sucht die Event Funktion nach Events des Namens beforeMenuDisplay und löst diese falls gefunden aus, in dem Fall währen das PluginMenuOrder und PluginMenuHide, die die Daten in der Theorie bearbeiten und anschließend werden die Daten wieder in die Menu Klasse eingeschweißt, ohne das die eigentliche Klasse zu extenden. Das hat dann wie gesagt den Vorteil das mehrere User/Module die Daten bearbeiten können, unabhängig davon was andere Plugins mit den Daten machen. Was für den Plugin Ersteller relevant ist sind ja nur die ID, der Link und dessen Name.

Rein theoretisch dürfte das auch mit Klassen und nicht nur mit Funktionen klappen, jedoch müsste man hier auf die Verwendung von call_user_func
verweisen, da diese dann Statische Klassen (Plugin Klasse) benötigt.

Meine Idee war es als erstes eine richtige Plugin Klasse zu schreiben die als Factory dient und die Plugins automatisch lädt und initialisiert. Und nachher auch nur ein Hook gesetzt wird an den die Plugins dann ausgeführt werden.

lg
07/13/2016 21:08 Mr.Tr33#6
Du baust ja jedes mal eine neue Instanz von Event auf, das doch falsch oder nicht? Wäre da eine statische Klasse nicht passender?

Bis jetzt habe ich es eigentlich verstanden, aber eins stört mich noch :D
Ich baue also in meinen Klassen überall Eventabfragen ein. Aber wo setze ich denn überall die Eventaktionen?
07/13/2016 22:31 paddelx3#7
Quote:
Originally Posted by Mr.Tr33 View Post
Du baust ja jedes mal eine neue Instanz von Event auf, das doch falsch oder nicht? Wäre da eine statische Klasse nicht passender?
Unsere Event Funktion ist ja keine Klasse somit haben wir auch im den Sinne keine richtige Instanz. static $events;. Dazu kannst du dir am besten mal die [Only registered and activated users can see links. Click Here To Register...] angucken. Man könnte nun auch hingehen und das ganze wie gesagt nach seinen belieben anpassen. Das ist ne Statische Klasse oder sonst was stecken. Das war auch mein erster Gedankengang als ich dein Thread gelesen hab, jedoch sind wir faule Leute und das währe nur unnötig mehr Arbeit.

Quote:
Originally Posted by Mr.Tr33 View Post
Bis jetzt habe ich es eigentlich verstanden, aber eins stört mich noch :D
Ich baue also in meinen Klassen überall Eventabfragen ein. Aber wo setze ich denn überall die Eventaktionen?
Das ist prinzipiell ganz einfach. Wie gesagt wir gehen davon aus, das unser Plugin immer eine Funktion hat, die die Daten bekommt. Diese eine Funktion nutzen wir als [Only registered and activated users can see links. Click Here To Register...]. Sprich diese wird aufgerufen und mit den Daten gefüttert, sobald ein Event triggert.

Nun weisen wir unser Plugin mit seiner Callback Funktion einen Event zu, das ganze passiert, sobald wir einen Callback mit angeben, der Wert der für den Callback genutzt wird aber immer noch NULL ist. Ein paar Beispiele wie wir unser Callback nun einen Event zuweisen: (oh seh gerade hab in meinen ersten Post paar schwere Fehler eingebaut :X, ist korrigiert)

PHP Code:
event('EventName', NULL, beispielCallbackFunctionOne($data) }); 
PHP Code:
event('EventName', NULL, beispielCallbackFunctionTwo($data) }); 
PHP Code:
event('EventName', NULL, beispielCallbackFunctionThree($data) }); 
Alles was wir jetzt noch machen müssen ist das Event zu triggern, indem wir die Daten übergeben. Das kann an einer beliebigen Stelle im Quellcode sein, da, wo wir's benötigen.

PHP Code:
$data= event('EventName', $data); 
07/14/2016 12:50 Mr.Tr33#8
Das du eine neue Instanz aufbaust kam ich darauf, weil du ständig new event geschrieben hast :D
Und faul sein heißt nicht, dass es direkt gut und schön ist :P

Ebenfalls hast du irgendwie nur dich wiederholt :D
Ich habe jetzt einfach mal was schönes zusammen geworfen:
Code:
class Controller {
	private $menu;

	function __construct() {
		$this->menu = new Menu();
	}

	function draw(){
		$this->menu->display();
	}
}

class Menu {
	private $menuData = [];

	public function display() {
		$this->menuData = event("menuBeforeDisplay", $this->menuData);

		foreach ($this->menuData as $item)
			echo "<li>".$item."</li>";
	}
}

class MenuOrder {
	private $data;

	function __construct($menuData) {
		
	}

	public function getData() {
		return $this->data;
	}
}
In Menu habe ich nur die Abfrage vom Event.
Wo aber setze ich den Event bzw. baue MenuOrder ein? Einfach stumpf nachdem ich die Menu Instanz aufgebaut habe?