liquid-development/game-tracker
3
0
Fork 0
Code Issues 14 Pull Requests 6 Actions Releases 1 Activity

Added comments for return value -1

Browse Source
This commit is contained in:
Felix Kirchner
2025-11-23 12:13:30 +01:00
parent acc5b0a3e9
commit fee5c57207
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
lib/presentation/views/main_menu/statistics_view.dart
View File

@@ -129,6 +129,7 @@ class _StatisticsViewState extends State<StatisticsView> {
final winner = game.winner; final winner = game.winner;
if (winner != null && winner.isNotEmpty) { if (winner != null && winner.isNotEmpty) {
final index = winCounts.indexWhere((entry) => entry.$1 == winner); final index = winCounts.indexWhere((entry) => entry.$1 == winner);
// -1 means winner not found in winCounts
if (index != -1) { if (index != -1) {
final current = winCounts[index].$2; final current = winCounts[index].$2;
winCounts[index] = (winner, current + 1); winCounts[index] = (winner, current + 1);
@@ -141,6 +142,7 @@ class _StatisticsViewState extends State<StatisticsView> {
// Adding all players with zero wins // Adding all players with zero wins
for (var player in players) { for (var player in players) {
final index = winCounts.indexWhere((entry) => entry.$1 == player.id); final index = winCounts.indexWhere((entry) => entry.$1 == player.id);
// -1 means player not found in winCounts
if (index == -1) { if (index == -1) {
winCounts.add((player.id, 0)); winCounts.add((player.id, 0));
} }
@@ -175,6 +177,7 @@ class _StatisticsViewState extends State<StatisticsView> {
final members = game.group!.members.map((p) => p.id).toList(); final members = game.group!.members.map((p) => p.id).toList();
for (var playerId in members) { for (var playerId in members) {
final index = gameCounts.indexWhere((entry) => entry.$1 == playerId); final index = gameCounts.indexWhere((entry) => entry.$1 == playerId);
// -1 means player not found in gameCounts
if (index != -1) { if (index != -1) {
final current = gameCounts[index].$2; final current = gameCounts[index].$2;
gameCounts[index] = (playerId, current + 1); gameCounts[index] = (playerId, current + 1);
@@ -187,6 +190,7 @@ class _StatisticsViewState extends State<StatisticsView> {
final members = game.players!.map((p) => p.id).toList(); final members = game.players!.map((p) => p.id).toList();
for (var playerId in members) { for (var playerId in members) {
final index = gameCounts.indexWhere((entry) => entry.$1 == playerId); final index = gameCounts.indexWhere((entry) => entry.$1 == playerId);
// -1 means player not found in gameCounts
if (index != -1) { if (index != -1) {
final current = gameCounts[index].$2; final current = gameCounts[index].$2;
gameCounts[index] = (playerId, current + 1); gameCounts[index] = (playerId, current + 1);
@@ -200,6 +204,7 @@ class _StatisticsViewState extends State<StatisticsView> {
// Adding all players with zero games // Adding all players with zero games
for (var player in players) { for (var player in players) {
final index = gameCounts.indexWhere((entry) => entry.$1 == player.id); final index = gameCounts.indexWhere((entry) => entry.$1 == player.id);
// -1 means player not found in gameCounts
if (index == -1) { if (index == -1) {
gameCounts.add((player.id, 0)); gameCounts.add((player.id, 0));
} }