上記のコメントは別として、sum は実際には Python の組み込み関数であり、探しているように見えるだけの機能を備えているため、上書きせずに識別子名として使用してください。代わりに使用してください。
また、すべての Python プログラマーが従うことを意図したスタイル ガイドもあります。これは、Python コードと、Perl や PHP などの他の言語で書かれたコードでよく遭遇する不可解なスラッジとをさらに区別するのに役立ちます。Python にはより高い標準があり、それを満たしていません。スタイル
したがって、不足している部分を埋めるためのいくつかの推測とともに、コードを書き直したものを次に示します。
from random import randint
CARD_FACES = {1: "Ace", 2: "2", 3: "3", 4: "4", 5: "5", 6: "6", 7: "7", 8: "8",
9: "9", 10: "10", 11: "Jack", 12: "Queen", 13: "King"}
def deal():
"""Deal a card - returns a value indicating a card with the Ace
represented by 1 and the Jack, Queen and King by 11, 12, 13
respectively.
"""
return randint(1, 13)
def _get_hand_value(cards):
"""Get the value of a hand based on the rules for Black Jack."""
val = 0
for card in cards:
if 1 < card <= 10:
val += card # 2 thru 10 are worth their face values
elif card > 10:
val += 10 # Jack, Queen and King are worth 10
# Deal with the Ace if present. Worth 11 if total remains 21 or lower
# otherwise it's worth 1.
if 1 in cards and val + 11 <= 21:
return val + 11
elif 1 in cards:
return val + 1
else:
return val
def show_hand(name, cards):
"""Print a message showing the contents and value of a hand."""
faces = [CARD_FACES[card] for card in cards]
val = _get_hand_value(cards)
if val == 21:
note = "BLACK JACK!"
else:
note = ""
print "%s's hand: %s, %s : %s %s" % (name, faces[0], faces[1], val, note)
# Deal 2 cards to both the dealer and a player and show their hands
for name in ("Dealer", "Player"):
cards = (deal(), deal())
show_hand(name, cards)
さて、私は夢中になって実際にすべてを書きました。別の投稿者が書いたように、sum(list_of_values) が良い方法ですが、実際にはブラック ジャックのルールには単純すぎます。