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