文字列インデックスが範囲外であるエラーPython forループ

Question

私の単純なブルートフォースパスワードプログラムはこれまでどおり完璧に動作しますが、実行するたびに常に最後の最後を出力し、文字列インデックスが範囲外です。すべてうまく動作し、私が望む出力が得られますが、whileループが閉じてプログラムが実行を停止するたびにエラーが発生します。

userpassword = raw_input("Enter a password: ") 

k = 0 
cyclenumb = 0 
newpasswordlist=[] 

lowercaselist=["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"] 
uppercaselist=["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"] 
numberslist=["1", "2", "3", "4", "5", "6", "7", "8", "9", "0"] 
symbolslist=["~", "`", "!", "@", "#", "$", "%", "^", "&", "*", "(", ")", "-", "_", "=", "+", "{", "[", "]", "}", "|", "\ ", ";", ":", "'", '"', ",", "<", ".", ">", "/", "?", " "] 

while cyclenumb <= 10000: 

    for x in userpassword[k]: 
     for z in lowercaselist: 
      if x in z: 
       newpasswordlist.append(z) 
       k +=1 
       print newpasswordlist 
      if x is not z: 
       for x in userpassword[k]: 
        for z in uppercaselist: 
         if x in z: 
          newpasswordlist.append(z) 
          k +=1 
          print newpasswordlist 
         if x is not z: 
          for x in userpassword[k]: 
           for z in numberslist: 
            if x in z: 
             newpasswordlist.append(z) 
             k +=1 
             print newpasswordlist 
           if x is not z: 
            for x in userpassword[k]: 
             for z in symbolslist: 
              if x in z: 
               newpasswordlist.append(z) 
               k +=1 
               print newpasswordlist 

    if userpassword == newpasswordlist: 
     break 

print newpasswordlist 
print "Here is your original password " + userpassword 

この簡単なエラーで私を助けてください。

Answer 1

kが長さuserpassword以上であることを検出しようとしても、このコードには何もありません。たとえば、kが増分されるたびにこれをチェックすることができます。大きすぎる場合はwhileループから抜け出してください。

Answer 2

このアプローチはあまり良くありません。 userpassword[k]は常に1文字であるため、for x in userpassword[k]は無意味です。次に、userpassword（文字列）とnewpasswordlist（これはリスト）を比較します。ですから、あなたはこのタスクに使用するアルゴリズムを再定義することをお勧めします。

Answer 3

kのようなことを行うことで、これをすべて取り除くことができます。私はあなたがなぜwhileループを持っているのかも分かりませんが、ここであなたの問題を修正しました。

for x in userpassword: 
    for z in lowercaselist: 
     if x is z: 
      newpasswordlist.append(z) 
      print newpasswordlist 
      continue 
     if x is not z: 
      for z in uppercaselist: 
       if x is z: 
        newpasswordlist.append(z) 
        print newpasswordlist 
        break 
       if x is not z: 
        for z in numberslist: 
         if x is z: 
          newpasswordlist.append(z) 
          print newpasswordlist 
          break 
         if x is not z: 
          for z in symbolslist: 
           if x is z: 
            newpasswordlist.append(z) 
            print newpasswordlist 
            break 
    if userpassword is newpasswordlist: 
     break