lib/data/dao/game_dao.dart

@@ -19,17 +19,17 @@ class GameDao extends DatabaseAccessor<AppDatabase> with _$GameDaoMixin {
   }
 
   /// Retrieves a [Game] by its [gameId].
-  Future<Game> getGameById(String gameId) async {
+  Future<Game> getGameById({required String gameId}) async {
     final query = select(gameTable)..where((g) => g.id.equals(gameId));
     final result = await query.getSingle();
 
     List<Player>? players;
-    if (await db.playerGameDao.hasGamePlayers(gameId)) {
-      players = await db.playerGameDao.getPlayersByGameId(gameId);
+    if (await db.playerGameDao.hasGamePlayers(gameId: gameId)) {
+      players = await db.playerGameDao.getPlayersByGameId(gameId: gameId);
     }
     Group? group;
-    if (await db.groupGameDao.hasGameGroup(gameId)) {
-      group = await db.groupGameDao.getGroupByGameId(gameId);
+    if (await db.groupGameDao.hasGameGroup(gameId: gameId)) {
+      group = await db.groupGameDao.getGroupByGameId(gameId: gameId);
     }
 
     return Game(
@@ -41,14 +41,16 @@ class GameDao extends DatabaseAccessor<AppDatabase> with _$GameDaoMixin {
     );
   }
 
-  Future<void> addGame(Game game) async {
+  /// Adds a new [Game] to the database.
+  /// Also adds associated players and group if they exist.
+  Future<void> addGame({required Game game}) async {
     await db.transaction(() async {
       for (final p in game.players ?? []) {
-        await db.playerDao.addPlayer(p);
-        await db.playerGameDao.addPlayerToGame(game.id, p.id);
+        await db.playerDao.addPlayer(player: p);
+        await db.playerGameDao.addPlayerToGame(gameId: game.id, playerId: p.id);
       }
       if (game.group != null) {
-        await db.groupDao.addGroup(game.group!);
+        await db.groupDao.addGroup(group: game.group!);
         await db.groupGameDao.addGroupToGame(game.id, game.group!.id);
       }
       await into(gameTable).insert(

lib/data/dao/group_dao.dart

@@ -20,12 +20,12 @@ class GroupDao extends DatabaseAccessor<AppDatabase> with _$GroupDaoMixin {
   }
 
   /// Retrieves a [Group] by its [groupId], including its members.
-  Future<Group> getGroupById(String groupId) async {
+  Future<Group> getGroupById({required String groupId}) async {
     final query = select(groupTable)..where((g) => g.id.equals(groupId));
     final result = await query.getSingle();
 
     List<Player> members = await db.playerGroupDao.getPlayersOfGroupById(
-      groupId,
+      groupId: groupId,
     );
 
     return Group(id: result.id, name: result.name, members: members);
@@ -33,7 +33,7 @@ class GroupDao extends DatabaseAccessor<AppDatabase> with _$GroupDaoMixin {
 
   /// Adds a new group with the given [id] and [name] to the database.
   /// This method also adds the group's members to the [PlayerGroupTable].
-  Future<void> addGroup(Group group) async {
+  Future<void> addGroup({required Group group}) async {
     await db.transaction(() async {
       await into(
         groupTable,
@@ -56,15 +56,18 @@ class GroupDao extends DatabaseAccessor<AppDatabase> with _$GroupDaoMixin {
 
   /// Deletes the group with the given [id] from the database.
   /// Returns `true` if more than 0 rows were affected, otherwise `false`.
-  Future<bool> deleteGroup(String id) async {
-    final query = (delete(groupTable)..where((g) => g.id.equals(id)));
+  Future<bool> deleteGroup({required String groupId}) async {
+    final query = (delete(groupTable)..where((g) => g.id.equals(groupId)));
     final rowsAffected = await query.go();
     return rowsAffected > 0;
   }
 
   /// Updates the name of the group with the given [id] to [newName].
   /// Returns `true` if more than 0 rows were affected, otherwise `false`.
-  Future<bool> updateGroupname(String id, String newName) async {
+  Future<bool> updateGroupname({
+    required String id,
+    required String newName,
+  }) async {
     final rowsAffected =
         await (update(groupTable)..where((g) => g.id.equals(id))).write(
           GroupTableCompanion(name: Value(newName)),

lib/data/dao/group_game_dao.dart

@@ -12,7 +12,7 @@ class GroupGameDao extends DatabaseAccessor<AppDatabase>
 
   /// Checks if there is a group associated with the given [gameId].
   /// Returns `true` if there is a group, otherwise `false`.
-  Future<bool> hasGameGroup(String gameId) async {
+  Future<bool> hasGameGroup({required String gameId}) async {
     final count =
         await (selectOnly(groupGameTable)
               ..where(groupGameTable.gameId.equals(gameId))
@@ -22,12 +22,12 @@ class GroupGameDao extends DatabaseAccessor<AppDatabase>
     return (count ?? 0) > 0;
   }
 
-  Future<Group> getGroupByGameId(String gameId) async {
+  Future<Group> getGroupByGameId({required String gameId}) async {
     final result = await (select(
       groupGameTable,
     )..where((g) => g.gameId.equals(gameId))).getSingle();
 
-    final group = await db.groupDao.getGroupById(result.groupId);
+    final group = await db.groupDao.getGroupById(groupId: result.groupId);
     return group;
   }
 

lib/data/dao/player_dao.dart

@@ -17,38 +17,47 @@ class PlayerDao extends DatabaseAccessor<AppDatabase> with _$PlayerDaoMixin {
   }
 
   /// Retrieves a [Player] by their [id].
-  Future<Player> getPlayerById(String id) async {
-    final query = select(playerTable)..where((p) => p.id.equals(id));
+  Future<Player> getPlayerById({required String playerId}) async {
+    final query = select(playerTable)..where((p) => p.id.equals(playerId));
     final result = await query.getSingle();
     return Player(id: result.id, name: result.name);
   }
 
   /// Adds a new [player] to the database.
-  Future<void> addPlayer(Player player) async {
+  /// If a player with the same ID already exists, updates their name to
+  /// the new one.
+  Future<void> addPlayer({required Player player}) async {
+    if (!await playerExists(playerId: player.id)) {
       await into(
         playerTable,
       ).insert(PlayerTableCompanion.insert(id: player.id, name: player.name));
+    } else {
+      await updatePlayername(playerId: player.id, newName: player.name);
+    }
   }
 
   /// Deletes the player with the given [id] from the database.
   /// Returns `true` if the player was deleted, `false` if the player did not exist.
-  Future<bool> deletePlayer(String id) async {
-    final query = delete(playerTable)..where((p) => p.id.equals(id));
+  Future<bool> deletePlayer({required String playerId}) async {
+    final query = delete(playerTable)..where((p) => p.id.equals(playerId));
     final rowsAffected = await query.go();
     return rowsAffected > 0;
   }
 
   /// Checks if a player with the given [id] exists in the database.
   /// Returns `true` if the player exists, `false` otherwise.
-  Future<bool> playerExists(String id) async {
-    final query = select(playerTable)..where((p) => p.id.equals(id));
+  Future<bool> playerExists({required String playerId}) async {
+    final query = select(playerTable)..where((p) => p.id.equals(playerId));
     final result = await query.getSingleOrNull();
     return result != null;
   }
 
-  /// Updates the name of the player with the given [id] to [newName].
-  Future<void> updatePlayername(String id, String newName) async {
-    await (update(playerTable)..where((p) => p.id.equals(id))).write(
+  /// Updates the name of the player with the given [playerId] to [newName].
+  Future<void> updatePlayername({
+    required String playerId,
+    required String newName,
+  }) async {
+    await (update(playerTable)..where((p) => p.id.equals(playerId))).write(
       PlayerTableCompanion(name: Value(newName)),
     );
   }

lib/data/dao/player_game_dao.dart

@@ -12,7 +12,7 @@ class PlayerGameDao extends DatabaseAccessor<AppDatabase>
 
   /// Checks if there are any players associated with the given [gameId].
   /// Returns `true` if there are players, otherwise `false`.
-  Future<bool> hasGamePlayers(String gameId) async {
+  Future<bool> hasGamePlayers({required String gameId}) async {
     final count =
         await (selectOnly(playerGameTable)
               ..where(playerGameTable.gameId.equals(gameId))
@@ -24,7 +24,7 @@ class PlayerGameDao extends DatabaseAccessor<AppDatabase>
 
   /// Retrieves a list of [Player]s associated with the given [gameId].
   /// Returns an empty list if no players are found.
-  Future<List<Player>> getPlayersByGameId(String gameId) async {
+  Future<List<Player>> getPlayersByGameId({required String gameId}) async {
     final result = await (select(
       playerGameTable,
     )..where((p) => p.gameId.equals(gameId))).get();
@@ -32,7 +32,7 @@ class PlayerGameDao extends DatabaseAccessor<AppDatabase>
     if (result.isEmpty) return <Player>[];
 
     final futures = result.map(
-      (row) => db.playerDao.getPlayerById(row.playerId),
+      (row) => db.playerDao.getPlayerById(playerId: row.playerId),
     );
     final players = await Future.wait(futures);
     return players.whereType<Player>().toList();
@@ -40,7 +40,10 @@ class PlayerGameDao extends DatabaseAccessor<AppDatabase>
 
   /// Associates a player with a game by inserting a record into the
   /// [PlayerGameTable].
-  Future<void> addPlayerToGame(String gameId, String playerId) async {
+  Future<void> addPlayerToGame({
+    required String gameId,
+    required String playerId,
+  }) async {
     await into(playerGameTable).insert(
       PlayerGameTableCompanion.insert(playerId: playerId, gameId: gameId),
     );

lib/data/dao/player_group_dao.dart

@@ -11,7 +11,7 @@ class PlayerGroupDao extends DatabaseAccessor<AppDatabase>
   PlayerGroupDao(super.db);
 
   /// Retrieves all players belonging to a specific group by [groupId].
-  Future<List<Player>> getPlayersOfGroupById(String groupId) async {
+  Future<List<Player>> getPlayersOfGroupById({required String groupId}) async {
     final query = select(playerGroupTable)
       ..where((pG) => pG.groupId.equals(groupId));
     final result = await query.get();
@@ -19,7 +19,7 @@ class PlayerGroupDao extends DatabaseAccessor<AppDatabase>
     List<Player> groupMembers = [];
 
     for (var entry in result) {
-      final player = await db.playerDao.getPlayerById(entry.playerId);
+      final player = await db.playerDao.getPlayerById(playerId: entry.playerId);
       groupMembers.add(player);
     }
 
@@ -28,7 +28,10 @@ class PlayerGroupDao extends DatabaseAccessor<AppDatabase>
 
   /// Removes a player from a group based on [playerId] and [groupId].
   /// Returns `true` if more than 0 rows were affected, otherwise `false`.
-  Future<bool> removePlayerFromGroup(String playerId, String groupId) async {
+  Future<bool> removePlayerFromGroup({
+    required String playerId,
+    required String groupId,
+  }) async {
     final query = delete(playerGroupTable)
       ..where((p) => p.playerId.equals(playerId) & p.groupId.equals(groupId));
     final rowsAffected = await query.go();
@@ -36,7 +39,10 @@ class PlayerGroupDao extends DatabaseAccessor<AppDatabase>
   }
 
   /// Adds a player to a group with the given [playerId] and [groupId].
-  Future<void> addPlayerToGroup(String playerId, String groupId) async {
+  Future<void> addPlayerToGroup({
+    required String playerId,
+    required String groupId,
+  }) async {
     await into(playerGroupTable).insert(
       PlayerGroupTableCompanion.insert(playerId: playerId, groupId: groupId),
     );