2015-06-22 48 views
-4

要重構下面的代碼,它是用Python編寫的:要重構下面的代碼,它是用Python編寫的:

def __init__(self): 
    self.var1= Module.ClassA() 
    self.var2= Module.ClassB() 
    self.var3= Module.ClassC() 
    self.var4= Module.ClassD() 
    self.var5= Module.ClassE() 
    self.var6= Module.ClassF() 
    self.var7= Module.ClassG() 
    self.var8= Module.ClassH() 
    self.var9= Module.ClassI() 
    self.var10= Module.ClassJ() 
    self.var11= Module.ClassK() 
    self.var12= Module.ClassL() 
    self.var13= Module.ClassM() 
    self.var14= Module.ClassN() 
    self.var15= Module.ClassO() 
    self.var16= Module.ClassP() 
    self.var17= Module.ClassQ() 
    self.var18= Module.ClassR() 
    self.var19= Module.ClassS() 
    self.var20= Module.ClassT() 
+0

極難讀取任何此代碼 –

+1

而你的問題是?請說明該代碼出了什麼問題,以及您想要達到的目標。 – Bakuriu

+0

實際代碼'Module','var's 1到20,'Class'es A到T中的實際名稱? – user2357112

回答

4

開始重構你有沒有給我們的代碼: ClassF是一個無用的名稱一個類將所有類重命名爲具有有意義的名稱。 var18是屬性的無用名稱,所以重構所有屬性以獲得有意義的名稱。

類需要引用其他不同類的實例的20個屬性:重構依賴關係,以便該類只引用實際直接使用的對象。

+0

在這裏,我給了例子,我不能顯示我的實際代碼。但我的要求只有這樣 –