[monster92]

Cheers if you can. I'm trying to see how I could my this program more readable. The aim is too make a slot machine which tells you, you win if you the same numbers.

import java.util.Random;

public class PP59 {

    public static void main(String[] args) {
        int Random1, Random2, Random3;
       


 


        Random generator = new Random();


        while (in.hasNextLine()) {
            
        

        Random1 = generator.nextInt(9);
        Random2 = generator.nextInt(9);
        Random3 = generator.nextInt(9);


        System.out.print(Random1);
        System.out.print(Random2);
        System.out.println(Random3);

        if (Random1 == Random2 && Random1 == Random3 || Random2 == Random1 && Random2 == Random3 || Random3 == Random1 && Random3 == Random2) {
            System.out.println("You won £50 :D/>");

        }
            if (Random1 == Random2 || Random2 == Random3 || Random1 == Random3) {
            System.out.println("You won £25 :D/>");

        }





    }
}
}

[guido-granobles]

In the line:

 while (in.hasNextLine())

Where is declared 'in' ?.

And this is most readable.

if ( (Random1 == Random2 && Random1 == Random3 ) || (Random2 == Random1 && Random2 == Random3)
		|| (Random3 == Random1 && Random3 == Random2) ) {

         System.out.println("You won £50 :D/>");
			
 }else if ( (Random1 == Random2) || (Random2 == Random3) || (Random1 == Random3) ) {

    System.out.println("You won £25 :D/>");

 }

[monster92]

Ahhh Cheers that looks a lot better. Thanks for the tips

[GregBrannon]

You don't need the extra blank lines; a source file header that describes what the file does, who wrote it and when would be nice; an appropriate comment now and then; and better variable names - avoid using the same subscripted name over and over.