Почему программа «шифрования» не работает с русским алфавитом?

Question

[Данил]

a = "абвгдеёжзийклмнопрстуфхцчшщъыьэюяабвгдеёжзийклмнопрстуфхцчшщъыьэюя"
stringToEncrypt = input("введи сообщение: ")
stringToEncrypt = stringToEncrypt.lower
shiftAmount = int(input("введи номер от 1-33, оно будет твоим ключом. "))
encryptedString = ""
for currentCharacter in stringToEncrypt:
    position = a.find(currentCharacter)
    newposition = position + shiftAmount
    encryptedString = encryptedString + a[newposition]
print("твое сообщение зашифровано", encryptedString)

Ошибку выдает:

Traceback (most recent call last):
  File "C:/Users/theda/Desktop/trash/random.py", line 6, in <module>
    for currentCharacter in stringToEncrypt:
TypeError: 'builtin_function_or_method' object is not iterable

Задача из книжки; спросил в школе, учитель сказал что она не знает питона, потомучто его нет в программе. В интернете искал, но ничего не понял.
По условиям задачи алфавит должен быть английский, но я попробовал на русском.

Comments

[Алан Гибизов]

Необходимо исправить вопрос в соответствии с правилами, в частности п.3.4, 3.6, 3.8

[Данил]

Алан Гибизов, простите, так правильно?

Answer 1

[Михаил Гайворонский]

Проблема в этой строчке: stringToEncrypt = stringToEncrypt.lower

Если ты всё правильно скопировал, то в ней ты не вызываешь метод lower(), а присваиваешь переменной stringToEncrypt ссылку на него. Т.е. правильно будет: stringToEncrypt = stringToEncrypt.lower()

Чтобы было проще, можно строку stringToEncrypt = stringToEncrypt.lower вообще удалить, и сразу написать:

stringToEncrypt = input("введи сообщение: ").lower()

А теперь подумай, зачем в переменной a словарь у тебя дважды повторяется?

Comments

[Данил]

Спасибо большое. Очень помогло, все заработало.
Алфавит два раза вводится потому что если доберется до конца первого алфавита, то смещение пойдет на второй алфавит. Например, если мне нужно будет зашифровать букву я

[SagePtr]

Данил, в пайтоне можно удвоить строку так:
a = "абвгдеёжзийклмнопрстуфхцчшщъыьэюя" * 2
Либо вместо дублирования - воспользоваться остатком от деления (кольцо вычетов), как обычно делают, если нужно зациклить массив.

[Михаил Гайворонский]

SagePtr, Данил, но и этого не нужно. Я не так давно выполнял такое же задание, моя функция дешифрования (что в принципе то же самое, что и шифрование в шифре Цезаря) выглядит так:

def decode_caesar(string, shift, alphabet="abcdefghijklmnopqrstuvwxyz"):
    coded_alphabet = alphabet[shift % len(alphabet):] + alphabet[:shift % len(alphabet)]
    return ''.join([coded_alphabet[alphabet.find(letter)] if letter in alphabet else letter for letter in string])

Хотя я вот только что понял, что можно и проще её сделать. Oh well.

[Данил]

SagePtr,
a = "абвгдеёжзийклмнопрстуфхцчшщъыьэюя" * 2
Спасибо, не знал.

воспользоваться остатком от деления (кольцо вычетов), как обычно делают, если нужно зациклить массив.

Простите, у меня черепашка(turtle) не запускается, а вы такие страшные вещи говорите.
Посмотрел что такое кольцо вычетов. Попытался почитать. Ладно сторожа тоже нужны.

[shurshur]

Данил, умные слова типа "колцо вычетов по модулю p" - это не так страшно. Это когда мы берём не все числа вообще, а только остатки от деления на p. И переопределяем основные операции: сложение берём по остатку, умножение тоже. Например, если p=7, то 2*4 mod 7 = 1.

В python так:

(2*4) % 7

Определённые подобным образом "сложение" и "умножение" обладают рядом свойств: коммутативность (a+b=b+a, a*b=b*a), ассоциативность (a+(b+c)=(a+b)+c) и некоторыми другими. Собственно, такое вот множество с двумя операциями и некоторыми особыми свойствами называют в алгебре кольцом.

Но всё это для задачи не очень важно. Тут достаточно просто брать остаток по длине словаря:

newposition = (position + shiftAmount) % len(a)

Так будет более разумно.

[Данил]

shurshur, здравствуйте.
Прочитал про функцию "len", да так правда удобнее, спасибо. Просто в учебнике Банкрашкова я пока до нее не дошел, а мне сказали что лучше сначала его пройти, а потом самому остальное начать читать. Когда я пробовал начинать функции и сразу запутался, папа сказал сначала понять основы, потом только углубляться, а то только голову в пустую забью. Спасибо вам за помощь и подсказку!

Answer 2

[dmshar]

Исправьте в вашем коде вот эту строчку
stringToEncrypt = stringToEncrypt.lower()
и начинайте уже учиться, учить Python, а не бездумно копипастить чужой код. Толку и пользы будет явно больше.

Comments

[Данил]

Простите, это я невнимательный, по заданию было upper(), а я подумал что лучше будет маленькие буквы и написал lower, а про скобки забыл. Я правда учусь только не всегда получается

Answer 3

[mayton2019]

Плох тот учитель что не видит простой шифр Цезаря. И какая разница Питон или не питон когда
просто надо спросить ученика как он вообще сам понимает свою разработку?

И удвоение алфавита здесь не нужно. Цезарь ведь поворачивает алфавит циклически. Берите остаток от деления на 33. А то что? Был-бы у вас алфавит в терабайт - то вы бы ставили двухтерабайтный диск чтобы решить такую задачу?