2 Commits

Author SHA1 Message Date
d411f58134 Changed icon for second statistics tile
All checks were successful
Pull Request Pipeline / test (pull_request) Successful in 2m21s
Pull Request Pipeline / lint (pull_request) Successful in 2m22s
2025-11-23 12:18:05 +01:00
fee5c57207 Added comments for return value -1 2025-11-23 12:13:30 +01:00

View File

@@ -89,7 +89,7 @@ class _StatisticsViewState extends State<StatisticsView> {
), ),
SizedBox(height: constraints.maxHeight * 0.02), SizedBox(height: constraints.maxHeight * 0.02),
StatisticsTile( StatisticsTile(
icon: Icons.casino, icon: Icons.percent,
title: 'Winrate per Player', title: 'Winrate per Player',
width: constraints.maxWidth * 0.95, width: constraints.maxWidth * 0.95,
values: winRates, values: winRates,
@@ -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));
} }