example-code-2e icon indicating copy to clipboard operation
example-code-2e copied to clipboard

More consistent way to get suit value and suits length

Open ricefryegg opened this issue 2 years ago • 1 comments

Change from using different way to get suit value than rank_value

suit_values = dict(spades=3, hearts=2, diamonds=1, clubs=0)

def spades_high(card):
    rank_value = FrenchDeck.ranks.index(card.rank)
    return rank_value * len(suit_values) + suit_values[card.suit]

To be more consistent. Also we can get suit_values from existing class variable FrenchDeck.suits.

def spades_high(card):
    rank_value = FrenchDeck.ranks.index(card.rank)
    suit_value = FrenchDeck.suits.index(card.suit)
    return rank_value * len(FrenchDeck.suits) + suit_value

ricefryegg avatar Apr 06 '22 00:04 ricefryegg

Also, I may need glasses, but in the listing page 5, the order of the suits is not the same as the order in suit_values on page 7.

Florimond avatar Aug 26 '22 16:08 Florimond