State Pattern so ok?

Hallo, ich habe nun ein State Pattern anhand eines Getränkeautomaten erstellt. Zusätzlich sollte es im 3 Schichten Model realisiert werden:

AutomatGUI -> Presentation (Froms)
Getraenkeautomat -> Bussines (State Pattern)
Product -> Data (hier nur ein Mockup)

Ich habe 3 Zustände realisiert, anfangs soll der Automat warten. Ein Zustand zum Rechnen der Beiträge und ein für die Produktausgabe. Jeder Zustand muss ja alle Ereignisse beachten die ausgelöst werden, wie z.B. GetChange() für die Geldrückgabe im StateAddMoney usw.

Da ich sowas zum Ersten mal gemacht habe, werde ich wahrscheinlich noch einige Fehler gemacht haben z.B. bei den Assoziation zwischen dem Getränkeautomaten. Also immer her mit Kritik und Verbesserungsvorschlägen 😃

Würde ich ja, aber ich wußte nicht wie man diese in Visio realisiert, deshalb hier als Funktion dargestellt.

Sieht für mich so aus. Kann der Aggregation/Komposition vernünftig darstellen? Falls ja, habe ich das bisher immer übersehen...

nein da hast du ncihts übersehen. im allgemeinen kann er eigendlich nur sehr sehr wenig darstellen. eigendlich ist er nur dazu gedacht, das man sich einen knappen überblick generieren lassen kann, wenn man nichts anderes mehr zur verfügung hat.

Ist die Beziehung (Assoziation) zwischen dem AutomatGUI zu Getraenkeautomat so ok?

Beim Getraenkeautomaten und der Produkt Klasse eine Komposition, ein Automat hat beliebig viele Getränke.

Ist die Beziehung (Assoziation) zwischen dem AutomatGUI zu Getraenkeautomat so ok?

Finde ich schon, da der Automat ja auch ohne Gui existieren könnte (ist dann halt nicht bedienbar).

Was mir noch aufgefallen ist (sehr unspektakulär), ist die Assoziation zwischen Automat und State. Sollte meines Wissens auch eine Komposition sein, da der Zustand ohne Automat nicht existieren kann (Oder überseh ich hier etwas?).
Zu den Zustandsklassen an sich kann ich mich nicht wirklich äußern, da es im UML richtig aussieht aber im Code? Keine Ahnung. Nehme mal an, auch richtig 😛

Was mir noch aufgefallen ist (sehr unspektakulär), ist die Assoziation zwischen Automat und State. Sollte meines Wissens auch eine Komposition sein, da der Zustand ohne Automat nicht existieren kann (Oder überseh ich hier etwas?).

..hab das im Netz überall so gesehen, auch hier:
http://www.dofactory.com/Patterns/PatternState.aspx

Gut, mag sein (auch wenn ich es nicht ganz verstehe).
Sollte jetzt aber auch nicht soooo relevant sein (Oder bist du gerade dabei UML richtig zu lernen? Dann sollte man das evtl. mal mit einem diskutieren, der UML bis ins letzte beherscht 😄).

Wie sieht die Beziehung zwischen Getraenkeautomat und der Product Klasse aus, ich habe eine Komposition verwendet. Ich glaube es müsste eher eine Aggregation sein, denn eine Komposition sagt aus das sie ohne Product nicht existieren kann, denn ein Automat funktioniert auch wenn er leer ist, nicht wahr?!?!?

Nein, aus meiner Sicht nicht. Ein Automat ohne zugeordnetes Produkt geht nicht. Leer wäre er, wenn ein Produkt zugeordnet ist, aber dessen Anzahl 0.

LaTino