Das deutsche Python-Forum

Code: Alles auswählen

dict = {'key2':'Wert2', 'key1':'Wert1','key3':'Wert3', 'key4':'Wert4'}

import sys

def buchslesen (key,value):
    Buch = open("datei2.txt", "r")
    Buch.readlines()
    return Buch

def worteingeben (dict):
    key = input("Geben Sie ein Wort ein: ")
    if key in dict:
        print ("Das deutsche Wort dafür lautet:"), dict[key]
        print (dict[key]);
    else:
        print ("Das Wort existiert nicht in meinem Wörterbuch")
        "einfügen" == input("Möchten Sie das Wort einfügen")
        buchschreiben(key,'value')
        dict.append = {'key5':'Wert5'}

print("Herzlich willkommen")
"begin" == input("Möchten Sie anfangen")
if ("ja"):
    print(" ")
    dict = buchslesen
    worteingeben(buchlesen)
else:("nein")
sys.exit()
ende(worteingabe)
print("schade... dann beim nächsten Mal")


def ende():
    sys.exit("message")
    os._exit()
 



def buchschreiben (key,value):
    Buch = open("datei2.txt", "w")
    Buch.write(key + " " + value + "\n")

Code: Alles auswählen

begin = 1
"begin" = 1

Code: Alles auswählen

dict = {'key2':'Wert2', 'key1':'Wert1','key3':'Wert3', 'key4':'Wert4'}
# "dict" überschreibt einen eingebauten Typ bei Python und ich ein schlechter Name.
# Jede halbwegs gute IDE zeigt das auch an und selbst beim Posten hier sollte auffallen,
# dass der Name anders eingefärbt wird als andere Namen.
#
# Es ist immer empfehlenswert, mit guten Beispielen zu arbeiten, wenn das hier also eine
# Liste von Vokabeln werden soll, dann also auch gleich passende Begriffe benutzen:
#
# vokabeln = {"vocable": "Vokabel"}
#
# Allerdings sollen Vokabeln ja wohl aus einer Datei "datei2.txt" gelesen werden,
# Da muss daher gar nichts initialisiert werden. Übrigens, zweimal ein und den selben
# Dateinamen zu benutzen sollte dazu führen, dass man den als Konstante vereinbart:
# 
# vokabeln = {}
#
# vokabeln_dateiname = "datei2.txt"
#

import sys
# "sys" - und auch das später benutzte "os" sind für dieses Programm unnötig.
# Der Import kann also weg.

def buchslesen (key,value):
# "buchslesen"? Der Blick für Kleinigkeiten ist bei der Software-Entwicklung essentiell,
# denn der Computer verzeiht keinen Schreibfehler und jedes Zeichen ist wichtig.
#
# Was sollen "key" und "value" als Parameter machen?
#
# Da es eine Konvention von Python ist, dort, wo man zwei Wörter benutzen würde, ein "_"
# zu benutzen, würde ich diese Funktion lieber so nennen:
#
# def buch_einlesen():
    
    Buch = open("datei2.txt", "r")
# eine weitere Konvention bei Python ist, lokale Variablen klein zu schreiben:
#
#    buch = open(vokabeln_dateiname)
#
# das "r" kann man übrigens weglassen. Da man Dateien auch wieder schließen muss, ist
# es besser, eine "with"-Anweisung zu benutzen, denn dann macht Python das automatisch,
# also:
#
#    with open(vokabeln_dateiname) as buch:
#
    Buch.readlines()
# Dieser Befehl verpufft einfach so, denn das, was er zurückgibt, wird sofort wieder
# vergessen. Gemeint ist sicherlich etwas wie dieses:
#    zeilen = buch.readlines()
#
    return Buch
