カードゲームのリストを別のリストに移動するPython

Question

私はPythonでポーカーゲームを作ろうとしています。使用済みのカードをリストから別のリストに移動して、使用済みのカードを保存したいカード。私はこのコードを使用しています：

deck = ['AS', 'KS', 'QS', 'JS', '10S', '9S', '8S', '7S', '6S', '5S', '4S', '3S', '2S',\ 
     'AD', 'KD', 'QD', 'JD', '10D', '9D', '8D', '7D', '6D', '5D', '4D', '3D', '2D',\ 
     'AC', 'KC', 'QC', 'JC', '10C', '9C', '8C', '7C', '6C', '5C', '4C', '3C', '2C',\ 
     'AH', 'KH', 'QH', 'JH', '10H', '9H', '8H', '7H', '6H', '5H', '4H', '3H', '2H'] 

used = [] 
p1 = [] 
p2 = [] 
a = 0 

while (a < 2): 
    drawn_card = random.choice(deck) 
    deck.append(drawn_card) 
    deck = [f for f in deck if f not in used] 
    p1.append(drawn_card) 
    a+=1 

# Don't read anything below this line 

card1 = p1[0] 
card2 = p1[1] 
f_card1 = "Name/Suit" 
f_card2 = "Name/Suit" 

# Formatting Player1's first drawn card 

if card1[0] == "A": 
    f_card1 = "Ace" 
elif card1[0] == "K": 
    f_card1 = "King" 
elif card1[0] == "Q": 
    f_card1 = "Queen" 
elif card1[0] == "J": 
    f_card1 = "Jack" 
else: 
    f_card1 = card1[0] 


if card1[1] == "S": 
    f_card1 = f_card1 + " of Spades" 
elif card1[1] == "D": 
    f_card1 = f_card1 + " of Diamonds" 
elif card1[1] == "C": 
    f_card1 = f_card1 + " of Clubs" 
else: 
    f_card1 = f_card1 + " of Hearts" 

# Formatting Player1's second drawn card 

if card2[0] == "A": 
    f_card2 = "Ace" 
elif card2[0] == "K": 
    f_card2 = "King" 
elif card2[0] == "Q": 
    f_card2 = "Queen" 
elif card2[0] == "J": 
    f_card2 = "Jack" 
else: 
    f_card2 = card2[0] 


if card2[1] == "S": 
    f_card2 = f_card2 + " of Spades" 
elif card2[1] == "D": 
    f_card2 = f_card2 + " of Diamonds" 
elif card2[1] == "C": 
    f_card2 = f_card2 + " of Clubs" 
else: 
    f_card2 = f_card2 + " of Hearts" 

print(f_card1) 
print(f_card2) 

while関数は私の問題がどこにありますか？私は時々、それらを印刷するときに同じカードを得ることができます。それをどうやって解決でき、私のミスはどこですか？事前のおかげで

Answer 1

あなたは自分のusedリストに描かれたカードを追加する必要はなく、背面にdeck：

used.append(drawn_card) #instead of deck.append(..) 

Answer 2

while (a < 2): 
    drawn_card = random.choice(deck) 
    used.append(drawn_card) 
    deck = [f for f in deck if f not in used] 
    p1.append(drawn_card) 
    a+=1 

used.append（drawn_card）

Answer 3

シンプルに他の人が示唆しているようにused.append(drawn_card)を使用するように修正しました。

CARDS_PER_HAND = 2 

random.shuffle(deck) 
hand = deck[:CARDS_PER_HAND] 

は、しかし、あなたはおそらく同じデッキから複数の手を扱っている： 良い方法は、ランダムにあなたのデッキをシャッフルし、それから、カードの必要数をスライスすることです。あなたはシャッフルされたデッキへのオフセットを追跡し、そこからスライスすることでそれを処理できますが、それは面倒です。それとも、deckのコピーを作成し、それをシャッフルし、それぞれの手のためのカードの必要数をオフにポップすることができます。

shuffled_deck = deck[:] # make a copy of deck 
random.shuffle(shuffled_deck) 
hand1 = [shuffled_deck.pop() for i in range(CARDS_PER_HAND)] 
hand2 = [shuffled_deck.pop() for i in range(CARDS_PER_HAND)] 
... etc. 

おそらく最もクリーンな方法は、イテレータから最初のnアイテムを取るためにitertools.islice()を使用することです：

from itertools import islice 

random.shuffle(deck) 
shuffled_deck = iter(deck) # returns an iterator 
hand1 = list(islice(shuffled_deck, CARDS_PER_HAND)) 
hand2 = list(islice(shuffled_deck, CARDS_PER_HAND)) 
... etc.