Переводчик кода Морзе Python

Aug 18 2020

Я учил себя Python в течение последних нескольких недель, меня интересуют криптография, коды и т. Д., Поэтому я подумал, что запуск переводчика кода Морзе будет хорошим проектом. Я знаю, что имена моих переменных могут быть разными, на самом деле это не шифрование, дешифрование и т. Д. Я в основном ищу совета о том, как сделать код чище и где я могу быть более эффективным.

Я думаю, что моя самая большая проблема заключается в том, что я не знаю, как обрабатывать входные данные в цикле while, как обычно. Проблема, с которой я столкнулся, заключалась в том, что я не мог проверить, было ли введено «e» или «d», так что это стало действительно шатким.

Я знаю, что могу улучшить:

  • Добавить цикл ввода
  • If, elif, else для действия
  • Сделать 'звук' фактическим логическим значением
  • Найдите фактическое время звука для dit и dah, но это не проблема кода
# Started: 08/17/2020
# Finished: 08/17/2020
# Takes an input message and outputs the message in morse code
# Keys taken from 'https://en.wikipedia.org/wiki/Morse_code'

from playsound import playsound
import time

# Dictionary that holds each letter and it's corresponding value
dict = {'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': '--..',
        '1': '.----', '2': '..---', '3': '...--', '4': '....-', '5': '.....', '6': '-....', '7': '--...', '8': '---..', '9': '----.', '0': '-----',
        ' ': '/', '.': '.-.-.-', ',': '.-.-', '?': '..--..', "'": '.----.', '!': '-.-.--', '/': '-..-.', '(': '-.--.', ')': '-.--.-',
        ':': '---...', ';': '-.-.-.', '=': '-...-', '+': '.-.-.', '-': '-....-', '_': '..--.-', '"': '.-..-.', '$': '...-..-', '@': '.--.-.'}

outputMessage = ""               # Holds our output message

# Sounds
sound = 'False'
dit = 'dit.wav'
dah = 'dah.wav'


def Encrypt(message):

    output = ''

    for char in message:
        if char in dict:
            output = output + dict[char]
            output = output + ' '

    return output


def Get_Key(val):
    for key, value in dict.items():
        if val == value:
            return key


def Decrypt(message):

    output = ''

    letters = message.split(' ')

    for letter in letters:
        temp = Get_Key(letter)
        output = output + temp

    return output


def Get_Inputs():
    # Get Inputs
    inputString = input('Enter a message to start.\n')
    action = input('(E)ncrypt or (D)ecrypt?\n')

    # Format Inputs
    message = inputString.lower().strip()
    action = action.lower().strip()

    return message, action


def Play_Sound(message):

    for char in message:
        if char == '.':
            playsound(dit)
        elif char == '-':
            playsound(dah)
        elif char == ' ':
            time.sleep(0.15)
        elif char == '/':
            time.sleep(0.30)


message, action = Get_Inputs()

if action == 'e' or action == 'encrypt':
    outputMessage = Encrypt(message)
elif action == 'd' or action == 'decrypt':
    outputMessage = Decrypt(message)
else:
    print('Error!')

print(outputMessage)

print('')
sound = input('Play sound? (T)rue / (F)alse\n')
if sound.lower().strip() == 't' or sound.lower().strip() == 'true':
    Play_Sound(outputMessage)

Ответы

6 RichardNeumann Aug 18 2020 at 19:25

Общий стиль

В вашем переводе dictиспользуются ключевое слово и строчные буквы. Подумайте о том, чтобы писать константы заглавными буквами и давать им выразительные имена, например MORSE_CODES = {...}.

Согласно PEP 8 , функции должны быть названы с использованием snake_case. CamelCaseзарезервировано для классов: outputMessageoutput_message, def Encrypt(...)def encrypt(...)и т. д.

Спектакль

Использование Get_Keyфункции не очень производительно, так как выполняет линейный поиск словаря. Просто переверните перевод dict один раз, а затем используйте его:

MORSE_ENCODING = {
    'a': '.-',
    'b': '-...',
    ...
}
MORSE_DECODING = {value: key for key, value in MORSE_ENCODING.items()}

...

        temp = MORSE_DECODING[letter]

Обработка ошибок

В настоящее время Encryptфункция автоматически пропускает все непереводимые символы. ValueError()Вместо этого подумайте о том, чтобы указать, что был предоставлен недопустимый ввод:

def encode(message):
    """Encodes a string into morse code."""

    code = ''

    for index, char in enumerate(message):
        try:
            code += MORSE_ENCODING[char.lower()]
        except KeyError:
            raise ValueError(f'Char "{char}" at {index} cannot be encoded.')

        code += ' '

    return code[:-1]  # Remove trailing space.


def decode(morse_code):
    """Decodes morse code."""

    message = ''

    for index, sequence in enumerate(morse_code.split()):
        try:
            message += MORSE_DECODING[sequence]
        except KeyError:
            raise ValueError(f'Cannot decode code "{sequence}" at {index}.')

    return message

Правильность

В Encryptнастоящее время ваша функция всегда возвращает конечный пробел. Вы можете избежать этого, вернувшись output[:-1].

Терминология

Преобразование кода Морзе в текст туда и обратно на самом деле не является шифрованием в собственном смысле. Возможно, вы захотите перефразировать {en,de}cryptс помощью {en,de}code.

Глобалы

Использование глобальных переменных, например, outputMessageможет иметь неприятные побочные эффекты, когда программа используется в качестве библиотеки. Весь код ниже def Play_Soundфункции должен перейти в def main()функцию, которую вы можете вызвать через

if __name__ == '__main__':
    main()

Внизу блока.