到處都是重復(fù)的代碼憔杨？方法體又臭又長看不懂眼刃？快來試試抽取方法（Extract Method）吧破托，保證藥到病除颊乘！
何為抽取方法
將重復(fù)的代碼或語義獨立的代碼抽取成一個方法
看好了：
Tom抓Jerry（又臭又長版）
class Tom {
/**
* Tom當前的位置
**/
private int curPos;
public void catchJerry() {
//只要沒抓到Jerry就一直抓
while(true) {
//找Jerry
//左看看
int posOfJerry = lookLeft();
//右看看
if(posOfJerry == -1)
posOfJerry = lookRight();
//上看看吮炕，下看看
//.....省略
//前往Jerry的位置
while(curPos <= posOfJerry) {
//邁前腿
curPos++;
//邁后腿 保持平衡
//....
}
//抬起爪子
upHands();
//揮爪
Object objInHand = waveHands();
//Jerry是否在手中
if(Jerry.equals(objInHand))
break;
}
}
}
又臭又長腊脱，兩個While的嵌套循環(huán)，讓人不明所以龙亲，連走路邁腿都寫在一塊陕凹，如果沒有注釋的話悍抑，絕對讓人無法理解
接下來見證重構(gòu)的奇跡
-
簡化邏輯：抓Jerry = 抓不到Jerry就一直抓
public void catchJerry() { //只要沒抓到Jerry就一直抓 while(!tryCatchJerry()); }catchJerry()只有一行了，此處應(yīng)有掌聲
但是我們又創(chuàng)建了又臭又長的方法——試圖抓住Jerry
private boolean tryCatchJerry() { //找Jerry //左看看 int posOfJerry = lookLeft(); //右看看 if(posOfJerry == -1) posOfJerry = lookRight(); //上看看杜耙，下看看 //.....省略 //前往Jerry的位置 while(curPos <= posOfJerry) { //邁前腿 curPos++; //邁后腿 保持平衡 //.... } //抬起爪子 upHands(); //揮爪 Object objInHand = waveHands(); //Jerry是否在手中 return Jerry.equals(objInHand)); } -
分而治之：試圖抓住Jerry = 找Jerry + 前往Jerry位置 + 揮爪抓住Jerry
private boolean tryCatchJerry() { //找Jerry并前往Jerry的位置 go(findPosOfJerry()); //揮爪抓Jerry return waveHandsCatchSomething(Jerry); }好的搜骡，tryCatchJerry簡單又清晰了
我對剩下的方法進行了微調(diào)，讓它們更有邏輯感
private int findPosOfJerry() { //左看看 int posOfJerry = lookLeft(); if(posOfJerry != -1) return posOfJerry; //右看看 posOfJerry = lookRight(); if(posOfJerry != -1) return posOfJerry; //上看看佑女，下看看 //.....省略 } private void go(int pos) { while(curPos <= pos) { //邁前腿 curPos++; //邁后腿 保持平衡 //.... } } private boolean waveHandsCatchSomething(Object something) { //抬起爪子 upHands(); //揮抓记靡，然后看看指定東西是否在手中 return something.equals(waveHands()); }
新版Tom抓Jerry
鐺鐺！掌聲在哪里团驱？
class Tom {
/**
* Tom當前的位置
**/
private int curPos;
public void catchJerry() {
//只要沒抓到Jerry就一直抓
while(!tryCatchJerry());
}
private boolean tryCatchJerry() {
//找Jerry并前往Jerry的位置
go(findPosOfJerry());
//揮爪抓Jerry
return waveHandsCatchSomething(Jerry);
}
private int findPosOfJerry() {
//左看看
int posOfJerry = lookLeft();
if(posOfJerry != -1)
return posOfJerry;
//右看看
posOfJerry = lookRight();
if(posOfJerry != -1)
return posOfJerry;
//上看看摸吠，下看看
//.....省略
}
private void go(int pos) {
while(curPos <= pos) {
//邁前腿
curPos++;
//邁后腿 保持平衡
//....
}
}
private boolean waveHandsCatchSomething(Object something) {
//抬起爪子
upHands();
//揮抓，然后看看指定東西是否在手中
return something.equals(waveHands());
}
}
何時應(yīng)該抽取方法
有重復(fù)代碼
方法體又臭又長看不懂
怎么抽取方法
IDEA提供了重構(gòu)的快捷方式和快捷鍵店茶，使用IDEA進行重構(gòu)蜕便，方便又快捷
-
選中要抽取的代碼
-
進行方法抽取 右鍵->選擇重構(gòu)->抽取->方法 （或者直接使用快捷鍵Ctrl + Alt + m）
自動檢測出個別重復(fù)代碼的細微差別，有些代碼可能只改動一兩個變量贩幻，IDEA會自動檢測出來轿腺，并在抽取方法時提醒我們，選擇左側(cè)Accept Signature Change（接受簽名變動）可以使抽取的方法自動替換更多的重復(fù)點
可以選擇替換掉所有的重復(fù)代碼（竟然有18處）
-
重構(gòu)——更改方法簽名
如果你對抽取出的方法的名字丛楚、參數(shù)族壳、返回值或是修飾符不滿意，不要使用Ctrl + R 修改趣些，IDEA提供了重構(gòu)方法——更改簽名（快捷鍵Ctrl + F6）
注意：方法的名字指的是方法做了什么仿荆，而非怎么去做，最好是 動詞+名詞 格式
比如：Tom.掃地() √
? Tom.掃地With掃把() ×
? Tom.用掃把掃地() ×
更多重構(gòu)手段
等我把這些文章全寫完發(fā)了坏平，再給你們提供鏈接
-
到處都是重復(fù)的代碼拢操？方法體又臭又長看不懂？快來試試抽取方法（Extract Method）吧舶替，保證藥到病除令境！
-
使用對象封裝參數(shù)集
答應(yīng)我，別再給方法弄一長溜的參數(shù)了顾瞪，好嗎舔庶？
-
使用查詢方法取代臨時變量
你這方法里怎么這么多臨時變量？我完全記不住每個變量是用來干什么的陈醒！
-
使用多態(tài)取代switch分支
“來看看外包老哥寫的超長switch語句”這種文章又登上首頁了
-
及時打斷if條件分支
外包老哥總是被黑的幕后黑手找到了惕橙！
-
轉(zhuǎn)移成員變量和函數(shù)
道理我都懂，可是你一直在用的成員變量為什么要讓我負責管理钉跷？
-
抽取類
總覺得弥鹦，是時候new一個替身出來幫我干活了
-
將數(shù)據(jù)的處理函數(shù)轉(zhuǎn)移給持有數(shù)據(jù)的類
或許，這個鍋應(yīng)該由你來背
-
向父類轉(zhuǎn)移成員變量和函數(shù)
所有子類都需要做的事爷辙，為什么不讓父類直接做了惶凝？
-
注意集合型成員變量的get/set方法吼虎！
有人動了我的集合對象，可是我卻不知道苍鲜！
技術(shù)不分領(lǐng)域，思想一脈相承玷犹，歡迎訪問橙味菌的博客
本文由博客一文多發(fā)平臺 OpenWrite 發(fā)布混滔！
