Q&A
正規表現なら一行で書ける気が。
前提・実現したいこと
pythonでユーザーがパスワードを入力する際に最低１０文字以上であること、大文字小文字数字を含んでいるかを確認するコードを作ろうとしているのですが、思い通りになりません。
発生している問題・エラーメッセージ
エラーメッセージ
該当のソースコード
python
1def checkio(password): 2 upp = ["A","B","C","D","E","F","G","H","I","J","K","L","M","N","O","P","Q","R","S","T","U","V","W","X","Y","Z"] 3 low = ["a","b","c","d","e","f","g","h","i","j","k","l","m","n","o","p","q","r","s","t","u","v","w","x","y","z"] 4 digit =[1,2,3,4,5,6,7,8,9,0] 5 up = False 6 lo = False 7 dig = False 8 if len(password)>=10: 9 10 for letter in password: 11 12 13 if letter in upp: 14 up =True 15 if letter in low : 16 lo = True 17 if letter in digit: 18 dig =True 19 if (up == True) and (lo == True) and (dig == True): 20 return True 21 return False 22print(checkio("aallllllllfjh1G"))
試したこと
このケースで、１０文字以上で大文字小文字数字を含んでいるのでTrueを出したいのですがFalseになってしまいます。どこか見落としているところがありましたらご指摘お願いいたします。
補足情報（FW/ツールのバージョンなど）
ここにより詳細な情報を記載してください。
回答4件
0
投稿2019/08/23 22:27
総合スコア22324
0
ベストアンサー
数字文字かの判定においてdigit=[1,2,...のように文字ではなく数値のリストを用いているのがうまくいかない原因です。該当文字かを判定するには正規表現を用いると少し簡潔に書けます。
Python
1import re 2 3def checkio(s): 4 if len(s) < 10: 5 return False 6 7 is_match = [0,0,0] # 大文字、小文字、数字なら各要素に1をセット 8 for c in s: 9 if re.match(r'[A-Z]',c): 10 is_match[0] = 1 11 elif re.match(r'[a-z]',c): 12 is_match[1] = 1 13 elif re.match(r'[0-9]',c): 14 is_match[2] = 1 15 # もし「上記のみで構成される」という制約をつけたいなら、以下のコメント部分を生かす 16 #else: 17 # return False # それ以外はだめ 18 19 return sum(is_match) == 3 # すべてを含む 20 21for s in ['1aA', '1aA2234567-','1aＡ22345678', '1aA2345678']: 22 print(s, checkio(s))
なお、pythonのis系関数は全角もTrueと判定するので注意が必要です。
Python
1print('１'.isdigit()) # True! 2print('Ａ'.isupper()) # True!
投稿2019/08/20 22:50
編集2019/08/20 22:54
総合スコア38258
なんだか面倒そうな表現になりそうなので妥協しました…
回答ありがとうございます！
正規表現をというモジュールを知らなかったのでとても勉強になりました！
もう一つ質問したいことがあります。
if re.match(r'[A-Z]',c):
is_match[0] = 1
elif re.match(r'[a-z]',c):
is_match[1] = 1
elif re.match(r'[0-9]',c):
is_match[2] = 1
この部分で、if,elif.elifと続いているところをif,if,ifではダメな理由を知りたいです。
すごく初歩的な質問ですみません！よろしくお願いします。
ダメではないですが無駄だからです。
たとえばもし英大文字なら小文字や数字でないことは明らかなので後続のif判定をする必要がないです。
なるほど！！ありがとうございます！！elifにすればそこを通さずに済むのですね！納得しました。
それは納得したのですが、if,if,ifにしたらTrueを出したいケースでFalseが出てしまいます、これはifだとムダだからという理由のほかに何かそれだとうまくいかない原因があると思うのですが、なぜ違う結果が出てしまうのですか？
ありがとうございます。
実際のコードとFalseが出てしまう具体的なケースが不明なので何ともいえませんが
３つの条件は互いに排他なので理屈ではif,if,ifでも同じ判定結果になるはずです。
丁寧なご回答感謝しています。ありがとうございます！！
0
以下のようにして、分岐条件を確認すると良いです。
python
1if letter in upp: 2 print("upp") 3 up =True 4if letter in low : 5 print("low") 6 lo = True 7if letter in digit: 8 print("dig") 9 dig =True
同じ型で比較してあげると良いですよ。
投稿2019/08/20 21:51
退会済みユーザー
総合スコア0
0
isupper、islower という関数があります
投稿2019/08/20 21:13
総合スコア87747
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。