pythonで、for文の中にwhile文のスクリプトが動きません

Question

python にて、乱数を生成し、その乱数が変数filter以下の場合にmain_listに組み込みたいのですが…

import random
max_num = 100
for i in range(10):
    main_list = []
    for j in range(10):
        rand_num = random.randint(0,max_num)
        filter = rand_num + 10
        judge = None        
        if filter <= max_num:
            judge = True
        elif filter >= max_num:
            judge = False
        while judge is True:
            main_list.append(rand_num)

このスクリプトを走らせるとpython自体が動かなくなってしまいます…　；(
汚いスクリプト、説明不足でしたらですいません。

windows10
python3.6.2
pycharmを使っています

Answer 1

値judgeは外因的に書き換えられることがないので、無限ループになります。

while judge is True:
main_list.append(rand_num)

この場合一度だけ処理が行われればいいので、if文で充分です。

import random

max_num = 100

for i in range(10):
    main_list = []
    for j in range(10):
        rand_num = random.randint(0,max_num)
        filter = rand_num + 10
       
        if filter <= max_num:
            main_list.append(rand_num)

ついでに

気になる点を列挙します。

• main_listが何回も上書きされてしまうこと

外側のループが繰り返される度に上書きされます。

• 条件が排他的なのにelifを使っていること

『もし〇〇ならば～、そうでなければ～』という排他的な条件の場合、if-elseが適しています。

• 条件式に xx is True が含まれていること

『「xxが真である」ことが真である』という意味になり冗長です。

• judgeに一時的にNoneを代入していること

Pythonでは変数の宣言はしなくて良いので、必要ないです。

ご不明な点はコメントにて遠慮なくご質問ください。

書いてみた

ちょっと要件が曖昧なので、かなり想像で埋めていますが、こんな感じでしょうか。

from random import randint

def my_rand():
    return randint(0, 100)

main_list = []
for i in range(10):
    inner_list = []

    for j in range(10):
        num = my_rand()
        if num <= 90:
            inner_list.append(num)

    main_list.append(inner_list)

print(main_list)

実行結果 Wandbox

[
[69, 37, 17, 18, 81, 64, 22, 49, 34], 
[36,  6, 15, 12, 25, 11, 28, 86, 82, 86], 
[27, 14, 39, 70, 56, 84, 86, 38,  6, 67], 
[34, 35, 26, 57, 23,  8, 25, 68, 10, 55], 
[40, 13, 60, 33,  0,  8, 88, 60], 
[21, 73, 16,  5, 84, 40, 15, 27,  6, 25], 
[52, 32, 53, 79, 10, 49, 47, 64,  4, 87], 
[46, 10, 68, 12, 26, 50, 70, 18, 56, 40], 
[49, 80, 56, 90, 65, 60,  4, 34, 30], 
[86, 60, 86, 80, 33, 21, 56, 46]
]

**註1: **見やすいように手動で成形しています。
**註2: **疑似乱数を用いているので、もちろん毎回結果は異なります。

Comments

[umyu]

>> LouiS0616さんへ
実行環境に依存しないように、テスト向けにrandom.seedがありまして、たとえばrandom.seed(42)などを記載しておくと、どの環境でも同じ結果を再現させることができます。

[LouiS0616]

@umyu さん
シード値が設定できることは存じておりますが、再現が必要というわけでもないので使いませんでした。
ともすれば回答するときだけ省いておいて、『もし再現したかったらシードにxxを設定してね』というのもありかもしれませんね。

[mkgrei]

from pprint import pprint
pprint(main_list)

すると成形無しで見やすくなります。

[LouiS0616]

@mkgrei さん
pprint、便利ですよね。実は回答のコードを出来るだけ平易にしようと言う狙いから、今回は手動での成形を選びました。(こっそり桁数も合わせています)

[MukeAtiek]

回答ありがとうございます！解決しました！

Answer 2

numpy.randomを見たほうがいいような。

https://docs.scipy.org/doc/numpy-1.14.0/reference/routines.random.html

Answer 3

入力サンプルと求めている出力結果を記載された方がベストな回答が付きやすいと思います。
0～ (max_num - 10)及びそれを含む範囲の乱数を取得したい時は以下のように記述できます。

rand_num = random.randint(0, max_num - 10 + 1)
main_list.append(rand_num)