4

私はリストの中からランダムに一つの要素を取り出して代入するコードを以下のように書きました

name = farm[randint(0,randint(0,len(farm)))]

リストは最初は空で、あとから要素を追加していくと言うものです。

しかし、
IndexError: list index out of range
と出てしまいます。
どのようにかけばエラーを避けられるでしょうか
よろしくお願いします。

2 件の回答 2

5

いくつか問題があると思います.

  • 空リストのランダムな要素を取り出す時にどういう結果を期待しているのでしょうか.今のコードでは空リストに対してインデックスでアクセスしているので IndexError が出るでしょう.どうも Python で関数としてこれを提供するものはないようで(python - Why list doesn't have safe "get" method like dictionary?), try-catch するか,事前に空かどうかのテストをし,よしなに処理をするのがいいと思います.
  • random.randint が入れ子で使われているのは意図があってのことでしょうか.単にランダムにひとつ選ぶなら一重で大丈夫そうに見えます.
  • また,このコードでは空でないリストに対してもIndexError が出る可能性があり,これは random.randint(a,b) が a ≤ i ≤ b なる i を返すからです.この場合でいえば len(farm) が返る場合があり,その時には farm[len(farm)] にアクセスが試みられて IndexError が出ます.この用途としては random.randrange を使うのがよいでしょうが,そもそもリストのランダムな要素にアクセスする場合は random.choicerandom.sample が適任かと思います.
1
  • 御丁寧に御助言ありがとうございました。無事解決いたしました
    – tasutasu
    Commented 2015年5月5日 13:52
1

@Yoshがいったように、これには、random.sample(population, k)random.choice(population)が便利です。

  • random.sample()は、リストからkサンプル数を返してくれます。
  • random.choice()は、リストから、ひとつだけのサンプルを返してくれます。

使い方の例:

import random

farm = [ "a", "b", "c", "d"]
one_sample = random.choice(farm)
two_samples = random.sample(farm, 2)

結果:

>>> one_sample
'b'

>>> two_samples
['c', 'a']

ちなみに、random.suffle()を使って、同じことができます

  • random.suffle(x)は、シーケンス x をインプレースにによって混ぜます。

例:

import random

farm = [ "a", "b", "c", "d"]

random.suffle(farm)
# 一つのサンプル
one_sample = farm[0]
two_samples = farm[:2]
1
  • お返事が遅れました。御丁寧にありがとうございます。
    – tasutasu
    Commented 2015年5月30日 3:07

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。