2

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

dict1 = {"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":"--.."}
def change():
    global dict1
    crypt = ""
    text = input("[*]Enter text\n[text]>>>")
    for i in text:
        b = i.upper()
        for j in dict1:
            if b == " ":
                crypt += " "
            elif b == j:
                crypt += str(dict1.get(b))
    print("[+]Crypt text:\n" + str(crypt))
change()

Если запустить данный код, выведется в консоль это:

[*]Enter text
[text]>>>hello stackoveflow 
[+]Crypt text:
......-...-..---                          ...-.--.-.-.----...-...-..-..---.--                          

Почему питон находит так много пробелов, и как исправить код так, что бы текст выводился через 1 пробел?

9
  • 4
    А вы зачем на пробел проверяете внутри цикла по словарю, вы пробел печатаете на каждой проверке в словаре. И кстати шифром это назвать сложно, расшифровать то это нельзя
    – Mike
    14 ноя 2017 в 17:35
  • 5
    И кстати, зачем вообще нужен цикл. вы можете получить просто dict[i.upper()] собственно словари для того и нужны
    – Mike
    14 ноя 2017 в 17:40
  • 5
    @True-hacker можно всё в одну строчку преобразование написать crypto = ''.join([dict1.get(c.upper(), ' ') for c in text]). А в вашем коде не верно что нужно сравнение на пробел вынести за цикл.
    – Arty
    14 ноя 2017 в 17:43
  • 3
    Мне кажется не с того вы свою карьеру "тру хакера" начинаете =) 14 ноя 2017 в 18:34
  • 3
    Напишите в вопросе, что это азбука Морзе, чтобы не вводить людей в заблуждение.
    – user239133
    14 ноя 2017 в 18:43

1 ответ 1

6

Пробел печатается много раз т.к. он добавляется внутри цикла, просто нужно проверку на пробел вынести за цикл. Но цикл не нужен вообще, не нужно итерировать по словарю и сравнивать каждый ключ, достаточно просто взять dict1.get(b), либо если всё-таки нужно проверить есть ли ключ в словаре то нужно писать if key in dict1. Можно всё преобразование очень кратко на Питоне написать (второй параметр get означает значение по умолчанию, т.е. то что вернуть если в словаре не найдено):

crypto = ''.join([dict1.get(c.upper(), ' ') for c in text])

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.