Need some help Please

[UMstudent]

I'm working on this program for class where I have to an amount of numbers that are randomly picked from 0 - 50. So what I have done is say there are 20 slots in my array so I'm trying to generator 20 random numbers for the array between 0-50 however I've ran into a few problems. Mainly figuring out how to make it print 20 seperate completely random numbers into the array. So here is what I have any help will be greatly appreciated. Thank you.

import java.util.Random;

public class Prog7_1

{
public static void main (String[] args)

{
int NUMBERS = 20, random, num;

Random generator = new Random();
{
for(random = 0; random < 20);
num = generator.nextInt(51);
random++;
}
int[] array = new int [NUMBERS];

for(int index = 0; index < NUMBERS; index++)
array[index] = num;

for(int value : array)
System.out.print(value + " ");
}
}

I'm guessing I have 2 problems 1. lays within array[index] = num; I've tried a few things to make it print out each seperate random number such as num++ but that just prints the first random number then adds 1 to it 20 times. Then second problem I'm guessing is with the loop that is making hte random numbers. Again thanks for any help.

I also just made a similar change with a while loop instead of a for loop and the program compiles but it just sits there when I run it and does nothing here is hte code I used for that one maybe that one is closer to my final program.

import java.util.Random;

public class Prog7_1

{
public static void main (String[] args)

{
int NUMBERS = 20, random = 0, num;

Random generator = new Random();


while(random < NUMBERS);
num = generator.nextInt(51);


//for(random = 0; random < 20);
//num = generator.nextInt(51);
//random++;

int[] array = new int [NUMBERS];

for(int index = 0; index < NUMBERS; index++)
array[index] = num;

random++;

for(int value : array)
System.out.print(value + " ");


}
}

[Ganon11]

import java.util.Random;
 
public class Prog7_1
{
    public static void main (String[] args)
    {
            int NUMBERS = 20, random, num; 
 
            Random generator = new Random();
            { // What is this bracket doing here?
            for(random = 0; random < 20); // Semicolon following the for statement will result in a pointless loop - in this case, an infinite loop, as random++ is outside the header.
                num = generator.nextInt(51); 
                        // You generate num, but you never do anything with it!
                random++; // Should be in the header.
            }
            int[] array = new int [NUMBERS];
 
            for(int index = 0; index < NUMBERS; index++)
            array[index] = num; // This will set each element of the array to the same value
 
            for(int value : array) // This is a bad header that I'm not sure I understand.
            System.out.print(value + "  "); // All you're doing here is printing value, not the value of the array.
    }
}

The above code needs some serious surgery. I think what you're trying to do can be accomplished in the following steps:

1) Declare variables - int num, final int NUMBERS, and int[] array.
2) for...loop from 0 to < NUMBERS (index i)
2) a. Generate a random number from 0 to 50 - store into num
2) b. Set the ith member of array to num (array[i] = num;).
2) c. Output array[i].

This is all that the above code was trying to accomplish, I think.

If you have to have unique random numbers, you can add the following logic:

2) d. for...loop from 0 to < i (index j)
2) d. 1. if array[j] == array[i], you have a repeat - you should generate a new random number. You can 'reset' this step of the array by decrementing i (i--) and then breaking from the second for... loop. The 1st loop will begin generating a random number for the same element.

[UMstudent]

The above code needs some serious surgery. I think what you're trying to do can be accomplished in the following steps:

1) Declare variables - int num, final int NUMBERS, and int[] array.
2) for...loop from 0 to < NUMBERS (index i)
2) a. Generate a random number from 0 to 50 - store into num
2) b. Set the ith member of array to num (array[i] = num;).
2) c. Output array[i].

This is all that the above code was trying to accomplish, I think.

If you have to have unique random numbers, you can add the following logic:

2) d. for...loop from 0 to < i (index j)
2) d. 1. if array[j] == array[i], you have a repeat - you should generate a new random number. You can 'reset' this step of the array by decrementing i (i--) and then breaking from the second for... loop. The 1st loop will begin generating a random number for the same element.[/quote]

For some reason whenever I set the first for loop 0 < 20 or random < NUMBERS i always get an error saying its expecting a ">" which i don't understand being that won't work cause 0 can't be greater then 20. Programming just kicks my ass lol.

[r035198x]

The above code needs some serious surgery. I think what you're trying to do can be accomplished in the following steps:

1) Declare variables - int num, final int NUMBERS, and int[] array.
2) for...loop from 0 to < NUMBERS (index i)
2) a. Generate a random number from 0 to 50 - store into num
2) b. Set the ith member of array to num (array[i] = num;).
2) c. Output array[i].

This is all that the above code was trying to accomplish, I think.

If you have to have unique random numbers, you can add the following logic:

2) d. for...loop from 0 to < i (index j)
2) d. 1. if array[j] == array[i], you have a repeat - you should generate a new random number. You can 'reset' this step of the array by decrementing i (i--) and then breaking from the second for... loop. The 1st loop will begin generating a random number for the same element.

For some reason whenever I set the first for loop 0 < 20 or random < NUMBERS i always get an error saying its expecting a ">" which i don't understand being that won't work cause 0 can't be greater then 20. Programming just kicks my ass lol.

Break it up. Write a method calles contains
public boolean contains(ArrayList array, int value) {
which returns true if value is a member of array.

Now your loop should look like this

 int index = 0; 
while(index < 20) {
        int x = (int)(Math.random() * 50);
        while(contains(array, x)) {
             x = (int)(Math.random() * 50);
        }
        array[index++] = x;
}
 

[UMstudent]

Thanks for your help I still wasn't able to get it lol I don't think i'll ever survive this java class but it was due last night and I just turned it what I had. Thanks though for helpin.