Ich fang mal mit meckern an
Offtopic zu den Benamungen:
- ich bekomm bei deutschen Bezeichnern irgendwie immer Pickel - aber das ist subjektiv
- ich finde, dass man bei nem I Irgendwas wohl kaum noch Schnittstelle in den Namen packen muss, dafür gibts ja vorn schon das I (das selbe bei Klassen).
Generell bin ich geschockt, wie viel Code man doch für eine solch einfache Anwendung schreiben kann, von der vermutlich 90% einfach nur boilerplate glue code ist.
Wenn du dir mal das Bildchen
hier anschaust, siehst du, dass bei MVP die View und das Model sich nicht kennen. Ist bei dir aber der Fall - siehe
TfoHauptFormular.SetzeAdressenDaten
.
Für
FuehreSchalterEintragNeuBearbeitenKommandoAusEreigniss
den Komponentennamen zu nehmen ist ne ziemliche Totsünde, wenn du
GUI und BL trennen willst (und wozu sollte man sonst MVP machen).
Entweder in 2 Methoden aufteilen oder per enum angeben ob neu oder edit.
Für eine eingehendere Analyse fehlt mir aktuell die Zeit.