Please help, java program terminating unexpectedly without reason

Question

ok, so I have a project I'm working on, here's its description:
Create a new project named FML_Pig where F is your first initial, M is your middle initial, and L is your last initial. For example, if your name is Alfred Edward Neuman then you would name your project AEN_Pig. If necessary, add a new Java file to this project named FML_Pig.
Add a java file to the project named FML_Dice.
Design and implement a Dice class as follows. The Dice class has 2 twenty‑sided dice with numbers on the faces (1 � 20) and 2 twenty-six-sided dice with letters on the faces (a � z).
import java.util.Random;
public class FML_Dice
private static Random gen = new Random();
private final static int NUM_SIDES = 20;
private int die1, die2, numSnakeEyes, numVowels, totalPoints;
private char die3, die4;
Part 1
Add a rollDice() method to your FML_Dice class that sets die1 and die2 to a random number between 1 and 20 and it sets die3 and die4 to a random character between �a� and �z�.
Helpful Hint:
dice3 = (char)(�a� + gen.nextInt(26)); // sets dice3 to a random char between �a� and �z�.
Part 2
Write a constructor that sets die1 and die2 to a random number between 1 and 20 and it sets die3 and die4 to a random character. Your constructor can simply call rollDice() to do this. It should also initialize numSnakeEyes, numVowels, and totalPoints to 0 (after you roll the dice).
Part 3
Write methods getDie1(), getDie2(), getDie3(), and getDie4() that returns the value of the respective die. Write a method numSnakes() that returns the number of snake eyes that have been rolled. Write a method numVowels() that returns the number of vowels that have been rolled. Write a method totalPts() that returns the totalPoints.
Part 4
Write a toString() method that overwrites the default toString() method. It should return a String representation of the value of all 4 die. For example, 17 2 h w.
Part 5
Write a method updateTotals that updates numVowels, numSnakeEyes, and totalPoints based on the current values of die1, die2, die3, and die4. numVowels should be incremented by 1 if either die3 or die4 are a vowel. However, if both die3 and die4 are vowels, the totalPoints should be reset to 0 and numVowels should be reset to 0 also. numSnakeEyes should be incremented by 1 if either die1 or die2 have a face value of 1 (this is not truly a snake eyes, but it gives better odds for the game). totalPoints should be incremented by the sum of die1 and die2 multiplied by (numSnakeEyes + 1).
Part 6
Write a main program (use the one in FML_Pig.java) that plays a game of Pig. In this game, two people play against each other. Player1 begins by rolling the dice. The total of die1 and die2 is added to his total score. Player1�s turn continues until he/she rolls a total of 4 vowels then the turn switches to Player2. Whenever a turn switches to the other player, the number of vowels is reset to zero. Also, if at any time both die3 and die4 are vowels, the total score is reset to 0 and the turn switches to the other player. (So the turn switches to the other player whenever the total number of vowels reaches four or there are two vowels rolled at the same time.)
If a player rolls snakeeyes (for our game, snakeeyes occurs whenever either of the dice have a face value of one � in reality, both die1 and die2 should have a face value of one but then snakeeyes would occur only once every 400 rolls), then his point values are doubled from that point on (all future rolls for the rest of the game, point values for this player are doubled). When a player rolls snake eyes again, point values are tripled for that roll and all future rolls. (Three snake eyes, quadrupled, etc.)
(continued on the next page)
Be sure to display the results of the roll for each turn.
First player to get to 2000 points wins the game and gets to oink like a pig.
Note: Both players need their own set of Dice since the Dice keeps track of the totalPoints, the number of snake eyes rolled so far, and the number of vowels rolled so far for that particular player. You can do this simply by declaring it that way in the main program:
Dice player1 = new Dice();
Dice player2 = new Dice();
You may add additional methods and instance variables to the class as needed. For example, I would probably write a private helper method isVowel() that is passed a char argument ch and returns true if ch is a vowel.
Also, you can have the computer play for both player 1 and player 2. Simply loop it until somebody wins. Print out the result for each turn including who rolled the dice (player 1 or player 2), what they rolled, how many snakeeyes do they have, how many points did they get for this turn, and how many total points do they have.
When you are completely finished, hand in just the java files (FLM_Dice & FLM_Pig). If you happened to write any other classes, make sure they are named with the FML format, and turn these in as well. I do not need EasyReader or p.
Oink! Oink!
This pig won the game --------->
so here's my code:
this isn't my code but is required to compile my code: // package com.skylit.io;
import java.io.*;
*  @author Gary Litvin
*  @version 1.2, 5/30/02
*  Written as part of
*  <i>Java Methods: An Introduction to Object-Oriented Programming</i>
*  (Skylight Publishing 2001, ISBN 0-9654853-7-4)
*   and
*  <i>Java Methods AB: Data Structures</i>
*  (Skylight Publishing 2003, ISBN 0-9654853-1-5)
*  EasyReader provides simple methods for reading the console and
*  for opening and reading text files.  All exceptions are handled
*  inside the class and are hidden from the user.
*  <xmp>
*  Example:
*  =======
*  EasyReader console = new EasyReader();
*  System.out.print("Enter input file name: ");
*  String fileName = console.readLine();
*  EasyReader inFile = new EasyReader(fileName);
*  if (inFile.bad())
*    System.err.println("Can't open " + fileName);
*    System.exit(1);
*  String firstLine = inFile.readLine();
*  if (!inFile.eof())   // or:  if (firstLine != null)
*    System.out.println("The first line is : " + firstLine);
*  System.out.print("Enter the maximum number of integers to read: ");
*  int maxCount = console.readInt();
*  int k, count = 0;
*  while (count < maxCount && !inFile.eof())
*    k = inFile.readInt();
*    if (!inFile.eof())
*      // process or store this number
*      count++;
*  inFile.close();    // optional
*  System.out.println(count + " numbers read");
*  </xmp>
public class EasyReader
  protected String myFileName;
  protected BufferedReader myInFile;
  protected int myErrorFlags = 0;
  protected static final int OPENERROR = 0x0001;
  protected static final int CLOSEERROR = 0x0002;
  protected static final int READERROR = 0x0004;
  protected static final int EOF = 0x0100;
   *  Constructor.  Prepares console (System.in) for reading
  public EasyReader()
    myFileName = null;
    myErrorFlags = 0;
    myInFile = new BufferedReader(
                            new InputStreamReader(System.in), 128);
   *  Constructor.  opens a file for reading
   *  @param fileName the name or pathname of the file
  public EasyReader(String fileName)
    myFileName = fileName;
    myErrorFlags = 0;
    try
      myInFile = new BufferedReader(new FileReader(fileName), 1024);
    catch (FileNotFoundException e)
      myErrorFlags |= OPENERROR;
      myFileName = null;
   *  Closes the file
  public void close()
    if (myFileName == null)
      return;
    try
      myInFile.close();
    catch (IOException e)
      System.err.println("Error closing " + myFileName + "\n");
      myErrorFlags |= CLOSEERROR;
   *  Checks the status of the file
   *  @return true if en error occurred opening or reading the file,
   *  false otherwise
  public boolean bad()
    return myErrorFlags != 0;
   *  Checks the EOF status of the file
   *  @return true if EOF was encountered in the previous read
   *  operation, false otherwise
  public boolean eof()
    return (myErrorFlags & EOF) != 0;
  private boolean ready() throws IOException
    return myFileName == null || myInFile.ready();
   *  Reads the next character from a file (any character including
   *  a space or a newline character).
   *  @return character read or <code>null</code> character
   *  (Unicode 0) if trying to read beyond the EOF
  public char readChar()
    char ch = '\u0000';
    try
      if (ready())
         ch = (char)myInFile.read();
    catch (IOException e)
      if (myFileName != null)
        System.err.println("Error reading " + myFileName + "\n");
      myErrorFlags |= READERROR;
    if (ch == '\u0000')
      myErrorFlags |= EOF;
    return ch;
   *  Reads from the current position in the file up to and including
   *  the next newline character.  The newline character is thrown away
   *  @return the read string (excluding the newline character) or
   *  null if trying to read beyond the EOF
  public String readLine()
    String s = null;
    try
      s = myInFile.readLine();
    catch (IOException e)
      if (myFileName != null)
        System.err.println("Error reading " + myFileName + "\n");
      myErrorFlags |= READERROR;
    if (s == null)
      myErrorFlags |= EOF;
    return s;
   *  Skips whitespace and reads the next word (a string of consecutive
   *  non-whitespace characters (up to but excluding the next space,
   *  newline, etc.)
   *  @return the read string or null if trying to read beyond the EOF
  public String readWord()
    StringBuffer buffer = new StringBuffer(128);
    char ch = ' ';
    int count = 0;
    String s = null;
    try
      while (ready() && Character.isWhitespace(ch))
        ch = (char)myInFile.read();
      while (ready() && !Character.isWhitespace(ch))
        count++;
        buffer.append(ch);
        myInFile.mark(1);
        ch = (char)myInFile.read();
      if (count > 0)
        myInFile.reset();
        s = buffer.toString();
      else
        myErrorFlags |= EOF;
    catch (IOException e)
      if (myFileName != null)
        System.err.println("Error reading " + myFileName + "\n");
      myErrorFlags |= READERROR;
    return s;
   *  Reads the next integer (without validating its format)
   *  @return the integer read or 0 if trying to read beyond the EOF
  public int readInt()
    String s = readWord();
    if (s != null)
      return Integer.parseInt(s);
    else
      return 0;
   *  Reads the next double (without validating its format)
   *  @return the number read or 0 if trying to read beyond the EOF
  public double readDouble()
    String s = readWord();
    if (s != null)
      return Double.parseDouble(s);
      // in Java 1, use: return Double.valueOf(s).doubleValue();
    else
      return 0.0;
}same with this:
public class p
     public static void l(String S)
      { System.out.println(S);}
     public static void o(String S)
      {System.out.print(S);}
     public static void l(int i)
      { System.out.println(i);}
     public static void o(int i)
      {System.out.print(i);}
     public static void l(boolean b)
      { System.out.println(b);}
     public static void o(boolean b)
      {System.out.print(b);}
    public static void l(char c)
      { System.out.println(c);}
     public static void o(char c)
      {System.out.print(c);}     
     public static void l(double d)
      { System.out.println(d);}
     public static void o(double d)
      {System.out.print(d);} 
    public static void l(Object obj)
      { System.out.println(obj.toString());}
     public static void o(Object obj)
      {System.out.print(obj.toString());}
     public static void l()
      {System.out.println();}
}       here's my code:
import java.util.*;
public class JMM_Pig
     public static void main(String[] args)
          int winner=0;
          int pts=0;
               JMM_Dice player1=new JMM_Dice();
               JMM_Dice player2=new JMM_Dice();
               p.l("Player 1 rolls the dice...");
               player1.rollDice();
               player1.updateTotals();
               boolean loop=true;
               while(loop)
               while(player1.numVowels()<=4)
                    p.l("Player 1 continues his turn...");
                    player1.rollDice();
                    player1.updateTotals();
                    p.l("Player 1 rolled a "+player1.getDie1()+" and a "+player1.getDie2()+" and a '"+player1.getDie3()+"' and a '"+player1.getDie4()+"'");
                    p.l("Player 1 rolled "+player1.currvowels()+" vowels.");
                    p.l("Player 1 has "+player1.totalvowels()+" total vowels.");
                    //if(player1.getMagic()==1)
                    //player1.totalPoints=(player1.totalPoints+player1.getDie1()+player1.getDie2())*player1.mult;
                    //pts=(player1.totalPts()+player1.getDie1()+player1.getDie2())*player1.getMult();
                    //player1.setTpts(pts);
                    //if(player1.getMagic()!=1)
                         //player1.totalPoints=player1.totalPoints+player1.getDie1()+player1.getDie2();
                    //pts=player1.totalPts()+player1.getDie1()+player1.getDie2();
                    //player1.setTpts(pts);
                    p.l("Player 1 has "+player1.numSnakes()+" snake eyes.");
                    p.l("Player 1 earned "+player1.ptsreturn()+" points this turn.");
                    p.l("Player 1 has "+player1.totalPts()+" total points.");
                    pts=0;
                    if(player1.totalPts()>=2000)
                         winner=1;
                         player1.setNv(4);loop=false;
                    if((player1.isavowel(player1.getadice()))&&player1.isavowel(player1.getadice2()))
                         player1.setTpts(0);player1.setNv(0);player1.setNv(4);
               p.l("Player 1's turn has ended...");
               player1.setNv(0);
               p.l("Player 2 rolls the dice...");
               player2.rollDice();
               player2.setTpts(player2.totalPts()+player2.getDie1()+player2.getDie2());
               while(player2.numVowels()<=4)
                    p.l("Player 2 continues his turn...");
                    player2.rollDice();
                    player2.updateTotals();
                    p.l("Player 2 rolled a "+player2.getDie1()+" and a "+player2.getDie2()+" and a '"+player2.getDie3()+"' and a '"+player2.getDie4()+"'");
                    p.l("Player 2 rolled "+player2.currvowels()+" vowels.");
                    p.l("Player 2 has "+player2.totalvowels()+" total vowels.");
                    //if(player1.getMagic()==1)
                    //player1.totalPoints=(player1.totalPoints+player1.getDie1()+player1.getDie2())*player1.mult;
                    //pts=(player1.totalPts()+player1.getDie1()+player1.getDie2())*player1.getMult();
                    //player1.setTpts(pts);
                    //if(player1.getMagic()!=1)
                         //player1.totalPoints=player1.totalPoints+player1.getDie1()+player1.getDie2();
                    //pts=player1.totalPts()+player1.getDie1()+player1.getDie2();
                    //player1.setTpts(pts);
                    p.l("Player 2 has "+player2.numSnakes()+" snake eyes.");
                    p.l("Player 2 earned "+player2.ptsreturn()+" points this turn.");
                    p.l("Player 2 has "+player2.totalPts()+" total points.");
                    pts=0;
                    if(player2.totalPts()>=2000)
                         winner=2;
                         player2.setNv(4);loop=false;
                    if((player2.isavowel(player2.getadice()))&&player2.isavowel(player2.getadice2()))
                         player2.setTpts(0);player2.setNv(0);player2.setNv(4);
               p.l("Player 2's turn has ended...");
               player2.setNv(0);
               if(player1.totalPts()>=2000)
                    winner=1;
                    loop=false;
               if(player2.totalPts()>=2000)
                    winner=2;
                    loop=false;
               }  //main loop
               if(winner==1)
                    p.l("It ended with the following statistics...");
                    p.l("Player 1's score was: "+player1.totalPts()+" and Player 2's score was: "+player2.totalPts());
                    p.l("Player 1 had: "+player1.numSnakes()+" snake eye(s) and Player 2 had: "+player2.numSnakes()+" snake eye(s)");
                    p.l("Player 1 wins, oink oink!");
               if(winner==2)
                    p.l("It ended with the following statistics...");
                    p.l("Player 1's score was: "+player1.totalPts()+" and Player 2's score was: "+player2.totalPts());
                    p.l("Player 1 had: "+player1.numSnakes()+" snake eye(s) and Player 2 had: "+player2.numSnakes()+" snake eye(s)");
                    p.l("Player 2 wins, oink oink!");
