Оптимизация кода простого калькулятора

@MonteKristo · Регистрация: 21.08.2015

Студворк — интернет-сервис помощи студентам

Доброго времени. Начал изучать вчера python. Сегодня решил написать простой калькулятор. Написал. Теперь хочу максимально упростить код, но не могу понять как сделать, чтобы функция возвращала введенные пользователем номера. Вот код:

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
while True:
    print("Настройки")
    print("Введите '+' для сложения")
    print("Введите '-' для вычетания")
    print("Введите '*' для умножения")
    print("Введите '/' для деления")
    print("Введите 'Выход' для выхода из калькулятора")
    user_input = input(":")
 
    def bring_numbers():
        num1 = float(input("Введите первое число: "))   #вот эту функцию хочу вызывать для каждого действия
        num2 = float(input("Введите второе число: "))
        return num1,num2
    
    
 
    if user_input == "Выход":
        break
    elif user_input == "+":
        #num1 = float(input("Введите первое число: "))
        #num2 = float(input("Введите второе число: "))
        bring_numbers()  #После ввода чисел, падает с ошибкой, что num 1 не определен, то есть числа не тянутся
        result = str(num1+num2)
        print ("Ответ: " + result)
    elif user_input == "-":
        num1 = float(input("Введите первое число: "))
        num2 = float(input("Введите второе число: "))
        result = str(num1-num2)
        print("Ответ: " + result) 
    elif user_input == "*":
        num1 = float(input("Введите первое число: "))
        num2 = float(input("Введите второе число: "))
        result = str(num1*num2)
        print("Ответ: " + result) 
    elif user_input == "/":            
        num1 = float(input("Введите первое число: "))
        num2 = float(input("Введите второе число: "))
        result = str(num1/num2)
        print("Ответ: " + result) 
    else:
        print("неизвестный ввод")

@Centrin0 · 22.05.2016, 09:09

MonteKristo, вы из функции возвращаете два числа, но потом их не храните. Те переменные num1, num2, они доступны только внутри самой функции. Читайте про область видимости переменных.

Python
1
2
num1, num2 = bring_numbers() 
result = str(num1+num2)

@Whost · 22.05.2016, 10:25

MonteKristo, и функцию лучше ставить в самом начале программы, а не в середине

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
operators = {'+' : 0, '-' : 1, '*' : 2, '/' : 3, 'Выход' : 4}
while 1:
    try:
        oper = operators[input('Введите операцию: ')]
    except KeyError:
        print('Неизвестный ввод')
        continue
    if oper == 4:
        break
    else:
        n1 = int(input('Введите число 1: '))
        n2 = int(input('Введите число 2: '))
        print('Oтвет: ', [n1 + n2, n1 - n2, n1 * n2, n1 / n2][oper])

@Semen-Semenich · 22.05.2016, 13:01

Сообщение от MonteKristo

bring_numbers() *#После ввода чисел, падает с ошибкой, что num 1 не определен, то есть числа не тянутся
* * * * result = str(num1+num2)

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

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
def bring_numbers():
        num1 = float(input("Введите первое число: "))
        num2 = float(input("Введите второе число: "))
        return num1,num2
 
while True:
    print("Настройки")
    print("Введите '+' для сложения")
    print("Введите '-' для вычетания")
    print("Введите '*' для умножения")
    print("Введите '/' для деления")
    print("Введите 'Выход' для выхода из калькулятора")
    user_input = input(":")
    if user_input == "Выход":
        break
    elif user_input == "+":
        x,y = bring_numbers() 
        result = str(x+y)
        print ("Ответ: " + result)
    elif user_input == "-":
        result = str(x+y)
        result = str(num1-num2)
        print("Ответ: " + result) 
    elif user_input == "*":
        result = str(x+y)
        result = str(num1*num2)
        print("Ответ: " + result) 
    elif user_input == "/":
        result = str(x+y)
        result = str(num1/num2)
        print("Ответ: " + result) 
    else:
        print("неизвестный ввод")

Добавлено через 8 минут
а вообще проще из функции вернуть результат. у тебя запрос данных и так идет в цикле

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
def bring_numbers(num1, num2,user_input):
    if user_input == "+":
        return num1+num2 
    elif user_input == "-":
        return num1-num2
    elif user_input == "*":
        return num1*num2
    elif user_input == "/":            
        return num1/num2
    else:
        print("неизвестный ввод")
            
while True:
    print("Настройки")
    print("Введите '+' для сложения")
    print("Введите '-' для вычетания")
    print("Введите '*' для умножения")
    print("Введите '/' для деления")
    print("Введите 'Выход' для выхода из калькулятора")
    user_input = input(":")
    
    if user_input == "Выход":
        break
 
    num1 = float(input("Введите первое число: "))
    num2 = float(input("Введите второе число: "))
    print ('Результат =',bring_numbers(num1, num2,user_input))

