Code-Redundanz in Controllern

[dako314]

Hallo zusammen,

ich baue gerade ein Contao-Bundle für eine zu entwickelnde Plattform. Hierzu ein kleines Beispiel:

Es gibt Beiträge. Diese Beiträge sind Kategorien zugeordnet. Jeder Benutzer darf nur bestimmte (von ihm gekaufte) Kategorien und deren Beiträge sehen. Außerdem gibt es so Dinge wie "Zuletzt angeschaute Beiträge", "Favorisierte Beiträge" oder "Abonnieren von Kategorien für E-Mail Benachrichtigung bei neuen Videos".

Alles schön und gut, die Datenstruktur, Controller und Models stehen schon.

Meine Frage ist, wo ich die Logik für die Abfragen/Permissions platziere, denn es ist nicht nur ein "Entry::findAll()", sondern ein Lookup in weiteren Tabellen. Dies möchte ich nicht in den Controllern jedes Mal wiederholen. Packe ich das als statische Funktionen in die Models? Oder was ist da Best-Practise? Wie kann ich hier Code-Redundanz vermeiden?

Danke für die Hilfe.

Viele Grüße
Daniel Koch

[BennyBorn]

Naja, ob es in Contao da wirklich eine Best Practice gibt bin ich mir nicht sicher.
An vielen Stellen wir ja einfach nur im entsprechenden Frontend-Modul die Rechtefrage geklärt.

Je nachdem wie Du alles bisher aufgebaut hast würde ich es persönlich entweder direkt in die Models mit reinpacken oder aber eben eine extra Permissions-Klasse bauen. Beides schon gemacht, je nach Use-Case unterschiedlich aber am Ende des Tages Code gespart und Übersichtlichkeit gewahrt

[dako314]

Naja, ob es in Contao da wirklich eine Best Practice gibt bin ich mir nicht sicher.
An vielen Stellen wir ja einfach nur im entsprechenden Frontend-Modul die Rechtefrage geklärt.

Je nachdem wie Du alles bisher aufgebaut hast würde ich es persönlich entweder direkt in die Models mit reinpacken oder aber eben eine extra Permissions-Klasse bauen. Beides schon gemacht, je nach Use-Case unterschiedlich aber am Ende des Tages Code gespart und Übersichtlichkeit gewahrt

Danke für die Antwort. Ich habe nun alles in die Model-Klassen gemacht, ansonsten hätte ich einigen redundanten Code. Ich glaube noch besser wäre echt eine extra Permission-Klasse, aber das wäre dann für das Projekt wahrscheinlich zu viel.