Python正規表現に関して

以下は、入力された文字列が大文字から始まり、小文字と数字を含み、かつ文字の長さ8以上かを判定する関数です。プログラムがあまり美しくないため、特に最初の大文字から始まるかの判定など、文字列の先頭を取ってきて正規表現を使っていますが、これ以外にもっと効率的な(大文字判定に限らず)書き方はありませんでしょうか？自分ならこう書くといったアイディアがありましたら、ぜひご教授ください。

python
1def strong_pw(string: str) -> str:
2    import re
3    re_starts_with_upper = re.compile(r'[A-Z]')
4    re_contains_alpha = re.compile(r'[a-z]+')
5    re_contains_num = re.compile(r'\d+')
6    if len(string) < 8:
7        return "パスワードが短すぎます。"
8    else:
9        mo_starts_with_upper = re_starts_with_upper.search(string[0])
10        if mo_starts_with_upper == None:
11            return "パスワードは大文字から始めてください。"
12        mo_contains_alpha = re_contains_alpha.search(string)
13        if mo_contains_alpha == None:
14            return "パスワードは小文字も含むようにしてください。"
15        mo_contains_num = re_contains_num.search(string)
16        if mo_contains_num == None:
17            return "パスワードは数字を含むようにしてください。"
18    
19    return "強度は十分です。"

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

回答2件

ベストアンサー

基本的には失敗したらエラー文字列を返すif文の羅列で組みます。提示のコードではelseは不要です。
なお、提示の場合はすべて正規表現でチェックできるので以下のようなループで処理することもできます。

Python
1import re
2
3def strong_pw(string: str) -> str:
4
5    checks = [(re.compile(r'.{8,}'),  "パスワードが短すぎます。"),
6              (re.compile(r'^[A-Z]'), "パスワードは大文字から始めてください。"),
7              (re.compile(r'[a-z]+'), "パスワードは小文字も含むようにしてください。"),
8              (re.compile(r'\d+'),    "パスワードは数字を含むようにしてください。")]
9
10    for m,mes in checks:
11        if m.search(string) is None:
12            return mes
13
14    return "強度は十分です。"
15
16print( strong_pw('Aa1'))
17print( strong_pw('aA1*****'))
18print( strong_pw('AA1*****'))
19print( strong_pw('Aaa*****'))
20print( strong_pw('Aa1*****'))