MCREvent - use enums for event types and object types
Description
E-mail vom 28.09.2022
Hallo zusammen,
ich würde gerne etwas mehr Struktur in die Klasse MCREvent bringen.
Objekte dieser Klasse werden von unserem Eventhandler verarbeitet.
Ein Event besteht aus einem EventTyp und einem ObjectTyp und kann als Hashtable weitere Daten aufnehmen.
Die Anzahl der EventTypes und ObjectTypes hat sich in letzter Zeit vergrößert und die Liste der Static-String_Konstanten wird immer länger. Um besser zwischen EventType und ObjectType unterscheiden zu können würde ich dafür gerne Enums an Stelle von Strings einführen.
Außerdem passt es konzeptionell besser, die Daten explizit als Hashtable/-Map innerhalb des Objektes zu definieren und nicht wie bisher das MCREvent von der Hashtable erben zu lasssen. Das sind aber eigentlich noch eine andere Baustelle.
Meiner Meinung nach gewinnt der Code dadurch und die Nutzung in den verschiedenen Klassen, die mit den Events arbeiten wird vereinfacht. Die Umstellung betrifft viele Klassen, ist aber nicht kompliziert.
E-mail vom 28.09.2022
Hallo zusammen,
ich würde gerne etwas mehr Struktur in die Klasse MCREvent bringen.
Objekte dieser Klasse werden von unserem Eventhandler verarbeitet.
Ein Event besteht aus einem EventTyp und einem ObjectTyp und kann als Hashtable weitere Daten aufnehmen.
Die Anzahl der EventTypes und ObjectTypes hat sich in letzter Zeit vergrößert und die Liste der Static-String_Konstanten wird immer länger.
Um besser zwischen EventType und ObjectType unterscheiden zu können würde ich dafür gerne Enums an Stelle von Strings einführen.
Außerdem passt es konzeptionell besser, die Daten explizit als Hashtable/-Map innerhalb des Objektes zu definieren und nicht wie bisher
das MCREvent von der Hashtable erben zu lasssen.
Das sind aber eigentlich noch eine andere Baustelle.
Meiner Meinung nach gewinnt der Code dadurch und die Nutzung in den verschiedenen Klassen, die mit den Events arbeiten wird vereinfacht.
Die Umstellung betrifft viele Klassen, ist aber nicht kompliziert.