Craps program win or lose?

[Ryan Gordon]

Hello, I am required to write a program that will play craps 3 times, I need a method that will ask the user to play the game and when the games are completed display how many times the play won and lost.
The variables I have to use are  boolean start, int die1, int die2, int sum, int valuePoint, and String status.
The methods that I have to use are public void play(), public void play(int die1, int die2), and public void rollDice().

I am having issues doing this, here is what I have so far.

import java.util.Scanner;
public final class RandomSumGame 
	extends Object
	{
		boolean start;
		private int die1;
		private int die2;
		int sum;
		int valuePoint;
		String status;
		
		public RandomSumGame() {
		super();
	}
	
	//method to start the game
	public final void play() {
			boolean start = true;
			int gamesPlayed = 0;
			
			while(gamesPlayed == 0)
			{
				System.out.println("Would you like to play craps? [y/n]? ");
				Scanner scan = new Scanner(System.in);
				String status = scan.nextLine();
				
				if (status.equalsIgnoreCase("y"))
				{
					gamesPlayed++;
				}
				else if (status.equalsIgnoreCase("n"))
				{
				start = false;
				gamesPlayed++;
				}
				else
				{
					System.out.println(" wrong input, please enter only 'y' or 'n' ");
				}
			}
		}
		
		//method to roll the dice
		public final void rollDice () {
			
			int die1 = (int)(Math.random()*6) + 1;
			int die2 = (int)(Math.random()*6) + 1;
			int sum = die1 + die2;
			System.out.println("Your dice roll is: " + sum);
		}
		
		//method to establish points and if user wins
		public final void play( int die1, int die2 )		{
			
			System.out.println("test");
			int sum = 0;
			if (sum == 2 || sum == 3 || sum == 12) {
				System.out.println("You lose ");
				if(sum == 7 || sum == 11) 
					System.out.println("You win ");
				if (sum == 4 || sum == 5 || sum == 6 || sum == 8 || sum == 9 || sum == 10)
					sum = valuePoint;
				if ( sum == valuePoint) 
					System.out.println("you win");
			}
		}
		
}

and

public final class RandomSumGameTest {
	public static void main(String[] args) {
		
		int die1 = (int)(Math.random()*6) + 1;
		int die2 = (int)(Math.random()*6) + 1;
		int sum = die1 + die2;
		
		RandomSumGame call = new RandomSumGame();
		RandomSumGame call2 = new RandomSumGame();
		RandomSumGame call3 = new RandomSumGame();
	
		call.play();
	
		call2.rollDice();
	
		call3.play(die1,die2);
	
	}
}

this is what the question looks like

4. [20 marks] Craps is a dice game where two dice are rolled. Each die has six faces representing values 1, 2, 3, 4, 5, or 6.

I. If the sum is 2, 3, or 12 (called craps), you lose;
II. If the sum is 7 or 11 (called natural), you win;
III. If the sum is any other value (4, 5, 6, 8, 9, or 10), a value point is established, and you continue to roll until you either roll a sum of the value point or a 7. If the sum of the new roll is equal to the value point, then you win; if the sum of the new roll is equal to 7, then you lose.
Remember, in option (III), you continue to roll until you get a 7 or the value point.
Sample runs:
• You rolled 5 + 6 = 11; you win
• You rolled 1 + 2 = 3; you lose
• You rolled 2 + 2 = 4; you establish the value point 4;
– Roll again 2 + 3 = 5; roll
– Roll again 2 + 1 = 3; roll
– Roll again 2 + 2 = 4; you win
• You rolled 2 + 6 = 8; you establish the value point 8;
– Roll again 4 + 4 = 8; you win
• You rolled 3 + 2 = 5; you establish the value point 5;
– Roll again 1 + 1 = 2; roll
– Roll again 2 + 2 = 4; roll
– Roll again 1 + 1 = 2; roll
– Roll again 3 + 4 = 7; you lose
Develop a program that plays craps with a player three times. At the end, the program prints the number of times the player won and the number of times the player lost.

There is a picture attached

[Dave Tolls]

What exact issues are you having?

[Ryan Gordon]

I am having trouble making the program tell me whether the user won or lost and how many times.

[Ryan Gordon]

