複数あるlistの判定について

Question

各リストの中にひとつでも10以上の数字がある場合はNGを出力するようなコードを書きたいのですが、うまくいきません。
どなたか教えていただければ幸いです。

lists [[1, 3, 5, 9], [3, 7, 11, 13], [1, 2, 5, 6], [3, 4, 11, 18]]

for list in lists:
    for number in list:
        if 0 < int(number) <= 10:
            continue
        else:
            print("NG")
            break
    print("OK")

上記の場合は、
OK
NG
OK
NG
となるようにしたいのですが、NGと出力したあとにOKが出てしまいます。

またこちらのOK, NGの代わりにTrue, Falseを返すことはできますか？
returnを置くとout of functionと出ますため、defで定義しなければいけないのでしょうか？

Answer 1

2 つの方法を書いてみました。
p.py

def check_lists(list_ary):
    for x in sum(list_ary, []):
        num = int(x)
        if num < 0 or 10 < num:
            return "NG"  # False
    return "OK" # True

def check_lists_x(list_ary):
    return {True} == set([0 < int(x) and int(x) <= 10 for x in sum(list_ary, [])])

list_ary = [[1, 3, 5, 9], [3, 7, 11, 13], [1, 2, 5, 6], [3, 4, 11, 18]]
print(check_lists(list_ary))
print(check_lists_x(list_ary))

list_ary = [[1, 3, 5, 9], [3, 7, 1, 3], [1, 2, 5, 6], [3, 4, 1, 8]]
print(check_lists(list_ary))
print(check_lists_x(list_ary))

実行例

参考情報

• Python 3 で flatten する方法いろいろ

https://qiita.com/hoto17296/items/e1f80fef8536a0e5e7db

Comments

[kentooo]

ありがとうございました！！

Answer 2

『一つでも条件を満たさなかったらアウト』というコードを組むときに、よく見る手段。

for list_ in lists:
    is_ok = True            # とりあえずOKにしとく

    for number in list_:
        if 0 < int(number) <= 10:
            continue 
        else:
            is_ok = False   # 一つでも満たさなかったら即Falseに
            break

    is_okが真なら"OK"、偽なら"NG"と出力

またこちらのOK, NGの代わりにTrue, Falseを返すことはできますか？

真偽値さえ固まれば出力は簡単なので、
判定結果をTrue／Falseで受け取る方法を先に考えた方が後々楽です。

returnを置くとout of functionと出ますため、defで定義しなければいけないのでしょうか？

return文は関数内でしか使えません。
これに回答しようとすると『関数とは何ぞや』という話から始めないといけないので、省きます。

その他のアプローチ

簡潔なコードになりそう順です。ただし簡単順ではありません。

• all関数を利用する
• 自ら関数を定義し、偽が定まった時点でreturn
• for-else文を利用し、else節に正常処理を仕込む

もう少しコーディングに慣れてきたら、それぞれ試してみると良いでしょう。

Comments

[kentooo]

ご丁寧に教えていただきありがとうございました。
初めにis_ok = TrueでとりあえずOKにしておくのはとても納得いきました。
アドバイスいただきました別の関数など、今後試してみたいと思います。

どうもありがとうございました。