Dungeon-API: Ersetze "Game.currentLevel().xyz()" durch "Game.xyz()" #1625
Comments
tgrothe
added a commit
to tgrothe/Dungeon
that referenced
this issue
Jan 4, 2025
Fixes Dungeon-CampusMinden#1625 Add a new method `randomTile(LevelElement elementType)` to the `Game` class. * **Game.java** - Add a new method `randomTile(LevelElement elementType)` that calls `currentLevel().randomTile(elementType)`. - Update the Javadoc for the new method to explain its purpose. * **GameTest.java** - Add a new test method to verify the functionality of `Game.randomTile(LevelElement elementType)`. --- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/Dungeon-CampusMinden/Dungeon/issues/1625?shareId=XXXX-XXXX-XXXX-XXXX).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Das Pattern kommt recht häufig vor:
Game.currentLevel().machWasMitDemLevel().Aus Architektursicht ist das Feature-Neid, Game hat etwas, auf dem ich direkt arbeiten will. Durch das direkte Abfragen zerstöre ich auch die Kapselung.
Aus Kundensicht will ich vom Game einfach nur ein zufälliges Tile bekommen. Ob da noch ein Level oder was involviert ist, interessiert mich eigentlich nicht (oder sollte mich nicht interessieren). Der zusätzliche Aufruf
.currentLevel()erzeugt mentalen Clutter ...Wenn ich das richtig sehe, gibt es eh nur ein aktuelles Level, oder? Ich würde die Methode
randomTile(X)inGameanbieten. Intern kann das ja direkt aufcurrentLevelweitergeleitet werden, aber ich würde dem Kunden damit einen zusätzlichen Schritt sparen.Das klappt freilich nicht mehr, wenn parallel mehrere Level existieren können und ich auf all diesen Leveln arbeiten könnte.
edit: es geht darum, die modellierung zu prüfen/zu überdenken und dann den code aufzuräumen, nicht nur einfach noch eine weitere schnittstelle anzubieten.
The text was updated successfully, but these errors were encountered: