Как убедиться, что функция класса не будет вызвана до тех пор, пока не будет вызвана другая функция класса?

Question

У меня есть объект класса, который создает некоторые поля данных:

class DataFields(object): 

    _fields_ = ['field_1', 'field_2', 'data_length'] 

    def __init__(self, data=None): 
     if data != None: 
      self.create_fields(data) 

    def create_fields(self, data): 
     i = 0 
     for field in self._fields_: 
      setattr(self, field, data[i]) 
      i += 1 

    def get_datalength(self): 
     return self.data_length 

Что является лучшим способом, чтобы убедиться, что функция get_datalength() нельзя назвать, если data_length поле не было создано (то есть, если не функция create_fields() был вызван один раз).

Я думал об использовании либо переменную, которая инициализируется в create_fields и проверяется в get_datalength() или try-except внутри функции get_datalength(). Каков самый питонический (или лучший) способ?

Answer 1

Я думаю, что самый вещий способ будет сгенерировано исключение:

def get_datalength(self): 
    try: 
     return self.data_length 
    except AttributeError: 
     raise AttributeError("No length call create_fields first") 

Простая причина: Нельзя запретить пользователю вызывать эту функцию на объекте. Либо пользователь получит AttributeError и не поймет, что происходит, либо вы предоставили собственный класс ошибок или, по крайней мере, сообщение об ошибке.

КСТАТИ: Это не вещий создание методов получения (нет таких вещей, как «частных членов») Если вам нужно сделать меньше работы на стоимость возвращения его взглянуть на @property декоратора

@property 
def datalength(self): 
    return do_some_stuff(self.data_length) 

Answer 2

Используя getattr со значением по умолчанию, вы можете вернуть None или какое-либо значение, если нет data_length атрибут еще в случае:

def get_datalength(self): 
    return getattr(self, 'data_length', None) 

Answer 3

используя исключение, вероятно, лучшим способом для того, что вы делаете, однако есть альтернативы, которые могут быть полезны, если вы будете использовать этот объект из интерактивной консоли:

def fn2(self): 
    print("this is fn2") 

class test: 
    def fn1(self): 
     print("this is fn1") 
     self.fn2 = fn2 
    def fn2(self): # omit this if you want fn2 to only exist after fn1 is called 
     print("Please call fn1 first") 

Я бы не рекомендовал это для повседневного использования, но в некоторых случаях это может быть полезно. Если вы опускаете определение fn2 внутри класса, то метод fn2 будет присутствовать только после вызова fn1. Для облегчения технического обслуживания коды вы можете сделать то же самое, как это:

class test: 
    def fn1(self): 
     print("this is fn1") 
     self.fn2 = self._fn2 
    def _fn2(self): 
     print("this is fn2") 
    def fn2(self): # omit this if you want fn2 to only exist after fn1 is called 
     print("Please call fn1 first") 

Если это будет использоваться внутри модуля, который будет импортироваться, то вы должны либо вызвать исключение или возвращаете действительное значение, как и другие ответы предложил.

Answer 4

Это может быть разрешено путем использования словаря в качестве переменной класса с именами методов в виде ключей.

called['method1'] 
called['method2'] 
called['method3'] 
... 

И установив ключ в этом вызове метода

class SomeClass(obj): 
    def method1(): 
     called['method1'] = 1 
    def method2(): 
     if method1 in called: 
      # continue