Shouldn't this method return `List<Player>` instead of `Map<Player, Integer>`?

[ntalbs]

ood-interview/tic_tac_toe/tictactoe/ScoreTracker.java

Lines 26 to 32 in bce98e5

	public Map<Player, Integer> getTopPlayers() {
	return playerRatings.entrySet()
	.stream()
	.sorted(Map.Entry.comparingByValue(Comparator.reverseOrder()))
	.map(Map.Entry::getKey)
	.collect(Collectors.toMap(player -> player, player -> playerRatings.get(player)));
	}

Inside the method, it sorts the players by their ratings in descending order, then collects them into a map. As it returns HashMap, the sorted order is not preserved. I think it should collect players into a list, so it can keep the sorted order. What do you think?

[fawazbokhari]

Agreed. I have updated getTopPlayers() to return List<Player> instead of Map<Player, Integer>. This preserves the sorted order and is more semantically appropriate for a method that returns "top players" in order. All existing tests pass with this change.