modul:m323:learningunits:lu01:loesungen:spaghetticode

Unterschiede

Hier werden die Unterschiede zwischen zwei Versionen angezeigt.

Link zu dieser Vergleichsansicht

Beide Seiten der vorigen Revision Vorhergehende Überarbeitung
modul:m323:learningunits:lu01:loesungen:spaghetticode [2026/08/18 10:17] – Codedarstellung in Tabelle und Typografie korrigiert adminmodul:m323:learningunits:lu01:loesungen:spaghetticode [2026/08/18 11:11] (aktuell) – Musterloesung via gh-Import aus dem solution-Branch admin
Zeile 14: Zeile 14:
 ===== 2. Musterlösung ===== ===== 2. Musterlösung =====
  
-<code python> +~~NOCACHE~~ 
-bestellungen = [ +=== main.py === 
-    {"artikel": "Maus",     "status": "offen",     "menge": 3,  "preis": 24.90}, +{{gh>https://github.com/templates-python/m323-lu01-a11-umsatz-refactoring/blob/solution/main.py}}
-    {"artikel": "Tastatur", "status": "storniert", "menge": 2,  "preis": 79.50}, +
-    {"artikel": "Monitor",  "status": "offen",     "menge": 0,  "preis": 249.00}, +
-    {"artikel": "Kabel",    "status": "geliefert", "menge": 10, "preis": 9.90}, +
-    {"artikel": "Dock",     "status": "offen",     "menge": 1,  "preis": 189.00}, +
-]+
  
 +Übrig bleiben genau die drei Grundstrukturen: Sequenz (Initialisierung und Rückgabe), Iteration (''for''), Selektion (''if'').
  
-def umsatz_alt(bestellungen): +===== 3. Was durch den Umbau verschwunden ist =====
-    """Ausgangsfassung - bleibt zum Vergleich stehen.""" +
-    i = 0 +
-    fertig = False +
-    total = 0 +
-    while not fertig: +
-        if i >= len(bestellungen): +
-            fertig = True +
-        else: +
-            b = bestellungen[i] +
-            if b["status"] == "storniert": +
-                i = i + 1 +
-                continue +
-            else: +
-                if b["menge"] <= 0: +
-                    i = i + 1 +
-                    continue +
-                else: +
-                    total = total + b["menge"] * b["preis"] +
-                    i = i + 1 +
-    return total+
  
 +  * **Der Index ''i''.** Damit auch jede Möglichkeit, ihn falsch zu initialisieren, an einer Stelle nicht hochzuzählen (Endlosschleife!) oder um eins danebenzuliegen.
 +  * **Die Flagge ''fertig''.** Die ''for''-Schleife kennt ihr Ende selbst.
 +  * **Die Sprünge.** Ohne ''continue'' liest sich der Ablauf von oben nach unten.
 +  * **Zwei Verschachtelungsebenen.** Die beiden Bedingungen sind zu einer benannten Frage zusammengefasst, die man laut vorlesen kann: //Wenn die Bestellung verrechenbar ist, zähle sie dazu.//
  
-def ist_verrechenbar(bestellung): +===== 4. Wie geprüft wird =====
-    """Selektion: Zaehlt diese Bestellung zum Umsatz?""" +
-    return bestellung["status"] != "storniert" and bestellung["menge"] > 0+
  
 +Die Gleichwertigkeit prüft ''test_umsatz_wie_referenz'' gegen ''umsatz_alt'' aus ''referenz.py'' – über vier Datensätze, inklusive leerer Liste, komplett stornierter Liste und ungültiger Mengen.
  
-def umsatz(bestellungen): +Die //Struktur// lässt sich mit einem gewöhnlichen Test nicht messen: Beide Fassungen liefern 362.70. Darum lesen zwei Tests die Datei ''main.py'' als Syntaxbaum ein:
-    """Sequenz - Iteration - Selektion, je genau einmal.""" +
-    total = 0 +
-    for bestellung in bestellungen: +
-        if ist_verrechenbar(bestellung): +
-            total += bestellung["menge"] * bestellung["preis"] +
-    return total+
  
 +=== main_test.py ===
 +{{gh>https://github.com/templates-python/m323-lu01-a11-umsatz-refactoring/blob/solution/main_test.py}}
  
-if __name__ == '__main__': +''test_iteration_ohne_while'' verlangt eine ''for''-Schleife und verbietet ''while'', ''test_keine_spruenge'' verbietet ''continue'' und ''break''. Beide prüfen zusätzlich das korrekte Ergebnis – sonst wären sie mit einer leeren Datei erfüllbar.
-    print(f"Umsatz alt: {umsatz_alt(bestellungen):.2f}")   # 362.70 +
-    print(f"Umsatz neu: {umsatz(bestellungen):.2f}")       # 362.70+
  
-    assert umsatz(bestellungen) == umsatz_alt(bestellungen) +Eine Abgabe, die richtig rechnet, aber die alte Struktur behält, kommt damit auf 9 von 11 Testpunkten.
-    assert umsatz([]) == umsatz_alt([]) == 0 +
-    assert umsatz([{"status": "storniert", "menge": 5, "preis": 10.0}]) == 0 +
-    assert umsatz([{"status": "offen", "menge": -2, "preis": 10.0}]) == 0 +
-    print("Beide Fassungen liefern dasselbe Ergebnis.") +
-</code> +
- +
-===== 3. Was durch den Umbau verschwunden ist ===== +
- +
-  * **Der Index ''i''.** Damit auch jede Möglichkeit, ihn falsch zu initialisieren, an einer Stelle nicht hochzuzählen (Endlosschleife!) oder um eins danebenzuliegen. +
-  * **Die Flagge ''fertig''.** Die ''for''-Schleife kennt ihr Ende selbst. +
-  * **Die Sprünge.** Ohne ''continue'' liest sich der Ablauf von oben nach unten. +
-  * **Zwei Verschachtelungsebenen.** Die beiden Bedingungen sind zu einer benannten Frage zusammengefasst, die man laut vorlesen kann: //Wenn die Bestellung verrechenbar ist, zähle sie dazu.// +
- +
-Übrig bleiben genau die drei Grundstrukturen: Sequenz (Initialisierung und Rückgabe), Iteration (''for''), Selektion (''if'').+
  
-===== 4. Antworten auf die Zusatzfragen =====+===== 5. Antworten auf die Zusatzfragen =====
  
 ==== Frage 1: Vergessenes Hochzählen ==== ==== Frage 1: Vergessenes Hochzählen ====
Zeile 99: Zeile 58:
 <code python> <code python>
 def umsatz_deklarativ(bestellungen): def umsatz_deklarativ(bestellungen):
-    return sum(b["menge"] * b["preis"] for b in bestellungen if ist_verrechenbar(b))+    return sum(b['menge'] * b['preis'] for b in bestellungen if ist_verrechenbar(b))
 </code> </code>
  
  • modul/m323/learningunits/lu01/loesungen/spaghetticode.txt
  • Zuletzt geändert: 2026/08/18 11:11
  • von admin