Python Kalkulator

Question

Co zmienić, co poprawić?

def dodawanie():
    x = float(input("Podaj pierwszą liczbę: "))
    y = float(input("Podaj drugą liczbę: "))
    print("Twój wynik to: ", x + y)

def odejmowanie():
    x = float(input("Podaj pierwszą liczbę: "))
    y = float(input("Podaj drugą liczbę: "))
    print("Twój wynik to: ", x - y)

def mnozenie():
    x = float(input("Podaj pierwszą liczbę: "))
    y = float(input("Podaj drugą liczbę: "))
    print("Twój wynik to: ", x * y)

def dzielenie():
    x = float(input("Podaj pierwszą liczbę: "))
    y = float(input("Podaj drugą liczbę: "))
    print("Twój wynik to: ", x / y)

def powrot():
    pow1 = input("Czy chcesz wrócić do menu?: ")
    if pow1 == "tak":
        menu()
    elif pow1 == "nie":
        print("Koniec programu")
    else:
        print("Tak/Nie")
        powrot()

def menu():
    print("---Witaj w kalkulatorze--- ")
    print("Wybierz co chcesz zrobić:")
    print("Dodawanie - 1")
    print("Odejmowanie - 2")
    print("Mnożenie - 3")
    print("Dzielenie - 4")
    cozrobic = int(input("Podaj co chcesz zrobić: "))
    if cozrobic == 1:
        dodawanie()
        powrot()
    elif cozrobic == 2:
        odejmowanie()
        powrot()
    elif cozrobic == 3:
        mnozenie()
        powrot()
    elif cozrobic == 4:
        dzielenie()
        powrot()
menu()

 

Answer 1

Możesz wyrzucić powtarzające się elementy z funkcji (pytanie o liczby)

def dodawanie(x,y):
    return x+y

i później

    x = float(input("Podaj pierwszą liczbę: "))
    y = float(input("Podaj drugą liczbę: "))
    cozrobic = int(input("Podaj co chcesz zrobić: "))
    if cozrobic == 1:
        print(dodawanie(x,y))
        powrot()

trochę printów można też zaoszczedzić używająć """

    print(
        """
        ---Witaj w kalkulatorze--- 
        Wybierz co chcesz zrobić:
        Dodawanie - 1
        Odejmowanie - 2
        Mnożenie - 3
        Dzielenie - 4
        """
        )

Przydałby się też jakiś warunek dla dzielenia przez zero

def dzielenie(x,y):
    if y == 0:
        return 'nie można dzielić przez zero'
    return x/y

 

Comments

Dziękuję bardzo, przyda się :)