[Refactoring]: BoardManager Klasse: Auslagerung der is_tile_free() Funktionalität #30

Open
opened 2026-02-16 17:45:18 +00:00 by FlorianReetz · 0 comments
Owner

Motivation

Aus Architekturgründen soll die is_tile_free() Funktion ausgelagert werden, da es sich teilweise um eine Regelabhänginge Fragestellung handelt. Die Regeln nach denen isFree? gültig ist, sind nicht universell und können sich von Spieltyp zu Spieltyp unterscheiden oder erweitern. Die Frage isFree? soll deshalb in die RuleSets (hier die SolitaireRuleSet) Klasse ausgelagert werden.

Ziel

  • BoardManager verliert die is_tile_free() Methode
  • SolitaireRuleSet erhält die is_tile_free() Methode

Scope

In diesem Issue
Migration der is_tile_free() Methode

Nicht in diesem Issue
Ausarbeitung oder Erweiterung der is_tile_free()

Akzeptanzkriterien

  • BoardManager kennt is_tile_free() nicht mehr
  • Alle Abfragen laufen über das RuleSet
  • Unit Tests wurden umgeschrieben
  • Alle Tests laufen grün, keine Warnings keine Erros im Projekt
### Motivation Aus Architekturgründen soll die `is_tile_free()` Funktion ausgelagert werden, da es sich teilweise um eine Regelabhänginge Fragestellung handelt. Die Regeln nach denen isFree? gültig ist, sind nicht universell und können sich von Spieltyp zu Spieltyp unterscheiden oder erweitern. Die Frage isFree? soll deshalb in die `RuleSets` (hier die `SolitaireRuleSet`) Klasse ausgelagert werden. ### Ziel - `BoardManager` verliert die `is_tile_free()` Methode - `SolitaireRuleSet` erhält die `is_tile_free()` Methode ### Scope **In diesem Issue** Migration der `is_tile_free()` Methode **Nicht in diesem Issue** Ausarbeitung oder Erweiterung der `is_tile_free()` ### Akzeptanzkriterien - [ ] `BoardManager` kennt `is_tile_free()` nicht mehr - [ ] Alle Abfragen laufen über das `RuleSet` - [ ] Unit Tests wurden umgeschrieben - [ ] Alle Tests laufen grün, keine Warnings keine Erros im Projekt
Sign in to join this conversation.
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Headframe_Studio/MamaMahjong#30
No description provided.