重構好還是不好？

考慮下面的代碼：重構好還是不好？

status = row[COL_STATUS] 

    if status == "idle": 
     row[COL_EDITABLE] = True 
     row[COL_FONTSTYLE] = pango.STYLE_NORMAL 
     row[COL_WEIGHT] = pango.WEIGHT_NORMAL 
    elif status == "DCed": 
     row[COL_EDITABLE] = True 
     row[COL_FONTSTYLE] = pango.STYLE_ITALIC 
     row[COL_WEIGHT] = pango.WEIGHT_NORMAL 
    else: 
     row[COL_EDITABLE] = False 
     row[COL_FONTSTYLE] = pango.STYLE_NORMAL 
     row[COL_WEIGHT] = pango.WEIGHT_BOLD

請問以下是淨有益的重構，在您看來？

d = {"idle": (True, pango.STYLE_NORMAL, pango.WEIGHT_NORMAL), 
     "DCed": (True, pango.STYLE_ITALIC, pango.WEIGHT_NORMAL), 
     None: (False, pango.STYLE_NORMAL, pango.WEIGHT_BOLD)} 
    e,f,w = d.get(status, d[None]) 
    row[COL_EDITABLE] = e 
    row[COL_FONTSTYLE] = f 
    row[COL_WEIGHT] = w

如果有更多的情況下，以上行組件編輯？

來源

2010-10-28 Claudiu

你在簡明性中所獲得的內容在可讀性方面會有所下降。在當前的例子中，我可以很容易地知道發生了什麼。在新代碼中，我必須考慮更加困難。

乘以接下來的千次編輯，你將會在你的手上產生一些嚴重的可維護性問題。

來源

2010-10-28 19:28:07

同意。第一個例子更容易閱讀。 – Geoff 2010-10-28 19:37:24

嗯我同意我認爲..會讓其他人發帖，但看着我的屏幕上的兩個並排我剛剛刪除了較短的版本 – Claudiu 2010-10-28 19:42:22

如何使用對象並做類似於「用子類替換類型代碼」？ http://www.refactoring.com/catalog/replaceTypeCodeWithSubclasses.html

來源

2010-10-28 19:31:30

重構好還是不好？

回答

相關問題