python3 トランプを引いたらデッキから削除したい

Question

デッキからユーザーに引いたカードを移して、
ユーザー手札から、存在するカードをデッキから削除したい。

#問題
削除するカードがが有りませんと、エラーが出てしまいます。

どうしたら良いでしょうか？
教えて頂きたいです。

cards=['♠A', '♥A', '♣A', '♦A', '♠2', '♥2', '♣2', '♦2', '♠3', '♥3', '♣3', '♦3', '♠4', '♥4', '♣4', '♦4', '♠5', '♥5', '♣5', '♦5', '♠6', '♥6', '♣6', '♦6', '♠7', '♥7', '♣7', '♦7', '♠8', '♥8', '♣8', '♦8', '♠9', '♥9', '♣9', '♦9', '♠10', '♥10', '♣10', '♦10', '♠J', '♥J', '♣J', '♦J', '♠Q', '♥Q', '♣Q', '♦Q', '♠K', '♥K', '♣K', '♦K']

use01=[]
use02=[]

def Sample04():
	use01.append(random.sample(cards, 2))
	
	for i in use01:
		for i2 in i:
			cards.remove(i2)
	#cards.remove(use01[0][i])


for i in range(6):
	Sample04()


print(use01,len(cards))

Answer 1

引いたカードを削除するのではなく、カードを一枚ずつイテレートするという発想で。

from itertools import islice
from random import shuffle


cards=['♠A', '♥A', '♣A', '♦A', '♠2', '♥2', '♣2', '♦2', '♠3', '♥3', '♣3', '♦3', '♠4', '♥4', '♣4', '♦4', '♠5', '♥5', '♣5', '♦5', '♠6', '♥6', '♣6', '♦6', '♠7', '♥7', '♣7', '♦7', '♠8', '♥8', '♣8', '♦8', '♠9', '♥9', '♣9', '♦9', '♠10', '♥10', '♣10', '♦10', '♠J', '♥J', '♣J', '♦J', '♠Q', '♥Q', '♣Q', '♦Q', '♠K', '♥K', '♣K', '♦K']


def Deck():
    D = cards[:]
    shuffle(D)
    yield from D


deck = Deck()
user01 = list(islice(deck, 6))
user02 = list(islice(deck, 6))

print('user01:', *user01)
print('user02:', *user02)
print('rest:', *deck)

Answer 2

もうちょっとスマートに書けそうな気がします。

import random

def make_deck():
    return [
        '♠A', '♠2', '♠3', '♠4', '♠5', '♠6', '♠7', '♠8', '♠9', '♠10', '♠J', '♠Q', '♠K',
        '♥A', '♥2', '♥3', '♥4', '♥5', '♥6', '♥7', '♥8', '♥9', '♥10', '♥J', '♥Q', '♥K',
        '♦A', '♦2', '♦3', '♦4', '♦5', '♦6', '♦7', '♦8', '♦9', '♦10', '♦J', '♦Q', '♦K',
        '♣A', '♣2', '♣3', '♣4', '♣5', '♣6', '♣7', '♣8', '♣9', '♣10', '♣J', '♣Q', '♣K',
        'JOKER', 'JOKER'
    ]

def draw_from_deck(deck, hand, n=1):
    for i in range(n):
        card = random.choice(deck)
        hand.append(card)
        deck.remove(card)

    return hand

def main():
    deck = make_deck()
    hand = draw_from_deck(deck, [], 5)
    print('deck:', deck)
    print('hand:', hand)

if __name__ == '__main__':
    main()

実行結果

deck: ['♠A', '♠2', '♠3', '♠4', '♠5', '♠7', '♠8', '♠9', '♠10', '♠J', '♠Q', '♠K', '♥A', '♥2', '♥3', '♥4', '♥5', '♥6', '♥7', '♥8', '♥9', '♥10', '♥J', '♥Q', '♥K', '♦A', '♦2', '♦3', '♦4', '♦5', '♦6', '♦7', '♦9', '♦10', '♦J', '♦Q', '♦K', '♣A', '♣3', '♣4', '♣5', '♣6', '♣7', '♣8', '♣9', '♣10', '♣Q', '♣K', 'JOKER']
hand: ['♦8', '♣J', 'JOKER', '♣2', '♠6']

カードのソートなどを見据えると、適切にクラス化するとさらに良さそうですね。

Comments

[mkgrei]

本当は人間のように一枚ずつ取り出す必要はなかったです。
シャッフルして、スライスすることと等価です。

[LouiS0616]

おっしゃる通りかと思います。
最初はsetで実装するのが妥当かと考えたのですが、そうするとジョーカーの扱いが面倒なんですよね。
実行速度も大してくわないので、直感的に一枚ずつ引くのもありかと思ったのです。

[mkgrei]

手札を追加するときにも使用できる汎用的な実装で素晴らしいと思います。

Answer 3

use01の中身を書き出して、監視してみてください。
2ラウンド目で、1ラウンド目に取り除いたカードを再度削除しようとするはずです。