本文背景为学习重构一书中的一些信息的记录。
该篇以一个影片租赁为背景。通过一层层的重构，最终实现相对来说比较完美的代码。
文中经典语句摘要：
“如果它没有坏，就不要动它 。” 用来形容我们工作中一些比较古董级的代码，或许写的很烂，但是基本功能是OK的，对于一个团队leader来说，不会主张你去修改他，对于一个保守级的开发来说，也懒的去修改他，虽然你个人任务修改它出错的概率近乎为0，但是仍旧是近乎。只愿我能在工作中永远保持一个敬畏之心，而非畏惧之心。
1、第一版，较多的信息冗余在一起，相对书中的第一版已经有了一些提升，加上了个人在初步做的一些重构，怎么说哥也是写过两年代码的人。简单重构还是了解一些的。
用户类，定义用户的名称以及租赁了哪些影片
package com.woniu.refactoring; import java.util.Enumeration;
import java.util.Vector; public class Customer {
private String _name;
private Vector _rentals = new Vector(); public String getName() {
return _name;
}
public void setName(String _name) {
this._name = _name;
}
public Vector getRentals() {
return _rentals;
}
public void setRentals(Vector _rentals) {
this._rentals = _rentals;
} public String statement() {
double totalAmout = 0;
int frequentRenterPoints = 0;
Enumeration rentals = _rentals.elements();
String result = "Rental Record for " + getName() + "\n";
while(rentals.hasMoreElements()) {
double thisAmout = 0;
Rental each = (Rental) rentals.nextElement();
//这里是个人阅读到这里时，认为需要重构的位置。用专业点的话叫提炼函数 TODO
// switch (each.getMovie().getPriceCode()) {
// case Movie.REGULAR:
// thisAmout += 2;
// if(each.getDaysRented() > 2) {
// thisAmout += (each.getDaysRented() - 2) * 1.5;
// }
// break;
// case Movie.NEW_RELEASE:
// thisAmout += each.getDaysRented() * 3;
// break;
//
// case Movie.CHILDENS:
// thisAmout += 1.5;
// if(each.getDaysRented() > 3) {
// thisAmout += (each.getDaysRented() - 3) * 1.5;
// }
// break;
// }
//上述方法，经过提炼后得到这个相对来说可以重用的一个函数amounFor
thisAmout = amountFor(each);
//这里是个人阅读到这里时，认为需要重构的位置。用专业点的话叫提炼函数 TODO
// frequentRenterPoints ++;
// if((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && each.getDaysRented() > 1) {
// frequentRenterPoints ++;
// }
//进过提炼后的积分计算方式
frequentRenterPoints = getFrequentRenterPoints(each);
result += "\t" + each.getMovie().getTitle() + "\t" + String.valueOf(thisAmout) + "\n";
totalAmout += thisAmout; } result += "Amount owed is " + String.valueOf(totalAmout) + "\n";
result += "You earned " + String.valueOf(frequentRenterPoints) + " frequent renter points";
return result;
}
/**
* 用于计算此次租赁产生的积分
* @param rental 租赁
* @return
*/
private int getFrequentRenterPoints(Rental rental) {
if((rental.getMovie().getPriceCode() == Movie.NEW_RELEASE) && rental.getDaysRented() > 1) {
return 2;//这里其实在最初我是没有想到可以直接用2来代替的
}
return 1;
} /**
* 用于计算此次租赁消费金额
* @param rental
* @return
*/
private double amountFor(Rental rental) {
double thisAmout = 0;
switch (rental.getMovie().getPriceCode()) {
case Movie.REGULAR:
thisAmout += 2;
if(rental.getDaysRented() > 2) {
thisAmout += (rental.getDaysRented() - 2) * 1.5;
}
break;
case Movie.NEW_RELEASE:
thisAmout += rental.getDaysRented() * 3;
break; case Movie.CHILDENS:
thisAmout += 1.5;
if(rental.getDaysRented() > 3) {
thisAmout += (rental.getDaysRented() - 3) * 1.5;
}
break;
}
return thisAmout;
} }
影片类，定义了影片的类型以及价格
package com.woniu.refactoring;
public class Movie {
public static final int CHILDENS = 2;
public static final int REGULAR = 0;
public static final int NEW_RELEASE = 1;
private String _title;
private int _priceCode;
public Movie(String title, int priceCode) {
_title = title;
priceCode = priceCode;
}
public int getPriceCode() {
return _priceCode;
}
public void setPriceCode(int _priceCode) {
this._priceCode = _priceCode;
}
public String getTitle() {
return _title;
}
}租赁类，记录影片被租赁的时间以及影片信息
package com.woniu.refactoring;
public class Rental {
private Movie _movie;
private int _daysRented;
public Rental(Movie movie, int daysRented) {
_movie = movie;
_daysRented = daysRented;
}
public Movie getMovie() {
return _movie;
}
public int getDaysRented() {
return _daysRented;
}
}第二版
由于在amountFor()方法中，并没有任何用户的信息，却在用户的类中实现了关于影片计算逻辑，金额的产生发生的位置应该是在租赁中产生的，因此，应该将其移至Rental中，使用（Move Mehtod）来进行处理。
用户类，封装了用户总消费额以及总积分的计算。同时，将关于租赁的单价计算与单积分计算进行了迁移。
package com.woniu.refactoring; import java.util.Enumeration;
import java.util.Vector; public class Customer {
private String _name;
private Vector _rentals = new Vector(); public String getName() {
return _name;
}
public void setName(String _name) {
this._name = _name;
}
public Vector getRentals() {
return _rentals;
}
public void setRentals(Vector _rentals) {
this._rentals = _rentals;
} public String statement() {
// double totalAmout = 0;
// int frequentRenterPoints = 0;
Enumeration rentals = _rentals.elements();
String result = "Rental Record for " + getName() + "\n";
while(rentals.hasMoreElements()) {
double thisAmout = 0;
Rental each = (Rental) rentals.nextElement();
//这里是个人阅读到这里时，认为需要重构的位置。用专业点的话叫提炼函数 TODO
// switch (each.getMovie().getPriceCode()) {
// case Movie.REGULAR:
// thisAmout += 2;
// if(each.getDaysRented() > 2) {
// thisAmout += (each.getDaysRented() - 2) * 1.5;
// }
// break;
// case Movie.NEW_RELEASE:
// thisAmout += each.getDaysRented() * 3;
// break;
//
// case Movie.CHILDENS:
// thisAmout += 1.5;
// if(each.getDaysRented() > 3) {
// thisAmout += (each.getDaysRented() - 3) * 1.5;
// }
// break;
// }
//上述方法，经过提炼后得到这个相对来说可以重用的一个函数amounFor
// thisAmout = amountFor(each);
thisAmout = each.getCharge();//这是第二版的重构，将金额计算与积分计算，移至了Rental中，因为这是在租赁过程中产生的
//这里是个人阅读到这里时，认为需要重构的位置。用专业点的话叫提炼函数 TODO
// frequentRenterPoints ++;
// if((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && each.getDaysRented() > 1) {
// frequentRenterPoints ++;
// }
//进过提炼后的积分计算方式
// frequentRenterPoints = getFrequentRenterPoints(each);
// frequentRenterPoints = each.getFrequentRenterPoints();//这是第二版的重构，将金额计算与积分计算，移至了Rental中，因为这是在租赁过程中产生的
result += "\t" + each.getMovie().getTitle() + "\t" + String.valueOf(thisAmout) + "\n";
// totalAmout += thisAmout;//2书中提到该处可以直接使用getCharge()获取金额，进而避免了局部变量的使用，但是，我没有找到关于多次计算的优化点，因此不敢王嘉苟同。 } result += "Amount owed is " + String.valueOf(getTotalCharge()) + "\n";//对于总金额计算单独提取方法，以备后用
result += "You earned " + String.valueOf(getTotalFrequentRenterPoints()) + " frequent renter points";
return result;
} /**
* 获取用户消费的总金额
* @return
*/
private double getTotalCharge() {
double totalAmout = 0;
Enumeration rentals = _rentals.elements();
while(rentals.hasMoreElements()) {
Rental rental = (Rental)rentals.nextElement();
totalAmout += rental.getCharge();
}
return totalAmout;
} /**
* 获取用户消费的总积分
* @return
*/
private int getTotalFrequentRenterPoints() {
int result = 0;
Enumeration rentals = _rentals.elements();
while(rentals.hasMoreElements()) {
Rental rental = (Rental)rentals.nextElement();
result += rental.getFrequentRenterPoints();
}
return result;
} /**
* 用于计算此次租赁产生的积分
* @param rental 租赁
* @return
*/
@Deprecated
private int getFrequentRenterPoints(Rental rental) {
if((rental.getMovie().getPriceCode() == Movie.NEW_RELEASE) && rental.getDaysRented() > 1) {
return 2;//这里其实在最初我是没有想到可以直接用2来代替的
}
return 1;
} /**
* 用于计算此次租赁消费金额
* @param rental
* @return
*/
@Deprecated
private double amountFor(Rental rental) {
return rental.getCharge();//这里其实没必要再次进行封装函数了。可以直接在循环中使用rental的getCharge方法进行计算。
// double thisAmout = 0;
// switch (rental.getMovie().getPriceCode()) {
// case Movie.REGULAR:
// thisAmout += 2;
// if(rental.getDaysRented() > 2) {
// thisAmout += (rental.getDaysRented() - 2) * 1.5;
// }
// break;
// case Movie.NEW_RELEASE:
// thisAmout += rental.getDaysRented() * 3;
// break;
//
// case Movie.CHILDENS:
// thisAmout += 1.5;
// if(rental.getDaysRented() > 3) {
// thisAmout += (rental.getDaysRented() - 3) * 1.5;
// }
// break;
// }
// return thisAmout;
} }
租赁类，增加了关于租赁单价的计算以及积分计算。
package com.woniu.refactoring;
public class Rental {
private Movie _movie;
private int _daysRented;
public Rental(Movie movie, int daysRented) {
_movie = movie;
_daysRented = daysRented;
}
public Movie getMovie() {
return _movie;
}
public int getDaysRented() {
return _daysRented;
}
/**
* 用户此次租赁积分的计算
* @param rental
* @return
*/
public int getFrequentRenterPoints() {
if((getMovie().getPriceCode() == Movie.NEW_RELEASE) && getDaysRented() > 1) {
return 2;//这里其实在最初我是没有想到可以直接用2来代替的
}
return 1;
}
/**
* 用于计算此次租赁消费金额
* 2关于这个方法，其实应该是在租赁关系发生时，已经需要计算好金额了。这样就可以避免了Customer中提到的优化点，
* 但是，这样同样会造成如果交易取消了，我浪费资源进行了计算。
* @param rental
* @return
*/
public double getCharge() {
double thisAmout = 0;
switch (getMovie().getPriceCode()) {
case Movie.REGULAR:
thisAmout += 2;
if(getDaysRented() > 2) {
thisAmout += (getDaysRented() - 2) * 1.5;
}
break;
case Movie.NEW_RELEASE:
thisAmout += getDaysRented() * 3;
break;
case Movie.CHILDENS:
thisAmout += 1.5;
if(getDaysRented() > 3) {
thisAmout += (getDaysRented() - 3) * 1.5;
}
break;
}
return thisAmout;
}
}原本到这里，我理解，此次重构已经结束了。但发现，后续还有新的重构方向。。。。。