@Flamme_W: Ja, da würde man eine ``while True:``-Schleife verwenden. Und ``try``/``except``/``else`` mit der Ausnahme die dabei auftreten kann, um die entsprechend zu behandeln. Und Du solltest da dann auch noch negative Zahlen nach der Umwandlung prüfen, damit niemand auf die Idee kommt -42 Flaschen Rotwein zu bestellen.
Konstantennamen werden per Konvention KOMPLETT_GROSS geschrieben und den Aufruf von `bestellung()` würde man noch durch ein ``if __name__ == "__main__":`` absichern, damit das nur passiert wenn man das Modul als Programm ausführt, aber nicht wenn man es importiert.
Die Klammern bei der Rechnung sind nicht notwendig, es gilt selbstverständlich Punkt- vor Strichrechnung.
`round()` benötigt sehr selten, nämlich nur dann, wenn man tatsächlich mit gerundeten Werten weiterrechnen will. Um ausgaben, oder allgemein Umwandlungen in Zeichenketten auf eine Nachkommaanzahl zu beschränken, verwendet man die Formatanweisungen bei den Platzhaltern bei Zeichenkettenformatierung. Hat zudem den Vorteil, dass man darüber auch hinbekommt, das der Betrag immer genau zwei Nachkommastallen, auch bei abschliessenden 0en. Also "1199.00 Euro" statt "1198.0 Euro" falls man 100 Flaschen Rotwein kauft.
Namen sollten keine Abkürzungen enthalten und nicht in komischen Wortreihenfolgen. Also nicht `zahl_rot`, sondern `rotwein_anzahl`.
Funktionen (und Methoden) werden üblicherweise nach der Tätigkeit benannt die sie durchführen, damit man sich leichter von eher passiven Werten unterscheiden kann. `bestellung` wäre beispielsweise ein Name der für eine Datenstruktur geeignet wäre die eine Bestellung repräsentiert. Also zum Beispiel eine Liste aus Bestellposten und Einzelpreisen, oder ein komplexeres Objekt das auch das Bestelldatum und den Kundennamen enthält und von dem man beispielsweise auch den Gesamtpreis abfragen kann.
Das vorhandene Programm entsprechend überarbeitet:
Code: Alles auswählen
#!/usr/bin/env python3
ROTWEIN_PREIS = 12.99
ROSEWEIN_PREIS = 9.98
WEISSWEIN_PREIS = 11.99
def erfrage_anzahl(eingabeaufforderung):
...
return 23 # TODO
def bestellen():
print("Wieviel Wein möchtest du?")
rotwein_anzahl = erfrage_anzahl("Anzahl Rotwein: ")
rosewein_anzahl = erfrage_anzahl("Anzahl Rosewein: ")
weisswein_anzahl = erfrage_anzahl("Anzahl Weißwein: ")
ergebnis = (
rotwein_anzahl * ROTWEIN_PREIS
+ rosewein_anzahl * ROSEWEIN_PREIS
+ weisswein_anzahl * WEISSWEIN_PREIS
)
print(f"Der Gesamtbetrag ist {ergebnis:.2f} Euro.")
if __name__ == "__main__":
bestellen()
Nach der Funktion für die Abfrage von einer Anzahl vom Benutzer, wäre die nächste Verbesserung die Preise nicht als einzelne Konstanten zu definieren, sondern als Datenstruktur. Eine Liste mit Tupeln aus Produktname und Preis oder ein Wörterbuch das Produktnamen auf Preise abbildet. Dann ist das leichter erweiterbar, und man muss in der `bestellen()`-Funktion nicht den immer gleichen Code für jedes einzelne Produkt schreiben, sondern kann einer Schleife über die Produkte schreiben und die abfragen.
Danach kann man sich überlegen ob es sinnvoll ist immer nach allen Produkten zu fragen, denn wenn es es in dem Laden irgendwann einmal 100 Weinsorten gibt, will der Kunde sicher nicht 99 mal 0 eingeben müssen, wenn er eine Flasche von einer Sorte kaufen will.
