Kommentare zu: Funktionen aufrufen, nur falls sie auch existieren

Von: maz

maz — Sat, 27 Feb 2010 16:49:22 +0000

Hallo, ich muss KingCrunch Recht geben. In PHP kann man viel anstellen, aber nicht alles ist sinnvoll. Ich finde, was man auf eine Weise in PHP aber nicht auch in anderen Sprachen umsetzen kann, sollte man sich zweimal überlegen. Z.B. ist Late Static Binding zwar auf den ersten Blick recht nützlich, aber jeder weiß, dass man für Vererbung/Polymorphie normalerweise ein Objekt benötigt (C++ Entwickler errinnern sich vielleicht an die vtable).
Durch das richtige Designmuster wird man nie auf Late Static Binduing angewiesen sein. Hinzu kommt, dass ich kein Fan von “Global State” bin und somit statische Dinge sowieso vermeide. (Kann hier nur empfehlen: http://www.youtube.com/watch?v=-FRm3VPhseI)
Um zum Beitrag noch was loszuwerden: Wieso setzt man kein Eventsystem a la Observer ein? Ein Plugin-System auf Funktionsbasis ist meiner Meinung nach keine gute Wahl, denn wie können sich hier mehrere Plugins für eine Aufgabe registrieren?

Von: Julian

Julian — Thu, 25 Feb 2010 23:35:44 +0000

Von Objektorientierung ist in dem Zusammenhang selbstverständlich nicht zu reden. OOP befasst sich mit Objekten (also Instanzen von Klassen) und weniger mit derlei statischen Konstrukten. Das ist oft eine Hintertür in meinen Augen.
Genau wie einige andere, würde ich in diesem Zusammenhang Fehlerbehandlung kritisieren – die ist nämlich nicht mehr gegeben. In welchem Zusammenhang darf man das Konstrukt verwenden? Nur mit Aufrufen von Plugin-Funktionen? Was ist, wenn ich es mit anderen Funktionen in Verbindung bringe .. schon entstehen Fehler.
Ich habe vor nicht all zu langer Zeit meinen Professor gefragt, wieso es (in Java) möglich ist, dass eine Instanz einer Klasse auf ein privates Attribut einer anderen Instanz derselben Klasse direkt zugreifen kann, wo man doch ohne direkten Zugriff und über Getter-Methoden eine viel bessere Kapselung erzwingen könnte. Die Antwort war in diesem Zusammenhang: “Programmcode soll sich nicht vor sich selbst schützen / kapseln, sondern vor äußeren Systemen” (sinngemäß). Damit möchte ich sagen, dass es nicht Sinn und Zweck ist, in einer Anwendung jede mögliche Fehlerquelle, die durch Fremdentwickler entsteht, auf Herz und Nieren zu prüfen. Plugins müssen nunmal korrekt entwickelt werden – es darf vor der Installation des Plugin durchaus eine Überprüfung stattfinden, aber eine vollständige Kapselung ist schier unmöglich, so dass immer Fehler auftreten können. Es ist also Aufgabe der Entwickler der Plugins, dieselben so zu entwickeln, dass sie den selben Standards wie die Anwendung selbst entsprechen.
Man muss hier eben ein Mittelmaß finden und meines Erachtens geht es schon zu weit, jede Funktion eines Plugins auf Existenz zu prüfen, denn man sollte davon ausgehen können, dass, wenn ein Plugin installiert wurde, entsprechende Funktionen zur Verfügung stehen. Dann muss man eben nur korrekte Abhängigkeiten beachten.
Übrigens weiß die Kern-Anwendung gar nicht von möglichen Funktionen, die ein Plugin zur Verfügung stellt. Nur Plugins können voneinander abhängen und optionale Funktionen aufrufen. Die Kern-Anwendung dagegen sollte ohne Plugins auskommen. Demnach ist es – unter Vorraussetzung, dass ein korrekt eingebundenes Plugin die Funktionen bereitstellt – eine Frage der Abhängigkeit von Plugins, ob ein Plugin eine bestimmte Funktion eines Anderen aufrufen kann. Kern-Funktionen sollten ja vorhanden bleiben.

Von: KingCrunch

KingCrunch — Thu, 25 Feb 2010 15:09:52 +0000

Andi:
1. Wenn eine Funktion nur optional aufgerufen werden soll, wäre ihr Fehlen kein Fehler.
2. Siehe 1 zB Widgets: Wenns fehlt, fehlts, aber gefährlich isses nicht

Von: Andi

Andi — Thu, 25 Feb 2010 08:08:14 +0000

Hm…also nachdem ich mir die ganze Diskussion durchgelesen habe, erschließt sich für mich immer noch nicht der Nutzen dieser eigentlich interessanten Idee.

Fazit bleibt für mich:
1. Fehlermeldungen haben Ihre Daseinsberechtigung und sind beim Debuggen unverzichtbar (siehe Julians Kommentar).
2. Die Funktion soll etwas tun (siehe Christopher K.)! Ich unterdrücke einen Fehler und meine Daten sind dazu noch unbrauchbar.
3. Gerade bei einem Plugin kann bei der Installation auf die Systemvoraussetzungen überprüft werden (metalguys Kommentar). -> http://php.net/manual/function.version-compare.php

Von: Simon

Simon — Wed, 24 Feb 2010 22:00:04 +0000

Das ist für mich kein OOP, das ist Vergewaltigung von Klassen.
Auch wenns nett aussieht, sauber ist das nicht

Von: DSB

DSB — Wed, 24 Feb 2010 21:25:11 +0000

Im genannten Kontext geht es ja darum PlugIns anzusteuern. Wenn diese nicht erreichbar sind, sollte das die Core-Funktionalität des Hauptskripts nicht beeinflussen, so dass die Vorgehensweise in Ordnung sein sollte.

Sicherlich könnte man jetzt einen Fall konstruieren, wo ein PlugIn von einem anderen abhängig ist, das übergeordnete nicht verfügbar ist und es dadurch im abhängigen PlugIn knallt. Aber dann sollte das abhängige PlugIn ebenfalls Prüfmethoden für das Vorhandensein des Haupt-PlugIns besitzen, welche dann greifen und es deaktivieren sollten.
Insofern betrachte ich die Vorgehensweise der Prüfung mit function_exists als legitime und schnelle Möglichkeit.
Just my 2 Cents.

Von: Christopher K.

Christopher K. — Wed, 24 Feb 2010 21:14:03 +0000

Was mir dabei aufgefallen ist:

Ich rufe eine Funktion ja meist nicht auf, weil es ganz nett wäre, sie aufzurufen, sondern weil ich sie brauche. Das heißt, wenn es sie nicht gibt, muss ich etwas tun. Sei es einen Fehler ausspucken, eine Exception werfen oder eine andere Funktion verwenden. Und das berücksichtigt diese Lösung hier nicht. Wenn ich mache if(function_exists()) kann ich per else irgendwas anderes tun.
Einfach im Code weiter zu gehen kann fatale Folgen haben, weil ich ja i.d.R. davon ausgehe, dass die Funktion etwas tut.

Was das Cachen angeht: Verlockende Idee, so ein Universalcache. Aber wie lange ist der Cache gültig? Was, wenn die Funktion außer einer Rückgabeberechnung noch etwas anderes tut, wie eine Datei schreiben? Was, wenn die Funktion etwas zeitabhängiges tut? Das muss man immer individuell machen mit dem Cachen, ein individueller Cache nutzt nur für einfache Fälle was.