이것은 데코레이터에 적절한 용도입니까?</p> <pre><code>class MyClass(object): def __init__(self, name): self.name = name self.df = pd.read_table(name) </code></pre> <p>이 같은 방법의 무리 :

Question

은 내가 이런 클래스가

가 결국

def valid_cond1(self): 
    # check if cond1 is satisfied with respect to self.df and return 
    # a DataFrame of rows not satisfying cond1 

def valid_cond2(self): 
    # same deal 

등, 나는이 방법의 각각의 출력을 먹고 싶어, 그리고 형식이 잘못된 행 목록을 클라이언트에 반환하십시오. 내가 지금처럼 구현을 조정하여 valid_cond1이 작업을 수행 할 수 있습니다

def valid_cond1(self): 
    # err_df = rows of self.df not meeting cond1 
    bad_lines = [] 
    for ix, val in err_df.iterrows(): 
     bad_lines.append("Error in line %s: %s. Cond1 not met.." % (ix,val)) 
    return bad_lines 

하지만 이러한 각 기능에 대해 동일한 로직을 작성하지 않으려는 (아마도 그들 중 아주 많습니다).

데코레이터를 사용할 수있는 곳입니까? 아니면 원하는 행동을 달성하기위한 또 다른 방법이 있습니까?

Answer 1

단순히 무난 무엇인가에 조건을 만들 수없는 경우 피하기 위해 이런 일을 할 수

def check_condition(self, condition): 
    # check the condition 
    return bad_lines # etc. 

def valid_cond1(self): 
    # define condition_1 
    return self.check_condition(condition_1) 

def valid_cond2(self): 
    return self.check_condition(condition_2) 

이 같은 뭔가를 장식으로 이동하지만, 아마도 더 일반적인 방법 않을 것 오류 인쇄 코드를 적어도 반복하십시오 :

@staticmethod 
def invalid_condition(err_df): 
    # bad lines stuff here 

def valid_cond1(self): 
    # calculate err_df 
    if err_df: 
     return self.invalid_condition(err_df) 

EDIT : 그냥 재미를 위해 꾸미기 버전. 나는 장식자를 사용하는 것으로 알려져있어 욕망을 이해할 수있다 :

from functools import wraps 

def print_error_info(func): 
    @wraps(func) 
    def wrapped(*args, **kwargs): 
     err_df = func(*args, **kwargs) 
     bad_lines = [] 
     for ix, val in err_df.iterrows(): 
      bad_lines.append("Error in line %s: %s. Cond1 not met.." % (ix,val)) 
     return bad_lines 
    return wrapped 

# use 
class MyClass: 
    # *snip* 

    @print_error_info 
    def valid_cond1(self): 
     # whatever you need 
     return err_df