Python: Wie viel Code packt man in ein try/except

Das kommt darauf an(tm).

Zum Bleistift ob "os.path.isfile()" auch noch 'ne Exception werfen kann 
(bei Symlink o.ä).

Wenn Code auf das Ergebnis der Funktion welche die Exeption auslösen 
kann angewiesen ist dann kannst du den auch gleich mit dort 
reinschreiben damit er nich mit ausgeführt wird.
Wenn deine Funktion bei einem Fehler allerdings sowieso abgebrochen 
wird, dann kannst du den Code auch ausserhalb stehen lassen und deine 
Funktion in der Exception beenden.
Bei deinem Beispiel ist die 1. Variante eher schlecht, wenn die Funktion 
noch andere als die abgefangene Exception auslösen kann.

Sascha

Georg schrieb:
> Klar, wenn man übersichtlich und lesbar programmiert, könnte ja ein
> anderer den Code verstehen.

Ich benutze return innerhalb von Funktionen am laufenden Band, gerade 
weil ich es lesbarer finde.

Irgendwelche Conditions prüfen und falls diese nicht erfüllt sind, 
sofort die Funktion mit einem 'return false' oder einem Fehlercode 
verlassen. Das ist doch viel lesbarer und sicherer als das über 
temporäre Variablen und if-else zu lösen. Und das nur, damit am Ende der 
Funktion ein einziges return steht?

Kaj G. schrieb:
>

1

>         if os.path.isfile(path + f):

2

>

Peng. {Hint: os.path.isfile(os.path.join(path, f))}

Ansonsten:

1

def find_file(path):

2

  try:

3

    return filter(os.path.isfile,

4

                  [os.path.join(path, f)

5

                   for f in os.listdir(path)])

6

  except FileNotFoundError, e:

7

    print str(e)

8

    return list()

oder, wenn Du keine Symlinks dabeihaben willst (os.path.isfile() gibt 
für Symlinks True zurück):

1

def find_file(path):

2

  try:

3

    return filter(lambda x: os.path.isfile(x)

4

                  and not os.path.islink(x),

5

                  [os.path.join(path, f)

6

                   for f in os.listdir(path)])

7

  except FileNotFoundError, e:

8

    print str(e)

9

    return list()

Sheeva P. schrieb:
> Peng. {Hint: os.path.isfile(os.path.join(path, f))}
Klaer mich auf, ich seh das Problem nicht.
Windows kann mit '/' im Pfad umgehen und da in path ein '/' am Ende 
steht, sehe ich nichts wo es knallen sollte.
Dazu macht es der zusaetzliche aufruf kein stueck lesbarer. :-/

Dazu ist der uebergebene Pfad an der Stelle immer relativ (Ja, das sieht 
man hier nicht).
Symlinks gibt es an der Stelle auch nicht (ja, ich weiss, das sieht man 
hier nicht). :)

Das mit filter und lambda ist ja auch ganz nett, macht es aber auch 
nicht lesbarer. :-/

Kaj G. schrieb:
> Das mit filter und lambda ist ja auch ganz nett, macht es aber auch
> nicht lesbarer. :-/

kann man doch aber was von lernen, wenn man nur in rare cases schlängelt 
;-)

Kaj G. schrieb:
> Sheeva P. schrieb:
>> Peng. {Hint: os.path.isfile(os.path.join(path, f))}
> Klaer mich auf, ich seh das Problem nicht.
> Windows kann mit '/' im Pfad umgehen und da in path ein '/' am Ende
> steht, sehe ich nichts wo es knallen sollte.
> Dazu macht es der zusaetzliche aufruf kein stueck lesbarer. :-/

Wenn aus welchem Grund auch immer mal kein '/', ':' oder '\' am Ende 
steht, funktioniert die Version mit os.path.join() trotzdem korrekt, 
unabhängig von Eingabe und Plattform. Es bleibt aber natürlich Dir 
überlassen, ob Du robusten, fehlertoleranten, wiederverwendbaren und 
pattformunabhängigen Code schreiben willst. ;-)

> Dazu ist der uebergebene Pfad an der Stelle immer relativ (Ja, das sieht
> man hier nicht).
> Symlinks gibt es an der Stelle auch nicht (ja, ich weiss, das sieht man
> hier nicht). :)

Das spielt auch gar keine Rolle.

> Das mit filter und lambda ist ja auch ganz nett, macht es aber auch
> nicht lesbarer. :-/

Ich finde die Variante mit filter() auf die List Comprehension lesbarer, 
aber das ist wohl Gewohnheitssache. YMMV.