Często powtarzana próba/z wyjątkiem Python

Question

Po pierwsze, nie jestem pewien, czy moje podejście jest właściwe, więc jestem otwarty na różne sugestie.

Jeśli instrukcje try/except są często powtarzane w kodzie?

try: 
    # Do similar thing 
    os.remove('/my/file') 
except OSError, e: 
    # Same exception handing 
    pass 

try: 
    # Do similar thing 
    os.chmod('/other/file', 0700) 
except OSError, e: 
    #Same exception handling 
    pass 

Na przykład za działania jednej linii można zdefiniować obsługi opakowanie wyjątku, a następnie przekazać funkcję lambda:

def may_exist(func): 
    "Work with file which you are not sure if exists.""" 
    try: 
     func() 
    except OSError, e: 
     # Same exception handling 
     pass 

may_exist(lambda: os.remove('/my/file')) 
may_exist(lambda: os.chmod('/other/file', 0700)) 

Czy to „rozwiązanie” po prostu zrobić rzeczy mniej oczywiste? Czy powinienem w pełni napisać wszystkie instrukcje try/except?

Answer 1

Najlepszym sposobem abstrakcyjnego obsługi wyjątków jest z menedżerem kontekstu:

from contextlib import contextmanager 
@contextmanager 
def common_handling(): 
    try: 
     yield 
    finally: 
     # whatever your common handling is 

następnie:

with common_handling(): 
    os.remove('/my/file') 

with common_handling(): 
    os.chmod('/other/file', 0700) 

ma to, że zaletą tego, że możesz umieścić pełne instrukcje i więcej niż jeden z nich w każdym wspólnym bloku obsługi.

Należy pamiętać, że konieczność ciągłego korzystania z tej samej obsługi sprawia wrażenie wyjątków, które powodują nadmierną obsługę. Czy na pewno potrzebujesz tak wiele?

Answer 2

Myślę, że twoje ogólne rozwiązanie jest w porządku, ale nie użyłbym tych lambd na dole. Ja polecam przechodzenia z funkcji i argumenty jak ten

def may_exist(func, *args): 
    "Work with file which you are not sure if exists.""" 
    try: 
     func(args) 
    except OSError, e: 
     # Same exception handling 
     pass 

may_exist(os.remove, '/my/file') 
may_exist(os.chmod, '/other/file', '0700') 

Answer 3

Prawdopodobnie byłoby czystsze aby may_exist dekorator:

from functools import wraps 
def may_exist(func): 
    @wraps(func): 
    def wrapper(*args, **kwds): 
     try: 
      return func(*args, **kwds) 
     except OSError: 
      pass 
    return wrapper 

Następnie można albo zrobić:

may_exist(os.remove)('/my/file') 
may_exist(os.chmod)('/other/file', 0700) 

na wezwanie jednorazowej lub:

remove_if_exists = may_exist(os.remove) 
... 
remove_if_exists('somefile') 

jeśli używasz go dużo.

Answer 4

Byłoby coś w tej pracy:

def may_exist(func, *func_args): 
    try: 
     func(*func_args) 
    except OSError as e: 
     pass