Dungeon: Modellierung von CombinableGUI, CraftingGUI und InventoryGUI

[cagix]

from #1562 (comment):

Insgesamt bin ich kein Freund von dieser Klassen Architektur. Ich finde CombinableGUI sollte eher ein Interface werden, statt wie jetzt eine Abstrakte Oberklasse von CraftingGUI & InventoryGUI. Dann könnte man schön ein Interface erstellen für die transferItem oder addItem Methoden. Aber vlt hat das ja auch einen Grund das es so gemacht wurde?

Die Modellierung sollte nochmal kritisch betrachtet werden. Dieses Issue entstand durch die (gefühlte) Notwendigkeit, die in der Klasse GUICombination aggregierten internen Elemente für den externen Zugriff als Referenz herauszureichen.

Damit wird die Kapselung quasi aufgehoben, die Nutzer können den internen Zustand der Klasse verändern, und die Nutzer sind gezwungen, mit den Daten selbst irgendetwas sinnvolles anzustellen...

das ist ein deutliches anzeichen für schlechte modellierung.