Python3パスワードが有効かを判定したいです。

Question

パスワードとして有効かを判定したい

コード
def password_check(i):
list1=["a","b","c","d","e","f","g","h"]
m=["A","B","C","D","E","F","G","H"]
n=[0,1,2,3,4,5,6,7,8,9]
p=["!","#","$","¥","%"]
list1+=m
list1+=n
list1+=p
for j in i:
if j in list1:
continue
return True
else:
return False

print(password_check("Caf02!"))

#パスワードがm,n,p,qのリストのいずれかを含んでいたらTrue、リスト以外が一つでも含まれていたらFalseと返したいのです
#が実行するとFalseと返ってきてしまいます。どうしたらよいかわからなくなってしまいました。よろしくお願いします。

Comments

[udon-ken]

インデントをきちんと付けて下さい。
あと、細かい話しですが、iとかjとかは通常整数型の変数に使います。
文字列に使うと他人からは非常に読みにくくなるのでやめた方がいいです。
変数名が思いつかないのなら、
for char in passwordとかでいいのではないかな。
こうすると何をしているのか誰が見てもわかりやすいです。

[dokugakusets]

ありがとうございます。iやjは整数型の変数用なのですね。勉強になりました！

Answer 1

toshikawaさんの回答をもとにした集合を使った例です。
それぞれの文字セットの集合の差をとることで簡潔に判定できます。

def password_check(passwd):
    import string

    strngs = set(string.ascii_letters+string.digits+'!#$¥%')
    passwd = set(passwd)

    return not bool(passwd - strngs)

print(password_check("Caf02!")) # True
print(password_check('Caf02__'))# False

Comments

[toshikawa]

すごいpythonicな回答ですね。集合の扱いは全然慣れていないので参考になります。

[can110]

標準で基本的なデータ型が用意されてて気軽に使えるのはPythonの魅力のひとつですね。

[dokugakusets]

わあ！すごいです！こうやってまとめてスッキリさせる方法があるのですね！
すごく見やすいです。
ありがとうございます！！！

Answer 2

def password_check(passwd):
    import string

    strngs = string.ascii_letters+string.digits+'!#$¥%'
    # strngs = abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789!#$¥%

    for s in passwd:
        if s in strngs:
            pass
        else:
            return False
    return True

print(password_check("Caf02!"))
print(password_check('Caf02__'))
"""
True
False
"""

Pythonだと、文字列はそのままforで取り出せますので、特にリストにしなくても大丈夫です。
また、inによる判定も大丈夫です。

あとは、便利なモジュールがありますので使っていきましょう。

Comments

[dokugakusets]

ありがとうございます！string関数は初めてです。すごくラクにアルファベットや数字が出せるのですね！すごく勉強になりました！それからforとreturnの位置が今まで悩んでいたことへの解決につながりました！！！

Answer 3

n=[0,1,2,3,4,5,6,7,8,9]

を

n=["0","1","2","3","4","5","6","7","8","9"]

に修正しましょう。