パスワードの強化検査 毎回同じ結果が出る エラーメッセージなし

Question

パスワードの強度を調べるプログラムで弱いのにInvalidとでないでValidと出てしまいます。
自分的にはこれで正しいように見えるのですが上手く行きません。
同じ出力になってしまうのです。
条件として
長さが6文字以上
英字と数字の両方を含む必要がある
同じ文字を3つ以上連続で使用してはいけない。

大文字と小文字の区別はいらないです。
この条件をすべて満たせばValidそうじゃない場合はInbvalidと出力したいです。

以下私が書いたプログラムです。　エラーメッセージはありませんでした。

t=list(input())

if len(t)<6:
    print("Invalid")
else:
    for i in range(len(t)):
        if t[i].isdecimal()==True:
            s=1
            
        else:
            b=1
    z=0
    if s==1 and b==1:
        j=0
        y=[0,0,0,0,0,0,0,0,0,0,0,0,0,0]
        p=len(t)-1
        for i in range(p):

            if t[i]==t[i+1]:
                y[j]+=1
            else :
                j+=1
        z=0
        for i in range(j):
            if y[i]>2:
                print("Invalid")
                z=1
        if z==0:
            print("Valid")     
    else :
        print("Invalid")

Comments

[yh1224]

作動しない、というのは具体的にどういう状況でしょうか？
特定の入力で正しい結果が出せないのであれば、その条件や例を示しましょう。:-)

(間違いを全部探して！という質問は丸投げとみなされてしまいますので。)

[退会済みユーザー]

言い方が悪かったですね
全ての入力に対し同じ出力になってしまうのです。

[yh1224]

とりあえず短い文字列は正しく結果が出るようです。

$ python test.py
123
Invalid

[attakei]

> 全ての入力に対し同じ出力になってしまうのです。
> 以下私が書いたプログラムです。　エラーメッセージはありませんでした。

試しに手元で動かしてみた範囲では、そんなことはなかったのですが。

abced
=> Invalid
abc123
=> Valid
abcdef
=> NameErrorで正常動作せず
123456
=> NameErrorで正常動作せず
aaaaaa
=> NameErrorで正常動作せず
aaabbbcccdddeeefffggghhhiiijjjkkklllmmmoooppp111rrrssstttuuuvvvxxxwwwyyyzzz01932
=> IndexErrorで正常動作せず
aaabcdef1
=> Valid

[退会済みユーザー]

エラーメッセージは作ったときのです
紛らわしくてすいません

[TakaiY]

みなさんが言うのは、「全ての入力に対し同じ出力になってしまう」と書いてあるが、実際に提示されたコードを実行してみると、入力によって異なる結果が出るので、貴方の質問が成立していないということです。
提示のコードにはいろいろ問題があるのですが、まずは認識が合っていないと先に進めません。
こういう場合よくあるのは、実行しているファイルと修正したファイルが異なる場合です。 修正して保存してないとか、修正して名前を変えたのに実行するときに変えてないとか、他の場所に保存していたとか。

[meg_]

> パスワードの強度を調べるプログラムで弱いのにInvalidとでないでValidと出てしまいます。
上記が発生した具体的な入力文字列は何ですか？

Answer 1

from itertools import groupby

t=input()

def chkpasswd(strng):

    def chk_same(strng):
        """３文字連続している場合True
        https://docs.python.org/ja/3/library/itertools.html#itertools.groupby
        """
        return sum([True for k, g in groupby(strng) for  j in g if len(list(g)) >= 3])
    
    def digit_string(strng):
        """数字と英文字が混在しているか確認
        一方でも含まれていないと、len()が0となることで判定
        """
        alph='abcedfghijklmnopqrstuvxyz'
        alphabet=alph+alph.upper()
        i , s =[int(i) for i in strng if ord(i) >=48 and ord(i) <= 57] ,[i for i in strng if i in alphabet ]
        return True if len(i) * len(s) else False

    # 判定本体
    # 入力値が６文字以上　かつ　数字と英文字が含まれている　かつ　同じ文字が３つ以上連続しない
    if len(strng) >= 6 and digit_string(strng) and not chk_same(strng):
        return 'Valid'
    else: return 'Invalid'

chkpasswd(t)

作ってみました。

あなたのプログラムだと
y=[0,0,0,0,0,0,0,0,0,0,0,0,0,0]によって暗黙的にパスワード長が１４文字に制限されていますね。

t=list(input())

valid = True
s = b = j = 0

if len(t)<6:  # 六文字以上か
    valid=False
else:
    for i in range(len(t)):  # 文字列判定
        if t[i].isdecimal()==True:
            s=1  # 数字 
        else:
            b=1  #　英文字

    if s==1 and b==1:  # 数字かつ英文字の場合
        p=len(t)-1
        for i in range(p):

            if t[i]==t[i+1]:  # 隣り合う文字が同じ場合
                j +=1  
                if j == 2:  # AAAとなった場合
                    valid=False
            else :  # 隣り合う文字が違う場合リセット
                j = 0
    
    else :  # どちらか一方の場合
        valid=False

print('Valid') if valid else print('Invalid')   # 全ての判定の結果により出力

条件式を生かしながら作り直すと、こんな感じでしょうか？
途中でprintすると直しづらいので、フラグを作ってやってみました。

私のより、綺麗ですね。????