是怎样？
重构前：
double getPrice() {
int basePrice = _quantity * _itemPrice;
double discountFactor;
if (basePrice > 1000) {
discountFactor = 0.95;
} else {
discountFactor = 0.98;
}
return basePrice * discountFactor;
}
重构后:
>```Java
double getPrice() {
return basePrice() * discountFactor();
}
private double discountFactor() {
if (basePrice() > 1000) {
return 0.95;
} else {
return 0.98;
}
}
private int basePrice() {
return _quantity * _itemPrice;
}
如何做？
- 先给这两个临时变量添加 final 修饰词确保他们只被赋值一次
final int basePrice = _quantity * _itemPrice;
final double discountFactor;
- 选中 basePrice， 右键 -> refactor -> Replace Temp With Query
double getPrice() {
final double discountFactor;
if (basePrice() > 1000) {
discountFactor = 0.95;
} else {
discountFactor = 0.98;
}
return basePrice() * discountFactor;
}
private int basePrice() {
return _quantity * _itemPrice;
}
- 运行测试。
- 接着开始替换discountFactor变量。这里不能直接用Replace Temp With Query, 先选中如下代码，
final double discountFactor;
if (basePrice() > 1000) {
discountFactor = 0.95;
} else {
discountFactor = 0.98;
}
用 Extract Method（cmd + opt + m） 将他们提炼到一个独立的方法中去， 由于后续还需要用到discountFactor的值，所以这里在Extract Method的时候，要提供一个返回值，不过android studio 会自动做完这个步骤。执行完成之后：
private double discountFactor() {
final double discountFactor;
if (basePrice() > 1000) {
discountFactor = 0.95;
} else {
discountFactor = 0.98;
}
return discountFactor;
}
运行测试。对discountFactor这个方法可以再简化一下，去掉临时变量，运行测试。
private double discountFactor() {
if (basePrice() > 1000) {
return 0.95;
} else {
return 0.98;
}
}
- 现在getPrice方法像这样：
double getPrice() {
final double discountFactor = discountFactor();
return basePrice() * discountFactor;
}
去掉临时变量，运行测试。
double getPrice() {
return basePrice() * discountFactor();
}
