【初心者】Python 正規表現によるパスワード形式の確認

Question

前提・実現したいこと

Pythonを使って、「パスワードが8文字以上、大文字、小文字、数字が1つ以上必ず含まれている」かどうかを確認するコードを書きました。
下記でうまくいったのですがこのコードであっていますか？また、よりいいコードがあれば教えてください。

ご回答宜しくお願いいたします。

該当のソースコード

import re

def password_check(password):
    if len(password) < 8:
        return False
    if not re.search(r'[A-Z]', password):
        return False
    if not re.search(r'[a-z]', password):
        return False
    if not re.search(r'[0-9]', password):
        return False
    return True
    
while True:
    password = input('パスワードを入力してください:')
    if password_check(password):
        break
    print('強力なパスワードを設定してください')
    
print('OK')

Answer 1

参考情報

• Validation of a Password - Python

https://stackoverflow.com/questions/41117733/

• Python初心者がパスワードチェッカーの作り方を考えてみた

https://muroiwataru.net/entry/python-pass-check

... コードであっていますか？ ...

不安になったｒ，ぜひ テストコードを書くとよいです。
コンソールからの手入力の繰り返しでもよいですが、
プログラムコード中に 入力例と その判定結果をたくさん書いておいて、
何度でも同じパターンで実行できるようにするとよいです。

sample.py

def password_check(password):
    return True

TESTS = [
    ["1", False],
    ["Aa12345678", True]
]

for p, expect in TESTS:
    result = password_check(p)
    if result != expect:
        print("#--- Error '{}': {} != {}".format(p, result, expect))

Answer 2

その方法で充分分かり易いと思います。
敢えて突っ込むとしたら、次の二点でしょうか。

• 利用できない文字についての条件が無い (aA1!"あ$%&' とかも通ってしまう)
• 再入力を促す際にパスワードの要件を説明した方が良い

---

これだけだと少し寂しいので、別の判定方法も紹介します。
紹介しておいて難ですが、お勧めはしません。コードの意図が通じづらいからです。

def password_check(password):
    return re.search(r'(?=.*[0-9])(?=.*[a-z])(?=.*[A-Z])[0-9a-zA-Z]{8,}', password)

Answer 3

合ってます。１つの正規表現にまとめるより、条件がわかりやすいです。
よりダイレクトに、

「パスワードが8文字以上、大文字、小文字、数字が1つ以上必ず含まれている」

をそのままコードにした、

def password_check(password):
    if len(password) >= 8 \
      and re.search('[A-Z]', password) \
      and re.search('[a-z]', password) \
      and re.search('[0-9]', password):
        return True
    return False

のほうが、さらにわかりやすいと思います。
（文字列中にバックスラッシュが無いので、raw文字列にする必要はないです。してもいいですけど）