Добавлено через 8 минут
прозевал исправить во всех условиях на
x,y = bring_numbers()
result = str(x (тут знак согласно условию) y)

@alex925 · 22.05.2016, 13:08

Для того, чтобы не было портянки условий используй словарь и код преобразится.

oldnewyear · 23.05.2016, 03:06

Python
1
2
user_input = input("Введите выражение: ")
print ('Результат =',eval(user_input))

Введите выражение: 1 + 6 * 6
Результат = 37

@alex925 · 23.05.2016, 07:02

oldnewyear, фу фу фу людей плохому учить

oldnewyear · 23.05.2016, 07:49

Сообщение от alex925

oldnewyear, фу фу фу людей плохому учить

А так?

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
while True:
    print("Настройки")
    print("Введите '+' для сложения")
    print("Введите '-' для вычетания")
    print("Введите '*' для умножения")
    print("Введите '/' для деления")
    print("Введите 'Выход' для выхода из калькулятора")
    user_input = input(":")
    
    if user_input == "Выход":
        break
 
    calc = {
    '+'  : lambda x,y: x + y,
    '-'  : lambda x,y: x - y,
    '*'  : lambda x,y: x * y,
    '/'  : lambda x,y: x / y}
 
 
    num1 = float(input("Введите первое число: "))
    num2 = float(input("Введите второе число: "))
    print ('Результат =',calc[user_input](num1, num2))

@Centrin0 · 23.05.2016, 08:16

lambda не нужны в таких количествах в консольном калькуляторе.

Не по теме:

Программирую на питоне и все еще удивляюсь таким конструкциям и применением словаря :D

@alex925 · 23.05.2016, 16:23

oldnewyear, так гораздо лучше.

Сообщение от Centrin0

lambda не нужны в таких количествах в консольном калькуляторе.

А почему бы и нет? Это не классы, которые без повода лучше не использовать. Тут все по делу.

@Vigi · 23.05.2016, 17:36

Немного скорректируем

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
calc = {
    '+': lambda x, y: x + y,
    '-': lambda x, y: x - y,
    '*': lambda x, y: x * y,
    '/': lambda x, y: x / y}
 
 
def foo(n1, n2, operation):
    return calc[operation](n1, n2)
 
 
while True:
    print("""
Настройки
Введите '-' для вычетания
Введите '*' для умножения
Введите '/' для деления
Введите 'Выход' для выхода из калькулятора
""")
    user_input = input(":")
    if user_input == "Выход":
        break
 
    for inc, i in enumerate(('num1', 'num2'), start=1):
        vars()[i] = float(input('Введите {}_е число: '.format(inc)))
 
    print('Результат =', foo(num1, num2, user_input))

@Whost · 23.05.2016, 22:05

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

Python
1
2
num1 = float(input())
num2 = float(input())

@alex925 · 23.05.2016, 22:09

А ещё лучше вот так просто:

Python
1
num1, num2 = [float(input('Введите число {}:'.format(i))) for i in range(1, 3)]

Компактно и главное не будет тормозить выполнение программы, потому что лукапы это очень дорогая операция + явное лучше не явного, такая магия не нужна тут.

@Whost · 23.05.2016, 22:26

alex925, так два значения всего, зачем так заморачиваться? Ну в крайнем случае вот так

Python
1
num1, num2 = float(input()), float(input())

Или я чего-то не понимаю?

@alex925 · 23.05.2016, 22:30

Я ни капли не заморочился, перемудрил Vigi, а у меня просто гибкий и масштабируемый код.

@Vigi · 24.05.2016, 03:12

Я особо то тоже не парился

просто написал возможный вариант создание переменных в рантайме, может в будущем и пригодится...

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@alex925 2742 / 2341 / 620 Регистрация: 19.03.2012 Сообщений: 8,830
	22.05.2016, 13:08
	Для того, чтобы не было портянки условий используй словарь и код преобразится. 0

@alex925 2742 / 2341 / 620 Регистрация: 19.03.2012 Сообщений: 8,830
	23.05.2016, 07:02
	oldnewyear, фу фу фу людей плохому учить 0

@Centrin0 4 / 4 / 5 Регистрация: 25.05.2014 Сообщений: 50
	23.05.2016, 08:16
	lambda не нужны в таких количествах в консольном калькуляторе. Не по теме: Программирую на питоне и все еще удивляюсь таким конструкциям и применением словаря :D 0

@alex925 2742 / 2341 / 620 Регистрация: 19.03.2012 Сообщений: 8,830
	23.05.2016, 22:30
	Я ни капли не заморочился, перемудрил Vigi, а у меня просто гибкий и масштабируемый код. 0

@Vigi 641 / 481 / 179 Регистрация: 28.05.2012 Сообщений: 1,419
	24.05.2016, 03:12
	Я особо то тоже не парился просто написал возможный вариант создание переменных в рантайме, может в будущем и пригодится... 0