I would like to make a method that will ask the user to play, then play a game, count the game and output the results. I get the concepts of counters but cant seem to get it going the way I need to...

[Carey Brown]

You don't need to say "extends Object", that is implied.

You have several member variables (aka "fields") such as "die1" that are hidden in your methods by local variables of the same name.

You increment gamesPlayed if the user enters "n".i

You should only create one Scanner for System.in and reuse it. This is done by declaring a static member variable just inside you class.

private static final INPUT = new Scanner( System.in );

You are using the old style Math.random(). Use the Random class instead, it is less error prone and usually more efficient.

Random rand = new Random();
...
die1 = rand.nextIn(6) + 1;

[Ryan Gordon]

I was doing one of the tutorials on the coderanch website that's where I got the extends object from I forgot to remove it, thank you I will work on this an see what I come up with

[Ryan Gordon]

Hello, I have a working program, but it does not meet the specifications. I added games played, loss count and win count. I need to make this program without those variables. I need to make use of boolean start and string status but not sure how to go about that, any suggestions?

import java.util.Random;
public class RandomSumgame1 {
	
	boolean start;
	int d1;
	int d2;
	int sum;
	int valuePloint;
	String status;
	int gamesPlayed = 3;
	int lossCount;
	int winCount;
	
	//method to roll the dice
	public static int rollDice() {
	Random random=new Random();
	return random.nextInt(6)+1;
	}//end method 1
	
	//method for playing game
	public void play() {
		//loop through games played
		for ( int i = 0; i < gamesPlayed; i++) {
			//set x and y equal to the result of each dice roll
			int x = rollDice();
			int y = rollDice();
			System.out.print("You rolled "+ x + "+" + y + " = "+(x+y)+";");
			int sum = x + y;
				//if user rolls a 2,3 or 12 they lose
				if(sum==2 || sum ==3 ||sum ==12) {
				lossCount++;
				System.out.println("you lose");
				//if user rolls a 7 or 11 they win
				}else if(sum==7 || sum==11) {
					winCount++;
					System.out.println("you win");
				}else {
					//if user rolls 4,5,6,8,9 or 10 they establish a value point
					System.out.println("your value point is "+sum+";");
			int valuePoint=sum;

				while(true) {
					System.out.print("Roll again ");
					x = rollDice();
					y = rollDice();
					System.out.print(x+"+"+y+" = "+(x+y)+";");
					sum =  x + y;
	//if user rolls a sum that is equal to the value point they win
					if(sum==valuePoint) {
					winCount++;
					System.out.println("you win");
					break;
					//if user rolls a 7 they lose
					}else if(sum==7) {
					lossCount++;
					System.out.println("you lose");
					break;
					
					}else {
					System.out.println("roll");
					continue;
					}
				}
			}
		}
		System.out.println("You won "+winCount + " times");
		System.out.println("You lost "+lossCount + " times");
		}//end method 2
	
	public void play(int d1, int d2) {
		
	}
}//end class

[Campbell Ritchie]

Not happy, I am afraid.
All your fields should have private access.
Why are you using a String for status? Are you using it at all in your latest version? Why was it a field rather than a local variable in the earlier versions? Does status mean win/lose? If so, ask your teacher why it has to be a String rather than an enum.
Your code isn't correctly indented; it is difficult to see where blocks start and finish. I know you often see code after a } but I think it is better always to push the enter key after a }. If you set up automatic indentation on your text editor that will help no end.
Don't create new Random objects. Use one Random object per application unless you have performance problems.
Comments like that in line 24 are useless.
Why have you got those local variables in lines 25‑26? Where are you using the two fields, if at all?
Why have you got while(true) in line 42? Can't you find a boolean expression to control that loop? Two break;s and a continue; in the same loop!
The method you are in there is much too long.

[Ryan Gordon]

I forgot about those fields, I corrected them.  It is supposed to be used somehow to test win/loss. I can ask why it has to be a String but I will still have to use String. I will fix my indentation. I guess instead of x and y I could have used d1 and d2. I didn't think of a better way to write line 42, I am struggling and really need to improve on my skill. I had a feeling that method was too long...
I'm just having trouble because its hard to write this program because I cant think of how to use boolean start and String status...