spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Hallo Community!

Seit ein paar Tagen sitze ich an einem Plugin für einen Minecraft-Server und mache mir große (vielleicht auch zu große ) Gedanken um das Code-Design. Bis jetzt verwalte ich Objekte wie Städte, Grundstücke und User mit Managern wie CityManager, PlotManager und UserManager. Dabei verwende ich keine Singletons, sondern reiche die Manager an Instanzen wie UserListener weiter, welche diese benötigen. Die Manager benötige ich nur, um Listen einfacher verwalten zu können.

Nun bemerke ich aber, dass diese Lösung nicht so geschickt aussieht wie anfangs gedacht, was mir Sorgen bereitet. Momentan benötigen nämlich die Hauptklassen, die für Spielmechaniken wie "Grundstück kaufen" verantwortlich sind, so ziemlich alle Manager-Instanzen.

Ein Beispiel:
Die Klasse Plot (= Grundstück) speichert die User-ID des Grundstückbesitzers. Wenn ich also in irgendeiner Klasse oder Methode eine Referenz zu einer Grundstück-Instanz habe und den Usernamen des Grundstückbesitzers herausfinden möchte, muss ich auch auf die UserManager-Instanz zugreifen, da ich zuerst über den Manager die User-Klasse erhalte:
Falls die User-Instanz nicht vorhanden ist, wird in der Datenbank nach der ID gesucht und eine neue User-Instanz erstellt mit den Variablen aus der Datenbankabfage.
Möchte ich herausfinden, in welcher Welt sich der User aufhält, benötige ich den WorldManager, welcher anhand der World-ID die World-Klasse ausgibt. Und so könnte das Spielchen weiterlaufen...

Sollte ich ein anderes Konzept anwenden?

Subclass Sandbox:
Alle Spiellogik-Klassen besitzen eine Referenz zur GamePlugin-Instanz, über die ich auf alle Manager zugreifen kann.

Singleton/Statische Klasse:
Ich greife global auf die Manager zu. Die verwendete API von Bukkit bietet selber eine globale Instanz an, um z. B. eine Liste aller Welten zu erhalten (Bukkit.getWorlds()). Wäre es sinnvoll, auch das Konzept zu verfolgen und GamePlugin zu einem Singleton zu machen?

Oder kennt jemand einen besseren Weg?

Nette Grüße
GlowDragon

Nach etwas Überlegung ist mir auch eingefallen, dass es überhaupt kein Problem ist, auf die Hauptklasse der API global zuzugreifen, da ich den Code dieser Klasse ja gar nicht kennen muss und im Falle eines Tests nicht mocken müsste. Die Abhängigkeit ist zwar nicht sichtbar, aber offensichtlich, da es sich ja bei dem Projekt um ein Plugin handelt. Werde also weiterhin keine Singletons für meinen eigenen Klassen nutzen.

Ich fühle mich aber immer noch unsicher, was meine Umsetzung des Projektes angeht. Ist es denn sinnvoll, für Listen wie Grundstücke, Städte, Welten, usw. einen Manager zu verwenden, wenn für jeweils eine Liste eine Tabelle in der Datenbank existiert?

Manager-Klassen werden ja auch kritisiert, vorallem wenn die Methoden dieser Manager auch in der Klasse stehen können, um die es eigentlich geht. Zum Beispiel verwenden manche Programmierer userManager.setPosition(User user, Position position) statt user.setPosition(Position position). Ich mache diesen Fehler aber nicht. Setze ich den Manager also dann richtig ein, wenn es nur eine Möglichkeit gibt, um die Position des Users zu verändern?

Mit den Begriffen habe ich mich noch nie beschäftigt. Ich danke dir für die Namensvorschläge! Nun hab ich folgende Klassen:
UserCache: Weil User erst bei Bedarf geladen werden.
PropertyRepository: Weil Grundstücke und Städte von Anfang an im Arbeitsspeicher liegen. Sie werden in einer Klasse gehalten, da sie meist zusammen gebraucht werden.
World..?..: Diese Klasse erstellt nur neue Welten und enthält zusätzliche Informationen zu diesen. Bin mir da bei der Namensgebung nicht sicher.

Das Problem ist, dass ich zusätzliche Informationen zur Welt speichern möchte und ich die World-Klasse nicht einfach bearbeiten kann, da sie zur Bukkit API gehört. Außerdem müssen Welten auf einem Minecraft-Server schon zu Beginn erstellt werden, zu einem anderen Zeitpunkt ist es nicht möglich.

Jetzt weiß ich, wie man das Konzept nennt, das ich anwende.

Zuerst war ich sketpisch, eine Referenz zum Manager in jede Plot-Instant zu packen, aber gerade hab ich mal recherchiert, wie viel Speicher eine Referenz verbraucht: Nur 4 oder 8 Bytes, jenachdem ob es sich um ein 32-Bit- oder 64-Bit-System handelt. Ich mach mich gleich an die Umsetzung.

Vielen Dank für die Antworten!