Was ist hier versuche ist ein modifizierter Vermittler-Entwurfsmuster.
Aus meiner Sicht ist es die Aufgabe des Vermittlers(Controllers) die Funktionen an die zuständigen Objekte zu delegieren. Damit vermeide ich wilde Verbindung zwischen den ALV-Eventhändler-Methoden und der relativ komplexen User-Hierarchie.jocoder hat geschrieben: ↑19.11.2019 11:59Was mir dabei aufgefallen ist, dass der Controller mehrere Aufgaben übernimmt (er hält die Referenzen der ALV-Klassen und steuert deren Verhalten?, die ALV-Klassen rufen die Controllermethoden INSERT_CHEF, UPDATE_CHEF bzw. INSERT_EMPL, UPDATE_EMPL auf? so meine Vermutung). Vielleicht wäre es besser, die Controllermethoden INSERT_CHEF, UPDATE_CHEF bzw. INSERT_EMPL, UPDATE_EMPL in eine eigene Klasse, die nur die Event-Handler-Methoden enthält, zu verschieben?
Vielen Dank für den Lob. Gedanken sollte sich jeder Entwickler vor dem Implementieren der Software machen :-)qyurryus hat geschrieben: ↑21.11.2019 09:14Zum einen erst ein mal Lob dafür, dass du dir Vorüberlegungen machst!
Ich würde dir generell empfehlen, alle Klassen komplett in Englisch zu benennen, statt einen Mischmasch anzuwenden.
Die Bedeutung von "Chef" wäre im englischen z.B. "Koch" (und ist im deutschen zu informell) - besser wäre vielleicht "manager" oder ggf. "employer". Dabei stellt sich auch die Frage, ob du mit "employer" wirklich Vorgesetzte meinst oder normale Mitarbeiter? Letzteres wäre "employee".