Question

我是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;
}

Answer 1

他们希望您替换阵列，如果您只是分配它就不会发生。请注意，返回数组仍然被认为是更好的方法。特别棘手：在你现有的代码中，你制作一个大小为5的数组，另一个大小为6.你自称是zahtzee，我们将使用5。

public static void throwDice(int [] dice) {     
    for (int x = 0; x < 5; x++)
        dice[x] = (int)(Math.random()*6+1);
}

Answer 2

解释为何无效：

您要做的事情：将骰子（您传入的参数）更改为等于滚动。基本上，（如果我在这里没有错）你试图使用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);
}

Answer 3

你的代码中存在很多错误。

在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();

如何从void方法显示数组？

3 个答案: