Tell me more ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite
1

I am trying to make an array of random numbers that range from 0-9 with the variable for this one being kickerNumbers. I get a compile error though saying incompatible types. I have tried changing the (int) to [int] like it said but I didn't think that would be right anyway and surly enough it wasn't. Is there another way to write this? I need it to act as the first four numbers of this to be 0-9 but the fifth number would need to be something smaller such as 0-4. for the 0-4 one I just made that a completely different variable. Was that a good choice? Here is the sample code for the lines that give the error. 

import java.util.Scanner;
import java.util.InputMismatchException;
public class CashBallTest
{
public static void main(String[]args)
{
    Scanner keyboard = new Scanner(System.in);
    int kicker;
    int[] kickerNumbers = (int)(Math.random()*0+9);
    int kickerPowerball=(int)(Math.random()*0+4);

The error reads: "\CashBallTest.java:9: incompatible types found : int required: int[] int[] kickerNumbers = (int)(Math.random()*0+9);"

I have also tried changing (int) to (int[]) but then it said it was an inconvertible type counting as a double when it needs to be an int[] but I don't see where it gets the double type from

share|improve this question
add comment (requires an account with 50 reputation)

5 Answers

up vote 4 down vote 
int[] kickerNumbers = (int)(Math.random()*0+9);

You can't assign an int to an int[]

Assuming that is supposed to be more the none number, it might be something like this

int[] kickerNumbers = new int[5];
for(int i = 0; i < kickerNumbers.length; i++) {
    kickerNumbers[i] = (int)(Math.random()*0+9);
}

On an unrelated note, why are you multiplying the Math.random()*0? That seems silly to me. If you want a number 1 through 10, I would do this: Math.random()*10 + 1

share|improve this answer
+1 for pointing out the random() issue. (and posting before me :) ) – Binyamin Sharet May 2 '11 at 21:48
add comment (requires an account with 50 reputation)
up vote 4 down vote
  1. This is an int array: int[] kickerNumbers but you assign it with int (not array) (int)(Math.random()*0+9);
  2. your random will return 9. always (operator precedence).

Create an array like this: int[] kickerNumbers = int[numberOfPlayers] and assign numbers in loop, or use ArrayList with numbers from 0-9 and call shuffle

share|improve this answer
add comment (requires an account with 50 reputation)
up vote 2 down vote

It may be work using your own random number generator so you can use nextInt and avoid some confusion.

Random r = new Random();
// four values [0, 9]
int[] kickerNumbers={r.nextInt(10), r.nextInt(10), r.nextInt(10), r.nextInt(10)};
// one value [0, 4]
int kickerPowerball = r.nextInt(5);
share|improve this answer
+1 for r.nextInt(). better than the games with Math.random() – Binyamin Sharet May 2 '11 at 22:34
And faster, but that doesn't matter here. ;) – Peter Lawrey May 2 '11 at 22:49
add comment (requires an account with 50 reputation)
up vote 0 down vote

You're attempting to assign an int to an array which is what is giving you the error.

what you want to do it use a loop to iterate through your array and assign random values to it, you mentioned that you wanted your numbers to be between 0-9 and 0-4 the following code will give you that result:

public static void main(String[] args) {
    int[] kickerNumbers = new int[4];
    for (int i = 0; i < kickerNumbers.length; i++) {
        kickerNumbers[i] = randomRange(0,9);
    }
    int kickerPowerball = randomRange(0,4);
}
public static int randomRange(int min, int max) {
    return (int) Math.round(((Math.random() * (max - min)) + min));
}

the code above includes a method that will return a integer between to given parameters.

share|improve this answer
add comment (requires an account with 50 reputation)
up vote 0 down vote

import java.util.Random; public class CashBallTest {

public static void main (String[]args){

    int[] kickerNumbers= new int[5];
    for (int i=0;i<4;i++)
      kickerNumbers[i]=randomize(9);
     kickerNumbers[4]=randomize(4);

    for (int i:kickerNumbers)
       System.out.println(i);
}

public static int  randomize(int k){
     Random rand=new Random();
     return rand.nextInt(k);
}

}

assuming you don't care for duplicates

sorry. i had not read answer from Peter..it is just the same

share|improve this answer
1  
You also should not get a new Random() every time. – Stephen P May 3 '11 at 0:56
add comment (requires an account with 50 reputation)

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.