Thread: Simulation for Grid Game?

I am trying to simulate a grid game using java (for loops, if/else). The game is played as follows: The player draws a card that has one of four colors (Red, green, blue, yellow). Then, the player rolls two dice and finds the product of the two numbers. The player tries to find the number on a grid with the multiples of numbers one through 6. Each number correlates with a color, and if the number the player gets matches the color you win. If not, you lose.

Here is the code I have attempted so far but I am getting errors I don't know how to fix, please help!

public class ColorGrid
{
public static void main(String args[])
{
//Pick a Color
int color=(int)(Math.random()*4)+1;
//Product of 2 Dice
int a=(int)(Math.random()*6)+1;
int b=(int)(Math.random()*6)+1;
int product=(a*b);
 
int trial=1;
int win=0;
int lose=0;
//Red
for(color=1,trial<=500,trial++);
{
	if(product==2,6,15,24)
	{
	win++;
	}
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	{
	lose++;
	}
}

Errors:

ColorGrid.java:16: error: not a statement
for(color=1,trial<=500,trial++);
                 ^
ColorGrid.java:16: error: ';' expected
for(color=1,trial<=500,trial++);
                              ^
ColorGrid.java:17: error: illegal start of expression
{
^
ColorGrid.java:18: error: ')' expected
	if(product==2,6,15,24)
	             ^
ColorGrid.java:18: error: ';' expected
	if(product==2,6,15,24)
	              ^
ColorGrid.java:18: error: illegal start of expression
	if(product==2,6,15,24)
	               ^
ColorGrid.java:18: error: ';' expected
	if(product==2,6,15,24)
	                ^
ColorGrid.java:18: error: illegal start of expression
	if(product==2,6,15,24)
	                  ^
ColorGrid.java:18: error: ';' expected
	if(product==2,6,15,24)
	                   ^
ColorGrid.java:18: error: illegal start of expression
	if(product==2,6,15,24)
	                     ^
ColorGrid.java:18: error: ';' expected
	if(product==2,6,15,24)
	                      ^
ColorGrid.java:22: error: illegal start of type
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	^
ColorGrid.java:22: error: <identifier> expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	            ^
ColorGrid.java:22: error: ';' expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	              ^
ColorGrid.java:22: error: illegal start of type
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	               ^
ColorGrid.java:22: error: <identifier> expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                ^
ColorGrid.java:22: error: ';' expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                 ^
ColorGrid.java:22: error: illegal start of type
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                   ^
ColorGrid.java:22: error: ';' expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                           ^
ColorGrid.java:22: error: <identifier> expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                          ^
ColorGrid.java:22: error: illegal start of type
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                            ^
ColorGrid.java:22: error: <identifier> expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                             ^
ColorGrid.java:22: error: ';' expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                               ^
ColorGrid.java:22: error: illegal start of type
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                                ^
ColorGrid.java:22: error: <identifier> expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                                  ^
ColorGrid.java:22: error: ';' expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                                   ^
ColorGrid.java:22: error: illegal start of type
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                                          ^
ColorGrid.java:22: error: <identifier> expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                                            ^
ColorGrid.java:22: error: ';' expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                                             ^
ColorGrid.java:22: error: illegal start of type
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                                               ^
ColorGrid.java:22: error: <identifier> expected
	else(product==!2)&&(product==!6)&&(product==!15)&&(product==!24)
	                                                                ^
ColorGrid.java:23: error: ';' expected
	{
	 ^
ColorGrid.java:24: error: illegal start of type
	lose++;
	    ^
ColorGrid.java:24: error: <identifier> expected
	lose++;
	      ^
ColorGrid.java:24: error: ';' expected
	lose++;
	       ^
ColorGrid.java:30: error: class, interface, or enum expected
}
^

I added it to my post, sorry about that.

in the for statement you need to separate with semicolons. for (color = 1; trial <=500; trial++) and not put a semicolon after the parentheses. In your if statements, you want to write != not ==!(int). Also in your compound else statement. Else is used to be done as long as the if statement is not used. It doesn't require any logic checks. If you want a logic check, you would need to use else if. And in your if statement, you cannot have commas like that to check multiple values.

for (color = 1; trial <= 500; trial++)
 
if ((product == 2) && (product == ... etc) // notice the double opening parentheses. This makes the compound if statement
 
else lose++ // or
 
else if ((product != 2) && (product != 4)) //

So, is it okay to have an if-else statement nested within a for loop?

Ok, now it is compiling but it is not completing 500 trials like I want it to, instead it is only completing one.

public class ColorGrid
{
public static void main(String args[])
{
//Pick a Color
int color=(int)(Math.random()*4)+1;
//Product of 2 Dice
int a=(int)(Math.random()*6)+1;
int b=(int)(Math.random()*6)+1;
int product=(a*b);
 
int trial=1;
int win=0;
int lose=0;
//Red
for(trial=1; trial<=500; trial++);
{
if(color==1)
	if((product==2)||(product==6)||(product==15)||(product==24))
	{
	win++;
	}
	else lose++;
//Blue
else if(color==2)
	if((product==4)||(product==12)||(product==20)||(product==36))
	{
	win++;
	}
	else lose++;
//Green
else if(color==3)
	if((product==3)||(product==10)||(product==16)||(product==30))
	{
	win++;
	}
	else lose++;
//Yellow
else if(color==4)
	if((product==1)||(product==5)||(product==9)||(product==9)||(product==18)||(product==25))
	{
	win++;
	}
	else lose++;
}
System.out.println("Number of Wins: "+win);
System.out.println("Number of Losses: "+lose);
 
}
}

Is it something with the for loop that is incorrect?

You didn't remove the semicolon from the end of the for loop. for (trial = 1; trial<=500;trial++) no semicolon. Other then that, you are sending the same numbers through the for loop, so even if it ran 500 times you would get either 500 wins or 500 losses.

That should not be the case, but I don't really know how to fix that. I should be getting a combination of wins and losses?

Norm, at first the output only showed:
Number of Wins: 0
Number of Losses: 1

Now, after removing the semi-colon, it is:

Number of Wins: 0
Number of Losses: 500

or vice versa.

But I need it to be a combination of wins and losses. What am I doing wrong?

You are assigning the values before the loop. Then going through the loop, with those assigned values.

How can I make it reassign the values each time it goes through the loop?

By removing the "How can" and the "?" from your last sentence.

Sorry hehe silly question! Thank you so much.