Auch mit dem Wörterbuch hat man immer noch eine Indirektion von Zeichenketten mit besonderer Bedeutung und dem Code für die Rechenart. Besser/direkter/einfacher wäre es die Rechenarten gleich als Paar von Rechensymbol und ausführbarer Operation (=Funktion) zu speichern und daraus auszuwählen.
Die erste Zuweisung an `new` finde ich zu versteckt. So Mehrfachzuweisungen würde ich eher machen wenn die Dinge etwas miteinander zu tun haben.
Wenn man die Zuweisung an `new` an den Ort verschiebt wo sie letztlich benötigt wird, sieht man relativ deutlich, dass das eigentlich in der ``while``-Bedingung verwendet werden sollte, statt das ganz am Ende der Schleife manuell zu testen. Und `new` als Name ist ja mittlerweile nicht mehr richtig.
``continue`` ist an sich schon unschön, weil das ein unbedingter Sprung ist, denn man nicht an der Einrückung erkennen kann. Und es ist problematisch wenn man Teile aus Schleifen in eigene Funktionen heraus ziehen will, oder Code am Ende *jeden* Schleifendurchlaufs hinzufügen muss. Hier ändert sich auch gar nichts am Programmablauf wenn man das einfach weg lässt.
Überarbeitet (ungetestet):
Code: Alles auswählen
#!/usr/bin/env python3
import random
from operator import add, mul as multiply
CALCULATION_TYPE = [("+", add), ("*", multiply)]
def make_formula(formula_type):
symbol, operation = formula_type
a = random.randint(1, 9)
b = random.randint(1, 9)
return f"{a} {symbol} {b}", operation(a, b)
def main():
print("Summen-/Produkt-Trainer (Ende mit x)")
is_running = True
while is_running:
formula, result = make_formula(random.choice(CALCULATION_TYPE))
while True:
answer = input(f"Wie lautet die Lösung von {formula} = ? ")
if answer == "x":
is_running = False
break
try:
is_correct = int(answer) == result
except ValueError:
print("Bitte nur ganze Zahlen eingeben.")
else:
if is_correct:
print("Die Lösung ist richtig!")
else:
print(
f"Die Lösung ist falsch!"
f" Richtig ist {formula} = {result}."
)
break
print("Auf Wiedersehen!")
if __name__ == "__main__":
main()
Das ist jetzt wieder eine Umsetzung wo die Benutzereingabe samt Umwandlung nicht sauber von der Auswertung getrennt ist. Finde ich deswegen problematisch, weil sobald das etwas grösser wird und/oder man das auch so sinnvoll auf Funktionen aufteilen will, eine offensichtliche Funktion so etwas wie ``def ask_number(prompt)`` ist, und das bekommt man aus dieser Programmstruktur nicht einfach herausgezogen.
Edit: Bezog sich alles auf den vorletzten Beitrag. Jetzt zum letzen: ``while new := True:`` ist umständlich für ``while True:``. Das `new` macht da keinen Sinn.
`sum_points` und `get_points` wären Funktionsnamen. Das zweite sollte eher `total_points` heissen, das erste eher `max_points` or `max_reachable_points`.
`nr_calcs` besteht aus zwei Abkürzungen und dazu noch aus einer Deutschen und einer Englischen. `number` abgekürzt ist `no`. Und `no` ist problematisch weil es für sich selbst eine Bedeutung hat und deshalb verwirrend werden kann. `calculation_count` oder `question_count` würde ich da verwenden.
Statt ``int(points/ 2)`` könnte man auch ``points // 2`` schreiben.
Die Punkte pro Aufgabentyp liessen sich in die Tupel aus der Lösung weiter oben einfach einbauen. So anonyme Tupel mit drei Werten sind dann aber für meinen Geschmack auch langsam die Grenze wo man besser auf `collections.namedtuple()` oder eine eigene Klasse umsteigt.
