Trait oder Elternklasse in einer Klassen Gruppe?

[Rolf B]

Hallo pl,

Es kommt immer darauf an, das Wesentliche zu erkennen.

was in diesem Fall ist: Du faselst.

Was Du zeigst, ist keine Alternative zu DI, sondern die in grellen Farben gezeichnete Darstellung des Problems, das DI lösen will: Festgebackene Abhängigkeiten, die Unit-Tests verhindern und Flexibilität nehmen. In deinem Fall ist es ein Zwieback: außer zur CGI Klasse gibt's auch noch die Abhängigkeit zum Sourcefile, d.h. das Unterschieben einer CGI-Klasse aus CGI_Emulator.php ist auch nicht mehr möglich.

Eigentlich ist es in deinem FW ja eine großartige Idee, die HTTP Request/Response Themen auszulagern. So kann man die Controllerklasse über automatisierte Tests laufen lassen und das CGI Umfeld simulieren. Aber genau dafür muss man das CGI Objekt injizieren können. Da ein Controller es vermutlich in 99% aller Fälle brauchen wird, dürfte es wenig sinnvoll sein, dieses Objekt so wie oben gezeigt lazy zu beschaffen. Das Beschaffen der CGI Instanz gehört in den Konstruktor. Man KANN es so bauen, dass eine fehlende CGI Injektion bewirkt, dass der Controller sich das CGI Objekt aus einer Default-Implementation holt. Aber niemals nicht so wie in deiner Methode. Die ist ein perfektes Beispiel für ignoriertes SRP.

Rolf