实现不违反封装的 OOP 委托

Question

这是一个关于赌博的简单场景。
有一个赌徒和骰子。 赌徒可以掷骰子。然后骰子值会改变

对于上述场景我创建了这个类图

如上图所示
赌徒可以掷骰子，它在Gambler.roll() 方法中实现
当赌徒掷骰子时，骰子值会发生变化，它在Dice.roll()方法中实现

使用java代码实现上述场景如下图 请注意此代码为伪代码，语法可能错误

class Gambler{
  private Dice dice;

  Gambler(Dice dice){
    this.dice = dice;
  }

  public void roll(){
    dice.roll();//delegation call
  }
}

/* --------------------------- */

import java.util.Random;
class Dice{
  private int faceValue;

  public int getFaceValue(){
    return faceValue;
  }


  public void roll(){
    //get random value between 1 ,6
    Random random = new Random();
    this.faceValue =  random.nextInt((6 - 1) + 1) + 1;
  }
}

我想知道
我的实现是否符合 OOAD 最佳实践且不违反封装？

特别是在将掷骰子从 Gambler 委托给 Dice 类时 Gambler 有 roll() 方法对吗？

Answer 1

你的设计基本上是正确的。唯一的问题是faceValue 属性前面缺少-。我不确定缺少的指标是否默认为某些东西（乍一看无法找到和指示）。无论如何，您应该添加它以使其明确。

另外，Dice 应重命名为 Die（单数）。

最后（更深入地研究您的代码）您需要一个包含 2 个元素的 Die 数组，而不仅仅是一个 dice，并且您需要掷两个骰子。

Answer 2

我认为骰子不是赌徒的一部分，赌徒只有掷骰子的行为。我认为下面的实现会更好：

class Gambler{
    //some field

    public int roll(Dice dice){
        return dice.roll();
    }
}

/* --------------------------- */

import java.util.Random;
class Dice{
     private int[] values = new int[]{1,2,3,4,5,6};

     public int roll(){
         //get random value between 1 ,6
         Random random = new Random();
         return values[random.nextInt((6 - 1) + 1) + 1];
     }
}

当然，这取决于实际需求。