Шифр Цезаря неправильно сдвигает буквы в алфавитном списке

Я прохожу курс по написанию шифра Цезаря. Выполнив задачу самостоятельно с немного другим кодом (который сработал), я теперь попытался точно воспроизвести код из курса, и он, похоже, не работает (ChatGPT говорит, что он тоже должен работать)

Если я выберу «декодирование», наберу букву «С» и сдвину ее на 2 (что станет -2), буква «а» будет отображаться правильно.

Но если я попытаюсь декодировать полное слово, например, выбрав «декодировать», написав «хур» и сдвинув на «3», оно должно показать «привет», но вместо этого покажет «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"

«ChatGPT говорит, что тоже должно работать» — ChatGPT ничего не понимает. Он просто принимает подсказку и выдает вероятный ответ.

Chris 25.08.2024 18:18
Почему в Python есть оператор "pass"?
Почему в Python есть оператор "pass"?
Оператор pass в Python - это простая концепция, которую могут быстро освоить даже новички без опыта программирования.
Некоторые методы, о которых вы не знали, что они существуют в Python
Некоторые методы, о которых вы не знали, что они существуют в Python
Python - самый известный и самый простой в изучении язык в наши дни. Имея широкий спектр применения в области машинного обучения, Data Science,...
Основы Python Часть I
Основы Python Часть I
Вы когда-нибудь задумывались, почему в программах на Python вы видите приведенный ниже код?
LeetCode - 1579. Удаление максимального числа ребер для сохранения полной проходимости графа
LeetCode - 1579. Удаление максимального числа ребер для сохранения полной проходимости графа
Алиса и Боб имеют неориентированный граф из n узлов и трех типов ребер:
Оптимизация кода с помощью тернарного оператора Python
Оптимизация кода с помощью тернарного оператора Python
И последнее, что мы хотели бы показать вам, прежде чем двигаться дальше, это
Советы по эффективной веб-разработке с помощью Python
Советы по эффективной веб-разработке с помощью Python
Как веб-разработчик, Python может стать мощным инструментом для создания эффективных и масштабируемых веб-приложений.
1
1
50
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 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}")

Ах! Большое спасибо, вы поняли

Gusm98 25.08.2024 14:14

@ Gusm98, отлично! Если это ответ на ваш вопрос, вы можете дополнительно принять его, нажав ✔️. Это сообщит другим пользователям, какой ответ вам больше всего подходит, отметит ваш вопрос как ответ на него в базе данных Stack Overflow и вознаградит вас и ответившего очками репутации.

Chris 25.08.2024 18:20

Другие вопросы по теме

Похожие вопросы

Замените ячейку в столбце на основе ячейки в другом столбце в полярном DataFrame
Python/win32crypt: «Доступ запрещен» при попытке сохранить сертификат в хранилище Windows «локального компьютера», но работает с хранилищем «текущий пользователь»
Как перебирать вложенные списки различной длины в цикле Ansible?
Изменение имени строки кадра данных Python Pandas
Недостаточно памяти для перезагрузки: torch.OutOfMemoryError: CUDA недостаточно памяти
Создайте столбцы 0–1 на основе другого фрейма данных с датами начала и окончания событий
Pandas .loc возвращает серию или плавает непоследовательно
Ошибочные результаты прокрутки pandas с временным окном, сгруппированным по кадру данных, импортированному из BigQuery
Как разграничить текст, хранящийся в переменной, для создания кадра данных в Python?
Как я могу настроить белое пустое пространство вокруг таблицы Matplotlib, чтобы оно соответствовало размеру таблицы?