Можно ли оптимизировать этот код?

Question

[yarmr]

Сам код:

tl = []
x = True
let = ['а', 'б', 'в', 'г', 'д', 'е', 'ё', 'ж', 'з', 'и', 'й', 'к', 'л', 'м', 'н', 'о', 'п', 'р', 'с', 'т', 'у', 'ф',
       'х', 'ц', 'ч', 'ш', 'щ', 'ъ', 'ы', 'ь', 'э', 'ю', 'я']
t = str(input())
o = []
od = []

for i in range(len(t)):
    tl.append(t[i])
print(tl)
for i in range(len(tl)):
    o.append(let.index(tl[i]) - 1)
print(o)
for i in range(len(o)):
    od.append(let[o[i] % 33])
print(od)

Этот код позволяет закодировать слова с помощю шифра цезаря, можно ли его ещё как-то оптимизировать

Answer 1

[Владимир Куц]

Можно

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

# t = str(input())
t = 'текст'
print(''.join([let[(let.index(x) - 1) % 33] for x in t]))
# сдйрс

Comments

[Сергей П]

Ну чего вы? Ни шанса для саморазвития человеку не оставили...

[Владимир Куц]

Сергей П, ну, - в энтерпрайзе задачи как правило посложнее будут...
Умный - в примере разберется. А любителям копипаста это все равно не поможет )

[Сергей П]

Владимир Куц, Да какой там энтерпрайз. Задача явно учебная, а топикстартер даже не потрудился изложить свои соображения по поводу ее решения.

[Владимир Куц]

Сергей П, да - вы правы

Answer 2

[Сергей П]

Конечно. Вот этот цикл совершенно не имеет никакого смысла:

for i in range(len(t)):
    tl.append(t[i])

Третий цикл тоже не нужен. Вы всё можете делать в одном цикле.
Или это вы разрешения спрашивали? Никто не запрещает.

Comments

[yarmr]

Нет, вопрос поставлен так из-за требований платформы

[Сергей П]

yarmr, требования платформы гласят, что этот ресурс - не решебник для задач и не бесплатная фриланс-биржа. Здесь готовы помочь с проблемой люд]ям, которые сами готовы прилагать усилия для решения. Вы в своём вопросе никак не показали:
- с чем именно у вас проблема
- что именно не понятно
- какие у вас гипотезы в плане подходов к решению вашей задачи

Из вашего вопроса понятно, что вы едва-ли знаете язык и понимаете как работают основные структуры данных и алгоритмические структуры. Если так, то, возможно вам следует подучить язык, а не кидаться решать задачи? А если уж вы и решаете задачи для обучения, то, наверно, стоило бы высказать свои размышления по поводу решения, чтобы вас можно было поправить, указать на ошибки, подтолкнуть к решению.

Ответ "Можно" был бы непродуктивным. Готовый правильный ответ в виде кода, я считаю, тоже вреден для пытающегося чему-то научиться.

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

Фактически вы проковыряли выключенным перфоратором дыру в стене и интересуетесь на профильном форуме строителей можно ли было \это сделать эффективнее. Конечно можно! Но если вы не знаете базовых принципов, то нужно начать с них.

Answer 3

[Vindicar]

Можно, если сделать допущение, что текст кодируется посимвольно, а не группами из N символов.

open_text = 'текст'  # открытый текст

# letters - именно строка, одна позиция - один символ
letters = ''.join(['а', 'б', 'в', 'г', 'д', 'е', 'ё', 'ж', 'з', 'и', 'й', 'к', 'л', 'м', 'н', 'о', 'п', 'р', 'с', 'т', 'у', 'ф',
       'х', 'ц', 'ч', 'ш', 'щ', 'ъ', 'ы', 'ь', 'э', 'ю', 'я'])
key = 5  # ключ к шифру - величина сдвига. Может быть >0 или <0
# генерируем таблицы подстановки
encrypt_table = str.maketrans(letters, letters[key:]+letters[:key])  # для шифрования
decrypt_table = str.maketrans(letters, letters[-key:]+letters[:-key])  # для дешифрования

cypher_text = open_text.translate(encrypt_table)
restored_text = cypher_text.translate(decrypt_table)

print(open_text)
print(cypher_text)
print(restored_text)