# Das ist genau so falsch, denn niemand will das Datei-Objekt haben. Zusammen mit
# "with", kann ich die Funktion wie folgt schreiben:
#
# def buch_einlesen():
#    with open(vokabeln_dateiname) as buch:
#       return buch.readlines() 
#
# Das ist aber noch nicht genug, denn wenn ich meine Vokabeln in einem dict halten
# will, muss ich die Liste der Zeilen aus der Datei in ein dict (ich meine hier jeweils
# den Python-Datentyp dict, nicht die falsch benannte Variable weiter oben) verwandeln.
# Dazu kann ich jede Zeile, in der hoffentlich zwei Wörter stehen mit "split" zerteilen.
# Da dummerweise in jeder Zeile noch ein Zeilenendzeichen ('\n') stehen kann, muss ich
# dieses mit "strip" entfernen, also so:
#
# def buch_einlesen():
#    with open(vokabeln_dateiname) as buch:
#       for zeile in buch.readlines():
#           englisches_wort, deutsches_word = zeile.strip().split() 
#           vokabeln[englisches_wort] = deutsches_word
#
# Nun brauche ich auch keine Rückgabe mehr.
#
# Übrigens, man muss natürlich "buch_einlesen" auch noch irgendwo aufrufen.

def worteingeben (dict):
# Was soll "dict" hier? Wenn schon eine globale Variable, dann soll man auch dazu
# stehen. Hier würde man sie nur mit einer lokalen Variable überschreiben. 
# Und wir erinnern uns an die Regel mit dem "_", ja?
#
    key = input("Geben Sie ein Wort ein: ")
# "key" ist ein schlechter Name. Der Prompt von input und auch der Name der Funktion
# suggerieren einen besseren Namen. Also warum nicht "wort", wo doch sonst auch alles
# deutsch ist?
#
    if key in dict:
        print ("Das deutsche Wort dafür lautet:"), dict[key]
        print (dict[key]);
# Diese drei Zeilen sind übrigens fast korrekt. Wenn man die Argumente bei "print"
# Klammern muss, was du ja herausgefunden oder übernommen hast, warum steht dann 
# das "dict[key]" nicht auch in der Klammer? Und was soll das ";" bewirken? Muscle
# memory durch Java-Programmierung? Kann ich mir aber eigentlich nicht vorstellen,
# denn dann wären auch die grundsätzlichen Konstrukte bei Python schon bekannt.
# Also: Das ";" kann einfach weg und die ")" beim ersten print muss nach ganz hinten.
# Und die dritte Zeile ist natürlich eigentlich redundant.
#
#   if wort in vokabeln:
#       print("Das deutsche Wort lautet", vokabeln[wort])
#
    else:
        print ("Das Wort existiert nicht in meinem Wörterbuch")
        "einfügen" == input("Möchten Sie das Wort einfügen")
# Dies macht syntaktisch keinen Sinn. Wieder verpufft ein Ausdruck, weil er nirgends
# gespeichert wird. Wenn ich eine Variable mit einem Wert vergleichen will, dann halte
# ich es für natürlicher, es auch so rum aufzuschreiben. Die Konstante nach vorne zu
# ziehen, ist ein C-ism, der daraus resultiert, dass man sich mit "=" und "==" leicht
# vertippen kann und es nicht merkt, weil C beides akzeptiert. Bei Python kann man 
# diesen Fehler nicht machen bzw. es ist ein echter Fehler. Wie auch immer, ich bin mir
# sicher, hier fehlt ein "if"
#
#   if input("Möchten Sie das Wort einfügen?") == "ja":

        buchschreiben(key,'value')
        dict.append = {'key5':'Wert5'}
# Wirklich? Key5 und Wert5? Müssten wir nicht noch nach der Übersetzung fragen? Warum
# all die konstanten Strings? Übrigens, da "buchschreiben" (welches "buch_schreiben")
# heißen sollte) ja die Datei ersetzt und nicht ergänzt, wäre es total praktisch, erst
# das Wort zu "dict" aka vokabeln hinzuzufügen und dann erst zu speichern.
#
#       vokabeln[wort] = input("Wie lautet die Übersetung?")
#       buch_abspeichern()

