Как я могу сделать мой код Python 3 более компактным? - PullRequest
Новая
       72

Как я могу сделать мой код Python 3 более компактным?

0 голосов
/ 05 февраля 2019

Это мой код, и я считаю его немного громоздким и повторяющимся."" "Клавиша UCIO = выбранная пользователем операция ввода x = глобальная переменная, первое число, с которым пользователь будет работать, y = глобальная переменная, второе число, с которым пользователь будет работать, z = локальная переменная, результирующий номер выбранной операции" ""

# Declaring the types of operates the user cold use
print("Select an operation")
print("1. Addition")
print("2. Subtraction")
print("3. Multiplication")
print("4. Division")
print("5. Power")

# Making the code look neater
print("")

# Gathering information from the user to calculate equation
UCIO = input("Enter an operation 1/2/3/4/5: ")
x = input("Enter your first number: ")
y = input("Enter your second number: ")

# Making the code look neater
print("")

# Calculating an equation with the operation "+"
if UCIO == "1":
    z = float(x) + float(y)
    print(x + " " + "+" + " " + y + " " + "=" + " " + str(z))

# Calculating an equation with the operation "-"
elif UCIO == "2":
    z = float(x) - float(y)
    print(x + " " + "-" + " " + y + " " + "=" + " " + str(z))

# Calculating an equation with the operation "*"
elif UCIO == "3":
    z = float(x) * float(y)
    print(x + " " + "*" + " " + y + " " + "=" + " " + str(z))

# Calculating an equation with the operation "/"
elif UCIO == "4":
    z = float(x) / float(y)
    print(x + " " + "/" + " " + y + " " + "=" + " " + str(z))

# Calculating an equation with the operation "^"
elif UCIO == "5":
    z = float(x) ** float(y)
    print(x + " " + "^" + " " + y + " " + "=" + " " + str(z))

1 Ответ

0 голосов
/ 05 февраля 2019

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

  1. Создать словарь, содержащий UCIO.С каждым UCIO, имеющим alias, message и рабочую функцию fct
  2. Напечатайте все возможные варианты, просматривая словарь и выбирая его message ключ
  3. Соберите свои входные данные, UCIO, x и y
  4. Если UCIO находится в возможных вариантах, используйте соответствующую рабочую функцию fct из словаря
  5. Остальноесообщить об ошибке 
ops = {}
ops['1'] = { 'alias': '+', 'message': 'Addition', 'fct': lambda x, y : x + y }
ops['2'] = { 'alias': '-', 'message': 'Substraction', 'fct': lambda x, y : x - y }
ops['3'] = { 'alias': '*', 'message': 'Multiplication', 'fct': lambda x, y : x * y }
ops['4'] = { 'alias': '/', 'message': 'Division', 'fct': lambda x, y : x / y }
ops['5'] = { 'alias': '^', 'message': 'Power', 'fct': lambda x, y : x ** y }

for k in ops.keys():
    print(f'{k}. {ops[k]["message"]}')

UCIO = input(f"Enter an operation {'/'.join(ops.keys())} : ")
x = input("Enter your first number: ")
y = input("Enter your second number: ")

if UCIO in ops and UCIO in ops:
    result = ops[UCIO]['fct'](float(x), float(y))
    print(f'{x} {ops[UCIO]["alias"]} {y} = {result}')
else:
    print('No candidates for the operation {UCIO}')
...