Как увеличить допустимое число вводимых знаков?

Question

[YOKARAMANE]

def encrypt(msg, key):
    encrypted=""
    for letter in msg:
        position = alphabet.find(letter)
        newPosition = position + key
        if letter in alphabet:
            encrypted = encrypted + alphabet [newPosition]
        else:
            encrypted = encrypted + letter
    return encrypted

@bot.message_handler(commands = ['encrypt'])
def msg_encrypt(message):
    error = 'Вы не указали параметры шифрования!\nПример: /encrypt РУДН 5'
    arg = get_arg(message.text)
    if len(arg) >= 2:
        try:
            key = int(arg[-1])
            bot.send_message(message.chat.id, f"{encrypt(ltstr(arg[0:-1]), key)}")
        except ValueError:
            bot.send_message(message.chat.id, error, parse_mode = 'Markdown')
    else:
        bot.send_message(message.chat.id, error, parse_mode = 'Markdown')

При вводе текста 'Привет, хабр, это тестовый текст' c шагом '5' возникает ошибка 'string index out of range'. Более короткие предложения проходят нормально. Как фиксить?

Comments

[antares4045]

В коде вопроса проблем не вижу. Косячит либо реализация алфавита, либо, что более вероятно, функция get_arg.
стактрейс у вас какой?

[YOKARAMANE]

antares4045,

Возникло исключение: IndexError
string index out of range

File "C:\Users\YOKA\Desktop\main.py", line 13, in encrypt
    encrypted = encrypted + alphabet [newPosition]
  File "C:\Users\YOKA\Desktop\main.py", line 53, in msg_encrypt
    bot.send_message(message.chat.id, f"{encrypt(ltstr(arg[0:-1]), key)}")
  File "C:\Users\YOKA\Desktop\main.py", line 86, in <module>
    bot.polling()

[Любитель поиграть в самолёты]

Поп правилам написания кода: именованные параметры надо передавать без пробелов.
В место
@bot.message_handler(commands = ['encrypt'])
и

bot.send_message(message.chat.id, error, parse_mode = 'Markdown')

Надо
@bot.message_handler(command =['encrypt'])
и

bot.send_message(message.chat.id, error, parse_mode='Markdown')

[antares4045]

YOKARAMANE, хм есть одно подозрение
попробуйте зашифровать букву 'я' с таким же ключом

[YOKARAMANE]

antares4045, да, Вы правы, та же самая ошибка)))

[antares4045]

Python Новичок, ну такое. Гвидо это сам придумал и кроме питонистов так никто не делает.

[Любитель поиграть в самолёты]

YOKARAMANE, нет это не влияет на ошибки. Просто так код читается легче так как он соответствует стандартам

[antares4045]

YOKARAMANE, не понял вопроса:
вы не поняли, что надо

newPosition = (position + key) % len(alphabet)

или что-то другое вылезло?

Answer 1

[antares4045]

Возможно это не финал решения, но из нашего общения в коментариях я вынес предположение, что строка у вас хранится в виде

"абвгдежзийклмнопрстуфхцчшщъыьэюя"

дальше суть проблемы:

position =alphabet.find('я') # 31
newPosition = position + key # 36
alphabet [newPosition] # ааааа! нет такой буквы

предпологаю, что вы хотели забацать шифр соломона, в котором при переходе через край счёт начинается заново

position =s.find('я') # 31
newPosition = (position + key) % len(alphabet) # 4
alphabet [newPosition] # д

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

А из-за чего сдвиг шифра неправильный без понятия: скиньте ка свой алфавит.

Comments

[YOKARAMANE]

Алфавит сделал единым, т.к. хотел чтоб автоматически распознавало язык без его выбора

alphabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyzАБВГДЕЁЖЗИЙКЛМНОПРСТУФХЦЧШЩЪЫЬЭЮЯабвгдеёжзийклмнопрстуфхцчшщъыьэюя"

[antares4045]

ну а что вы ожидали
"я" № 117
(177 + 5) % 118 = 4
№4 -- заглавная латинская "E"
если вы хотите, чтобы оно прям по канонам шпарило, тогда нужно сильно деривиализировать алгоритм

[YOKARAMANE]

antares4045, Вас понял. Буду переделывать под разные алфавиты, спасибо!

[antares4045]

YOKARAMANE,

alphabet = [
    ''.join(chr(i) for i in range(ord('A'), ord('Z') + 1)),
    ''.join(chr(i) for i in range(ord('А'), ord('Я') + 1)), 
    ''.join(chr(i) for i in range(ord('a'), ord('z') + 1)),
    ''.join(chr(i) for i in range(ord('а'), ord('я') + 1))
]

def encodeLetter(letter, key):
    for subAlphabet in alphabet:
        index = subAlphabet.find(letter)
        if index != -1:
            return subAlphabet[(index + key) % len(subAlphabet)]
    return letter

def decodeLetter(letter, key):
    for subAlphabet in alphabet:
        index = subAlphabet.find(letter)
        if index != -1:
            return subAlphabet[(index + len(subAlphabet) - key) % len(subAlphabet)]
    return letter

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