代码重构总结(上篇)

jorwen_fang

浏览: 51644 次
性别:
来自: 上海

最近访客更多访客>>

fujj8337

TomTianTao

snowstorm

pengfeifei26

博主相关

博客

微博

相册

留言

关于我

文章分类

社区版块

存档分类

博客分类：

Java

重构

重构手法千变万化，如果你看过《重构_改善既有代码的设计》这本书一定会很有感触，此文我从这本书中挑选个人觉得比较有用的几点，一些我在工作中觉得比较实用的重构手法。

（01） - 方法提炼

有句话叫，当你为了给一段代码写注释的时候，你就能考虑把他提炼一下了。
来看一个实际例子，登录
配合idea的Refactor->Extract->Method

原始代码如下：

public String login()

{

//错误信息

String error = "";

//1 显示验证码框

flag = "1";

//2 检查用户名密码为空

returnInfoBean = new ReturnInfoBean();

if (StringUtils.isBlank(custId))

{

error = "用户名不能为空";

}

else if (StringUtils.isBlank(custPassword))

{

error = "登录密码不能为空";

}

if(StringUtils.isNotBlank(error))

{

returnInfoBean.setSuccess(false);

returnInfoBean.setReturnMessage(error);

return "login";

}

 

try

{

//3 调用恒生登录

HsDsApi hsApi = new HsDsApi(";P            003", "integrate/login", LOGGER);

hsApi.put("logtype", logtype);

hsApi.put("lognumber", custId);

hsApi.put("password", custPassword);

if (StringUtils.isNotBlank(certificateType)) hsApi.put("certificatetype", certificateType);

HsResponse respx = hsApi.callHsApi();

returnInfoBean.setSuccess(respx.isSuccess());

if (respx.isSuccess())

{

//3.1 保存用户基本信息存入session

String sessionKey = respx.getString("sessionkey");

ICTLoginBean lb = new CTLoginBeanImpl();

lb.setUserInfo(getUserInfo(sessionKey));

lb.setAccountUtil(getAccountUtil(sessionKey));

request.getSession().setAttribute(ICTLoginBean.sessionID, lb);

}

else

{

//3.2 登陆失败返回错误代码并处理

throw new RuntimeException("【" + respx.getCode() + "】" + respx.getMessage());

}

} catch (Exception r)

{

//3.3 调用失败

error = "系统异常，请稍候再试";

LOGGER.error("调用恒生登录失败", r);

}

 

//4 是否登陆成功

if (StringUtils.isNotBlank(error))

{

returnInfoBean.setSuccess(false);

returnInfoBean.setReturnMessage(error);

return "login";

}

else

{

return "first";

}

}

其中2和3.1提炼到了private方法， 3提炼到了别的类里，因为明显这是个公共方法

查看

打印?

public String login()

{

returnInfoBean = new ReturnInfoBean();

returnInfoBean.setSuccess(true);

//1 显示验证码框

flag = "1";

 

//2 检查用户名密码为空

if (!checkLogin()) return "login";

 

try

{

//3 调用恒生登录

HsResponse respx = HsCommonAPi.login(logtype, custId, custPassword, certificateType);

if (respx.isSuccess())

{

//3.1 保存用户基本信息存入session

saveInfoToSession(respx.getString("sessionkey"));

}

else

{

//3.2 登陆失败返回错误代码并处理

throw new RuntimeException("【" + respx.getCode() + "】" + respx.getMessage());

}

}

catch (Exception r)

{

//3.3 调用失败

returnInfoBean.setSuccess(false);

returnInfoBean.setReturnMessage("系统异常，请稍候再试");

LOGGER.error("调用恒生登录失败", r);

}

 

//4 是否登陆成功

return returnInfoBean.isSuccess()?"first":"login";

}

（02） - 分解临时变量

一个变量变用在多个地方，而且每次使用含义都不同，这个时候可以考虑把他分解

重构后

（03） - 以查询取代临时变量

如果一个临时变量在方法里赋值有点复杂，而且可能还会被复用。可以提炼到方法里可以说是第1话的子范例

重构后

（04） - 引入解释性变量

