pycharm提示This inspection detects instance attribute definition outside __init__ method

示例代碼：

class MiNiCarStore(CarStore):

    def createCar(self, typeName):
        self.carFactory = CarFactory()  # 會出現下劃線提示This inspection detects instance attribute definition outside __init__ method
        return self.carFactory.createCar(typeName)

緣由是：根據 SRP(Single Pesponsibility Principle, SRP) 原則，這個類原本就會承擔某個界面邏輯，那麼它就不該該再承擔「初始化」的職責，初始化的工做應該在另外一個單獨的類中完成，這樣能讓代碼更可測（也就是更好寫單元測試）

可改寫以下：

class MiNiCarStore(CarStore):
	
	def __init__(self):
        self.carFactory = None
        
    def createCar(self, typeName):
        self.carFactory = CarFactory()
        return self.carFactory.createCar(typeName)

也能夠在settings -> editor -> inspections -> python 取消勾選提示

擴展閱讀

單一職責原則(Single Pesponsibility Principle, SRP)
單一職責有兩個含義： 一個是避免相同的職責分散到不一樣的類中， 別一個是避免一個類承擔太多職責
爲何要遵照SRP呢?

• 能夠減小類之間的耦合
  若是減小類之間的耦合，當需求變化時，只修改一個類，從而也就隔離了變化；若是一個類有多個不一樣職責，它們耦合在一塊兒，當一個職責發生變化時，可能會影響到其餘職責。
• 提升類的複用性 修改電腦比修理電視機簡單多了。主要緣由就在於電視機各個部件之間的耦合性過高，而電腦則不一樣，電腦的內存、硬盤、聲卡、網卡、鍵盤燈等部件均可以很容易地單獨拆卸和組裝。某個部件壞了，換上新的便可。上面的例子就體現了單一職責的優點。因爲使用了單一職責，使得‘組件’能夠方便地‘拆卸’和‘組裝’。 不遵照SRP會影響對類的複用性。當只須要用該類的某一個職責時，因爲它和其餘的職責耦合在一塊兒，也就很難分離出。 遵照SRP在實際代碼開發中有沒有什麼應用？有的。以數據持久層爲例，所謂的數據持久層主要指的是數據庫操做，固然，還包括緩存管理等。這時就須要數據持久層支持多種數據庫。應該怎麼作？定義多個數據庫操做類？想法已經很接近了，再進一步，就是使用工廠模式。 工廠模式(Faction)容許你在代碼執行時實例化對象。它之因此被稱爲工廠模式是由於它負責‘生產對象’。以數據庫爲例，工廠須要的就是根據不一樣的參數，生成不一樣的實例化對象。最簡單的工廠就是根據傳入的類型名實例化對象，如傳入MySQL，就調用MySQL類並實例化，若是是SQLite，則調用 SQLite的類並實例化，甚至還能夠處理TXT、Execl等‘類數據庫’。