Pythonについて

パスワード文字列 pを引数にとり、ユーザが正しいパスワードをコンソールから入力したら Trueを返すが、そうでない場合は最大で合計3度までユーザに再入力を促し、正しいパスワードが一度も入力されなかった場合に限り False を返す関数 auth (p) を定義せよ

動作例 > はプロンプト、下線部分はユーザの入力を表す)

Python
1
2> auth ('sesame') パスワードを入力してください: apple
3
4パスワードを入力してください: sesame
5
6True
7
8> auth('apple')
9
10パスワードを入力してください: sesame
11
12パスワードを入力してください: orange
13
14パスワードを入力してください: banana
15
16False
17

自分の解答例

Python
1
2def auth(p):
3 i=1
4 while i<=3:
5  a=str(input("パスワードを入力してください:"))
6  if a==p:
7   print(True)
8   break
9  else:
10   i+=1
11   continue
12 print(False)
13

これだとTrueの時にもFalseが表示されます。どうしたらいいですか？

meg_

2021/10/15 10:30

コードはインデントが判るように「コードの挿入」で記入してください。

syota44

2021/10/15 10:38

すみません。やり直しました。

行動規範の内容に同意します

回答6件

while ループを break で抜けているため、
print(False) も実行されています。
break を return にすると意図した動作になると思われます。

Python
1def auth(p):
2    i=1
3    while i<=3:
4        a = str(input("パスワードを入力してください: "))
5        if a==p:
6            print(True)
7            #break
8            return
9        else:
10            i += 1
11            continue
12    print(False)

ちなみに range を使うともっと簡単にループを書けますよ

Python
1def auth(p):
2    for i in range(3):
3        if str(input("パスワードを入力してください: ")) == p:
4            print(True)
5            return
6    print(False)

投稿2021/10/15 11:08

nemoplus

総合スコア161

break の代わりに return でいいんじゃね?

あと continue はいらないんじゃね?

投稿2021/10/15 11:08

takasima20

総合スコア7464

takasima20

2021/10/15 11:10

ついでにいうと else なしで i+=1 はインデント上げていいんじゃね?

takasima20

2021/10/15 11:12

まあ、nemoplusさんの回答のように range 使えばそのへんのメンドーはないスけどね。

行動規範の内容に同意します

これだとTrueの時にもFalseが表示されます。どうしたらいいですか？

Trueのときに関数を抜けるようにするとFalseは表示されません。

Python
1def auth(p):
2 i=1
3 while i<=3:
4  a=str(input("パスワードを入力してください:"))
5  if a==p:
6   print(True)
7   return
8  else:
9   i+=1
10   continue
11 print(False)

投稿2021/10/15 11:03

meg_

総合スコア10760

Python
1
2def auth(p):
3 i=1
4 while i<=3:
5  a=str(input("パスワードを入力してください:"))
6  if a==p:
7   return True
8   break
9  else:
10   i+=1
11   continue
12 return False
13