クラス変数が増加しない

Question

「20枚コインがある、先攻後攻が1～4枚ずつ交互に取る。20枚目を取ったら負け」というゲーム」をpythonで実装しようとして、以下のようなコードを書きました。
（環境はPyCharm Community Edition 2020.2.2 x64、Anaconda導入済み)

import random


class Player(object):

    CARD_DECK = list(reversed([i for i in range(1, 21)]))

    def __init__(self, get_card_list=None, win_count=0):
        self.win_count = win_count
        if get_card_list is None:
            self.get_card_list = []

    def draw_card(self, draw_choice=None):
        if draw_choice is None:
            draw_choice = [1, 2, 3, 4]
        draw_card_count = random.choice(draw_choice)
        for i in range(draw_card_count):
            if len(self.CARD_DECK) == 0:
                break
            draw_card = self.CARD_DECK.pop()
            self.get_card_list.append(draw_card)
        if 19 in self.get_card_list:
            self.win_count += 1


first_player = Player()
second_player = Player()

for i in range(100):
    while len(first_player.CARD_DECK) > 0:
        first_player.draw_card()
        second_player.draw_card()

print(first_player.get_card_list)
print(second_player.get_card_list)
print(first_player.win_count)

こちらが期待しているのは、PlayerClassをそれぞれfirst_player, second_playerにインスタンス化し、draw_cardメソッドでクラス変数で宣言した20枚のカードデッキがなくなるまでお互い交互に1～4枚カードを引き、最終的に１９を所有しているplayerの勝利数:win_countが増えることです。そして以上をfor文で1000回ほど繰り返すことでどちらのplayerの勝利数が多いのかをインスタンス変数のwin_countで表示したいのですが、どうも繰り返し処理がうまくいってないような気がします。
どうにかアドバイスお願いします。

Comments

[TakaiY]

どのような問題があって、何を聞きたいのでしょう？

[meg_]

> どうも繰り返し処理がうまくいってないような気がします。
そう思われた理由は何でしょうか？ そこを質問に書いてくれないと課題が分かりません。

Answer 1

• クラス変数のCARD_DECKの内容は、1回のゲームごとに([20, 10, ……, 1]のリストに)初期化する必要がある。
• インスタンス変数のget_card_listの内容は、1回のゲームごとに[]に初期化する必要がある。
• 19を引いて勝敗が決まったらすぐにCARD_DECKを[]にしないと、1回のゲームで複数回win_countを増やしてしまうことがある。

import random


class Player(object):

    def __init__(self, get_card_list=None, win_count=0):
        self.win_count = win_count
        self.get_card_list = get_card_list

    def reset_card_deck():
        Player.CARD_DECK = list(reversed([i for i in range(1, 21)]))

    def draw_card(self, draw_choice=None):
        if draw_choice is None:
            draw_choice = [1, 2, 3, 4]
        draw_card_count = random.choice(draw_choice)
        for _ in range(draw_card_count):
            if len(Player.CARD_DECK) == 0:
                break
            draw_card = Player.CARD_DECK.pop()
            self.get_card_list.append(draw_card)
        if 19 in self.get_card_list:
            Player.CARD_DECK = []
            self.win_count += 1


first_player = Player()
second_player = Player()

for _ in range(10000):
    Player.reset_card_deck()
    first_player.get_card_list = []
    second_player.get_card_list = []
    while len(Player.CARD_DECK) > 0:
        first_player.draw_card()
        second_player.draw_card()

#    print(first_player.get_card_list)
#    print(second_player.get_card_list)

print(f"1st win {first_player.win_count}, 2nd win {second_player.win_count}")

1st win 4998, 2nd win 5002