使用迭代器从甲板上分发卡片

Question

我编写了一个函数，允许使用迭代器向玩家分发卡片。我仍然不确定在这里使用迭代器是否是一个好主意。你能提供另一个实现，即在玩家之间分发卡片吗？或者你能纠正我的功能，使它更简洁吗？

import random


mydeck = [r+s for r in '23456789TJQKA' for s in 'SHDC']

def deal(numhands, n=5, deck=mydeck):
 if numhands*n> len(deck):
    return ('One or more parameters of the function might be incorrect')
 random.shuffle(deck)
 iterator=iter(deck)
 result=[]
 for player in range(numhands):
    interim=[]
    for card in range(n):
        interim.append(next(iterator))
    result.append(interim)
 return result

数字-玩家的数量

N-手中的牌数

一副52张牌

Answer 1

• 您可以使用列表理解和切片，这是更“毕多尼”。
• 您可以考虑使用安全的随机数生成器(random.SystemRandom())来防止游戏中的欺骗行为。
• 您应该将全局变量视为一个不断更改的变量，它可能会导致代码并行执行方面的问题。
• 您应该引发异常，而不是返回硬编码错误消息。

'''Card Dealer'''

import random

DECK = [r+s for r in '23456789TJQKA' for s in 'SHDC']

def deal(numhands, handsize=5, deck=DECK):
    '''Returns a list of hands'''
    if numhands*handsize > len(deck):
        raise IndexError("Too many cards requested")
    shuffled = list(deck)
    random.SystemRandom().shuffle(shuffled)
    return [shuffled[i:i+handsize] for i in range(0, numhands*handsize, handsize)]