重構是在編寫代碼後在不更改代碼的外部行為的前提下通過更改代碼的內部結構來改進代碼的過程。
一、何時需要重構
1、代碼中存在重復的代碼;
如果類中有重復的代碼塊,需將其提煉出一個新的獨立方法,如果是不同類中具有相同的代碼,將其提煉成一個新類。
2、過大的類和過長的方法;
過大的類往往是類抽象不合理的結果,類抽象不合理將降低代碼的復用率。當看到一個過長的方法時,需要想辦法將其劃分為多個小方法。
方法中的代碼行數越多,則方法越難理解。我們推薦每個方法中只包含 20-25 行代碼。但有些人說 1-10 行更合理,這只是些個人喜好,沒有硬性的規則。抽取方法是最常見的重構方式之一。如果你發現一個方法過長,或者已經需要一個注釋來描述它的目的了,那麼你就可以應用抽取方法了。人們總是會問一個方法到底多長合適,但其實長度並不是問題的根源。當你在處理復雜的方法時,跟蹤所有局部變量是最復雜和消耗時間的,而通過抽取一個方法可以節省一些時間。可以使用 Visual Studio 來抽取方法,它會幫助你跟蹤局部變量,並將其傳遞給新的方法或者接收方法的返回值。
3、牽一發而動全身的修改;
修改一個小功能,或增加一個小功能時,就引發一次代碼地震,也許是你的設計抽象度不夠理想,功能代碼太過分散所引起的。
4、類之間需要過多的通訊;
A 類需要調用B 類的過多方法訪問B 的內部數據,是不是這兩個類根本就不應該分開。
5、過度耦合的信息鏈;
代碼中常常會使用中間層來達到松耦合的目的,但過多的中間層,層層相連,是否需要考慮減少中間層。
6、功能相似的類或方法;
7、不完美的設計;
8、缺少必要的注釋;
9、始終控制類的大小
超大的類在嘗試做太多的事情,這違反了單一職責原則(SRP),也就是面向對象設計原則 SOLID 中的 S。
為什麼一定要將兩個職責分離到單獨的類中呢?因為每一個職責都是變化的中心。在需求變更時,這個變更將會出現在負責該職責的類中。如果一個類承擔了多個職責,就會有一個以上的原因導致其變化。如果一個類有多重職責,則說明這些職責已經耦合到了一起。並且某個職責的變化將有可能削弱或限制這個類滿足其他職責的能力。這種耦合將會導致非常脆弱的設計,進而在職責發生變化時,設計可能被意想不到的破壞了。
下面兩條可以歸為重寫范疇,代碼規范問題。
10、避免過多的參數
通過聲明一個類來代替多個參數。創建一個類,用於包含所有的參數。通常來講,這是一個較好的設計,並且這個抽象非常的有價值。
11、避免復雜的表達式
復雜的表達式意味著其背後隱藏了一些涵義,我們可以通過使用屬性來封裝這些表達式,進而使代碼更易讀些。
if(product.Price>500 && !product.IsDeleted && !product.IsFeatured && product.IsExported) { // do something }
二、C# VS2010中的幾點重構
1、提取方法重構
2、重命名重構
3、封裝字段重構
4、提取接口重構
5、移除參數重構
6、重新排列參數重構