标签：封装 抽取 vat 参数 alt sof 逻辑 equal ide
到处都是重复的代码？方法体又臭又长看不懂？快来试试抽取方法（Extract Method）吧，保证药到病除！
将重复的代码或语义独立的代码抽取成一个方法
看好了：
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的嵌套循环，让人不明所以，连走路迈腿都写在一块，如果没有注释的话，绝对让人无法理解
简化逻辑：抓Jerry = 抓不到Jerry就一直抓
public void catchJerry() {
//只要没抓到Jerry就一直抓
while(!tryCatchJerry());
}catchJerry()只有一行了，此处应有掌声
但是我们又创建了又臭又长的方法——试图抓住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简单又清晰了
我对剩下的方法进行了微调，让它们更有逻辑感
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());
}铛铛！掌声在哪里？
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());
}
}有重复代码
方法体又臭又长看不懂
IDEA提供了重构的快捷方式和快捷键，使用IDEA进行重构，方便又快捷
选中要抽取的代码
进行方法抽取 右键->选择重构->抽取->方法 （或者直接使用快捷键Ctrl + Alt + m）
自动检测出个别重复代码的细微差别，有些代码可能只改动一两个变量，IDEA会自动检测出来，并在抽取方法时提醒我们，选择左侧Accept Signature Change（接受签名变动）可以使抽取的方法自动替换更多的重复点
可以选择替换掉所有的重复代码（竟然有18处）
重构——更改方法签名
如果你对抽取出的方法的名字、参数、返回值或是修饰符不满意，不要使用Ctrl + R 修改，IDEA提供了重构方法——更改签名（快捷键Ctrl + F6）
注意：方法的名字指的是方法做了什么，而非怎么去做，最好是 动词+名词 格式
比如：Tom.扫地() √
? Tom.扫地With扫把() ×
? Tom.用扫把扫地() ×
等我把这些文章全写完发了，再给你们提供链接
到处都是重复的代码？方法体又臭又长看不懂？快来试试抽取方法（Extract Method）吧，保证药到病除！
使用对象封装参数集
答应我，别再给方法弄一长溜的参数了，好吗？
使用查询方法取代临时变量
你这方法里怎么这么多临时变量？我完全记不住每个变量是用来干什么的！
使用多态取代switch分支
“来看看外包老哥写的超长switch语句”这种文章又登上首页了
及时打断if条件分支
外包老哥总是被黑的幕后黑手找到了！
转移成员变量和函数
道理我都懂，可是你一直在用的成员变量为什么要让我负责管理？
抽取类
总觉得，是时候new一个替身出来帮我干活了
将数据的处理函数转移给持有数据的类
或许，这个锅应该由你来背
向父类转移成员变量和函数
所有子类都需要做的事，为什么不让父类直接做了？
注意集合型成员变量的get/set方法！
有人动了我的集合对象，可是我却不知道！
技术不分领域，思想一脉相承
标签：封装 抽取 vat 参数 alt sof 逻辑 equal ide
原文地址：https://www.cnblogs.com/chengweijun/p/12165739.html
