[Tornel]

I'm trying to write a program in Java that can take specific high and low limits to a range of numbers, and still include those limits in the random number. I had an idea of how to do it, but I'm doing something wrong with it. Here is my code, maybe you guys can figure out what I'm doing wrong? Thanks

import java.util.Random;

public class RandomNumbers
{
public static void main(String[] args)
{
Random rand= new Random();

double low =Keyboard.readDouble("Enter the low limit for your random number(double)");
double high = Keyboard.readDouble("Enter the high limit for your random number(double)");

int low2 = Keyboard.readInt("Enter the high limit for your random number(int)");
int high2 = Keyboard.readInt("Enter the high limit for your random number(int)");

double x= randomDouble(rand, low, high);
int y= randomInt(rand, low2, high2);


System.out.println("Here is your random number(double): " + x);
System.out.println("Here is your random number(int): " + y);
}

public static double randomDouble(Random rand, int low, int high)
{
Random rand= new Random();
return ((high-low) * rand.nextDouble + low);

}

public static int randomInt(Random rand, int low2, int high2)
{
return rand.nextInt(high2 - low2 + 1) + low2;
}
}

[Advertisements]

public static int randomInt(Random rand, int low2, int high2)
{
return rand.nextInt(high2 - low2 + 1) + low2;
}

You don't need to accept an instance of the Random class here, just create your own:

public int randomInt(int min, int max) {
	return new Random().nextInt(max - min + 1) + min;
}

public static double randomDouble(Random rand, int low, int high)
{
Random rand= new Random();
return ((high-low) * rand.nextDouble + low);

}

You're doing a lot of weird stuff here. First of all, your trying to create an instance of the Random class when you already are accepting one as a parameter. Then, you forgot parentheses after rand.nextDouble. There are a lot of ways to generate a random double, here's a simple approach (which I think you were trying to do):

public double randomDouble(int min, int max) {
	return ((max - min + 1) * new Random().nextDouble()) + min;
}

You could also use your randomInt(int, int) method:

public double randomDouble(int min, int max) {
	return randomInt(min, max) + new Random().nextDouble();
}

[destin]

public static int randomInt(Random rand, int low2, int high2)
{
return rand.nextInt(high2 - low2 + 1) + low2;
}

You don't need to accept an instance of the Random class here, just create your own:

public int randomInt(int min, int max) {
	return new Random().nextInt(max - min + 1) + min;
}

public static double randomDouble(Random rand, int low, int high)
{
Random rand= new Random();
return ((high-low) * rand.nextDouble + low);

}

You're doing a lot of weird stuff here. First of all, your trying to create an instance of the Random class when you already are accepting one as a parameter. Then, you forgot parentheses after rand.nextDouble. There are a lot of ways to generate a random double, here's a simple approach (which I think you were trying to do):

public double randomDouble(int min, int max) {
	return ((max - min + 1) * new Random().nextDouble()) + min;
}

You could also use your randomInt(int, int) method:

public double randomDouble(int min, int max) {
	return randomInt(min, max) + new Random().nextDouble();
}

[Tornel]

Thanks! I got it now