# Critique my first Java Program!

• 01-25-2012, 08:41 PM
Lucid15
Critique my first Java Program!
Our first assignment in my computer science was to make the game Pig. It was the very first thing we did in the class and i was hoping to get some feedback on the program i wrote for it. For those of you who dont know how the game works:
The game of Pig is a very simple jeopardy dice game in which two players race to reach 100 points. Each turn, a player repeatedly rolls a die until either a 1 is rolled or the player holds and scores the sum of the rolls (i.e. the turn total). At any time during a player's turn, the player is faced with two decisions:

roll - If the player rolls a
1: the player scores nothing and it becomes the opponent's turn.
2 - 6: the number is added to the player's turn total and the player's turn continues.
hold - The turn total is added to the player's score and it becomes the opponent's turn.
Also, i should note our professor has wrote a quick shortcut code library, so you may notice that in there, specifically for printLine etc..
Code:

```public class Pig extends ljing.Program { public static void main(String[] args) {                 new Pig().run(); } public void run() {                 int PlayerOne = 0;                 int PlayerTwo = 0;                 boolean player1turn = true;                 java.util.Random die = new java.util.Random();                 int turnPoints = 0;                 boolean player2turn = false;                 // need to modify keep going game condition                 while (PlayerOne <=100 && PlayerTwo <= 100){                 while (player1turn) {                         int roll = die.nextInt(6) + 1;                         if (roll == 1) {                                 System.out.println("Sorry P0, you rolled a " + roll);                                 System.out.println("Score: " + PlayerOne + "- " + PlayerTwo);                                 turnPoints = 0;                                 player2turn = true;                                 player1turn = false;                                 break;                         } else {                                 System.out.println("P0,You rolled a " + roll);                                 turnPoints += roll;                                 printLine("You have " + turnPoints + " points this turn");                                 if (readBoolean("keep going?"))                                 {                                         continue;                                 }                                                                         else {                                                 PlayerOne += turnPoints;                                                 printLine("Score: " + PlayerOne + "- " + PlayerTwo);                                                 player1turn = false;                                                 player2turn = true;                                                 turnPoints = 0;                                                 if (PlayerOne >= 100){                                                         printLine("Congratulations P0! You win!");                                                         System.exit(0);                                                                                                                 }                                                 break;                                                                                                 }                                         }                                 }                                                         while (player2turn) {                         int roll = die.nextInt(6) + 1;                         if (roll == 1) {                                 System.out.println("Sorry P1, you rolled a " + roll);                                 System.out.println("Score: " + PlayerOne + "- " + PlayerTwo);                                 turnPoints = 0;                                 player1turn = true;                                 player2turn = false;                                 break;                         } else {                                 System.out.println("P1,You rolled a " + roll);                                 turnPoints += roll;                                 printLine("You have " + turnPoints + " points this turn");                                 if (readBoolean("Keep Going?"))                                 {                                         continue;                                 }                                 {                                         if (PlayerTwo >= 100) {                                                 printLine("Congratulations, player1! You Win! Exiting...");                                                 System.exit(0);                                         } else {                                                 // if no roll                                                 PlayerTwo += turnPoints;                                                 printLine("Score: " + PlayerOne + "- " + PlayerTwo);                                                 player1turn = true;                                                 player2turn = false;                                                 turnPoints = 0;                                                 if (PlayerTwo >= 100) {                                                         printLine("Congratulations P1! You win!");                                                         System.exit(0);                                                                                                                 }                                                 break;                                                                                         }                                 }                         }                 } } } }```
Thank you for all your feedback!
• 01-25-2012, 10:14 PM
DarrylBurke
Re: Critique my first Java Program!
• 01-26-2012, 08:07 AM
Lucid15
Re: Critique my first Java Program!
Wait what is db? Sorry newb here
• 01-26-2012, 09:56 AM
DarrylBurke
Re: Critique my first Java Program!
My initials

db