После вызова функции вылетает программа?

Question

[AnnoyingWolf]

После вызова функции и ввода туда 2-ух числовых значений закрывается программа

def ans_input():
    ans1 = float(input('Введите первое число:'))
    ans2 = float(input('Введите другое число:'))
while True:
    user_input = input('Ввод:')
    if user_input == 'выйти':
        break
    elif user_input == 'прибавить':
        ans_input()
        result = (ans1 + ans2)
        print('Ответ', result)
    elif user_input == 'отнять':
        ans_input()
        result = (ans1 - ans2)
        print('Ответ', result)
input('ENTER')

Но когда я вручную ввожу без использования функции,тогда всё работает прекрасно

elif user_input == 'прибавить':
        ans1 = float(input('Введите первое число:'))
        ans2 = float(input('Введите другое число:'))
        result = (ans1 + ans2)
        print('Ответ', result)

Answer 1

[Сергей Горностаев]

Переменные ans1 и ans2 локальные и существуют только в пределах функции ans_input.

def ans_input():
    ans1 = float(input('Введите первое число:'))
    ans2 = float(input('Введите другое число:'))
    return ans1, ans2


while True:
    user_input = input('Ввод:')
    if user_input == 'выйти':
        break
    elif user_input == 'прибавить':
        ans1, ans2 = ans_input()
        result = (ans1 + ans2)
        print('Ответ', result)
    elif user_input == 'отнять':
        ans1, ans2 = ans_input()
        result = (ans1 - ans2)
        print('Ответ', result)
input('ENTER')

Comments

[AnnoyingWolf]

И как их вывести за пределы функции,чтобы при этом они не выводились в начале кода?

[Влад Григорьев]

AnnoyingWolf, return ans1, ans2 ???

[AnnoyingWolf]

Влад Григорьев, не помогает

[Сергей Горностаев]

AnnoyingWolf, я дополнил ответ. Но это ведь основы основ. Если вы будете ответы на такие элементарные вопросы искать на Тостере, не быть вам программистом.

[AnnoyingWolf]

Сергей Горностаев, Спасибо большое,помогло))Я правильно понял,что переменным ans1 и ans2 поочередно присваиваются значения функции ans_input()?

[Сергей Горностаев]

AnnoyingWolf, функция возвращает кортеж, который распаковывается в две переменные.

[AnnoyingWolf]

Сергей Горностаев, понятно,ещё раз спасибо.

[AnnoyingWolf]

Я узнал что переменные в функции можно просто объявить глобальными,и всё, не нужно даже return писать

[Сергей Горностаев]

AnnoyingWolf, технически можно, но идея плохая. Использования глобальных переменных надо избегать.

[AnnoyingWolf]

Сергей Горностаев, можно поподробнее почему?

[Сергей Горностаев]

AnnoyingWolf, причин огромное количество. Например

• Отсутствие контроля доступа;
  
• Образование неявных связей и непредсказуемых состояний;
  
• Засорение пространства имён;
  
• Усложнение контроля за освобождение и выделением памяти;
  
• Ухудшение читаемости и сопровождаемости кода;
  
• И прочее, и прочее.
  

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

Всё это подробно описывается в каждом учебнике. И давно уже перешло в статус аксиом, как и использование безусловных переходов.

[AnnoyingWolf]

Сергей Горностаев, Спасибо большое)

Answer 2

[Антон Федорян]

def ans_input():
    ans1 = float(input('Введите первое число:'))
    ans2 = float(input('Введите другое число:'))
    return ans1 + ans2


while True:
    user_input = input('Ввод:')
    if user_input == 'выйти':
        break
    elif user_input == 'прибавить':
        result = ans_input()
        print('Ответ', result)
    else:
        pass
input('ENTER')

Comments

[AnnoyingWolf]

Нужно не только для 1 действия,а чтобы между значениями ans1 и ans2 можно было проводить операции

[Антон Федорян]

AnnoyingWolf, для чего их ввод в функцию тогда выносить?

[AnnoyingWolf]

Антон Федорян, для того чтобы сократить код и не писать каждый раз 2 input'а.Основная цель именно сократить код,чтобы просто требовались 2 ввода от пользователя,а дальше они обрабатывались в зависимости от выбора(прибавить/отнять)

[Сергей Горностаев]

AnnoyingWolf,

from operator import add, sub

commands = {
    'прибавить': add,
    'отнять': sub,
}

def ans_input(op):
    ans1 = float(input('Введите первое число:'))
    ans2 = float(input('Введите другое число:'))
    return op(ans1, ans2)


while True:
    user_input = input('Ввод:')
    if user_input == 'выйти':
        break
    else:
        if user_input not in commands:
            continue
        result = ans_input(commands[user_input])
        print('Ответ', result)

input('ENTER')