Übungsprojekt meinAdressbuch

[ltownatze]

Mal ne kleine Zwischenfrage:
Ich konnte in der Python-Dokumentation nichts in der Richtung finden und langsam glaube ich das es da auch nichts zu finden gibt bevor ich also weiter suche bis ich alt und grau bin hier meine Frage:

Mal angenommen ich habe 2 Funktionen: 'main()' und 'open_addressbook()':

Code: Alles auswählen

def main():
    [...]
    open_addressbook()
    [...]

def open_addressbook():
    addressbook_filename = 'einschönerdateiname'

und ich möchte die Variable 'addressbook_filename', nach dem Aufruf von open_addressbook(), in main() weiterverwenden.
Wie könnte ich hier vorgehen? Ist sowas überhaupt möglich?

[cofi]

Ist sowas überhaupt möglich?

Nein, gluecklicherweise geht das nicht. Du suchst Rueckgabewerte.

[ms4py]

Ist sowas überhaupt möglich?

Nein, gluecklicherweise geht das nicht. Du suchst Rueckgabewerte.

Also mit Rückgabewerten lautet die Antwort eigentlich schon "Ja"

Code: Alles auswählen

def main():
    [...]
    addressbook_filename = open_addressbook()
    [...]

def open_addressbook():
    addressbook_filename = 'einschönerdateiname'
    [...]
    return addressbook_filename

[Hyperion]

Also mit Rückgabewerten lautet die Antwort eigentlich schon "Ja"

Bezogen auf "die Variable" im Ursprungspost aber dann eben doch "nein"

[ltownatze]

Mir fällt es noch etwas schwer mich präzise Auszudrücken.. aber ich wurde irgendwie Verstanden

Mit 'return' hatte ich auch schon rumgespielt, bin aber nicht auf diese Idee gekommen:

Code: Alles auswählen

addressbook_filename = open_addressbook()

Außerdem hat mich die Python-Doku hier etwas verwirrt.

[Hyperion]

Außerdem hat mich die Python-Doku hier etwas verwirrt.

Respekt für das Lesen der Doku, aber Du solltest erst einmal hier gucken Das sind wirklich absolute Baics.

[cofi]

Mit 'return' hatte ich auch schon rumgespielt, bin aber nicht auf diese Idee gekommen:

Code: Alles auswählen

addressbook_filename = open_addressbook()

Um das klarzustellen: `addressbook_filename` ist NICHT dieselbe Variable, wie in der Funktion, sondern enthaelt den Rueckgabewert von `open_addressbook`, d.h. du kannst sie a) aendern, ohne den Wert in der Funktion zu aendern und b) aendert sich der Wert in der Funktion, dann aendert sich nicht der Wert dieser Variablen (es sei denn, du weisst ihr wieder den Rueckgabewert der Funktion zu).

[Hyperion]

Was mir hier gerade noch auffällt: Wieso definiert man denn einen Dateinamen innerhalb einer Funktion? Irgend wie kein guter Stil...

Vielleicht noch mal ein simpleres Beispiel, um klar zu stellen, dass Namen jeweils in ihrem Scope eindeutig sind:

Code: Alles auswählen

In [1]: def foo():
   ...:     a = "test"
   ...:     return a
   ...:

In [2]: a = 3

In [3]: b = foo()

In [4]: print a, b
------> print(a, b)
(3, 'test')

Das 'a' auf Modulebene verweist auch nach dem Aufruf von foo() weiterhin auf 3. Dafür wird der Rückgabewert von foo() an den Namen 'b' gebunden.

[ltownatze]

Danke für die ausführlichen und anschaulichen Erklärungen, ich habs jetzt endlich begriffen.

Wieso definiert man denn einen Dateinamen innerhalb einer Funktion? Irgend wie kein guter Stil...

Eigentlich ging es mir nicht um den Dateinamen sondern um ein Dictionary das aus einer Datei ausgelesen wird und dem/der Namen/Variable/Referenz (k.A. was jetzt der korrekte Begriff ist) 'addressbook' zugeordnet wird. In der Praxis sieht das so aus:

Code: Alles auswählen

def main():
    [...]
    addressbook = open_addressbook(addressbook_filename)
    [...]
def open_addressbook(addressbook_filename):
    try:
        fp = open(addressbook_filename, 'r')
        addressbook = json.load(fp)
        fp.close()
    except IOError:
        addressbook = {}
    return addressbook

Ist es in diesem Fall guter Stil?

[Hyperion]

Datein würde ich immer mit with öffnen:

Code: Alles auswählen

with open(filename, "r") as infile:
    addressbook = json.load(infile)

Damit sparst Du Dir das close() und bist auch sicher, dass die datei auch bei einer Exception geschlossen wird (wozu Du sonst ein finally benötigen würdest!)

Sieht doch so ganz gut aus

Einzig die Rückgabe eines leeren Dicts bei einem Fehler könnte man überdenken. So bekommt man ja keine Rückmeldung, ob das geklappt hat oder nicht!

Ich würde dann eher die Exception beim Auruf der Funktion abfangen und ggf. dort eine Meldung ausgeben. Mit einem leeren Dict kannst Du dann ja immer noch weiterarbeiten.