input
retorna uma string, mas você está comparando o retorno dela com um número inteiro (na linha if opcao == 1
). Para fazer esta comparação da forma correta você pode transformar o retorno em um número (usando int(input(...))
, como sugeriu a outra resposta), ou compará-lo com uma string: if opcao == '1'
.
Mas acho que tem um jeito melhor. Repare que há muita repetição no código: todas as operações pedem que se digite os valores, e as mensagens são praticamente as mesmas, só mudando o nome da operação. Uma forma melhor seria parametrizar tudo isso e também ler os 2 valores antes de cada função.
Outro detalhe é que você está fazendo return print(...)
, só que print
retorna None
e não faz sentido retornar o resultado de um print
(ou você retorna o valor, ou imprime ele). No caso de funções, eu prefiro que elas retornem o valor e quem as chamou que decida o que fazer com o retorno (podendo inclusive imprimi=lo).
Enfim, uma sugestão de melhoria:
def calculadora(operacoes):
print("Ola! \n" "Bem vindo a calculadora digital em python! \n" "Por favor, digite a opção:")
print('\n'.join(f'Opção {i}: {op[0]}' for i, op in operacoes.items()))
opcao = input()
if opcao in operacoes:
nome_operacao, verbo, funcao = operacoes[opcao]
print(f'Vamos {verbo}!')
print(f'Agora, digite os valores a serem usados para a {nome_operacao}')
n = int(input())
m = int(input())
print(f'O resultado é {funcao(n, m)}')
else:
print('opção inválida')
def multiplicacao(n, m):
return n * m
def soma(n, m):
return n + m
def subtracao(n, m):
return n - m
def divisao(n, m):
return n / m
operacoes = {
'1': ('soma', 'somar', soma),
'2': ('subtração', 'subtrair', subtracao),
'3': ('multiplicação', 'multiplicar', multiplicacao),
'4': ('divisão', 'dividir', divisao)
}
calculadora(operacoes)
Agora a função calculadora
recebe um dicionário contendo as opções e as respectivas operações (na verdade, cada opção é mapeada para uma tupla contendo os textos relativos ao nome da operação e a função a ser chamada).
Depois, eu leio os números e passo eles como argumentos da função. Repare que cada função retorna o resultado da operação, e quem chamou a função se encarrega de imprimir o valor final (nada de return print
, que como eu já disse, não faz sentido).
Desta forma fica mais flexível (a função calculadora
pode receber um dicionário com operações completamente diferentes e ainda sim funcionará da mesma forma) e evita as repetições de código (não preciso mais ler os 2 números dentro de cada função, pois elas apenas recebem os valores que já foram lidos anteriormente - e veja também que nesse caso converti os valores para int
, caso contrário você estaria fazendo as operações com strings). Se quiser, pode usar também float
em vez de int
, caso o programa aceite números com casas decimais.