Vorbereitung auf meine Pythonklausur

[sharete]

@sharete: Man muss das Modul vorher natürlich importieren.

Habe es ausprobiert, im codeskulptor ( kennt ihr das eigentlich?) zeigt er mir dann an:

Line 1: ImportError: No module named itertools

Hier nochmal der ganze Code:

Code: Alles auswählen

from itertools import compress

X = [3,6,4,2,8,5,5,1]
 
Y = [1,0,0,1,1,1,0,0]
 
sum(itertools.compress(X, Y))

Ich hätte jetzt noch eine weitere Frage. Und zwar sollen wir ein Additionsprogramm schreiben, dass so lange ausgeführt wird, bis der Benutzer für X & Y jeweils eine 0 eingibt. Dies habe ich so geschrieben:

Code: Alles auswählen

def addition():
    value1 = int(raw_input("Eingabe"))
    value2 = int(raw_input("Eingabe"))
    
    while value1 != 0 and value2 != 0:
        print "x =",value1
        print "y =",value2
        print "x+y =",value1+value2
        value1 = int(raw_input("Eingabe"))
        value2 = int(raw_input("Eingabe"))
addition()

Wieso bricht er es aber schon ab, wenn ich für einen value eine 0 eingebe, also z.B. bei 0+5 ? Ich habe in der while Schleife doch die Bedingung and mit reingebracht?

LG

[BlackJack]

@sharete: Was ergibt denn der Audruck bei ``while`` als Ergebnis wenn `value1` gleich 0 ist und `value2` gleich 5 ist? Setz die Werte doch mal ein und rechne das aus. Und dann mach Dir klar bei welchem Ergebnis der Schleifenkörper betreten wird und bei welchem nicht.

[sharete]

@sharete: Was ergibt denn der Audruck bei ``while`` als Ergebnis wenn `value1` gleich 0 ist und `value2` gleich 5 ist? Setz die Werte doch mal ein und rechne das aus. Und dann mach Dir klar bei welchem Ergebnis der Schleifenkörper betreten wird und bei welchem nicht.

Also muss ich aus dem and einfach ein oder machen

Code: Alles auswählen

def addition():
    value1 = int(raw_input("Eingabe"))
    value2 = int(raw_input("Eingabe"))
   
    while value1 != 0 or value2 != 0:
        print "x =",value1
        print "y =",value2
        print "x+y =",value1+value2
        value1 = int(raw_input("Eingabe"))
        value2 = int(raw_input("Eingabe"))
addition()

[BlackJack]

@sharete: Wenn in allen möglichen Fällen das richtige passiert, dann ja. Es gibt vier Fälle. `value1` kann 0 oder ≠0 sein, und `value2` kann 0 oder ≠0 sein. Wenn Du alle vier Fälle durchrechnest, und da immer der richtige Wert für die Bedingung heraus kommt, dann passt das so. Falls nicht, dann nicht.

Warum hast Du das denn jetzt so umgestellt? Eine ``while True:``-Schleife die bei der entsprechenden Bedingung abgebrochen wird, wäre hier sinnvoller und entspräche mehr dem was man da eigentlich machen möchte. Zudem sollte man den Code für die Eingabe der beiden Werte nicht zweimal im Programm stehen haben. Code- und Datenwiederholungen machen Programme aufwändiger zu ändern und dabei erhöht sich auch die Gefahr das man Fehler macht, die dann wiederum schwieriger zu finden sind.

[sharete]

Es hat so gepasst, also lasse ich das jetzt mal so stehen .

Mal zu einer anderen Aufgabe, die mich ein paar Haare gekostet haben. Ich poste einfach mal die Aufgabenstellung und meinen Code mit dazu, mit der bitte, mal drüber zu schauen. Und zwar die Aufgabe:

Code: Alles auswählen

Schreiben Sie ein Programm, dass die Nettosumme eines Bankkontos basierend auf Einträgen aus der Transaktionsdatei berechnet, die an der Konsole eingegeben werden
Das Format der Transaktionsdatei soll wie folgt aussehen:
H 100
S 200
H bedeutet Haben und S bedeutet Soll.
Angenommen, die Eingabe für das Programm sieht wie folgt aus:
H 300
H 300
S 200
H 100
(Beachten Sie, dass der Buchstabe vom Betrag jeweils durch ein Blank getrennt ist)
Die Ausgabe des Programmes sollte dann wie folgt aussehen:
500

Mein Code der bis dato so funktioniert, wobei ich aber denke, dass es einfacher geht:

Code: Alles auswählen

