之前的重构系列中,介绍了书中提到的重构基础,以及识别代码的坏味道。今天继续第三更,讲述那些重构手法(上)。看看哪些手法对你的项目能有所帮助…

对函数进行整理,使之更恰当的包装代码。
改造前:
void printInfoAndDetail() {
this.printInfo();
System.out.println("this is detail name:" + _name);
System.out.println("this is detail account:" + _account);
}
改造后:
void printInfoAndDetail() {
this.printInfo();
this.printDetail();
}
void printDetail() {
System.out.println("this is detail name:" + _name);
System.out.println("this is detail account:" + _account);
}
动机:
控制函数的粒度,函数粒度很小,那么被复用的机会就更大;其次会使高层函数读起来就像一系列注释,再次,如果函数都是细粒度,那么函数的覆盖也会更容易。
一个函数多长才算合适?其实长度不是关键问题,关键在于函数名和函数本体之间的语义距离。
做法:
1、创造一个新函数,根据这个函数意图来命名(以它”做什么“来命名,而不是”怎样做“命名)。
只要新的函数名能够以更好的方式昭示代码意图,你也应该提炼他(就算代码只是一条消息,或一个函数调用)。但如果你想不出一个更有意义的名称,就别动。
2、将提炼出来额代码从源函数复制到新建的目标函数中。
3、检查变量。检查提炼出的代码是否引用了源代函数的局部变量或参数。以被提炼函数中是否含有临时变量。
难点:
这个重构手法的难点就在于局部变量的控制,包括传进源函数的参数和源函数所有声明的临时变量。
改造前:
int getRating() {
return isGe5() ? 2 : 1;
}
boolean isGe5(){
return _num >= 5;
}
改造后:
int getRating() {
return _num >= 5 ? 2 : 1;
}
动机:
移除非必要的间接层。当然间接层有其价值,但不是所有的间接层都有价值,可以去除那些无用的间接层。
做法:
1、检查函数,确定他不具备多态性。如果有子类继承了这个函数,那就不能将此函数内联。因为子类无法覆盖一个根本不存在的函数。如例子中,子类可以重写isGe5(),但内敛之后的_num > 5 ? 2 : 1是无法重写的,除非你重写了getRating()。
2、找出函数的所有被调用点,将这个函数的所有被调用点都替换为函数本体。
改造前:
double price = order.price();
return price > 1000;
改造后:
return order.price() > 1000
改造前:
double price = _qu * _item;
if(price > 1000){
return price * 0.95;
} else {
return price * 0.98;
}
改造后:
if(getPrice() > 1000){
return getPrice() * 0.95;
} else {
return getPrice() * 0.98;
}
double getPrice(){
return _qu * _item;
}
改造前:
if((platform.indexOf("mac") > -1)
&& (platform.indexOf("ie") > -1)
&& resize > 0
){
// todo...
}
改造后:
final boolean isMac = (platform.indexOf("mac") > -1;
final boolean isIe = (platform.indexOf("ie") > -1;
final boolean resized = resize > 0;
if( && isIe && resized){
// todo...
}
改造前:
double temp = 2 * (_h + _w);
System.out.println(temp);
temp = _h * _w;
System.out.println(temp);
改造后:
final double temp = 2 * (_h + _w);
System.out.println(temp);
final double area = _h * _w;
System.out.println(area);
改造前:
int discount(int inputVal) {
if(inputVal > 50) inputVal -= 2;
}
改造后:
int discount(int inputVal) {
int result = inputVal;
if(inputVal > 50) result -= 2;
}
“决定把责任放在哪儿”。
如果一个类有太多的行为,或如果一个类于另一个类有太多合作而形成高度耦合,尝试搬移函数。将旧函数变成一个单纯的委托函数,或是将旧函数完全移除。
改造前:
class Account {
private AccountType _type;
private int _dayOverdrawn;
double overdraftCharge(){
if(_type.isPremium()) {
double result = 10;
if(_dayOverdrawn > 7) result += (_dayOverdrawn - 7) * 0.85;
return result;
} else {
return _dayOverdrawn * 1.75;
}
}
}
改造后:
class Account {
private AccountType _type;
private int _dayOverdrawn;
double overdraftCharge(){
return _type.overdraftCharge(_dayOverdrawn);
}
}
class AccountType {
double overdraftCharge(int daysOverdrawn){
if(isPremium()) {
double result = 10;
if(dayOverdrawn > 7) result += (dayOverdrawn - 7) * 0.85;
return result;
} else {
return dayOverdrawn * 1.75;
}
}
}
如果一个字段,在其所驻类之外的另一个类中有更多函数使用了它,就要考虑搬移这个字段。这里的使用可能是设值,取值函数间接进行的。
改造前:
class Account {
private AccountType _type;
private int _rate;
double overdraftCharge(double amount, int days){
return _rate * amount * days / 365;
}
}
改造后:
class Account {
private AccountType _type;
double overdraftCharge(){
return _type.getRate() * amount * days / 365;
}
}
class AccountType {
private double _rate;
void setRate(double r) {
this._rate = r;
}
void getRate() {
return _rate;
}
}
建立一个新类,将相关的字段和函数从旧类搬移到新类。一个类应该是一个清楚的抽象,处理一些明确的责任。
改造前:
class Account {
private String personName;
private String personPhone;
private double money;
public String getAccountInfo(){
return personName + ",联系方式:" + personPhone + ",余额:" + money;
}
}
改造后:
class Account {
private Person person = new Person();
private double money;
public String getAccountInfo(){
return person.getPersonName() + person.personPhone() + ",余额:" + money;
}
}
class Person {
private String personName;
private String personPhone;
public String getPersonName(){
return "联系人:" + personName;
}
public String getPersonPhone(){
return "联系方式:" + personPhone;
}
}
将这个类的所有特性搬移到另一个类中,然后移除原类。与Extract Class相反。
“封装”即使不是对象的最关键特性,也是最关键特性之一。“封装”意味着每个对象都应该尽可能少了解系统的其他部分。
改造前:
class Person {
private Department department;
public Department getDepartment(){
return department;
}
}
class Department {
private Person manager;
public Person getManager(){
return manager;
}
}
// 如果客户希望知道某人的经理是谁,那他的调用关系是:
xxx.getDepartment().getManager();
// 暴露了部门和经理的委托关系
改造后:
class Person {
private Department department;
public Department getDepartment(){
return department;
}
public Person getManager(){
return department.getManager();
}
}
class Department {
private Person manager;
public Person getManager(){
return manager;
}
}
// 如果客户希望知道某人的经理是谁,那他的调用关系是:(隐藏了Department)
xxx.getManager();
某个类做了过多的简单委托动作。
当你需要为提供服务的类增加一个函数,但你无法修改这个类。如果你只使用这个函数一次,那么额外编码工作没什么大不了,升值可能根本不需要原本提供服务的那个类。然而,如果你需要多次使用这个函数,就得不断重复这些代码。重复代码是软件万恶之源。
改造前:
Date newStart = new Date(pre.getYear(), pre.getMethod(), pre.getDate() + 1);
改造后:
Date newStart = nextDay(pre);
private static Date nextDay(Date arg) {
return new Date(arg.getYear(), arg.getMethod(), arg.getDate() + 1);
}
如真实项目中的案例:
BeanUtil.copyProperties(),原始方法该行为需要抛异常,且被建议不再使用该方法进行bean复制。
于是引入外加函数:
class BeanUtilExt {
public static void copyProperties(Object target, Object source) {
try {
BeanUtil.copyProperties()
} catch (Exception) {
// ignored...
}
}
}
这种方式个人不推荐。
当你需要为提供服务的类提供一些额外函数,但你无法修改这个类。
改造前:
private int _low, _high;
boolean includes(int arg){
return arg >= _low && arg <= _high;
}
改造后:
private int _low, _high;
boolean includes(int arg){
return arg >= getLow() && arg <= getHigh();
}
int getLow(){
return _low;
}
int getHigh(){
return _high;
}
直接访问变量好处:代码容易阅读。
间接访问变量好处:子类可以通过重写(覆盖)一个函数而改变获取数据的途径。
开发初期,你往往决定以简单的数据项表示简单的情况。但是,随着开发的进行,你可能会发现,这些简单的数据项不再那么简单了。比如你一开始会用字符串来表示“电话号码”,但是随后你会发现,电话号需要“格式化”,“抽取区号”之类的特殊行为。如果这样的数据项只有一两个,你还可以把相关函数放进数据项所属的对象里,但是Duplicate Code和Feature Envy很快就会表现出来。这时,你就应该将数值变为对象。
你从一个类衍生出许多批次相等的实例,希望将它们替换为同一个对象。
你有一个引用对象,很小且不可变,而且不易管理。
你有一个数组,其中的元素各自代表不同的东西。
改造前:
String[] row = new String[3];
row[0] = "liver";
row[1] = "15";
改造后:
Performance row = new Performance();
row.setName("liver");
row.setWins(15);
有一些领域数据置身于GUI组件中,而领域函数需要访问这些数据。
将该数据复制到一个领域对象中。建立一个Observer模式,用以同步领域对象和GUI对象内的重复数据。可以使用事件监听器,诸如JAVAFX中的Property。
两个类都需要使用对方特性,但其间只有一条单向链接。
两个类之间有双向关联,但其中一个类如今不再需要另一个类的特性。
改造前:
double count(double a, double b){
return a * 0.95 * b;
}
改造后:
double count(double a, double b){
return a * RATE_CONSTANT * b;
}
static final double RATE_CONSTANT = 0.95;
即面向对象的首要原则之一:封装,或称为“数据隐藏”。
改造前:
public String _name;
改造后:
private String _name;
public String getName() {
return _name;
}
public void setName(String name) {
this._name = name;
}
让这个函数返回该集合的一个只读副本,并在这个类中提供添加/移除集合元素的函数。
改造前:
class Person {
List<String> classes;
public List<String> getClasses(){
return classes;
}
public void setClasses(List<String> cls){
this.classes = cls;
}
}
改造后:
class Person {
List<String> classes;
public List<String> getClasses(){
return classes;
}
// setter方法隐藏,避免用户修改集合内容而一无所知
private void setClasses(List<String> cls){
this.classes = cls;
}
public void addClass(String cls) {
classes.add(cls);
}
public void removeClass(String cls) {
classes.remove(cls);
}
}
主要用来应对传统编程环境中的记录结构。
类中有一个数值类型码,但它并不影响类的行为。
改造前:
class Person {
public static final int O = 0;
public static final int A = 1;
public static final int B = 2;
public static final int AB = 3;
@Getter
@Setter
private int _bloodGroup;
public Person(int bloodGroup) {
_bloodGroup = bloodGroup;
}
}
改造后:
class Person {
public static final int O = BloodGroup.O.getCode();
public static final int A = BloodGroup.A.getCode();
public static final int B = BloodGroup.B.getCode();
public static final int AB = BloodGroup.AB.getCode();
@Getter
private int _bloodGroup;
public Person(int bloodGroup) {
_bloodGroup = BloodGroup.code(bloodGroup);
}
public void setBloodGroup(int arg) {
_bloodGroup = BloodGroup.code(arg);
}
}
class BloodGroup {
public static final BloodGroup O = new BloodGroup(0);
public static final BloodGroup A = new BloodGroup(1);
public static final BloodGroup B = new BloodGroup(2);
public static final BloodGroup AB = new BloodGroup(3);
private static final BloodGroup[] _values = {A, A, B, AB};
@Getter
private final int _code;
private BloodGroup(int code) {
_code = code;
}
public static BloodGroup code(int arg) {
return _values[arg];
}
}
你有一个不可变的类型码,他会影响类的行为。
你有一个类型码,它会影响类的行为,但你无法通过继承手法消除他。
到此仅汇总了一半的重构手法,个人觉得部分重构手法是以牺牲一定的代码阅读性为代价。且书中提到的多数重构手法还是要视具体编程场景而定。避免错误引用。
重构手法和设计模式一样,均为编程模式中的最佳实践。是符合大多数场景和行为的思想或方法的总结。记住是大多数。了解最佳实践有助于提高平常的编码习惯以及提升代码的维护性,可修改性。但如果被错误引用,程序将因为过度设计或引用而变得臃肿。
持续更新中…将继续更新重构手法(下)…
我有一个模型:classItem项目有一个属性“商店”基于存储的值,我希望Item对象对特定方法具有不同的行为。Rails中是否有针对此的通用设计模式?如果方法中没有大的if-else语句,这是如何干净利落地完成的? 最佳答案 通常通过Single-TableInheritance. 关于ruby-on-rails-Rails-子类化模型的设计模式是什么?,我们在StackOverflow上找到一个类似的问题: https://stackoverflow.co
如何在buildr项目中使用Ruby?我在很多不同的项目中使用过Ruby、JRuby、Java和Clojure。我目前正在使用我的标准Ruby开发一个模拟应用程序,我想尝试使用Clojure后端(我确实喜欢功能代码)以及JRubygui和测试套件。我还可以看到在未来的不同项目中使用Scala作为后端。我想我要为我的项目尝试一下buildr(http://buildr.apache.org/),但我注意到buildr似乎没有设置为在项目中使用JRuby代码本身!这看起来有点傻,因为该工具旨在统一通用的JVM语言并且是在ruby中构建的。除了将输出的jar包含在一个独特的、仅限ruby
在rails源中:https://github.com/rails/rails/blob/master/activesupport/lib/active_support/lazy_load_hooks.rb可以看到以下内容@load_hooks=Hash.new{|h,k|h[k]=[]}在IRB中,它只是初始化一个空哈希。和做有什么区别@load_hooks=Hash.new 最佳答案 查看rubydocumentationforHashnew→new_hashclicktotogglesourcenew(obj)→new_has
我将应用程序升级到Rails4,一切正常。我可以登录并转到我的编辑页面。也更新了观点。使用标准View时,用户会更新。但是当我添加例如字段:name时,它不会在表单中更新。使用devise3.1.1和gem'protected_attributes'我需要在设备或数据库上运行某种更新命令吗?我也搜索过这个地方,找到了许多不同的解决方案,但没有一个会更新我的用户字段。我没有添加任何自定义字段。 最佳答案 如果您想允许额外的参数,您可以在ApplicationController中使用beforefilter,因为Rails4将参数
我的主要目标是能够完全理解我正在使用的库/gem。我尝试在Github上从头到尾阅读源代码,但这真的很难。我认为更有趣、更温和的踏脚石就是在使用时阅读每个库/gem方法的源代码。例如,我想知道RubyonRails中的redirect_to方法是如何工作的:如何查找redirect_to方法的源代码?我知道在pry中我可以执行类似show-methodmethod的操作,但我如何才能对Rails框架中的方法执行此操作?您对我如何更好地理解Gem及其API有什么建议吗?仅仅阅读源代码似乎真的很难,尤其是对于框架。谢谢! 最佳答案 Ru
我的假设是moduleAmoduleBendend和moduleA::Bend是一样的。我能够从thisblog找到解决方案,thisSOthread和andthisSOthread.为什么以及什么时候应该更喜欢紧凑语法A::B而不是另一个,因为它显然有一个缺点?我有一种直觉,它可能与性能有关,因为在更多命名空间中查找常量需要更多计算。但是我无法通过对普通类进行基准测试来验证这一点。 最佳答案 这两种写作方法经常被混淆。首先要说的是,据我所知,没有可衡量的性能差异。(在下面的书面示例中不断查找)最明显的区别,可能也是最著名的,是你的
几个月前,我读了一篇关于rubygem的博客文章,它可以通过阅读代码本身来确定编程语言。对于我的生活,我不记得博客或gem的名称。谷歌搜索“ruby编程语言猜测”及其变体也无济于事。有人碰巧知道相关gem的名称吗? 最佳答案 是这个吗:http://github.com/chrislo/sourceclassifier/tree/master 关于ruby-寻找通过阅读代码确定编程语言的rubygem?,我们在StackOverflow上找到一个类似的问题:
我目前正在使用以下方法获取页面的源代码:Net::HTTP.get(URI.parse(page.url))我还想获取HTTP状态,而无需发出第二个请求。有没有办法用另一种方法做到这一点?我一直在查看文档,但似乎找不到我要找的东西。 最佳答案 在我看来,除非您需要一些真正的低级访问或控制,否则最好使用Ruby的内置Open::URI模块:require'open-uri'io=open('http://www.example.org/')#=>#body=io.read[0,50]#=>"["200","OK"]io.base_ur
我正在使用RubyonRails3.2.2,我想从我的模型/类中“提取”一些方法。也就是说,在不止一个类/模型中,我有一些方法(注意:方法与用户授权相关,并被命名为“CRUD方式”),这些方法实际上是相同的;所以我认为DRY方法是将这些方法放在“共享”模块或类似的东西中。实现该目标的常见且正确的方法是什么?例如,我应该将“共享”代码放在哪里(在哪些目录和文件中)?如何在我的类/模型中包含提到的方法?你有什么建议?注意:我正在寻找“RubyonRails制作东西的方式”。 最佳答案 一种流行的方法是使用ActiveSupport关注点
前言作为一名程序员,自己的本质工作就是做程序开发,那么程序开发的时候最直接的体现就是代码,检验一个程序员技术水平的一个核心环节就是开发时候的代码能力。众所周知,程序开发的水平提升是一个循序渐进的过程,每一位程序员都是从“菜鸟”变成“大神”的,所以程序员在程序开发过程中的代码能力也是根据平时开发中的业务实践来积累和提升的。提高代码能力核心要素程序员要想提高自身代码能力,尤其是新晋程序员的代码能力有很大的提升空间的时候,需要针对性的去提高自己的代码能力。提高代码能力其实有几个比较关键的点,只要把握住这些方面,就能很好的、快速的提高自己的一部分代码能力。1、多去阅读开源项目,如有机会可以亲自参与开源