我正在尝试用简单,简单的java制作一个cee-lo程序。我只是在学习。但是,当我到达我的瞬间时。 (我已将其简化为测试)它总是返回false。我似乎无法弄清楚为什么。它甚至可以显示正确的数据,但是当它进行比较时,它就会失败。
public class ceeLo
{
public static void main (String [] args)
{
Scanner scan= new Scanner (System.in);
int [] die = new int [3];
int answer;
boolean roll = true;
boolean qualifed;
boolean instantW;
boolean instantL;
do
{
System.out.println("Roll the dice?");
answer = scan.nextInt ();
if (answer == 0)
roll= false;
else
{
int i;
for (i = 0; i < die.length; i++)
{
die[i]= rollin();
System.out.println(diceTxt(die[i]));
}
qualifed = (qualify (die));
System.out.println("Qualified = " + qualifed);
instantW = (easyW (die));
System.out.println("Instant win = " + instantW);
}
}
while (roll);
}
// Generate random numbers for the roll
public static int rollin ()
{
Random rand = new Random();
int die= rand.nextInt(6);
return die;
}
//Check if dice qualify with pair
public static boolean qualify (int [] die)
{
boolean qualify;
//Pair Qualifying roll
if (die[0] == die[1] || die[0] == die[2] || die[1] == die[2])
qualify = true;
else
qualify = false;
return qualify;
}
//Check if instant win
public static boolean easyW (int [] die)
{
boolean instantW;
// show contents of die [x] for testing
System.out.println (die[0] + "" + die[1] + "" + die[2]);
if (die[0] > 2 && die [1] > 2 && die[2] > 2)
instantW = true;
else;
instantW = false;
return instantW;
}
}
答案 0 :(得分:7)
在其他之后删除分号;它应该是其他的
我猜原因是,
instantW = false;
被视为单独的声明而不是else块的一部分。这就是为什么instantW始终被赋值为false并返回false的原因。
使用{}定义块总是更好,即使它们是单行也是如此。这是我的偏好。
正如Greg Hewgill所建议的那样,使用单一陈述instantW = die[0] > 2 && die [1] > 2 && die[2] > 2;
会比if / else更好。
答案 1 :(得分:3)
编写布尔方法的更好方法是做类似
的操作boolean easyW(int[] die)
{
return (die[0] > 2 && die[1] > 2 && die[2] > 2);
}
甚至更好(更一般)
boolean easyW(int[] die)
{
for(int roll : die)
{
if(roll < 2)
{
return false;
}
}
return true;
}
但在你的情况下,你有一个;
。修正版:
public static boolean easyW (int [] die)
{
boolean instantW;
// show contents of die [x] for testing
System.out.println (die[0] + "" + die[1] + "" + die[2]);
if (die[0] > 2 && die [1] > 2 && die[2] > 2)
instantW = true;
else
instantW = false;
return instantW;
}