Thread: Help me

I tried to create a box(3*3) and addition of horizontal,vertical and diagonal value should be equal to 15.
But Random class generate 0 and minus values and the program does not work as I expected.Please help me to correct my program.

import java.util.*;
class mb {
	public static void main(String args[]){
		int ran[][]=new int[3][3];
		boolean H[]=new boolean[3];
		boolean V[]=new boolean[3];
		boolean HV[]=new boolean[2];
		final int size =3;
		Random r=new Random();
		boolean Equal;		
 
			boolean s_con=true;
			for(int i=0;i<size; i++){
				for(int j=0; j<size; j++)
					ran[i][j]=0;	
			}						
				M1:for(int i=0; i<size; i++){
					M2:for(int j=0; j<size; j++){					
						int x=r.nextInt();
						x=x%9+1;
						boolean condition=true;		
						L1:for(int n=0; n<size; n++){
							L2:for(int m=0; m<size; m++){
								if(x==ran[n][m]){
									condition=false;
									break L1;
								}
							}	
						if(condition)
							ran[i][j]=x;
						if(!condition)						
							break M1;
					}			
				}	
		H[0]=ran[0][0]+ran[0][1]+ran[0][2]==15;
		H[1]=ran[1][0]+ran[1][1]+ran[1][2]==15;
		H[2]=ran[2][0]+ran[2][1]+ran[2][2]==15;
 
		V[0]=ran[0][0]+ran[1][0]+ran[2][0]==15;
		V[1]=ran[0][1]+ran[1][1]+ran[2][1]==15;
		V[2]=ran[0][2]+ran[1][2]+ran[2][2]==15;
		HV[0]=ran[0][0]+ran[1][1]+ran[2][2]==15;
		HV[1]=ran[2][0]+ran[1][1]+ran[0][2]==15;
 
		Equal=(H[0]&&H[1]&&H[2]&&V[0]&&V[1]&&V[2]&&HV[0]&&HV[1]);
		}		
		for(int i=0; i<size;i++){
			System.out.print("\t");
				for(int j=0; j<size; j++)
					System.out.print(ran[i][j]+" ");		
				System.out.println();
			}
	}
}

Random class generate 0 and minus values

Yes nextInt() will return 0s. I don't know about minus values. Can you show this happening in your code by adding println() statements to print them out?

program does not work as I expected

No, they often don't. The computer does what you tell it to, not what you expect it to.
Can you explain what the program does that you don't expect and what you want it to do.

Your values for ran are changing every time it's run.

What is s_con ever used for?

Also, why is equal being set to

H[0]&&H[1]&&H[2]&&V[0]&&V[1]&&V[2]&&HV[0]&&HV[1]);

I thought booleans were only set to true or false.

Originally Posted by Norm

No, they often don't. The computer does what you tell it to, not what you expect it to.
Can you explain what the program does that you don't expect and what you want it to do.

Haha you said it.

OK the nextInt needs to be given a range I don't know what range it is in your case it could be 1-9, 1-15, 0-15 I don't believe you said. I'm guessing from your code (x=x%9+1) it's 1-9 so lets assume that for this post.

you would need 9 numbers I don't think you can set a range(I may be wrong here) so you just add the minimum number you'd expect so in this case 1 so you would say

int x=r.nextInt(9);
x+=1;

or

int x=r.nextInt(9) + 1;

That should give you just numbers in the range 1-9 you can adapt this obviously.

But I'm not really sure what you want it to do are you checking if the lines add up to 15 or are you trying to ensure they add up 15?