Skip to main content
conteúdo editado
Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

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=loimprimí-lo).

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).

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 imprimí-lo).

Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

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.