cms2day Forum - Community & Support rund ums cms2day

Hello,

Ich erlaube mir, mal langer Zeit mal wieder was zu schreiben und dann ist es auch gleich noch negativ. Bitte hasst mich nicht.

Ich gebe meine persönliche Meinung ab:
Ich erachte diese Klasse als (eventuell noch?) sinnlos. Es ergibt keinen Sinn, eine Klasse zu schreiben, in der man einen einfachen Query für eine relationale Datenbank in PHP-Syntax schreibt anstelle direkt per SQL.

Wenn tatsächlich ein komplexes SQL auf einem relationalem Modell geschrieben wird, wird immer zuerst die Query geschrieben und getestet. Erst dann wird das Query in die Programmlogik eingebunden und die Verarbeitung auf das Resultat entwickelt. Also warum sollte man die SQL-Syntax dann noch in die PHP-Syntax umschreiben?

Aus meiner Sicht ist die Syntax in PHP nicht einfacher wie direkt über SQL.

require_once 'QueryBuilder.class.php';
$queryBuilderSELECT = new lkdevelopment\QueryBuilder();
$queryBuilderSELECT->select("bestellungID", "bestellungDatum")->from("bestellungen", "b")->where("b.bestellungID = :bid")->groupBy("b.bestellDatum")->limit(1, 3);
echo $queryBuilderSELECT;

ist weder kürzer noch übersichtlicher als
SELECT bestellungID,bestellungDatum FROM `bestellungen` as b WHERE b.bestellungID = :bid GROUP BY b.bestellDatum LIMIT 1,3

---

Zum Vergleich zu den von dir genannten Frameworks: Doctrine zum Beispiel ist ein ORM. Das bedeutet, dessen Nutzen ist, dass ein Objektmodell in ein relationales Modell gespeichert und wieder geladen werden kann. Es dient nicht dazu, ein einfaches Statement auf einem relationalem Modell zu bilden sondern es transformiert die objektorientierte Welt in die relationale Welt.

Nichts für ungut, nur eine bescheidene Meinung.

Cheers,
Prime

Muss ich mich Prime anschließen. Wenn OOP oder auch nur irgendwo in die Richtung, dann richtig. Und genau aus diesem Grund baut cms2day 5 auch noch auf die alte Variante, da der Aufwand viel zu enorm wäre, alles umzuschreiben. Was bringt es da, so was zu nutzen? Meiner Meinung nach auch etwas sinnlos.

Ich denke, dass es auf der einen Seite eine wirklich tolle Idee ist und Du auch daran bleiben solltest,
jedoch für Cms2Day ist dies einfach nicht von Nützen...

Zitat von »$this->getUser();«

Nun, Geschmack liegt bekanntlich im Auge des Betrachters. Eigentlich hat die Klasse auch noch einen weiteren Zweg, da sie auch zeitgleich für das Ausführen der Querys zuständig ist. Das habe ich hier aber bewusst mal raus genommen.

Dies lässt sich auch ohne diese Klasse mit den Bordmitteln von PHP realisieren - Deshalb sehe ich auch hier keinen Vorteil. Sinnvoll wäre es, wenn dies Datenbankunabhängig wäre, das heisst einen richtigen DB-Layer.

Zitat von »$this->getUser();«

Weil er einfacher zu lesen ist bzw. schneller [...]

Auf diese Aussage möchte ich dich zitieren: "Nun, Geschmack liegt bekanntlich im Auge des Betrachters." Ich erachte diese Schreibart als weder lesbarer noch schneller. Bei den Beispielen sind es die einfachsten zu schreibenden Queries. Sobald die Queries komplexer werden (z.B. mit verarschalteten Subselects, Exist-Statements oder mit Unions oder gar case-statements) würde das meines Erachtens noch klarer werden.

Zitat von »$this->getUser();«

Es bringt nur kleinere Vorteile von einem ORM (z.b. das einfache und verständliche Bauen von Querys)

Dies ist aus meiner Sicht keines der "kleineren" Vorteile eines ORMs. Ein ORM hat eine Aufgabe - Das Mapping zwischen OOM und RM. Das Bauen von Queries ist hierbei lediglich eine Begleiterscheinung.

Nun, für uns bleibt es, wie Matze es schon gesagt hat in der alten Variante - Oldschool.

Prime kennt den Verlauf der Entwicklung und auch die Beweggründe.

Es arbeiten nicht nur Profis in Agenturen mit cms2day, sondern auch User mit "PHP Kenntnissen".

Diesen ist der bisherige Aufbau von cms2day schon immer sehr entgegengekommen, da diese mehr damit anfangen konnten, bei evtl eigenen Anpassungen oder Modulen.

Auch die Profis und die der Agenturen lieben die übersichtlichen Funktionen und den Quellcode.

Schon vor langem hatten wir darüber diskutiert, sind trotzdem weiter den alten Pfad gegangen und er wird dankbar angenommen und auch nur positiv bewertet.

Alt Bewertes hat immer noch seinen Bestand und auch seine Begründung. Es ist deswegen ja nicht schlechter. Nur anders und warum sollten wir nicht anders sein.

Wie schon erwähnt ist Alles Geschmackssache.