Fehlende Methoden für Games Datenbank inplementieren #76
3 Participants
Notifications
Due Date
No due date set.
Blocks
Depends on
#78 Game zu Match umbenennen
liquid-development/game-tracker
#20 GameHistoryView anpassen
liquid-development/game-tracker
Reference: liquid-development/game-tracker#76
Reference in New Issue
Block a user
Delete Branch "feature/74-fehlende-methoden-für-games-datenbank-inplementieren"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fehlende Methoden für Games Datenbank inplementieren
Zugehörige Issue(s):
Closes #74
Beschreibung
Eine klare und prägnante Übersicht über die vorgenommenen Änderungen. Erläutere nicht nur das was gemacht wurde, sondern auch warum.
Änderungen
Es wurden folgende neue Methoden inkl. Tests implementiert:
setWinner(): Setzte den Gewinner eines SpielsgetWinner(): Bekomme den Gewinner eines Spiels alsPlayer-Objekt (Nullable)removeWinner(): Entfernt den Gewinner eines SpielshasWinner(): Gibt zurück, ob ein Spiel einen Gewinner hatupdateGameName(): Ändert den Namen eines SpielsupdateGroupOfGame(): Ändert die einem Spiel zugewiesene GruppeupdatePlayersFromGame(): Ändert die einem Spiel zugewiesenen SpielerAußerdem
addGames(),addPlayers()undaddGroups()-Methoden angepasst um sie besser von den Methoden zum Hinzufügen eines einzelnen Objekts zu unterscheidenZusätzliche Anmerkungen
Ich hab bisschen verkackt und die Änderungen auf der Branch zu #3 gemacht, deswegen sind hier jetzt auch die Änderungen davon drin
@@ -66,3 +66,4 @@/// Adds a new [Game] to the database./// Also adds associated players and group if they exist./// If a game, player, or group already exists, it will be replaced.Future<void> addGame({required Game game}) async {warum steht hier if a game player or group exists it will be replaced? bezieht sich das auf das game, bzw. die group/den player im game? Vorhin meintest du doch das replace funktioniert nicht so
Das Replacen funktioniert nicht bei den Verbindungstabellen, aber bei Entity-Tabellen schon. Bei den Verbindungstabellen sind beide Schlüssel zusammen primary key und deswegen kann da nichts ersetzt werden, weil die primary keys beim ersetzen gleich bleiben, nur die anderen attribute ändern sich
@@ -16,0 +14,4 @@/// [GroupGameTable]. If there is already group associated to the game,/// it will be replaced.Future<void> addGroupToGame({required String gameId,sind hier nicht addGroupToGame und updateGroupOfGame redundant? Zumindest steht bei addGroupToGame ja, dass es replaced wird, wenn's existiert
@@ -137,2 +137,4 @@}});test('Updating the games players works coreclty', () async {*correctly
digga warum mache ich das immer falsch omg
Ich muss gleich nochmal was überprüfen, weil mir gerade eingefallen ist, dass ggf. das doppelte hinzufügen von spielen dazu führen kann, das mehrere gruppen oder spieler zu einem spiel gehören
Fehlende Methoden für Games Datenbank inplementierento WIP: Fehlende Methoden für Games Datenbank inplementierenWIP: Fehlende Methoden für Games Datenbank inplementierento Fehlende Methoden für Games Datenbank inplementieren@@ -0,0 +21,4 @@}class _CreateGameViewState extends State<CreateGameView> {/// Reference to the app databaseWarum so viele useless Kommentare? ChatGPT?
Ja war wo anders auch so, finde das auch zu viel, er meinte beim anderen mal aber zum verständnis und so
also ich check das ja an sich, wenn das denn nur halt überall so wäre
finde es gibt auf jeden fall parts in dem code die schwerer zu verstehen sind als das und garkeine kommentare haben
Finde das irgendwie geiler wenn jede variable da oben ne beschreibung hat, wenn man über sie hovert auch (mit den
///), also finde das sinnvollerWarum dann aber nicht überall? oder kommt das im refactor?
Ja würde ich gerne machen, auch als vorlage für alle kommenden Variablen, Methoden & Klassen
ups hahaha
wollte approven auf handy und hab das falsche gedrückt 😅
aber ist ja eh schon gemerged