0

这是一个非常奇怪的错误,我不知道发生了什么。基本上我的代码是最重要的,它还没有完全完成,但无论如何。对于那些玩过顶级王牌的人,我的代码模拟了两只手之间的牌传递。但是,如果您查看if len(Your_Hand)== 0:我使用 len() 打印这些手的长度的第 129 行 (),它们不会相加。即 len() 表示读取列表时显然不正确的长度?

谢谢,如果您能看一下并尝试帮助我了解正在发生的事情,那将是很棒的!

这是我的代码:

#top trumps stuff
import random
import time

pack = [("harry Potter",2,3,4,5),
         ("Hermione Granger", 5,6,7,8),
         ("Ron Weasley", 12, 13,4,5),
         ("Neville Longbottom", 1,1,1,1),
         ("Ginny Weasley",2,3,4,5),
         ("Draco Malfoy",3,6,7,8) ]

pile_1 = []
pile_2 = []
clash_pile = []
Your_Hand = 0
Opponent_Hand = 0
end_loop=0
catagory_exchange = {"brains":0,
                     "knowledge": 1 ,
                      "cunning": 2, 
                      "evil":3}

choice = 0
acceptable_answers = catagory_exchange.keys()
proceed = 0
clash = 0 
end_game = 0

#Shuffles the pack
random.shuffle(pack)

# Deals the cards
Dealer_count =0
while Dealer_count <len(pack):
    pile_1.append(pack[0])
    pack.remove(pack[0])
    pile_2.append(pack[0])
    pack.remove(pack[0])

#Asks Which pile they'd like
Which_Pile = raw_input(">> Which Pile would you like 1 or 2?")

while end_loop  != 1:
    if Which_Pile == "1":
        Your_Hand = pile_1
        Opponent_Hand = pile_2
        print "Lets now start!"
        end_loop = 1

    elif Which_Pile == "2":
        Your_Hand = pile_2
        Opponent_Hand = pile_1
        print "Lets now start!"
        end_loop = 1
    else:
        Which_Pile =  raw_input("You didn't pick 1 or 2! Pick again!!")

while end_game != 1:

    print len(Your_Hand)
    print Your_Hand

    print len(Opponent_Hand)
    print Opponent_Hand

    print Your_Hand[0]
    choice = raw_input(">> choose a catagory, brains, knowledge, cunning, evil - ")

    if choice in acceptable_answers:
        proceed = 1
    else:
        print "Thats not an catagory"

    if proceed == 1:

        if Your_Hand[0][catagory_exchange[choice]] > Opponent_Hand[0][catagory_exchange[choice]]:
            if clash == 1:
                print "You won the cards from previous rounds aswell!!"
                clash = 0
                clash_pile[:] = []

            else:
                print "You won the card you received %s" % Opponent_Hand[0][0]
            Your_Hand.append(Opponent_Hand[0])
            Your_Hand.append(clash_pile)
            Opponent_Hand.remove(Opponent_Hand[0])
            Your_Hand.append(Your_Hand[0])
            Your_Hand.remove(Your_Hand[0])

        elif Your_Hand[0][catagory_exchange[choice]] < Opponent_Hand[0][catagory_exchange[choice]]:
            if clash ==1:
                print "You lost the cards from previous rounds aswell"
                clash = 0
                clash_pile[:] = []

            else:
                print "You lost the card"
            Opponent_Hand.append(Your_Hand[0])
            Your_Hand.append(clash_pile)
            Your_Hand.remove(Your_Hand[0])
            Opponent_Hand.append(Opponent_Hand[0])
            Opponent_Hand.remove(Opponent_Hand[0])

        if Your_Hand[0][catagory_exchange[choice]] == Opponent_Hand[0][catagory_exchange[choice]]:
            clash =1
            print "They both have the same value"
            print "They have been added to a pile which you will win when you win the next round"
            clash_pile.append(Your_Hand[0])
            clash_pile.append(Opponent_Hand[0])
            Opponent_Hand.remove(Opponent_Hand[0])
            Your_Hand.remove(Your_Hand[0])

if len(Your_Hand)== 0:
    print "Oh no! You have run out of cards! You lose!!"

if len(Opponent_Hand)==0:
    print "Well done! Your opponent has run out of cards! You win!!"
4

2 回答 2

4

几轮之后,我的手牌长度为 3,由一张罗恩韦斯莱牌和两张空单组成:[('Ron Weasley', 12, 13, 4, 5), [], []]。我猜你不想在那里有那些空列表。

罪魁祸首似乎是这一行:

Your_Hand.append(clash_pile)

这总是会在您的手上添加一个空列表。

你可能想要extend而不是append. 此外,您应该在将冲突堆重置为之前将冲突堆添加到获胜者的手上[]

