这是一种很不雅观的方式来完成我想要完成的事情。很好奇什么是更好的方法

Question

基本上，它想要完成的就是找到胜率最多的球队。这是通过对每个球员的制胜球进行合计，并根据他们来自哪一支球队来判断，为该队计数来发现的。

我通过遍历所有玩家并找到唯一的队名来创建作为队对象的队列表来实现这一点。

然后我查了一下球员名单，如果球员所在的球队和现在的球队一样，就会为他们的制胜球加分。

一个小任务总共有四个for循环。看起来很恶心。

    /**
     * Returns the team with the most wins
     */
    public Team getTeamWithMostWins() {
        Team teamWithMostWins = new Team();
        List<Team> teams = new List<Team>();

        if (!players.isEmpty()) {

            // Compile the List of teams
            for (int i = 0; i < players.size(); i++) {
                if (!teams.contains(players.get(i).getTeam())) {
                    teams.add(new Team(players.get(i).getTeam()));
                }
            }

            // Set the wins for the teams
            for (int i = 0; i < players.size(); i++) {
                String team = players.get(i).getTeam();
                int winningGoals = players.get(i).getWinningGoals();

                // Go through the teams List to set the points
                for (int j = 0; j < teams.size(); j++) {
                    // If the current player's team is equal to the current team in the loop
                    if ((teams.get(j).getName()).equals(team)) {
                        teams.get(j).incrementWinsBy(winningGoals);
                    }
                }
            }

            int mostWins = teams.get(0).getWins();

            // Find the team with the most wins
            for (int i = 1; i < teams.size(); i++) {
                if (teams.get(i).getWins() > mostWins) {
                    teamWithMostWins = teams.get(i);
                }
            }

        }
        else {
            teamWithMostWins = null;
        }

        return teamWithMostWins;
    }

Answer 1

正如Jordan Denison在评论中指出的，您可以使用for-each循环。参见下面的示例。

此外，目前你将只得到最后一个队比第一个队有更多的胜利。为了得到拥有最多胜利的队伍，您必须更新最多胜利：

int mostWins = teams.get(0).getWins();

// Find the team with the most wins
for(Team team : teams) {
    if (team.getWins() > mostWins) {
        teamWithMostWins = team;
        mostWins = team.getWins(); // <--- Update the most wins every time you find a team with more wins
    }
}

此外，考虑使用其他答案中所示的地图。