有方法的掷骰子

Question

我目前正在从我的学习书中做一些练习，其中一项任务是：编写一个骰子类“骰子”，其值在 1-6 之间。还应该有一个选择随机值的构造函数，以及一个也选择随机值的方法 roll()。还应该创建方法 getValue()，该方法将用于检索显示的值。为这个程序编写一个测试类。

Edit* 我将随机生成器移到构造函数中，将 roll 方法留空。构造函数已经随机化了，roll()方法应该怎么做？

这是我目前的代码：

public class Dice {

    int value;
    int currentRoll;

    public Dice() {
        Random rand = new Random();
        this.value = 1;
        this.currentRoll = rand.nextInt(6) + 1;
    }

    public int roll() {
        Random rand = new Random();
        this.currentRoll = rand.nextInt(6) + 1;
        return this.currentRoll;
    }

    public int getValue() {
        return this.currentRoll;
    }
}

我不明白的是：为什么要在构造函数和 roll() 方法中随机取值？另外，我错过了什么？

Answer 1

要求是构造函数和roll() 方法都选择一个随机值。他们做同样的事情。所以为了避免不必要的代码重复，构造函数可以调用roll方法等。

警告：在构造函数中调用可覆盖的方法是坏习惯。 This post explains 为什么会这样。

您的代码可能如下所示：

class Dice {

    private int value;

    // You could create the Random instance here once, instead of everytime
    // recreating it
    private Random r = new Random();

    public Dice() {
        roll();
    }

    // You could also change the return type to and int and return this.value
    public final void roll() {
        this.value = this.r.nextInt(6) + 1;
    }

    public int getValue() {
        return this.value;
    }
}

---

我不知道我是否完全同意这些要求，但既然它们是要求，你应该承认它们。

Answer 2

为什么要在构造函数中选择一个随机值？好吧，坦率地说，因为这是练习的要求。
他们为什么有这个要求？如果我不得不猜测，那是为了模拟一个骰子总是有一些面朝上的事实（即，有一个值），无论你是否明确滚动它，但如果你想要一个明确的答案，你'必须询问这本书的作者他或她的想法。

您可以通过在构造函数中调用roll 来实现。另外，请注意，您与已初始化但从未使用过的 value 成员存在冗余：

public class Dice {
    private int currentRoll;

    //Constructor
    public Dice() {
        roll();
    }

    // methods...