Question

这是一个yahtzee游戏，这个方法应该计算并返回五个骰子的值。但这是一种愚蠢的方式，有一个switch语句，通过参数传递用户选择的类别，然后是每个可能类别的for循环。有没有比我的设计理念更简单的方法呢？

private int assignScoreToCategory(int category)
{
    int computedScore = 0;
    println("dice: "+dice1+" "+dice2+" "+dice3+" "+dice4+" "+dice5);

    //  Switches on the category the user has selected.
    switch (category)
    {
    case ONES: 
        for (int i = 0; i < 4; i++){
            if (diceArray[i] == ONES){
                println(computedScore);
                computedScore++;
            }
        }break;

    case TWOS: 



        break;
    case THREES: break;
    case FOURS: break;
    case FIVES: break;
    case SIXES: println("cat 6");   break;
    case UPPER_SCORE: break;
    case UPPER_BONUS: break;
    case THREE_OF_A_KIND: break;
    case FOUR_OF_A_KIND: break;
    case FULL_HOUSE: break;
    case SMALL_STRAIGHT: break;
    case YAHTZEE: break;
    case CHANCE: break;
    case LOWER_SCORE: break;
    case TOTAL: break;
    default: return 0; 
    }
    return computedScore;
}

Answer 1

您最应该使用enum代替int。然后，作为进一步的改进，将switch语句中包含的逻辑移动到枚举的实例方法中。然后，您只需通过一次方法调用即可替换switch。有很好的语法为每个枚举成员提供不同的实现。将这个想法与一个通用的方法实现结合起来，尽可能地借助一些实例变量来覆盖尽可能多的地方：

enum Category {
  private final int score;
  ... 
  ONES(1) { 
    public int score() { 
      int computedScore = 0;
      for (int i = 0; i < 4; i++) {
       if (diceArray[i] == ONES) {
          println(computedScore);
          computedScore++;
       }
      }
      return computedScore;
    }
  },
  ...,
  SIXES(6),
  ...;

  private Category(int score) { this.score = score; }

  public int score() { return this.score; }
}

我想给你一个更全面的例子，但你没有提供很多具体的逻辑。

我是否以正确的方式使用switch语句

1 个答案: