hironsan / uecda-pyclient Goto Github PK

２次元配列で表すかそれともビット列で表すか？

ビット列の方が高速に演算はできると思うが、わかりやすさに難がある。

OptionParserは非推奨なので、ArgParserを使う

CardSetに委譲しているだけなのに必要か

suitsの値が適当。

2の冪乗の値を足し合わせるようにする。

基本的には英単語にする。

• kaidan -> sequence
• shibari -> lock
• kakumei -> revolution

NormalCardSetでfind_groupsを実行してもジョーカーを使用している。
例えば、以下のような手札の時、

cards[0][1] = 1
cards[1][1] = 1

find_groupsを実行すると以下のような結果になってしまう。

S3
D3
S3 D3
D3 JOKER

ジョーカーは使うべきではない。

カード交換のメソッドに交換枚数を渡せていない

Suitを４つまでしか用意していないので5枚だしされると落ちる。

役を保持するのにMeldクラス、それらのコレクションとしてMeldsクラスがあると便利かも。
役の間で強さを比較しやすい。

C3C4C5 < C4C5C6みたいな

カードを出せない。

場のカード状態(階段、枚数組)によって分岐をしている。
テストがしにくいので、Stateパターンを用いて分岐を無くし、カード選択処理を委譲したい。

出せないカードを削除していない

C4Joker(C5)C6の階段のとき

card_num      : 3
is_joker_only : False
is_kaidan     : True
is_pair       : False
max card      : JOKER(C5)
min card      : C4
suits         : 3

Jokerを最強カードとしているためこのようなことが起きる。table_cardの変更が必要。