Python 入力形式の違いにエラーを返す方法

Question

前提・実現したいこと

Pythonで５つの異なる数字の入力を求めているため、同じ数字の入力はできないというメッセージをだして、入力しなおす形にしたいです。

発生している問題・エラーメッセージ

同じ文字が2回3回あると何度もメッセージが表示されます。
'42377'など4桁目と5桁めが同じ場合は受け付けられてしまいます。

該当のソースコード

def f():         
    while True:
        n = input("５つの異なる数字を入れてください： ")
        nList =list(n)
        try:
            int(n)
            if len(n)== 5:
                for i in range (5):
                    if nList.count(nList[i]) == 1:
                        return n
                    else:
                        print("同じ数字は使えません.")                        
            else:
                print("5桁の数字が必要です")
        except ValueError:
            print("数字を入れてください")
        except:
            print("入力したものは受け付けできませんでした")
f()

試したこと

printがレンジの中にはいっているため、メッセージは複数回出ているのだと思います。
他にどういうやり方があるのかが思い当たりません。
どうぞよろしくお願いします。

Comments

[meg_]

「'42377'など4桁目と5桁めが同じ場合は受け付けられてしまいます。」 ← 理由は分かっていますか？
「printがレンジの中にはいっているため、メッセージは複数回出ているのだと思います。」 ← ”思う”ではなくデバッグして確かめましょう。

Answer 1

if len(n) == len(set(n)):
    return n
else:
    print("同じ数字は使えません.") 

ではいかがでしょうか

Comments

[doigao]

確かに一番シンプルで使いやすいです。ありがとうございました。

Answer 2

p.py

def check(s):
  try:
    num = int(s)
  except ValueError:
    raise Exception("数字を入れてください。")
  if num < 0:
    raise Exception("数字だけを入れてください。")

  if len(s) != 5:
    raise Exception("5桁の数字が必要です")
  if len(set(list(s))) != 5:
    raise Exception("同じ数字は使えません.")
  
def f():
    while True:
      n = input("５つの異なる数字を入れてください： ")
      try:
        check(n)
        return n
      except Exception as e:
        print(e)

f()

実行例

-1234。 01234 なども正しく処理できています。

Comments

[doigao]

ありがとうございます。正しく作動します。
ただ一つの関数にするようにとのことだったので、参考にしてやってみます。

[katoy]

> ... 正しく作動します ...
+1234 がどう処理されるかも確認してみてください。

[doigao]

確かに-1234だと
数字だけを入れてください
となるのに
+1234だと受け付けます。どうしてなのでしょうか？

[katoy]

自分でバグを見つけ、それを自分で解決していくことが大事です。
len("+1234") は 5 だからです。
set(s) の要素がすべて 数字 であることのチェックをする必要があります。
len(str(int(s)) == 5 というチェック方法にしても良いかもしれません。

[doigao]

ありがとうございました。いろいろやってみます。

Answer 3

次の箇所で、最初の文字が 1 文字のみであれば処理を終えてしまっていることが原因です:

                for i in range (5):
                    if nList.count(nList[i]) == 1:
                        return n
                    else:
                        print("同じ数字は使えません.")                        

「いま何をしているかよくわからない」 を防ぐために

• 正常処理と例外処理 を考え、例外処理への分岐判定を if の条件にし、
  例外処理 が終わった時点で continue や raise, return などを用いて
  正常処理を終わらせてしまいます
• ブロックの入れ子は 2 段階までに制限するようにします

例:

def validate(nList):
    for i in range (5):
        if nList.count(nList[i]) != 1:
            raise Exception()


def f():         
    while True:
        n = input("５つの異なる数字を入れてください： ")
        nList = list(n)
        print(nList)
        try:
            int(n)
        except ValueError:
            print("数字を入れてください")
            continue
        if len(n) != 5:
            print("5桁の数字が必要です")
            continue
        try:
            validate(nList)
        except Exception:
            print("同じ数字は使えません.")                        


f()

実行結果:

$ python test.py
５つの異なる数字を入れてください： 1234a
['1', '2', '3', '4', 'a']
数字を入れてください
５つの異なる数字を入れてください： 1234
['1', '2', '3', '4']
5桁の数字が必要です
５つの異なる数字を入れてください： 11111
['1', '1', '1', '1', '1']
同じ数字は使えません.
５つの異なる数字を入れてください： 42377
['4', '2', '3', '7', '7']
同じ数字は使えません.
５つの異なる数字を入れてください： 42378
['4', '2', '3', '7', '8']
５つの異なる数字を入れてください：

Comments

[doigao]

ありがとうございます。正しい答え（異なる5桁の数字）を入れた後もループをくりかえしてしまうようですが、参考にします。

[y_shinoda]

質問の内容に対する回答ではないのでそこは敢えてそのままにしましたが、
正しい答えを入力した場合に終了させたい場合は
while ブロックの最後に return を追加してください

[doigao]

了解しました。ご丁寧な解説ありがとうございました。

Answer 4

同じ文字が2回3回あると何度もメッセージが表示されます。

ループの中にprintがあるからだと思います。
一回のみ表示させたいのであれば、ループ結果がすべて真の時にのみ n を返すようにすればいいと思います。

##'42377'など4桁目と5桁めが同じ場合は受け付けられてしまいます。
nList.count(nList[i])
はiがいくつnListに含まれるかを表しています。上記コードのforループではiに0~4までの値しか入っていないので、恐らく5以上の値が二つ含まれているときには受け付けられてしまいます。
for i in range (5):
を
for i in range (10):
に変えればよいと思います。

間違っている点、不明瞭な点があったら教えていただければ幸いです。