Seite 1 von 1

Nochmal größere Umstrukturierung

Verfasst: Mi 6. Dez 2017, 11:57
von shadowcat
Im Nachhinein hab ich mich gefragt, warum ich das nicht gleich von Anfang an gemacht habe. :-? Vermutlich weil ich anfangs die Helper-Klassen nicht alle neu machen wollte. Das hat sich jetzt aber sowieso ergeben.

Also folgendes:
  • Ich führe jetzt einen Namespace ein.
  • Alle Klassen werden umbenannt. (Spart später Tippselei.)
Namespace(s):
CAT
CAT\Helper
CAT\Backend
...

Klassennamen:
Aus CAT_Object (Basisklasse für alles) wird CBase.

Bei allen anderen fällt der bisherige Präfix weg, aus CAT_Helper_Directory wird z.B. CDirectory im Namenspace CAT\Helper.

Für die einfache Verwendung ("Kurznamen") schreibt man an passender Stelle sowas wie:

Code: Alles auswählen

use CAT\Helper\Directory as Directory;
Dann kann man die Klasse verwenden als (Beispiel):

Code: Alles auswählen

$files = Directory::getFiles(...);
Ohne das use einfach voll referenzieren:

Code: Alles auswählen

$files = \CAT\Helper\Directory::getFiles(...);
Edit: "C"-Präfix entfernt.

Re: Nochmal größere Umstrukturierung

Verfasst: Mi 6. Dez 2017, 18:53
von shadowcat
Ich glaube das C-Präfix ist doch keine so gute Idee... Das macht z.B. Probleme beim Auflösen der Route...

(Beispiel: http.../backend/media --> bisher CAT_Backend_Media::index(), jetzt müßte überall noch ein C mit rein...)

Außerdem heißen ein paar Klassen "hinten" gleich, z.B. CAT_Page und CAT_Helper_Page, da muß man ein bißchen aufpassen.

Re: Nochmal größere Umstrukturierung

Verfasst: Di 30. Jan 2018, 14:25
von shadowcat
Ich werde mal das Dwoo nach CAT/vendor verschieben, also per Composer integrieren. Ich muß nur noch überlegen wie ich das dann mit den Plugins mache. Also unseren eigenen. Die gehören ja nicht ins vendor-Verzeichnis.

Re: Nochmal größere Umstrukturierung

Verfasst: Mi 31. Jan 2018, 18:39
von cwsoft
Aus der ZEN of Python: Namespaces are one honking great idea -- let's do more of those!

Gute Entscheidung. Kann man die aktuelle 2.0 irgendwo Codetechnisch einsehen, bzw. Ist das Github Repo noch aktuell.

Re: Nochmal größere Umstrukturierung

Verfasst: Mi 31. Jan 2018, 19:16
von shadowcat
Im Augenblick ist es nicht so ganz aktuell, fürchte ich, aber ich sehe zu, dass ich in den nächsten Tagen mal alles committe.

Re: Nochmal größere Umstrukturierung

Verfasst: Do 1. Feb 2018, 12:50
von shadowcat
Das Repo ist jetzt weitestgehend aktuell.

Re: Nochmal größere Umstrukturierung

Verfasst: So 4. Feb 2018, 11:53
von cwsoft
Danke. Sieht bisher recht vielversprechend aus.