Sprawdź, czy plik jest czytelny w Pythonie: spróbuj lub czy/else?

Question

Mam następujący kod:

import glob, os 
for file in glob.glob("\\*.txt"): 
    if os.access(file, os.R_OK): 
     # Do something 
    else: 
     if not os.access(file, os.R_OK): 
      print(file, "is not readable") 
     else: 
      print("Something went wrong with file/dir", file) 
     break 

Ale nie jestem do końca pewien, czy to dobry sposób, żeby to zrobić. Czy lepiej jest używać błędu try i catch? Jeśli tak, jak mogę spróbuj dla czytelności? Zwróć uwagę na break w mojej instrukcji else. Jak tylko plik nie może zostać odczytany, chcę przerwać pętlę.

Answer 1

Dla mnie, używając try-except w tym samym zakresie, w którym użyto if-else, nie zyskuje czytelności. Wartością wyjątków jest to, że mogą one zostać przechwycone na wyższym poziomie w drzewie wywołań.

Poruszanie się tylko jeden poziom, unikamy oświadczenie break:

import glob, os 
try: 
    for file in glob.glob("\\*.txt"): 
     with open(file) as fp: 
      # do something with file 
except IOError: 
    print("could not read", file) 

Ale prawdziwy geniusz wyjątków jest, gdy kod po prostu znika:

# Operate on several files 
# SUCCESS: Returns None 
# FAIL: Raises exception 
def do_some_files(): 
    for file in glob.glob("\\*.txt"): 
     with open(file) as fp: 
      # do something with file 

Teraz jest odpowiedzialność programu wywołującego wyświetlać przydatny komunikat o błędzie po awarii. Usunęliśmy odpowiedzialność za uporanie się z awarią całkowicie poza tym kodem i całą inną dziedziną.

W rzeczywistości można całkowicie przenieść odpowiedzialność z naszego programu i do tłumacza. W takim przypadku interpreter wyświetli kilka przydatnych komunikatów o błędach i zakończy nasz program. Jeśli domyślny komunikat Pythona jest wystarczająco dobry dla użytkowników, nie zaleca się sprawdzania błędów w ogóle. Tak więc, oryginalny scenariusz staje:

import glob, os 
for file in glob.glob("\\*.txt"): 
    # Do something 

Answer 2

try: 
     # check to see if file is readable 
     with open(filename) as tempFile: 





except Exception as e: 
     print e 
     # here you can modify the error message to your liking 

Zwykle to robię. Jest wytrzymała i prosta

Answer 3

w Pythonie kultury, jest to bardziej powszechne ask forgiveness, not permission, więc lepiej jest złapać wyjątek:

for filename in glob.glob('*.txt'): 
    try: 
     with open(filename) as fp: 
      # work with the file 

    except IOError as err: 
     print "Error reading the file {0}: {1}".format(filename, err) 
     break 

ten sposób można również uniknąć podwójnego sprawdzenia czy warunki wyścigu.

Answer 4

bardziej wyraźny sposób, aby sprawdzić, czy file jest w rzeczywistości plik, a nie katalog na przykład, że jest czytelny:

from os import access, R_OK 
from os.path import isfile 

file = "/some/path/to/file" 

assert isfile(file) and access(file, R_OK), \ 
     "File {} doesn't exist or isn't readable".format(file)