不解释自己看例子，目的为了代码有更好地可读性

重构后

（05） - 使代码更简洁

这部分功能，欢迎大家投稿

if ( flag == 1 ){

return true;

}

else{

return false;

}

重构后

1.return flag == 1;

（06） - 尽早结束非正常逻辑

if(taskList != null && !taskList.isEmpty()){

 

//正常逻辑代码

 

return count;

}

重构后

int count = 0;

if(taskList == null || taskList.isEmpty()){

return 0;

}

 

//正常逻辑代码

 

return count;

除了return，抛出异常也是种结束方式

（07） - 满足条件立刻跳出循环

public boolean contain(int year, Month month, int day) {

boolean found = false;

for (IPolyDate date : dateList) {

if (date.same(year, month.getMonth(), day)) {

found = true;

break;

}

}

 

return found;

}

重构后

public boolean contain(int year, Month month, int day) {

for (IPolyDate date : dateList) {

if (date.same(year, month.getMonth(), day)) {

return true;

}

}

 

return false;

}

不但代码简洁还能提高效率

（08） - 使用数据构分离公共逻辑，避免重复（坚持DRY原则）

DRY原则：不要重复粘帖你自己的代码。Don't repeat yourself

public void testGetIntPart() throws Exception {

assertEquals("0", digitTransform.getIntPart("0.01");

assertEquals("1", digitTransform.getIntPart("1.2");

assertEquals("1234", digitTransform.getIntPart("1234");

assertEquals("1", digitTransform.getIntPart("1.01");

assertEquals("0", digitTransform.getIntPart("0.01");

assertEquals("11111", digitTransform.getIntPart("11111");

assertEquals("1000", digitTransform.getIntPart("1000.11");

}

重构后

　public void testGetIntPart() throws Exception {

　　String[][] cases = new String[][] { { "0.01", "0" }, { "1.2", "1" },

{ "1234", "1234" }, { "1.01", "1" }, { "0", "0" },

{ "11111", "11111" }, { "1000.11", "1000" } };

 

　　for (int i = 0, len = cases.length; i < len; i++) {

　　　assertEquals(cases[1], digitTransform.getIntPart(cases[0]));

　　}

　}

（09） - 公共逻辑提炼

public int getRemainMinutes(int hour, int minute) {

if (this.fromAfterEqual(hour, minute)) {

return (toHour * 60 + toMinute) - (fromHour * 60 + fromMinute);

} else if (this.toAfterEqual(hour, minute)) {

return (toHour * 60 + toMinute) - (hour * 60 + minute);

}

else {

return 0; //(toHour * 60 + toMinute) - (toHour * 60 + toMinute);

}

}

重构后

private int getMinutes(int startHour, int startMinute, int endHour,

int endMinute) {

int minutes = 0;

minutes = (endHour * 60 + endMinute) - (startHour * 60 + startMinute);

 

return minutes;

}

 

public int getRemainMinutes(int hour, int minute) {

int startHour = toHour;

int startMinute = toMinute;

 

if (this.fromAfterEqual(hour, minute)) {

startHour = fromHour;

startMinute = fromMinute;

} else if (this.toAfterEqual(hour, minute)) {

startHour = hour;

startMinute = minute;

}

 

return this.getMinutes(startHour, startMinute, toHour, toMinute);

}

（10） - 何时需要何时创建

public static Date parseDate(String date) throws ;P arseException {

SimpleDateFormat formatter = new SimpleDateFormat("yyyy-MM-dd");

 

if ((date == null) || (date.equals(""))) {

return null;

}

else {

try {

return formatter.parse(date);

} catch (ParseException pe) {

throw new ;P arseException(pe.getMessage(), pe.getErrorOffset());

}

}

}

重构后

public static Date parseDate(String date) throws ;P arseException {

if ((date == null) || (date.equals(""))) {

return null;

}

else {

try {

SimpleDateFormat formatter = new SimpleDateFormat("yyyy-MM-dd");

return formatter.parse(date);

} catch (ParseException pe) {

throw new ;P arseException(pe.getMessage(), pe.getErrorOffset());

}

}

}