Learn String Manipulation by Building a Cipher - Step 86

Tell us what’s happening:

somehow my code raised an error before the test could happen
instructions: Next, modify your `encryption` and `decryption` variables by calling `encrypt` and `decrypt`, respectively, in place of `vigenere`.

``````text = 'Hello Zaira!'
custom_key = 'python'

def vigenere(message, key, direction=1):
key_index = 0
alphabet = 'abcdefghijklmnopqrstuvwxyz'
final_message = ''

for char in message.lower():

# Append any non-letter character to the message
if not char.isalpha():
final_message += char
else:
# Find the right key character to encode/decode
key_char = key[key_index % len(key)]
key_index += 1

# Define the offset and the encrypted/decrypted letter
offset = alphabet.index(key_char)
index = alphabet.find(char)
new_index = (index + offset*direction) % len(alphabet)
final_message += alphabet[new_index]

return final_message

# User Editable Region

def encrypt(message, key):
return encryption(message, key)

def decryption(message, key):
return decryption(message, key, -1)

encryption = encrypt(text, custom_key)
print(encryption)
decryption = decrypt(encryption, custom_key, -1)
print(decryption)

# User Editable Region

``````

User Agent is: `Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:128.0) Gecko/20100101 Firefox/128.0`

Challenge Information:

Learn String Manipulation by Building a Cipher - Step 86

`decrypt` function does not exist

I would reset this and try again. Just edit the function calls and do no edit the function definitions (`def`)

this is my updated code, but it says my decrypt takes 2 parameters but 3 were given

``````text = 'Hello Zaira!'
custom_key = 'python'

def vigenere(message, key, direction=1):
key_index = 0
alphabet = 'abcdefghijklmnopqrstuvwxyz'
final_message = ''

for char in message.lower():

# Append any non-letter character to the message
if not char.isalpha():
final_message += char
else:
# Find the right key character to encode/decode
key_char = key[key_index % len(key)]
key_index += 1

# Define the offset and the encrypted/decrypted letter
offset = alphabet.index(key_char)
index = alphabet.find(char)
new_index = (index + offset*direction) % len(alphabet)
final_message += alphabet[new_index]

return final_message
def encrypt(message, key):
return vigenere(message, key)

def decrypt(message, key):
return vigenere(message, key, -1)

encryption = encrypt(text, custom_key)
print(encryption)
decryption = decrypt(encryption, custom_key, -1)
print(decryption)
``````

The ‘decrypt’ takes in two arguments. The last argument here is not needed.

tysm, it seems logical after all

1 Like