Python 我的洗牌计划只给我球杆作为套装

我必须为我要上的编程课做一副洗牌牌牌，我真的被难住了。我的大部分代码都是不必要的，因为我们正在学习函数。我想我所有的东西都是对的，但我的输出只是给了我球杆作为套装

import random

def define_cards():
    rank_string = ("ace","two","three","four","five","six","seven","eight","nine","ten","jack","queen","king")
    suit_string = ("clubs","diamonds","hearts","spades")
    cards = []
    for suit in range(4):
        for rank in range(13):
            card_string = rank_string[rank] + " of " + suit_string[suit]
            cards.append(card_string)
        return cards

def create_deck(deck):
    for i in range(52):
        deck.append(i)
        return

def shuffle_deck(deck):
    random.shuffle(deck)
    return

def deal_card(deck):
    return deck.pop(0)

deck=[]

deck = define_cards()
shuffle_deck(deck)
print ("The first 10 cards are:")
for i in range(10):
    card = deal_card(deck)
    print (card)

结果应该是一样的，但是随机的。

---

请帮帮我

在define_cards函数中

return

应该在for循环之外

创建甲板功能也一样

def define_cards():
    rank_string = ("ace","two","three","four","five","six","seven","eight","nine","ten","jack","queen","king")
    suit_string = ("clubs","diamonds","hearts","spades")
    cards = []
    for suit in range(4):
        for rank in range(13):
            card_string = rank_string[rank] + " of " + suit_string[suit]
            cards.append(card_string)
    return cards

---

在define_cards函数中，返回应在for循环之外

创建甲板功能也一样

def define_cards():
    rank_string = ("ace","two","three","four","five","six","seven","eight","nine","ten","jack","queen","king")
    suit_string = ("clubs","diamonds","hearts","spades")
    cards = []
    for suit in range(4):
        for rank in range(13):
            card_string = rank_string[rank] + " of " + suit_string[suit]
            cards.append(card_string)
    return cards

---

create_deck

功能是否与

deck=range（52）

@Brady创建deck的最佳方法是

itertools.product

。如果您愿意，您可以阅读它。

create_deck

功能是否与

deck=range（52）

@Brady创建deck的最佳方法是

itertools.product

。如果你愿意，你可以阅读它。另外，对于

create_deck

函数也有相同的注释。我很确定

create_deck（）

是不必要的-事实上，它从未被调用过！你说得对@Ken。但是为了调试起见，我提到了它。另外，对于

create\u deck

函数也有相同的注释。我非常确定

create\u deck（）

是不必要的-事实上，它从未被调用过！你说得对@Ken。但只是为了调试，我提到了它。