Java BlueJ Help

Question

In the following code i need to find the round Number (yahtzee game) with a max rounds at 13 and once the limit is reached the round returns to 1 but is initialized to 0. I am having trouble where i am using the JUnit testing and the round number is suppose to be one but returns zero.
/** The Match class is the main ``controller'' of the yahtzee match. 
A Match sequence is as follows: each player gets to play their turn in
the first round,
then each player gets to play their turn in the second round, and so on
until all MAXROUNDS
(13) rounds have been played for the first game.  Then each player gets
to play the first round for the second game, and so on...
The Match class controls the following:<P>
    <UL>
    <LI> which Player plays next, and which is the current Player;
        <LI> the round number (0 through MAXROUNDS-1);
    <LI> the game  number (0 through MAXGAMES-1).
        </UL>
public class Match {
  /** The maximum number of players allowed per match. This
      value must be 5 when the class is submitted. */
  public final static int MAXPLAYERS = 5;
  /** The maximum number of games per match. This value must be 5 when
      the class is submitted. */
  public final static int MAXGAMES = 5;
  /** The maximum number of rounds per game. This value must be 13 when
      the class is submitted. */
  public final static int MAXROUNDS = 13;
  private Player[]players;
  private int currentPlayer;
  private int numPlayer;
  private int numGames;
  private int numRound;
  /** Constructor for the match.  Initialises the number of players, the
    game number, the round number, and the current player number all  to
    zero.
  public Match() {
      players = new Player[MAXPLAYERS];
      numPlayer = 0;
      numGames = 0;
      numRound = 0;   
      currentPlayer = 0;   
  /** Instance method which adds a named player into the set of players
      and adds one to the total number of players.
  public void addPlayer(String name) {
      Player details = new Player(name);
      if(numPlayer < MAXPLAYERS){
          players[numPlayer++] = details;}
  /** Instance method which returns the number of players.
      The number returned is in the range 0 to MAXPLAYERS.
      Returns zero only when no players have yet been added through
      addPlayer().
  public int getNumPlayers () {
      return numPlayer;
  /** Instance method which returns the Player who's turn it is. */
  public Player getCurrentPlayer () {
      return players[currentPlayer];
  /** Instance method which returns the specified Player.
      @param playerNum numerical identifier for the player.  Must be
      in the range 0 to n-1, where n is the number of players in the game.
  public Player getPlayer (int playerNum) {
      return players[playerNum];
  /** Instance method which advances to the next Player.  If this is the
    last Player then advance to the next round. If this is the last
        round, then advance to the next game.
  public void nextPlayer() {     
      currentPlayer++;
      if(currentPlayer == numPlayer){currentPlayer = 0;numRound++;};
  /** Instance method which returns the number of the current game.
      The number returned is 0 or greater (zero for the first game). 
      Note that the graphical user
      interface is responsible for quitting the game when the game number
      advances to MAXGAMES
  public int getGameNum () {  
      return numGames;
  /** Instance method which returns the number of the current round.
      The number returned is in the range 0 to MAXROUNDS-1. */
  public int getRoundNum() {
      int result = 0;
      if(numRound == 0){numGames++;};
      if(numRound <= MAXROUNDS-1){     
      result =  numRound;}
      return result;
}

Answer 1

If I get you right, I'd change two things:
1.) Have nextPlayer() do the round check, i.e.
if (++numRound == MAXROUNDS) {
  numRound = 0;
}2.) Have getRoundNum() just return numRound + 1 (making the 0-based variable a 1-based counter)
If there's no urge for you to have numRound 0-based, you can also make it 1-based.