凯撒密码错误地移动了字母列表中的垃圾

Question

我正在学习一门课程，你可以在其中编写凯撒密码。在我自己使用稍微不同的代码（有效）完成任务后，我现在尝试完全复制课程中的代码，但它似乎不起作用（ChatGPT 说它也应该有效）

如果我选择“解码”，输入字母“C”并将其移位 2（将变为 -2），它会正确显示字母“a”。

但是如果我尝试解码一个完整的单词，例如通过选择“解码”，写入“khoor”并移动“3”，它应该显示“hello”，但显示“hklro”。

这是代码：

alphabet = ['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']

direction = input("Type 'encode' to encrypt, type 'decode' to decrypt:\n").lower()
text = input("Type your message:\n").lower()
shift = int(input("Type the shift number:\n"))

def caesar(encode_or_decode,original_text, shift_amount):
    output_text = ""
    for letter in original_text:
        if encode_or_decode == "decode":
            shift_amount *= -1
        shifted_position = alphabet.index(letter) + shift_amount
        shifted_position %= len(alphabet)
        output_text += alphabet[shifted_position]
    print(f"Here is the {encode_or_decode}d result: {output_text}")

caesar(original_text=text, shift_amount=shift, encode_or_decode=direction)

我的输入是

"decode", "khoor", "3"

Answer 1

您在

encode_or_decode

循环内测试

for

，如果是

decode

，则在每次迭代中不断将

shift_amount

乘以

-1

，这会导致解码时出现错误结果。将支票移到循环之外，应该没问题：

def caesar(encode_or_decode,original_text, shift_amount):
    output_text = ""
    if encode_or_decode == "decode": # encode_or_decode check moved here
        shift_amount *= -1
    for letter in original_text:
        shifted_position = alphabet.index(letter) + shift_amount
        shifted_position %= len(alphabet)
        output_text += alphabet[shifted_position]
    print(f"Here is the {encode_or_decode}d result: {output_text}")