def habensoll():
    eingabe = 0
    gesamt = 0
    while eingabe!="abbruch":
        eingabe = raw_input("Haben oder Soll")
        
        if eingabe[0] == "H":
            eingabe=eingabe.split("H")
            eingabe.pop(0)
            umwandeln = int(eingabe[0])
            gesamt = gesamt + umwandeln
         
        elif eingabe[0] == "S":
            eingabe = eingabe.split("S")
            eingabe.pop(0)
            umwandeln2 = int(eingabe[0])
            gesamt = gesamt - umwandeln2
            
    print gesamt
habensoll()

[BlackJack]

@sharete: Beim einen sagst Du es funktioniert doch so und beim anderen fragst Du obwohl es doch so funktioniert was man besser machen kann? Denn auch hier fällt mit als erstes auf, das man Code und Daten nicht wiederholen sollte. Im ``if``- und im ``elif``-Zweig steht nahezu der gleiche Code und 'H' und 'S' stehen mehr als einmal im Quelltext.

Durchnummerierte Namen sind auch keine gute Idee. Warum diese sinnlose 2 bei `umwandeln2`? `umwandeln` hätte vollkommen gereicht. Zudem ist `umwandeln` eher ein Name für eine Funktion, die normalerweise nach der Tätigkeit benannt werden die sie durchführen.

Auch hier wäre wieder eine ``while True``-Schleife angebracht, die bei der Eingabe von 'abbruch' dann mit ``break`` verlassen wird.

An einen Namen sollten nur Werte des gleichen (Duck-)Typs gebunden werden. Und nicht wie bei `eingabe` mal eine Zahl und mal eine Zeichenkette. Wenn man für einen Namen noch keinen konkreten Wert hat, dann gibt es den speziellen Wert `None`. Das weiss der Leser dann auch gleich was Sache ist und fragt sich nicht ob ausgerechnet die 0 eine besondere Bedeutung hat.

Das mit dem Aufteilen an 'S' oder 'H' und dann entfernen des ersten Elements, das *immer* eine leere Zeichenkette ist, und das dann auch noch mit `pop()` statt mit ``del`` ist extrem umständlich und ja eigentlich so gar nicht das offensichtliche was man hier machen würde. Das wäre *am* Leerzeichen die beiden Werte zu trennen und die beiden Elemente zwei sinnvollen Namen zuweisen. Dann kann man das zweite Element in eine Zahl umwandeln und braucht dann nur noch entscheiden ob man es von der Gesamtsumme abziehen oder es addieren soll.

Ungetestet:

Code: Alles auswählen

def habensoll():
    gesamt = 0

    while True:
        eingabe = raw_input('Haben oder Soll')
        if eingabe == 'abbruch':
            break

        haben_oder_soll, wert = eingabe.split()

        if haben_oder_soll == 'H':
            faktor = 1
        elif haben_oder_soll == 'S':
            faktor = -1
        else:
            faktor = 0

        gesamt += int(wert) * faktor
           
    print gesamt

[sharete]

Sorry, ich wollte dich nicht angreifen, indem ich das so stehen lasse .
Also zu der Aufgabe davor, wenn du sagt mit while True wäre der Code besser durchdacht, meinst du das so?

Code: Alles auswählen

def addition():
    value1 = int(raw_input("Eingabe"))
    value2 = int(raw_input("Eingabe"))
    while True:
        print "x =",value1
        print "y =",value2
        print "x+y =",value1+value2
        value1 = int(raw_input("Eingabe"))
        value2 = int(raw_input("Eingabe"))
        if value1 == 0 and value2 == 0:
            break
addition()

Zu der letzten Aufgabe bezüglich des Haben Soll fällt mir auf, dass in meinem Code das irrelevant ist, ob ich H100 oder H 100 schreibe und bei dir ist es zwingend notwendig mit Leerzeichen zu schreiben. Ist ja nicht weiter schlimm, jedoch wüsste ich gerne, wodurch das verursacht wird?!

[BlackJack]

@sharete: Ja, das ist schon fast perfekt. Jetzt musst Du nur noch die Eingabe *vor* der Schleife entfernen und die Eingabe *in* der Schleife vor die Rechnung verschieben.

Das mit dem nötigen Leerzeichen wird dadurch verursacht, dass `split()` an Leerzeichen (und anderen ”whitespace”-Zeichen) aufteilt. Die Aufgabenstellung garantiert das Leerzeichen ja. Wenn das nicht der Fall wäre, dann würde man aber gar kein `split()` verwenden, sondern einfach das erste Zeichen mit ``eingabe[0]`` erfragen und den Rest mit ``eingabe[1:]``. Man sollte in dem Fall vorher noch einen Test setzen ob die Zeichenkette überhaupt etwas enthält, weil ``eingabe[0]`` zu einer Ausnahme führt wenn der Benutzer einfach die Eingabetaste betätigt ohne etwas einzugeben.