print("Herzlich willkommen")
# Yeah, kein Fehler :)
"begin" == input("Möchten Sie anfangen")
# Das verpufft wieder wirkungslos.
if ("ja"):
# Das ist syntaktisch wieder Müll, wird aber wohl von Python geschluckt, da fast alles
# (außer False, None, "", 0 und leeren Listen) als wahr gilt und somit "ja" den
# Wahrheitswert "True" hat (was man mit bool("ja") prüfen kann). Hier fehlt die Variable,
# in der man sich die Frage von einer Zeile davor gemerkt hat.
#
    print(" ")
# kann man machen, muss man aber nicht...

    dict = buchslesen
# Hier wird dict die Funktion selbst zugewiesen, nicht das Ergebnis des Funktionsaufrufs.
# Da wir aber inzwischen buch_lesen() als Funktion, die eine globale Variable verändert
# und daher nichts zurückgeben muss definiert haben, kann das so aussehen:
#
#   buch_einlesen()
    worteingeben(buchlesen)
# wenn überhaupt, dann wäre hier "dict" richt.
#
#   wort_eingeben()
else:("nein")
# das ("nein") verpufft. Noch schlimmer, es verhindert, dass man hinter else: einen
# eingerückten Block von weiteren Anweisungen eingeben kann. Genau das will man aber
sys.exit()
# Wie oben erwähnt, "sys.exit" ist unnötig. 
ende(worteingabe)
# Insbesondere, wenn man danach noch versucht eine Funktion aufzurufen, die noch
# 2x das selbe versucht.
#
print("schade... dann beim nächsten Mal")
# und danach dann versucht, etwas auszugeben. Einzig diese Zeile kann stehen bleiben.

def ende():
    sys.exit("message")
    os._exit()
# Diese Funktion kann weg und "os" wäre eh ein Fehler, da dieses Modul nie importiert
# wurde. Der "_" macht übrigens klar, dass das eine interne FUnktion ist, von der man
# die Finger lassen soll. 

def buchschreiben (key,value):
# Hier will ich dies haben:
#
# def buch_abspeichern():
#
    Buch = open("datei2.txt", "w")
# Wir erinnern uns an "with"?
#
#    with open(vokabeln_dateiname) as buch:
#
    Buch.write(key + " " + value + "\n")
# Nun müssen wir alle Einträge in einer Schleife abspeichern:
#
#       for englisches_wort, deutsches_word in vokabeln.items():
#           buch.write(englisches_wort + " " + deutsches_word + "\n")
#
# Und das war's dann auch schon.
# Ach ja, diese Funktion muss noch über das Hauptprogramm geschoben werden, sonst findet
# Python sie nicht, bzw. das Hauptprogramm, also die Anweisungen, die nicht in einem "def"
# stehen sollte immer ganz unten in einer Datei stehen.

Code: Alles auswählen

"Immer wenn du den Programmablauf nicht verstehst oder Fehler suchst streue prints in den Code ein

Code: Alles auswählen

print("Herzlich willkommen")
# Meines Wissens nach wird "==" benutzt um herrauszufinden ob etwas True oder False ist. Sowie dass die Variable welche einen Wert erhalten soll nicht in "" gesetzt wird.
"begin" == input("Möchten Sie anfangen")
# if was ja?
if ("ja"):
    print(" ")
# ab hier wird dein das in der ersten Zeile angelegte dict überschrieben mit <function buchslesen at 0x26b50d8>
    dict = buchslesen
    worteingeben(buchlesen)
# Ein else sieht imho folgender Maßen aus:
# else:
#    ...
#    ...
else:("nein")
sys.exit()
ende(worteingabe)
print("schade... dann beim nächsten Mal")

Code: Alles auswählen

# Vokabelprogramm - Python 2.7.2.1
vokabeln = {}
vokabeln_dateiname = "c:\datei.txt"

def buch_abspeichern():
    buch = open(vokabeln_dateiname)
    with open(vokabeln_dateiname) as buch:
     buch.write(key + " " + value + "\n")
     for englisches_wort, deutsches_word in vokabeln.items():
        buch.write(englisches_wort + " " + deutsches_word + "\n")
    print("Wort gespeichert")
    wort_eingeben()

