質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.92%

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

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 204

doigao

score 2

前提・実現したいこと

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

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

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

該当のソースコード

def f():         
    while True:
        n = input("5つの異なる数字を入れてください: ")
        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がレンジの中にはいっているため、メッセージは複数回出ているのだと思います。
他にどういうやり方があるのかが思い当たりません。
どうぞよろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • meg_

    2020/07/11 18:53

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

    キャンセル

回答 4

checkベストアンサー

+2

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

ではいかがでしょうか

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/11 20:32

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

    キャンセル

0

同じ文字が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):
に変えればよいと思います。

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

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

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

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

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

例:

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


def f():         
    while True:
        n = input("5つの異なる数字を入れてください: ")
        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
5つの異なる数字を入れてください: 1234a
['1', '2', '3', '4', 'a']
数字を入れてください
5つの異なる数字を入れてください: 1234
['1', '2', '3', '4']
5桁の数字が必要です
5つの異なる数字を入れてください: 11111
['1', '1', '1', '1', '1']
同じ数字は使えません.
5つの異なる数字を入れてください: 42377
['4', '2', '3', '7', '7']
同じ数字は使えません.
5つの異なる数字を入れてください: 42378
['4', '2', '3', '7', '8']
5つの異なる数字を入れてください:

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/11 20:20

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

    キャンセル

  • 2020/07/11 20:23

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

    キャンセル

  • 2020/07/11 20:35

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

    キャンセル

0

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("5つの異なる数字を入れてください: ")
      try:
        check(n)
        return n
      except Exception as e:
        print(e)

f()

実行例
イメージ説明

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/07/11 20:39

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

    キャンセル

  • 2020/07/11 20:52

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

    キャンセル

  • 2020/07/11 21:04

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.92%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る