Python類__init__佈局？

Question

在蟒蛇，它是壞的形式寫一個__init__定義，如：在初始化的變量是對函數的調用返回一個字符串

class someFileType(object): 
    def __init__(self, path): 
     self.path = path 
     self.filename = self.getFilename() 
     self.client = self.getClient() 
     self.date = self.getDate() 
     self.title = self.getTitle() 
     self.filetype = self.getFiletype() 
    def getFilename(self): 
     '''Returns entire file name without extension''' 
     filename = os.path.basename(self.path) 
     filename = os.path.splitext(filename) 
     filename = filename[0] 
     return filename 
    def getClient(self): 
     '''Returns client name associated with file''' 
     client = self.filename.split() 
     client = client[1] # Assuming filename is formatted "date client - docTitle" 
     return client 

？或者它被認爲是懶惰的編碼？無論何時我想引用文件的某些方面，主要是爲了將something.filetype當作something.getFiletype()來救我。

此代碼是按客戶端將文件分類到文件夾中，然後通過文檔類型和基於文件名中數據的其他操作。

Answer 1

不，我不明白爲什麼這將是不好的形式。事實上，創建實例時只計算一次這些值可能是一個好主意。

你也可以直到需要通過使用緩存property小號推遲算了一筆賬：

class SomeFileType(object): 
    _filename = None 
    _client = None 

    def __init__(self, path): 
     self.path = path 

    @property 
    def filename(self): 
     if self._filename is None: 
      filename = os.path.basename(self.path) 
      self._filename = os.path.splitext(filename)[0] 
     return self._filename 

    @property 
    def client(self): 
     '''Returns client name associated with file''' 
     if self._client is None: 
      client = self.filename.split() 
      self._client = client[1] # Assuming filename is formatted "date client - docTitle" 
     return self._client 

現在，訪問somefiletypeinstance.client將引發self.filename按需計算，以及高速緩存它自己的計算結果。

在這種特定情況下，您可能還想要.path屬性;一個與清除緩存值二傳手：

class SomeFileType(object): 
    _filename = None 
    _client = None 

    def __init__(self, path): 
     self._path = path 

    @property 
    def path(self): 
     return self._path 

    @path.setter 
    def path(self, value): 
     # clear all private instance attributes 
     for key in [k for k in vars(self) if k[0] == '_']: 
      delattr(self, key) 
     self._path = value 

    @property 
    def filename(self): 
     if self._filename is None: 
      filename = os.path.basename(self.path) 
      self._filename = os.path.splitext(filename)[0] 
     return self._filename 

    @property 
    def client(self): 
     '''Returns client name associated with file''' 
     if self._client is None: 
      client = self.filename.split() 
      self._client = client[1] # Assuming filename is formatted "date client - docTitle" 
     return self._client 

因爲property基於緩存確實增加了一些複雜性的開銷，你需要考慮，如果真的是值得的;對於你的具體，簡單的例子，它可能不是。您的屬性的計算成本的確非常低，除非您計劃創建大量這些類，否則與必須維護按需緩存屬性的精神成本相比，提前計算屬性的開銷可以忽略不計。

Answer 2

你的代碼是做兩個不同的東西：

a）簡化通過暴露某些計算的屬性變量，而不是函數的類的API。

b）預計算它們的值。

第一個任務是屬性是什麼;一個簡單的使用將使你的代碼更簡單，而不是更復雜，（同樣重要）將使意圖更加清晰：

class someFileType(object): 
    @property 
    def filename(self): 
     return os.path.basename(self.path) 

然後，您可以編寫var.filename，你會動態計算從路徑的文件名。

@ Martijn的解決方案增加了緩存，同時也負責b部分（預計算）。在你的例子中，至少，計算結果很便宜，所以我沒有看到這樣做的好處。

相反，緩存或預計算引起一致性問題。考慮以下片段：

something = someFileType("/home/me/document.txt") 
print something.filename # prints `document` 
... 
something.path = "/home/me/document-v2.txt" 
print something.filename # STILL prints `document` if you cache values 

最後一條語句應該打印什麼？如果你緩存你的計算，你仍然會得到document而不是document-v2！除非你是某些沒有人會嘗試更改基本變量的值，否則需要避免緩存或採取措施確保一致性。最簡單的方法是禁止修改path - 那性質designed to do.

結論的事情之一：使用屬性來簡化你的界面。不要緩存計算，除非由於性能原因而需要這些計算。如果你緩存，採取措施確保一致性，例如通過使底層值爲只讀。

PS。這些問題類似於數據庫規範化（非規範化設計引發一致性問題），但是在Python中，您有更多的資源來保持同步。