Java code help?

Question

System.out.print("Enter your play (R, P, S): ");
personPlay=Keyboard.readString();

//Use a method of the String class to make the personPlay all UPPERCASE
personPlay.toUpperCase();

//Use Math.random() to generate computer's play as either 0, 1 or 2
//Store result in computerInt
computerInt=(int)Math.random()*3;
//Use else-if to convert 0, 1 or 2 to initialize computerPlay to R, P or S
if(computerInt==0)
computerPlay="R";
else
{ if(computerInt==1)
computerPlay="P";

if(computerInt==2)
computerPlay="S";
}

System.out.println("The computer played: " + computerPlay);

/*-------------------------------------------------------
every time i compile and run the program, it always outputs "The computer played: R" ....so there is something wrong with my if-else statements, but i can't figure out what?

Believe it or not-- I have studied java code and understand how Math.random works....

Answer 1

I just see som bad syntax... I'm a little rusty, but shouldn't it read:

if(computerInt==0) {
computerPlay="R";
}
if(computerInt==1){
computerPlay="P";
}
if(computerInt==2){
computerPlay="S";
}

Plus the randomization string should probably be:

computerInt=Math.floor (Math.random()*2) ;

Answer 2

Math.random() returns numbers between 0 and 1. you should have a multiplier first, say 100. if you want the generated values limited to a certain number, use the modulo operator.
for example, you want a number to be a maximum of 2 only, you may write:
computerInt = ((Math.random())*100)%2;

Answer 3

The cast is binding more tightly than the multiplication. I think you want computerInt=(int)(Math.random()*3);

Answer 4

I think your problem is in your use of Math.random().....if I'm not mistaken it should be variable = (int)(3*Math.random); but your way might work too, I've always put parentheses around the scaler multiplication.

Answer 5

daa has it right. The logic is weird, but should work.