def buch_einlesen():
     buch = open(vokabeln_dateiname)
     with open(vokabeln_dateiname) as buch:
       for zeile in buch.readlines():
         englisches_wort, deutsches_word = zeile.strip().split()
         vokabeln[englisches_wort] = deutsches_word
     wort_eingeben()

def wort_eingeben():
     wort = input("Geben Sie ein Wort ein: ")
     if wort in vokabeln:
        print("Das deutsche Wort lautet", vokabeln[wort])
     else:
        print ("Das Wort existiert nicht in meinem Wörterbuch")
        if raw_input("Möchten Sie das Wort einfügen? (ja/nein)") == "ja":
          vokabeln[wort] = raw_input("Wie lautet die Übersetung?")
          buch_abspeichern()

print("Herzlich willkommen")
if raw_input("Möchten Sie anfangen? (ja/nein)") == "ja":
   buch_einlesen()
else:("nein")
print("schade... dann beim nächsten Mal")

Code: Alles auswählen

In [8]: a, b = "hello hallo".split()

In [9]: a, b = "hello hallo foo".split()
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
/home/nelson/src/Python/forum/<ipython-input-9-b6c7c598eafe> in <module>()
----> 1 a, b = "hello hallo foo".split()

ValueError: too many values to unpack (expected 2)

In [10]: a, b = " ".split()
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
/home/nelson/src/Python/forum/<ipython-input-10-b27890d4aff3> in <module>()
----> 1 a, b = " ".split()

ValueError: need more than 0 values to unpack

Code: Alles auswählen

VOCABULARIES = {
    "hello": "Hallo",
    "world": "Welt",
    ...
}

Code: Alles auswählen

In [14]: def foo():
   ....:     bar()
   ....:     

In [15]: def bar():
   ....:     foo()
   ....:     

Code: Alles auswählen

RuntimeError: maximum recursion depth exceeded

Code: Alles auswählen

vokabeln_dateiname = r"c:\datei.txt"

Code: Alles auswählen

    buch = open(vokabeln_dateiname)
    with open(vokabeln_dateiname) as buch:

Code: Alles auswählen

>>> 
*** Remote Interpreter Reinitialized  ***
>>> 
Herzlich willkommenHerzlich willkommen
 
Traceback (most recent call last):
  File "<module1>", line 41, in <module>
  File "<module1>", line 29, in buch_einlesen
  File "<string>", line 530, in Win32Input
  File "<string>", line 1, in <module>
NameError: name 'house' is not defined

Code: Alles auswählen

#
#!/usr/bin/env python

# Vokabelprogramm - Python 2.7.2.1
vokabeln = {}
vokabeln_dateiname = r"c:\datei.txt"

def buch_abspeichern():
    buch = open(vokabeln_dateiname)
    with open(vokabeln_dateiname) as buch:
     buch.write(key + " " + value + "\n")
     for englisches_wort, deutsches_wort in vokabeln.items():
        buch.write(englisches_wort + " " + deutsches_wort + "\n")

def buch_einlesen():
     buch = open(vokabeln_dateiname)
     with open(vokabeln_dateiname) as buch:
        for zeile in buch.readlines():
            englisches_wort, deutsches_wort = zeile.strip().split()
            vokabeln[englisches_wort] = deutsches_wort
     wort = input("Geben Sie ein Wort ein: ")
     if wort in vokabeln:
        print "Das deutsche Wort lautet", vokabeln[wort]
     else:
        print  "Das Wort existiert nicht in meinem Wörterbuch"
     if raw_input("Möchten Sie das Wort einfügen? (ja/nein)") == "ja":
        vokabeln[wort] = raw_input("Wie lautet die Übersetung?")
        buch_abspeichern()

print "Herzlich willkommen"
if raw_input ("Möchten Sie anfangen? (ja/nein)") == "ja":
   print " "
   buch_einlesen()
else: "nein"
print "schade... dann beim nächsten Mal"
#sys.exit()