LU01.A11 - Spaghetticode entwirren

Die Funktion umsatz_alt in referenz.py berechnet den Umsatz aus einer Liste von Bestellungen. Sie ist korrekt, aber niemand liest sie gern: eine while-Schleife mit Abbruchflagge, ein von Hand hochgezählter Index, continue-Sprünge und drei Verschachtelungsebenen.

Diese Datei ist der Massstab: Ihre Fassung muss für jede Eingabe exakt dasselbe liefern. Verändern dürfen Sie sie nicht.

referenz.py

"""
Die urspruengliche Fassung der Umsatzberechnung - NICHT VERAENDERN.
 
Sie ist korrekt, aber niemand liest sie gern: eine while-Schleife mit
Abbruchflagge, ein von Hand hochgezaehlter Index, continue-Spruenge und drei
Verschachtelungsebenen.
 
Ihre Fassung in main.py muss fuer jede Eingabe dasselbe Ergebnis liefern.
Diese Datei wird nicht gelintet und nicht bewertet.
"""
 
BESTELLUNGEN = [
    {'artikel': 'Maus', 'status': 'offen', 'menge': 3, 'preis': 24.90},
    {'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},
]
 
 
def umsatz_alt(bestellungen):
    """Berechnet den Umsatz - so, wie man es besser nicht schreibt."""
    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
 
 
if __name__ == '__main__':
    print(f'Umsatz alt: {umsatz_alt(BESTELLUNGEN):.2f}')

main.py

"""
LU01.A11 - Spaghetticode entwirren
 
Bringen Sie die Umsatzberechnung aus referenz.py auf die drei Grundstrukturen
der strukturierten Programmierung zurueck - Sequenz, Selektion und Iteration -
ohne ihr Verhalten zu veraendern.
 
Verboten sind in dieser Datei: while, continue und break.
"""
 
from referenz import BESTELLUNGEN
 
 
def ist_verrechenbar(bestellung):
    """
    Entscheidet, ob eine Bestellung zum Umsatz zaehlt.
 
    Eine Bestellung zaehlt, wenn sie nicht storniert ist und
    die Menge groesser als 0 ist.
 
    :param bestellung: dict mit artikel, status, menge, preis
    :return: True oder False
    """
    # TODO Beide Bedingungen zu einer Aussage zusammenfassen und zurueckgeben
 
 
def umsatz(bestellungen):
    """
    Summiert Menge mal Preis ueber alle verrechenbaren Bestellungen.
 
    Verwenden Sie eine for-Schleife direkt ueber die Elemente, ohne Index und
    ohne Abbruchflagge, und hoechstens eine Verschachtelungsebene darin.
 
    :param bestellungen: Liste von Bestellungen
    :return: Umsatz als Zahl, bei leerer Liste 0
    """
    # TODO Sequenz - Iteration - Selektion, je genau einmal
 
 
def main():
    """Gibt den Umsatz der Beispielbestellungen aus."""
    print(f'Umsatz neu: {umsatz(BESTELLUNGEN):.2f}')
 
 
if __name__ == '__main__':
    main()

1. Analysieren: Markieren Sie im Code von referenz.py, wo Sequenz, Selektion und Iteration vorkommen. Welche Stellen entsprechen keiner der drei Grundstrukturen, sondern imitieren einen Sprung?
2. Umbauen: Implementieren Sie in main.py die beiden vorgegebenen Funktionen. Vorgaben:
   • Die Schleife läuft direkt über die Elemente (for bestellung in bestellungen), ohne Index und ohne Abbruchflagge.
   • Kein continue, kein break, kein while.
   • Höchstens eine Verschachtelungsebene innerhalb der Schleife.
   • Die Bedingung, ob eine Bestellung überhaupt zählt, steckt in ist_verrechenbar und gibt True oder False zurück.
3. Begründen: Halten Sie in zwei bis drei Sätzen fest, welche Fehlerquellen im Ausgangscode durch den Umbau verschwunden sind.

Umsatz neu: 362.70

1. Der Ausgangscode zählt i an drei verschiedenen Stellen hoch. Was passiert, wenn man eine dieser Stellen vergisst? Probieren Sie es in einer Kopie aus.
2. Das Theorem von Böhm und Jacopini besagt, dass jeder Algorithmus mit den drei Grundstrukturen auskommt. Wie stützt Ihr Umbau diese Aussage?
3. Für Schnelle: Formulieren Sie die Anforderung an umsatz deklarativ – als Satz über das Ergebnis, ohne Schleife und Zwischensumme.

1. Akzeptiere das Classroom-Assignment
2. Klone dein persönliches Repository in die Entwicklungsumgebung
3. Analysiere referenz.py und beantworte Teil 1
4. Implementiere ist_verrechenbar und umsatz in main.py
5. Lokal prüfen mit pytest und python _run_pylint.py
6. Pushen – die Bewertung läuft automatisch

Die Abgabe des Codes erfolgt als Push in das persönliche GitHub-Repository. Die Analyse, die Begründung und die Zusatzfragen geben Sie zusätzlich in Moodle ab.

© Kevin Maurizi