オリジナルポーカーの役判定について

Question

大学の授業でお金を題材にしたオリジナルポーカーをunityを使って作成しています。

プレーヤーは７つの項目の山札からカードを１枚ずつ引きます。
それぞれの項目の金額の合計や組み合わせで役が決定します。
例として貯金２万円、住宅５万円、生活費３万円、衣服２万円、趣味２万円、交際費２万円、を引いた場合はロイヤルライフという役になります。
そこで役判定のコードを書いているのですが、それぞれの山札から引いた７枚の手札を１０種の役と比べるコードが思いつきません。
どのような方法があるのか、また、どのような書き方になるのか教えていただけないでしょうか。

public class NewBehaviourScript : MonoBehaviour
{
public enum PokerHand
{
ロイヤルライフフラッシュ,ロイヤル将来メザッシュ,現状充実ウイッシュ,金貯まりペア,FULLハウス,健康ペア,
ハッピー３カード,見栄ペア,人生ダイナシペア,健全家計ストレート,
}
class life {
int saving =2;
int home = 5 ;
int life = 3;
int clothes = 3;
int hobby = 2;
int dating = 2;
}
class syourai
{
int saving = 4;
int home = 5;
int life = 3;
int clothes = 1;
int hobby = 2;
int dating = 1;
}
class jyuuzitu
{
int saving = 1;
int home = 5;
int life = 3;
int clothes = 2;
int hobby = 3;
int dating = 2;
}
class kenzen
{
int saving = 4;
int home = 5;
int life = 3;
int clothes = 1;
int hobby = 2;
int dating = 1;
}
class fullhouse
{
int home = 9;
int life = 2;
int clothes = 0;
}

class happy3
{
    int clothes = 3;
    int hobby = 3;
    int dating = 3;
}
class kanetamari
{
    int saving = 6;
    int home = 2;
    }
class kenkou
{
    int life = 3;
    int hobby = 2;
    }
class mie
{
    int life = 7;
    int clothes = 5;
    }
class dainashi
{
    int hobby = 5;
    int dating = 5;
}

}
自分はクラスを用いて判断しようとしたのですがこの先の判定が思いつきませんでした。

Answer 1

まず、強い役が完成していたら、それより弱い役は判定する必要がないかと思います。
なので一番強い役から順に判定していきます。

ひとまずロイヤルライフフラッシュかどうかだけを判定できるようにしましょう。

役の判定１つにつき、１つのメソッドを作るとよいでしょう。

//ロイヤルライフフラッシュが成立しているかどうかの判定
//成立していたらTrue,不成立ならFalseが返ってきます
bool RoyalLifeFlush( List<Card> Hands )
{
     //手札に特定のカードが存在するか調べる

　  //貯金2万のカードを持っているかどうか
     bool hasSavings2 = Hands.Exists(card=> card.type == CardType.貯金 && card.value == 2) 

    //住宅5万のカードを持っているかどうか
     bool hasHome5 = Hands.Exists(card=> card.type == CardType.住宅 && card.value == 5)

    ...中略

     //全部Trueだったらロイヤルライフフラッシュ成立
     return hasSavings2 && hasHome5 && .....
}

//ロイヤル将来メザッシュが成立しているかどうかの判定
bool RoyalFutureMezash( List<Card> Hands )
{
}

PokerHand HandJudgment()
{
      if( RoyalLifeFlush(Hands) ) return PokerHand.ロイヤルライフフラッシュ;
      if( RoyalFutureMezash(Hands) ) return PokerHand.ロイヤル将来メザッシュ;
}

もっとスマートに書けますが、とりあえずわかりやすさ重視にしました。

Cardクラスは、CardTypeというEnumと、int Valueを持っています。

一旦これぐらいでどうでしょうか？

ここからメソッドではなく、判定用のクラスを作って、判定用クラスの配列をfor文で回して、
それだけで判定できるようになったらOKかなと思います

もしわからなければ更に追記していきますので、何度か質問してください。

Comments

[kante]

分かりやすい丁寧な回答ありがとうございます！「判定用のクラスを作って、判定用クラスの配列をfor文で回して、それだけで判定できるようになったらOKかなと思います」ここの部分の意味が分からないのですが、配られた手札とすべてのメソッドの内容をfor文で判定するということでしょうか？また、
PokerHand HandJudgment()
{
if( RoyalLifeFlush(Hands) ) return PokerHand.ロイヤルライフフラッシュ;
if( RoyalFutureMezash(Hands) ) return PokerHand.ロイヤル将来メザッシュ;
}ここの部分は判定する部分ではないのでしょうか？

[ganta7188]

それについては、一旦完成したあとの発展なので、まずは気にしなくて大丈夫です。
僕が書いた方法は初心者向けなので、本来はもっとクラスを上手く使ったほうが、
追加、変更、修正が簡単になるという話ですね。

どういうことか想像付きにくいと思いますので一応クラスでやる方法を書いておきますが、
まずはメソッドでやる方法を試してからのほうが良いかなと思います。

HandJudgementerというクラスを作り、
それに、それぞれの役のデータを入れることで、あとでまとめてfor文で処理できるようにします。

```C#
var RoyalLifeFlush = new HandJudgementer(PokerHand.ロイヤルライフフラッシュ); RoyalLifeFlush.Add(CardType.貯金, 2);
RoyalLifeFlush.Add(CardType.住宅, 5);
･･･中略

var RoyalFutureMezash = new HandJudgementer(PokerHand.ロイヤル将来メザッシュ);
RoyalFutureMezash .Add(CardType.貯金, 4);
RoyalFutureMezash .Add(CardType.住宅, 5);
･･･中略

JudgementerList.Add(RoyalLifeFlush);
JudgementerList.Add(RoyalFutureMezash);
･･･中略

for(int i=0; i < JudgementerList.Count; i++)
{
//Judgeという共通のメソッドで、全ての役を判定
bool seiritsu = JudgementerList[i].Judge(Hands);

　//成立していた場合は、成立役名を取得して、for文を抜ける
if(seiritsu)
{
seiritsuYaku = JudgementerList[i].pokerHand;
break;
}
}
```
みたいなイメージです。
こうすることでメソッドを作るよりは短くなりますし、後から修正箇所を見つけやすくなります。
また、例えば「ジョーカーを持っていた場合は、持っていないカードに置き換えて判定する」
などの仕様が入った場合に、メソッドがたくさんある場合はすべてのメソッドに修正を加える必要があります。
ですがクラスを使って共通のJudgeメソッドを使う場合、Judgeメソッドにジョーカーの処理を追加するだけでよくなります。

こんな感じで、クラスを使うとそれ以外にも色々と利点がありますので、
良かったら勉強を進めてみてください。