and dice class:
import java.util.*;
public class JMM_Dice {
     private static Random gen = new Random();
    private final static int NUM_SIDES = 21;
    private int die1, die2, numSnakeEyes, numVowels, totalPoints;
    private char die3, die4;
    private int dice=0;
       //dice=
       private int dice2=0;
       //dice2=;
       private String adice="";
       //adice=;
       private String adice2="";
       //adice2;
       private int magic=0;
       private int mult=0;
       private int currvowels=0;
       private int totalvowels=0;
  public static EasyReader key = new EasyReader();
  public JMM_Dice()
      dice=0;dice2=0;
       String alpha="abcdefghijklmnopqrstuvwxyz";
      adice="";adice2="";
       rollDice();
       numSnakeEyes=0;numVowels=0;totalPoints=0;
  public void setTpts(int tpts)
       this.totalPoints=tpts;
  public void setNv(int nv)
       this.numVowels=nv;
  public int getDie1()
       return dice;
  public int getDie2()
       return dice2;
  public String getDie3()
       return adice;
  public String getDie4()
       return adice2;
  public int numSnakes()
       return numSnakeEyes;
  public void setSnakes(int s)
       this.numSnakeEyes=s;
  public String getadice()
       return this.adice;
  public String getadice2()
       return this.adice2;
  public int numVowels()
       return numVowels;
  public int getMagic()
       return this.magic;
  public int getNv()
       return this.numVowels;
  public void setMagic(int mag)
       this.magic=mag;
  public int getMult()
       return this.mult;
  public void setMult(int m)
       this.mult=m;
  public int totalPts()
       return totalPoints;
  public String toString()
       return dice+""+dice2+""+adice+""+adice2;
  public boolean isavowel(String str)
       if(str.equals("a")||str.equals("e")||str.equals("i")||str.equals("o")||str.equals("u"))
            return true;
       else
            return false;
  public int ptsreturn()
       return dice+dice2;
  public int currvowels()
       int tmp=currvowels;
       currvowels=0;
       return tmp;
  public int totalvowels()
       return numVowels;
  public void updateTotals()
       if(adice.equalsIgnoreCase("a")||adice.equalsIgnoreCase("e")||adice.equalsIgnoreCase("i")||adice.equalsIgnoreCase("o")||adice.equalsIgnoreCase("u")||adice2.equalsIgnoreCase("a")||adice2.equalsIgnoreCase("e")||adice2.equalsIgnoreCase("i")||adice2.equalsIgnoreCase("o")||adice2.equalsIgnoreCase("u"))
            numVowels++;currvowels++;
       if((isavowel(adice))&&isavowel(adice2))
            totalPoints=0;numVowels=0;
       if(dice==1||dice2==1)
            numSnakeEyes++;
       int fd=dice;
       int sd=dice2;
       int sum=fd+sd;
       totalPoints+=sum*(numSnakeEyes+1);
  public void rollDice()
       dice=gen.nextInt(20)+1;
       dice2=gen.nextInt(20)+1;
       String alpha="abcdefghijklmnopqrstuvwxyz";
       adice=String.valueOf(alpha.charAt(gen.nextInt(26)));
       adice2=String.valueOf(alpha.charAt(gen.nextInt(26)));
  public static void sleep (int wait)
  {     long timeToQuit = System.currentTimeMillis() + wait;
       while (System.currentTimeMillis() < timeToQuit)
            ;   // take no action
}the program works fine except it's supposed to terminate when one of the two AIs get a score of 2000, right now it terminates no matter what score the AIs get and no matter what I've tried to do it keeps doing that...can someone please tell me why it's terminating so oddly? Thanks! :)

Answer 1

Here's how my code works in a nutshell, the main program starts with the boolean loop=true;
               while(loop)
               {then comes the loops for the two AIs, first the player 1 AI goes with this loop: while(player1.numVowels()<=4)
               {there the player 1 roles the dice and it keeps going until player 1 gets a total of 2000 points at which time this is supposed to execute: if(player1.totalPts()>=2000)
                         winner=1;
                         player1.setNv(4);loop=false;
                    }, player1.setNv(4); sets numVowels to 4 so that the inner loop exits, loop=false exits the other loop and winner=1; specifies that player 1 won which is used outside of the loops here: if(winner==1)
                    p.l("It ended with the following statistics...");
                    p.l("Player 1's score was: "+player1.totalPts()+" and Player 2's score was: "+player2.totalPts());
                    p.l("Player 1 had: "+player1.numSnakes()+" snake eye(s) and Player 2 had: "+player2.numSnakes()+" snake eye(s)");
                    p.l("Player 1 wins, oink oink!");
               if(winner==2)
                    p.l("It ended with the following statistics...");
                    p.l("Player 1's score was: "+player1.totalPts()+" and Player 2's score was: "+player2.totalPts());
                    p.l("Player 1 had: "+player1.numSnakes()+" snake eye(s) and Player 2 had: "+player2.numSnakes()+" snake eye(s)");
                    p.l("Player 2 wins, oink oink!");
               } the same thing happens for player 2 if player 1 didn't already get 2000 points...but the if statement despite everything pointing to the variables containing the right values don't seem to be working, as shown by the example program output I posted it just ends at any random number...hopefully this helps make figuring out what's wrong easier :) ...so can anyone please help me out? thanks! :)