«Pop index out the range» для колоды карт

Question

У меня в настоящее время есть код для создания колоды карт, для перетасовки колоды и создания руки из этой колоды.

from random import randint 

def make_deck(): 
    deck = [] 
    for suit in suits: 
     for rank in ranks: 
      deck.append((suit,rank)) 
    return deck 

suits = ['spades','hearts','diamonds','clubs'] 
ranks = ['ace','two','three','four','five','six','seven','eight','nine','ten','jack','queen','king'] 

deck = make_deck() #list of cards 

def shuffle(deck): 
    for k in range(100): # do it 100 times 
     card = deck.pop(randint(0,51)) 
     deck.append(card) 

def make_hand(): 
    hand = [] 
    for k in range(2): 
     card = deck.pop(randint(0,51)) 
     hand.append(card) 
    return hand 

hand = make_hand() 

Когда я запускаю программу, печать deck даст мне список карт, как мне нравится, и тогда, когда я пытаюсь тасует колоду, я получаю ошибку о том, что индекс популярности находится вне диапазона. Не совсем уверен, почему он окажется вне индекса, так как диапазон (0,51) представляет все 52 карты колоды.

Такая же ошибка возникает при попытке использовать функцию make_hand(), хотя и не всегда. Я неправильно использую метод pop? Что мне нужно изменить, чтобы работала функция shuffle, и hand всегда будет возвращать две разные карты из колоды?

Answer 1

Это безопаснее, чтобы написать это как

card = deck.pop(randint(0,len(deck)-1)) 

Хотя это быстрее, если вы сохраните это значение где-то.

Но это также указывает на проблему, с которой сталкивается ваша колода. Правильно ли вы настроили петли в создании колоды? Возможно, вы создаете менее 52 карт, поэтому, когда вы генерируете целое число на верхнем конце, вы получаете эту ошибку. Это также кажется вероятным, если это случается иногда в make_hand (где вы повторяете дважды) и постоянно в случайном порядке (где вы повторяете 100 раз).

Если вы намерены перетасовать карты после того, как раздались руки, вам определенно нужно использовать len() или отслеживать размер колоды каким-либо другим способом. И даже в функции make_hand вы выталкиваете одну карту, а затем все еще пытаетесь сделать из списка размером 52, даже если вы вытащили карту.

Answer 2

pop мутирует список, удаляя объект из списка. Таким образом, длина списка станет меньше, чем 52, и в итоге будет выведена ошибка. Вместо 51 используйте длину списка каждый раз len(deck).

Для перетасовки, вы могли бы также использовал функцию shuffle из библиотеки random:

from random import shuffle 
## later in the code 
shuffle(deck) 

Answer 3

В make_hand вы берете 2 карты из колоды. Первый из них всегда будет успешным, но второй должен работать с колодой, содержащей только 51 карточку, в то время как все еще есть шанс на 1 из 52, чтобы попытаться сделать pop(52), что не удастся.

Я не могу воспроизвести ту же ошибку с помощью функции shuffle, если только я не передам ей колоду, которая была использована в make_hand, поэтому я предполагаю, что это то, что вы сделали.

Просто убедитесь, что в обеих функциях вы можете использовать randint(0, len(deck)-1).