fix(analyze): show correctly hiding

crispgm · Jul 1, 2024 · 5d381eb · 5d381eb
1 parent 7b189fe
commit 5d381eb
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/internal/operator/double_player_rank.go b/internal/operator/double_player_rank.go
@@ -230,9 +230,13 @@ func (o *DoublePlayerRank) Output() {
 
 	header := []string{"#", "Name", "Events", "Games", "Win", "Loss", "Draw", "WR%", "ELO", "KRP", "ATSA", "ITSF"}
 	table := [][]string{}
-	for i, d := range sliceData {
+	index := 1
+	for _, d := range sliceData {
+		if !o.options.ShowInactive && d.Inactive {
+			continue
+		}
 		item := []string{
-			fmt.Sprintf("%d", i+1),
+			fmt.Sprintf("%d", index),
 			d.Name,
 			fmt.Sprintf("%d", d.EventsPlayed),
 			fmt.Sprintf("%d", d.GamesPlayed),
@@ -246,6 +250,7 @@ func (o *DoublePlayerRank) Output() {
 			fmt.Sprintf("%d", d.ITSFPoints),
 		}
 		table = append(table, item)
+		index++
 	}
 	// }}}
 

diff --git a/internal/operator/single_player_rank.go b/internal/operator/single_player_rank.go
@@ -192,12 +192,13 @@ func (o *SinglePlayerRank) Output() {
 
 	header := []string{"#", "Name", "Events", "Games", "Win", "Loss", "Draw", "WR%", "ELO", "KRP", "ATSA", "ITSF"}
 	table := [][]string{}
-	for i, d := range sliceData {
-		if o.options.ShowInactive && d.Inactive {
+	index := 1
+	for _, d := range sliceData {
+		if !o.options.ShowInactive && d.Inactive {
 			continue
 		}
 		item := []string{
-			fmt.Sprintf("%d", i+1),
+			fmt.Sprintf("%d", index),
 			d.Name,
 			fmt.Sprintf("%d", d.EventsPlayed),
 			fmt.Sprintf("%d", d.GamesPlayed),
@@ -211,6 +212,7 @@ func (o *SinglePlayerRank) Output() {
 			fmt.Sprintf("%d", d.ITSFPoints),
 		}
 		table = append(table, item)
+		index++
 	}
 	// }}}