其他观察结果可能与您的原始问题无关:

  • if可能if Your_Hand[0][catagory_exchange[choice]] == Opponent_Hand[0][catagory_exchange[choice]]:应该是elif
  • 无论谁实际上赢得了冲突,您都会在冲突堆中获得牌
  • 当我拿走对手的最后一张牌时,游戏并没有结束
  • 如果每张牌都进入冲突,玩家就没有什么可打的了(尽管这可能是顶级王牌固有的问题)
于 2013-03-05T21:16:57.157 回答
1

我重写了你的程序,它现在对我有用。我发现了几件事:

最大的问题是@Kevin 在他的回答中指出的问题:您使用list.append()而不是list.extend()clash_pile.

此外,category_exchange包含偏离 1 的值。每张卡片中的第 0 项是一个字符串,选择brains会导致卡片名称之间的字符串比较。(当我不断地和 Neville Longbottom 赢牌时,我很困惑!)

我重写了移动卡片的代码。我不是先调用.append()然后调用.remove(),而是调用.pop()删除值并返回它,然后在内部进行.append()。当这一切都在一行时,您不太可能看到有人只更改两行代码之一的错误。

此外,我不是在找到可接受的输入时标记变量来发出信号,而是使用无限循环并在找到可接受的输入时简单地中断。

此外,我没有使用标志变量来表示“冲突堆中有卡片”之类的东西,或者检查卡片列表上的长度为 0,而是将其更改为直接检查列表值。

另外,由于我很懒,我让程序接受一个类别的数字。而不是打字knowledge,你可以只打字2,它就可以工作。

如果您有任何问题,请告诉我。

#top trumps stuff
import random
import time

pack = [
    ("Harry Potter",2,3,4,5),
    ("Hermione Granger", 5,6,7,8),
    ("Ron Weasley", 12, 13,4,5),
    ("Neville Longbottom", 1,1,1,1),
    ("Ginny Weasley",2,3,4,5),
    ("Draco Malfoy",3,6,7,8)
]

pile_1 = []
pile_2 = []
clash_pile = []
category_exchange = {
    "brains" : 1,
    "knowledge" : 2 ,
    "cunning" : 3, 
    "evil" : 4,
}

#Shuffles the pack
random.shuffle(pack)

# Deals the cards
while pack:
    pile_1.append(pack.pop(0))
    pile_2.append(pack.pop(0))

#Asks Which pile they'd like

while True:
    Which_Pile = raw_input(">> Which Pile would you like 1 or 2?")
    if Which_Pile.strip() == "1":
        Your_Hand = pile_1
        Opponent_Hand = pile_2
        print "Lets now start!"
        break
    elif Which_Pile.strip() == "2":
        Your_Hand = pile_2
        Opponent_Hand = pile_1
        print "Lets now start!"
        break
    else:
        print "You didn't pick 1 or 2! Pick again!!"

while True:
    if not Your_Hand:
        print "Oh no! You have run out of cards! You lose!!"
        break

    if not Opponent_Hand:
        print "Well done! Your opponent has run out of cards! You win!!"
        break

    print "\n"

    print "DEBUG: Opponent: len: %d  top_card: %s" % (len(Opponent_Hand), str(Opponent_Hand[0]))
    print "DEBUG: You: len: %d  top_card: %s" % (len(Your_Hand), str(Your_Hand[0]))

    print "Here is your top card: '%s'" % str(Your_Hand[0])

    while True:
        choice = raw_input(">> choose a category, brains, knowledge, cunning, evil: ")

        if choice in category_exchange:
            i = category_exchange[choice]
            break
        else:
            try:
                i = int(choice)
            except ValueError:
                pass
            if i in category_exchange.values():
                break
        print "That's not a category"


    print "Opponent had: %s" % str(Opponent_Hand[0])

    if Your_Hand[0][i] > Opponent_Hand[0][i]:
        if clash_pile:
            print "You won the cards from previous rounds as well!!"
        else:
            print "You won the card!"

        Your_Hand.append(Opponent_Hand.pop(0))
        Your_Hand.extend(clash_pile)
        clash_pile = []
        Your_Hand.append(Your_Hand.pop(0))

    elif Your_Hand[0][i] < Opponent_Hand[0][i]:
        if clash_pile:
            print "You lost the cards from previous rounds as well"
        else:
            print "You lost the card!"

        Opponent_Hand.append(Your_Hand.pop(0))
        Opponent_Hand.extend(clash_pile)
        clash_pile = []
        Opponent_Hand.append(Opponent_Hand.pop(0))

    else:
        assert Your_Hand[0][i] == Opponent_Hand[0][i]
        print "Both cards have the same value."
        print "They have been added to a pile which will go to the next winner."
        clash_pile.append(Your_Hand.pop(0))
        clash_pile.append(Opponent_Hand.pop(0))
于 2013-03-05T21:52:55.917 回答