1~nまでの乱数を重複しないようにリスト化したい

Question

###前提・実現したいこと
1nまでの整数乱数を重複しないようにリストに格納したいです。
よろしくお願いします。
###発生している問題・エラーメッセージ
ループ回数が安定せず、例えば40回行うように入力しても2428回くらいしかループしてくれません。
↓実際に走らせたた画面です。
http://i.imgur.com/um03XCT.png

###該当のソースコード

import random

print "please enter the number of peoples"
ppl = input()

randList = []

x = 1
for x in xrange(ppl):
	temp = random.randint(1,ppl)

	if temp not in randList:
		randList.append(temp)
		x += 1

print randList

###試したこと
for文の中のif文を消して、重複を気にせずそのままリスト化すると設定回数行われたのでif文が原因ということまではわかりました。

Answer 1

結果がおかしいのは
for x in xrange(ppl):
とすると
if temp not in randList:
の結果が False でリストrandList に値が追加されない時も含めて、ループが ppl 回しかまわらない為です。

このアルゴリズムで行いたいのであれば、
for x in xrange(ppl):
の部分を
while len(randList) != ppl:
等として、randListのサイズがpplになるまでループを回すとよいのではないでしょうか。

また、random パッケージにはリストをシャッフルする random.shuffle()という関数があるので

import random
ppl = input()
randList = range(ppl)
random.shuffle(randList)
print randList

と書くこともできますよ

Comments

[AZDDI]

なるほど、見えないところで規定回ループはされてたんですね！ありがとうございます。
shuffle、知らなかったです。活用させていただきます。

Answer 2

この目的なら shuffle を使ったほうが手っ取り早い気がします。

randList = range(ppl)
random.shuffle(randList)
print(randList)

Comments

[AZDDI]

shuffleってものがあるんですね、確かにこっちの方が早そうです！回答ありがとうございました。