最實用的10個重構小技巧排行榜,您都用過哪些呢?
LZ最近一直在研究虛擬機器原始碼,可惜目前還只是稍微有一點點頭緒,無法與各位分享,龐大的JAVA虛擬機器原始碼果然不是一朝一夕能搞定的,LZ汗顏。
本次我們拋開JAVA虛擬機器原始碼這些相對底層的東西,LZ來與各位探討一下幾個程式碼重構的小技巧,這些內容部分來自於書籍當中,部分來自於LZ維護專案當中的一些實踐經驗。如果猿友們曾經用過這種手法,也不妨參與到文章的留言當中,將你的小心得、小體會共享與他人,也可以拿來衝擊LZ自己定義的排行榜,LZ不甚歡迎。
重構的手法有很多種,相對而言,一篇文章的涵蓋量自然是無法提到所有,LZ這裡也只能提出一些平時會經常使用的一些手法,像一些比較高階的手法,各位有興趣的可以去找一些專門的書籍涉獵。
另外還有一點,由於LZ是做JAVA開發的,因此部分重構小技巧可能與JAVA語言,或者說與面向物件的語言息息相關,不過大多數技巧,無論是面向過程的語言,還是面向物件的語言,都是可以相互通用的。
廢話不多說,我們來看看實用重構技巧的排行榜吧。
No.1:重複程式碼的提煉
重複程式碼是重構收效最大的手法之一,進行這項重構的原因不需要多說。它有很多很明顯的好處,比如總程式碼量大大減少,維護方便,程式碼條理更加清晰易讀。
它的重點就在於尋找程式碼當中完成某項子功能的重複程式碼,找到以後請毫不猶豫將它移動到合適的方法當中,並存放在合適的類當中。
小例項
class BadExample { public void someMethod1(){ //code System.out.println("重複程式碼");/* 重複程式碼塊 */ //code } public void someMethod2(){ //code System.out.println("重複程式碼");/* 重複程式碼塊 */ //code } } /* ---------------------分割線---------------------- */ class GoodExample { public void someMethod1(){ //code someMethod3(); //code } public void someMethod2(){ //code someMethod3(); //code } public void someMethod3(){ System.out.println("重複程式碼");/* 重複程式碼塊 */ } }
No.2:冗長方法的分割
有關冗長方法的分割,其實有時候與重複程式碼的提煉是有著不可分割的關係的,往往在我們提煉重複程式碼的過程中,就不知不覺的完成了對某一個超長方法的分割。倘若在你提煉了大部分的重複程式碼之後,某一些冗長方法依然留存,此時就要靜下心來專門處理這些冗長方法了。
這其中有一點是值得注意的,由於我們在分割一個大方法時,大部分都是針對其中的一些子功能分割,因此我們需要給每一個子功能起一個恰到好處的方法名,這很重要。可以說,能否給方法起一個好名字,有時候能體現出一個程式猿的大致水準。
小例項
class BadExample {
public void someMethod(){
//function[1]
//function[2]
//function[3]
}
}
/* ---------------------分割線---------------------- */
class GoodExample {
public void someMethod(){
function1();
function2();
function3();
}
private void function1(){
//function[1]
}
private void function2(){
//function[2]
}
private void function3(){
//function[3]
}
}
No.3:巢狀條件分支的優化(1)
大量的巢狀條件分支是很容易讓人望而卻步的程式碼,我們應該極力避免這種程式碼的出現。儘管結構化原則一直在說一個函式只能有一個出口,但是在這麼大量的巢狀條件分支下,讓我們忘了這所謂的規則吧。
有一個專業名詞叫衛語句,可以治療這種恐怖的巢狀條件語句。它的核心思想是,將不滿足某些條件的情況放在方法前面,並及時跳出方法,以免對後面的判斷造成影響。經過這項手術的程式碼看起來會非常的清晰,下面LZ就給各位舉一個經典的例子,各位可以自行評判一下這兩種方式,哪個讓你看起來更清晰一點。
小例項
class BadExample {
public void someMethod(Object A,Object B){
if (A != null) {
if (B != null) {
//code[1]
}else {
//code[3]
}
}else {
//code[2]
}
}
}
/* ---------------------分割線---------------------- */
class GoodExample {
public void someMethod(Object A,Object B){
if (A == null) {
//code[2]
return;
}
if (B == null) {
//code[3]
return;
}
//code[1]
}
}
No.4:巢狀條件分支的優化(2)
此處所說的巢狀條件分支與上面的有些許不同,它無法使用衛語句進行優化,而應該是將條件分支合併,以此來達到程式碼清晰的目的。由這兩條也可以看出,巢狀條件分支在編碼當中應當儘量避免,它會大大降低程式碼的可讀性。
下面請尚且不明覺厲的猿友看下面這個典型的小例子。
小例項
class BadExample {
public void someMethod(Object A,Object B){
if (A != null) {
if (B != null) {
//code
}
}
}
}
/* ---------------------分割線---------------------- */
class GoodExample {
public void someMethod(Object A,Object B){
if (A != null && B != null) {
//code
}
}
}
No.5:去掉一次性的臨時變數
生活當中我們都經常用一次性筷子,這無疑是對樹木的摧殘。然而在程式當中,一次性的臨時變數不僅是對效能上小小的摧殘,更是對程式碼可讀性的褻瀆。因此我們有必要對一些一次性的臨時變數進行手術。
小例項
class BadExample {
private int i;
public int someMethod(){
int temp = getVariable();
return temp * 100;
}
public int getVariable(){
return i;
}
}
/* ---------------------分割線---------------------- */
class GoodExample {
private int i;
public int someMethod(){
return getVariable() * 100;
}
public int getVariable(){
return i;
}
}
No.6:消除過長引數列表
對於一些傳遞了大批引數的方法,對於追求程式碼整潔的程式猿來說,是無法接受的。我們可以嘗試將這些引數封裝成一個物件傳遞給方法,從而去除過長的引數列表。大部分情況下,當你嘗試尋找這樣一個物件的時候,它往往已經存在了,因此絕大多數情況下,我們並不需要做多餘的工作。
小例項
class BadExample {
public void someMethod(int i,int j,int k,int l,int m,int n){
//code
}
}
/* ---------------------分割線---------------------- */
class GoodExample {
public void someMethod(Data data){
//code
}
}
class Data{
private int i;
private int j;
private int k;
private int l;
private int m;
private int n;
//getter&&setter
}
No.7:提取類或繼承體系中的常量
這項重構的目的是為了消除一些魔數或者是字串常量等等,魔數所帶來的弊端自不用說,它會讓人對程式的意圖產生迷惑。而對於字串等型別的常量的消除,更多的好處在於維護時的方便。因為我們只需要修改一個常量,就可以完成對程式中所有使用該常量的程式碼的修改。
順便提一句,與此類情況類似並且最常見的,就是Action基類中,對於INPUT、LIST、SUCCESS等這些常量的提取。
小例項
class BadExample {
public void someMethod1(){
send("您的操作已成功!");
}
public void someMethod2(){
send("您的操作已成功!");
}
public void someMethod3(){
send("您的操作已成功!");
}
private void send(String message){
//code
}
}
/* ---------------------分割線---------------------- */
class GoodExample {
protected static final String SUCCESS_MESSAGE = "您的操作已成功!";
public void someMethod1(){
send(SUCCESS_MESSAGE);
}
public void someMethod2(){
send(SUCCESS_MESSAGE);
}
public void someMethod3(){
send(SUCCESS_MESSAGE);
}
private void send(String message){
//code
}
}
No.8:讓類提供應該提供的方法
很多時候,我們經常會操作一個類的大部分屬性,從而得到一個最終我們想要的結果。這種時候,我們應該讓這個類做它該做的事情,而不應該讓我們替它做。而且大部分時候,這個過程最終會成為重複程式碼的根源。
小例項
class BadExample {
public int someMethod(Data data){
int i = data.getI();
int j = data.getJ();
int k = data.getK();
return i * j * k;
}
public static class Data{
private int i;
private int j;
private int k;
public Data(int i, int j, int k) {
super();
this.i = i;
this.j = j;
this.k = k;
}
public int getI() {
return i;
}
public int getJ() {
return j;
}
public int getK() {
return k;
}
}
}
/* ---------------------分割線---------------------- */
class GoodExample {
public int someMethod(Data data){
return data.getResult();
}
public static class Data{
private int i;
private int j;
private int k;
public Data(int i, int j, int k) {
super();
this.i = i;
this.j = j;
this.k = k;
}
public int getI() {
return i;
}
public int getJ() {
return j;
}
public int getK() {
return k;
}
public int getResult(){
return i * j * k;
}
}
}
No.9:拆分冗長的類
這項技巧其實也是屬於非常實用的一個技巧,只不過由於它的難度相對較高,因此被LZ排在了後面。針對這個技巧,LZ很難像上面的技巧一樣,給出一個即簡單又很容易說明問題的小例子,因為它已經不僅僅是小手段了。
大部分時候,我們拆分一個類的關注點應該主要集中在類的屬性上面。拆分出來的兩批屬性應該在邏輯上是可以分離的,並且在程式碼當中,這兩批屬性的使用也都分別集中於某一些方法當中。如果實在有一些屬性同時存在於拆分後的兩批方法內部,那麼可以通過引數傳遞的方式解決這種依賴。
類的拆分是一個相對較大的工程,畢竟一個大類往往在程式中已經被很多類所使用著,因此這項重構的難度相當之大,一定要謹慎,並做好足夠的測試。
No.10:提取繼承體系中重複的屬性與方法到父類
這項技巧大部分時候需要足夠的判斷力,很多時候,這其實是在向模板方法模式邁進的過程。它的例項LZ這裡無法給出,原因是因為它的小例項會毫無意義,無非就是子類有一樣的屬性或者方法,然後刪除子類的重複屬性或方法放到父類當中。
往往這一類重構都不會是小工程,因此這一項重構與第九種類似,都需要足夠的謹慎與測試。而且需要在你足夠確認,這些提取到父類中的屬性或方法,應該是子類的共性的時候,才可以使用這項技巧。
結束語
由於LZ目前的工作就是維護一個相對古老的專案,因此上面這十種手法,LZ幾乎都已經一一嘗試過了,可喜的是效果都還不錯。
限於最後兩種與實際情況的聯絡太過緊密,因此LZ無法給出簡單的例項,不過後面兩種畢竟不是常用的重構手法,因此也算是可以接受了。不過不常用不代表不重要,各位猿友還是要知道這一點的。另外LZ還要說的是,上面的例項只是手法的一種簡單展示,實際應用當中,程式碼的結構可能是千奇百怪,但卻萬變不離其宗。因此只要抓住每種手法的核心,就不難從這些亂軍叢中安然穿過。
好了,本次的小分享到此結束,希望各位猿友如果覺得有所收穫,可以推薦一下鼓勵下LZ,順便也讓更多的人看到。這樣的話,或許我們每一個接手的專案程式碼,都不至於十分的糟糕了,也算是給像LZ這樣的專案維護者一條生路吧。
版權宣告
作者:zuoxiaolong(左瀟龍)
出處:部落格園左瀟龍的技術部落格--http://www.cnblogs.com/zuoxiaolong
您的支援是對博主最大的鼓勵,感謝您的認真閱讀。
本文版權歸作者所有,歡迎轉載,但未經作者同意必須保留此段宣告,且在文章頁面明顯位置給出原文連線,否則保留追究法律責任的權利。