【カードゲーム】カードを引く（or配る）メソッドはどこに実装すべきか。

Question

はじめまして、java教材向けに簡単なカードゲームを制作しました。制作後、もうすこしリファクタリング＆拡張性のあるコードにできないものかと思い、この場を借りて質問させて頂くことにしました。

ざっとですが、下のようなクラスがあります。

• Cardクラス（カード）
  -カードの数値
  -カードのスート（ダイヤ、スペードなど）
  -カードの表示値 （数値は１でも表示はAなど）
• Deckクラス（カードの山）
  -Card52
  -shuffle() カードをシャッフル
• Playerクラス （プレイヤー）
  -Card[]（手札分）
  -cardCount （手持ちカード枚数）
  -addCound()（手持ちカード枚数カウンタを増やす）
• Gameクラス (ベースクラス：main()があります)
  -Deck mainDeck （カードの山）
  -cardMax （現在の山の枚数）
  -hitCard() （カードを引く）

【カードを引く】というメソッドは、main()のあるGameクラス内に記載し、ゲームループから呼び出しました。

static void hitCard(Player p) {
	//山の一番最後からカードを一枚取って、プレイヤーのカード配列に入れる
	p.cards[p.cardCount]=mainDeck.cards[cardMax - 1];
	//手持ちカード枚数カウンタを増やす
	p.addCount();
	//山の枚数を減らす
	cardMax--; 
}

と、こんな感じです。
上記コード＆記載場所で動作的には何ら問題はないです。

ここからが質問なのですが、Gameクラスに【カードを引く】というメソッドを書くのはどうなんでしょう？

例えば、複数のカードゲームを選択してプレイできるように今後プロジェクトを改造する場合を想定したら、【カードを引く】メソッドはどこに実装すべきだと思いますか？

今考えているのは、Gameクラスをmain()持ちではないクラスに変更し、各ゲームはGameクラスを継承した物にするという作りに。スーパークラスのGameクラスの中に【カードを引く】メソッドを実装するのが妥当なのかな？と考えています。

皆さんの思考をお聞かせ頂きたいです。

Answer 1

僕はオブジェクト指向を使う主な目的は、コードの書き方を現実世界の表現に近づけるためのモノと考えています。（DDDで言うユビキタス言語です）そういう意味で、Gameがカードを引くメソッドを持っているとゲームがカードを引くとコード上は表現されます。コード上現実世界を表現できていますか？これをコードにすると

// ゲームはプレイヤーのためにカードを引く(Game hits card for player)
game.hitCard(player);

と表現されるはずです。

これよりは、カードを引くプレイヤーがカードを引くの方がそれっぽいですよね。現実世界でカードを引くのはプレイヤーですから。Gameにはカードを配るメソッドが定義されており、そこでカードの配り方が定義されていたら下記のように実装されるでしょう。

public class Game {

  public void deal(){
    Card card = drawFromDeck();
    player.hand(card); // Gameがカードを配る
  }

  private Card drawFromDeck() {
    // デッキからカードを選ぶ実装例
    return deck.cards[random];
  }

}

ところで、プレイヤーに配るカードは、カードゲームのルール毎に異なりませんか？例えばポーカーと七並べのルールで遊べたとして、1枚ずつ配るのか、数枚まとめて配るのか、はカードゲームのルールに定義します。Gameにカードを配るを実装するよりも、Ruleにカードを配るを実装した方がそれらしくなります。

こうしていくことで、コードにドメインの知識を語らせる事ができるでしょう。

Comments

[Jake]

コメントありがとうございます。

書かれている通り、「Gameがカードを引く（配る）」というカタチになっている、というのも懸念点の１つでした。

まとめて配る場合や順に配る場合といろいろパターンがありますが、基本的には１枚ずつ配ることの繰り返しですので、仮想ゲームマスターに配ってもらう、という感覚で作っていたのですが、書かれている内容を見ると、方向性がクリアになってきたような気がします。

ありがとうございます。

[退会済みユーザー]

GameMasterは良いクラス名ですね。Ruleがカードを配るのもあとになって変な気がしました。GameMasterがRuleを知っていて、そのRuleに合わせてカードを配るんですね。

Answer 2

質問に対するリファクタリングを考えてみました。
1-Gameクラスを抽象クラス化
2-抽象メソッドdoTurnを追加→ゲーム内の状態を変化させるメソッド、今回のカードを引く等の行為など
3-抽象メソッドjudgeを追加→ゲーム状態変化に伴う勝敗判定
4-Gameクラスのmain内で2.3を呼ぶ
5-Gameを実装したCardGameを実装

Comments

[Jake]

コメントありがとうございました。