我是java的初学者,我正在尝试制作Yahtzee的游戏,并且我需要从void方法中随机掷骰子作为数组。有人可以向我解释为什么这不起作用吗?
import java.util.Arrays;
public class YatzeeGame {
public static void main(String[] args) {
// TODO Auto-generated method stub
int[] diceRolls = new int[5];
diceRolls = throwDice(diceRolls);
System.out.println(display(diceRolls));
}
public static void throwDice(int [] dice) {
int [] roll = {(int)(Math.random()*6+1),
(int)(Math.random()*6+1),(int)(Math.random()*6+1),
(int)(Math.random()*6+1),(int)(Math.random()*6+1),
(int)(Math.random()*6+1)};
dice = roll;
}
public static String display(int [] dice) {
String str = Arrays.toString(dice);
str = str.replace("[", "");
str = str.replace("]", "");
str = str.replace("," , " ");
return str;
}
答案 0 :(得分:2)
他们希望您替换阵列,如果您只是分配它就不会发生。请注意,返回数组仍然被认为是更好的方法。特别棘手:在你现有的代码中,你制作一个大小为5的数组,另一个大小为6.你自称是zahtzee,我们将使用5。
public static void throwDice(int [] dice) {
for (int x = 0; x < 5; x++)
dice[x] = (int)(Math.random()*6+1);
}
答案 1 :(得分:1)
解释为何无效:
您要做的事情:将骰子(您传入的参数)更改为等于滚动。基本上,(如果我在这里没有错)你试图使用throwDice更改diceRolls。
你实际在做什么:你已经通过diceRolls
并说“在这里,我们称之为骰子”。然后,在你的功能结束时,你基本上说“骰子不再意味着骰子了。骰子现在意味着滚动”。这意味着diceRolls仍然没有改变。
您需要更改dice
的实际值,而不是更改骰子。
例如:
public static void throwDice(int[] dice) {
// change the actual values of dice, instead of changing dice
dice[0] = (int) (Math.random() * 6 + 1);
dice[1] = (int) (Math.random() * 6 + 1);
dice[2] = (int) (Math.random() * 6 + 1);
dice[3] = (int) (Math.random() * 6 + 1);
dice[4] = (int) (Math.random() * 6 + 1);
}
答案 2 :(得分:1)
你的代码中存在很多错误。
在throwDice
方法中,dice
是一个局部变量,因此将其更改为roll
,这是另一个局部变量,不会影响该方法之外的任何内容。
此外,您的返回类型为void
,因此您无法使用该方法设置任何变量。
您可以使用返回int[]
的方法:
public static int[] throwDice() {
int[] roll = new int[6];
for (int i = 0; i < 6; i++) {
roll[i] = (int) (Math.random() * 6) + 1;
}
return roll;
}
然后使用它:
int[] diceRolls = throwDice();