Возврат значения в другой класс и сохранение

Question

Предположим, что дан класс Die и он содержит случайное значение для шестигранного кубика.

Другой класс PairOfDice должен получить доступ к getvalue в Die и сохранить два значения кубика.

Ошибка: не удается найти символ при выполнении PairOfDice.

Как решить эту проблему?И есть ли другие предложения для кода Java?

public class Die {
public static Random rand = new Random();
private int sides; // Number of sides
private int value; // Die's value
public Die() {
    sides = 6;
    roll();
}
public void roll() {
    value = rand.nextInt(sides) + 1;
}
public int getSides() {
    return sides;
}
public int getValue() {
    return value;
}

Второй класс дан:

public class PairOfDice {
private int dieOne;
private int dieTwo;
public void main(String[] args){
    Die die;
    die = new Die();
}
private void dieOne(int value){
    dieOne = die.getValue();
}
private void dieTwo(int value){
    dieTwo = die.getValue();
}
public int getDieOneValue(){
    return dieOne;
}
public int getDieTwoValue(){
    return dieTwo;
}
}

Answer 1

Этот квест должен быть обобщен: я написал класс Die с двумя открытыми конструкторами.Если конструктор не имеет параметра, размер штампа по умолчанию равен шести, иначе вы можете иметь любое количество сторон.Затем я написал класс Dices с двумя конструкторами.Первый из них содержит количество кубиков (с 6 сторонами), а второй - Список кубиков с предпочтительными сторонами.Если вы хотите узнать, как обобщить проблему (любую проблему), вы можете проверить мой код.(Конечно, это может быть сделано более эффективно и более элегантно, но вот простой код):

import java.util.ArrayList;
import java.util.List;
import java.util.Random;

class Die {
    private Random RAND = new Random();
    private int noOfSides;
    private int value;

    // Default constructor without the parameter of sides gives the six sized die
    public Die() {
        this.noOfSides = 6;
    }

    // The constructor WITH number of sides
    public Die(int noOfSides) {
        this.noOfSides = noOfSides;
    }

    // rolling the die
    public void roll() {
        this.value = RAND.nextInt(noOfSides) + 1;
    }

    public int getValue() {
        if (value == 0) roll(); // if the die is never rolled -> roll it!
        // else return the rolled value
        return value;
    }

    // just for curiosities
    public int getNoOfSides() {
        return noOfSides;
    }

    public String toString() {
        return "Die has the " + noOfSides + " sides, and the last roll value was " + getValue();
    }
}

class Dices {

    private int noOfDices;
    private List<Die> myDices = new ArrayList<Die>();

    // NO constructor without the number of dices
    private Dices() {
    }

    public Dices(int noOfDices) {
        this.noOfDices = noOfDices;

        // example is for 6 sided dices
        for (int i = 0; i < noOfDices; i++) {
            getMyDices().add(new Die());
        }
    }

    // example with the list of dices with predefined sizes
    public Dices(List<Die> myDices){
        this.myDices = myDices;
    }

    public List<Die> getMyDices() {
        return myDices;
    }

    public String toString() {
        String s = "";
        for (Die die : getMyDices()) {
            s = s + die + "\n";
        }
        return s;
    }
}

public class Answer {

    public static void main(String[] args) {

        //test with two dices (6 size):
        Dices twoDices = new Dices(2);
        System.out.println(twoDices);

        //test with 4 dices size 3, 7, 9, 22
        Dices fourDices = new Dices
                (List.of(new Die(3),
                        new Die(7),
                        new Die(9),
                        new Die(22)));
        System.out.println(fourDices);
    }
}

Вы можете видеть, если кубик никогда не бросается, getValue сначала бросьте кубик, затем верните значение.В противном случае вы можете бросить кубик, и значение будет сохранено в значение частного поля ...

Answer 2

Предположим, что класс Die работает как положено.

Итак, можно подумать об этих изменениях в PairOfDice, которые, вероятно, решат немедленную проблему.

public class PairOfDice {
  private Die dieOne = new Die();
  private Die dieTwo = new Die();

  public static void main(String[] args) {
    PairOfDice pair = new PairOfDice();
    System.out.printf("Pair: %d - %d%n", pair.getDieOneValue(), pair.getDieTwoValue());
  }

  public int getDieOneValue() {
     dieOne.roll();
    return dieOne.getValue();
  }

  public int getDieTwoValue() {
    dieTwo.roll();
    return dieTwo.getValue();
  }
}

Класс PairOfDice должен содержать ссылки на две кости (в переменных экземпляра private Die).

Методы getDieXValue() работают с переменными экземпляра и возвращают значение после генерации roll().

Теперь вопрос заключается в том, требуется ли хранить values двух кубиков или просто иметь доступ к возможности получить значения. Если действительно нужно хранить значения, то можно сделать:

public class PairOfDice {
  private int dieOneValue;
  private int dieTwoValue;

  public PairOfDice {
    Die die = new Die();

    // get a value for the first die
    die.roll();
    dieOneValue = die.getValue();

    // get a value for the 2nd die
    die.roll();
    dieTwoValue = die.getValue();
  }

  public int getDieOneValue() {
    return dieOneValue;
  }

  ...

Лично, если кто-то собирается создавать объекты, то храните и используйте объекты.