Python Code unsauber?

bool_again1 = True
bool_again2 = True

while bool_again1:

    bool_again2 = True

    while bool_again2:
        int_operator = int(input("Do you want to add(1), subtract(2), multiply(3) or divide(4) two numbers: "))

        if int_operator < 1 or int_operator > 4:
            print("Invalid Input")
        else:
            bool_again2 = False

    bool_again2 = True

    int_number1 = int(input("First number: "))
    int_number2 = int(input("Second number: "))

    match int_operator:
        case 1:
            print("The solution is: ", int_number1 + int_number2)
        case 2:
            print("The solution is: ", int_number1 - int_number2)
        case 3:
            print("The solution is: ", int_number1 * int_number2)
        case 4:
            print("The solution is: ", int_number1 / int_number2)

    while bool_again2:
        int_again = int(input("Do you want to use the calculator again ( Yes(1), No(2) ): "))

        if int_again == 1:
            bool_again2 = False
        elif int_again == 2:
            bool_again1 = False
            bool_again2 = False
        elif int_again != 1 or int_again != 2:
            print("Invalid Input")

Ich bin ziemlich neu bei der Programmierung und habe dieses einfache Rechenprogramm geschrieben. Allerdings möchte ich erfahrenen Programmierer fragen, wie ich diesen Code besser schreiben könnte oder ob er ok ist, da er meiner Meinung nach sehr unsauber aus sieht (durch die vielen verschiedenen "again" Variabelen, etc)

Danke

Answer 1

[Kelec]

Mir würde jetzt direkt bei dem einfallen, dass du die Schleifen mit again_2 weglassen könntest und durch continue ersetzen könntest. Also bei ungültiger Eingabe keine Schleife sondern continue aufrufen, das würde die erste Schleife komplett ersetzen.

Da die int Funktion in Python Exceptions werfen kann kannst du diese ebenfalls abfangen und nutzen. So würde dein Programm einfach abbrechen.

Die 2te while ist zwar nicht wirklich redundant wird aber meist so gelöst, dass ungültige Eingaben zB einfach als ein Nein gewertet werden. Hängt eben vom Programm ab.

Ansonsten wären diese While Blöcke auch Platz für eigene Funktionen also zB eine Funktion getOperator, welche solange die Eingabe liest bis ein gültiger Operator gewählt wurde. Dadurch sparst du dir die verschachtelten Schleifen und die extra Variable was das einfacher lesbar macht.

Answer 2

[Schachpapa]

Vernünftige Variablennamen sind die halbe Miete für Lesbarkeit.

statt bool_again1 vielleicht notDone

statt bool_again2 vielleicht inputInvalid

Den letzten Block würde ich so schreiben:

cont = ""
while cont not in ("1", "2"):
  cont = input("Do you ...")
  if cont == "2":
     notDone = False
  # wenn connt == 1 ist, wird der große Block wiederholt

Erfahrung ist die Summe aller selbst gemachten (und beseitigten) Fehler ;-)

Je mehr Programme du schreibst, umso besser wird dein Stil.

Answer 3

[darrenefecto]

Anstatt if und else kannst du if (!bool_again) return bool_again = False;

Und Kommentare schreiben macht alles immer verständlicher

Woher ich das weiß: Hobby – Mehrjährige Erfahrung in Spieleprogrammierung (C#, C++)