Skip to content

Commit d3f9640

Browse files
committed
Fix a bug where upper case letters were not found in the alphabet
1 parent 59c427f commit d3f9640

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

password-storage/caesar_cipher.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,17 +13,19 @@ def encrypt(original_message, key):
1313
'u', 'v', 'w', 'x', 'y', 'z')
1414

1515
encrypted_message = ''
16+
key = alphabet.index(key)
1617

1718
# Encrypts the original message then stores in the return variable
18-
for letter in original_message:
19+
for letter in original_message.lower():
1920
encrypted_letter = ''
20-
if letter.lower() not in alphabet:
21+
letter_index = alphabet.index(letter) + 1
22+
if letter not in alphabet:
2123
encrypted_letter = letter
22-
elif alphabet.index(letter) + key >= len(alphabet):
24+
elif letter_index + key >= len(alphabet):
2325
encrypted_letter = \
24-
alphabet[alphabet.index(letter) + key - len(alphabet)]
26+
alphabet[letter_index + key - len(alphabet)]
2527
else:
26-
encrypted_letter = alphabet[alphabet.index(letter) + key]
28+
encrypted_letter = alphabet[letter_index + key]
2729
encrypted_message += encrypted_letter
2830

2931
return encrypted_message

0 commit comments

Comments
 (0)