系列博客
1. 改善代码设计 —— 优化函数的构成(Composing Methods)
2. 改善代码设计 —— 优化物件之间的特性(Moving Features Between Objects)
3. 改善代码设计 —— 组织好你的数据(Composing Data)
4. 改善代码设计 —— 简化条件表达式(Simplifying Conditional Expressions)
5. 改善代码设计 —— 简化函数调用(Making Method Calls Simpler)
6. 改善代码设计 —— 处理概括关系(Dealing with Generalization)
1. Parameterize Method (令函数携带参数) 解释:
"令函数携带参数" 并不是简单的让你在函数里加上参数, 如果函数里需要某个参数, 我们谁都会加上它. 你可能发现这样的几个函数: 它们做着类似的事情, 只是因为极少的几个值导致函数的策略不同, 这时可以使用 Parameterize Method 消除函数中那些重复的代码了, 而且可以用这个参数处理其它更多变化的情况.
下面有一个非常简单的例子.
冲动前:public double FivePercentRaise()
{
_salary *= 1.05;
return _salary;
}
public double TenPercentRaise()
{
_salary *= 1.10;
return _salary;
} 冲动后:public double Raise(int percent)
{
_salary *= (1 + percent / 100);
return _salary;
} 2. Replace Parameter with Explicit Methods (用明确的函数取代参数) 解释:
Replace Parameter with Explicit Methods 与 Parameterize Method 是相反的操作. 如果函数内根据不同的参数作出了不同的行为, 那么可以考虑使用明确的函数取代参数. 不过这一条很少用到, 很少出现类似下面例子的垃圾代码, 建议使用 C# 里的属性, 即使是"冲动后"的代码, 在实际中也不是很多见.
冲动前:public void Set(string name, int value)
{
if(name.Equals("height"))
{
_height = value;
return;
}
if(name.Equals("width"))
{
_width = value;
return;
}
} 冲动后:public void SetHeight(int height)
{
_height = height;
}
public void SetWidth(int width)
{
_width = width;
} 3. Preserve Whole Object (保持对象完整) 解释:
你从某个物件中取出若干值, 将这些值作为参数传递给某个函数, 为何不考虑下把整个物件作为传递进去呢? 当然具体情况具体分析, 有时你确实需要按你原来做的那样做.
冲动前:int height = rectangle.Height;
int width = rectangle.Width;
int area = CalculateArea(height, width);
冲动后:
int area = CalculateArea(rectangle); 4. Replace Parameter with Methods (以函数取代参数) 解释:
如果函数的某个参数值可以通过方法直接获得, 这种情况下, 很多时候函数不应该通过参数来获取该值, 应该在函数里直接通过方法获取需要的值.
冲动前:double basePrice = basePrice * num;
double discount = GetDiscount();
double charge = DiscountPrice(basePrice, discount); 冲动后:double basePrice = basePrice * num;
double charge = DiscountPrice(basePrice); 5. Introduce Parameter Object (引入参数对象) 解释:
你是否发现有几个参数总是同时的出现在一些函数的参数列表里? 这几个参数有一个响亮的臭名 —— 数据泥团(Data Clump), Introduce Parameter Object 让你将数据泥团封装成一个对象, 从而在原先的函数直接传入整个对象即可, 以后还可以对这个对象进行扩展.
我不主张对于所有的数据泥团都这么做, 哪怕这个数据泥团出现过几十次上百次, 如果数据泥团的各个数据直接的联系并不是那么紧密, 它们不能用一个物件笼统的包含它们, 这样的情况下"引入参数对象"可能并不是很适合.
6. Remove Setting Method (移除设值函数) 解释:
这一条没什么好说的, 如果你的类中某个值域只应该在对象初始化时被设值, 那么类中就不应该再有关于这个值域的设值函数, 应该通通把它们删掉.
7. Hide Method (隐藏某个函数) 解释:
这一条也没什么好说的, 如果你的类中某个函数只可能用于类本身的函数调用, 其它类从来没有用到过, 最好将这个函数的访问权限设置为 private. 当然这一步要谨慎, 现在没有被其它类调用过, 不代表以后不会!
8. Replace Constructor with Factory Method (用工厂函数取代构造函数) 解释:
"以工厂函数取代构造函数" 让我想起了 "以多态取代条件", 如果你在创建对象的时候不是仅仅对它做简单的构造, 例如你构造一个 Employee 对象, Employee 可能是工程师, 销售员, 也有可能是设计师, 应该根据不同的类别的职工创建 Employee 对象.
9. Encapsulate Downcast (封装向下转型动作) 解释:
要找到需要进行 Encapsulate Downcast 重构的函数并不困难, 如果你的函数返回的是 object 类型, 在别的地方调用这个函数获取一个 object 之后还要进行一下转型(如将这个 object 转成 int 类型), 你应该在函数里封装一下转型的操作.
冲动前:public object LastPerson()
{
return _persons.LastElement();
}
冲动后:
public Person LastPerson()
{
return (Person)_persons.LastElement();
} 10. Replace Error Code with Exception (用异常取代错误码) 解释:
你的函数返回的是一个 int 型, 你约定返回 1 代表这个函数运行正常, 返回 –1 代表某个地方出错了, 错误码就是指这里的 "1" 和 "-1". 你想想你的函数返回的是错误码, 在调用这个函数的时候极有可能可能还需要判断你返回的是什么错误码, 如果你收到的是 –1 就停止程序的运行, 书中形容这样的做法很有趣: 就好像因为错过一班飞机而自杀一样, 如果真那么做, 哪怕我是猫, 我的九条命 (注: 猫生命力比较强) 也早就赔光了.
别忘了 C# 里的异常处理, 这是一个很重要的概念, 是时候使用它们了.
11. Replace Exception with Test (用测试取代异常) 解释:
"异常处理" 是一个很不错的功能, 但不要滥用, 对于肯定不会出现异常的代码块就不要将它们放入 try...catch 中.
对于在一定条件下代码会抛出异常的某些情况, 建议使用 Replace Exception with Test, 你可以在可能抛出异常的代码前先测试一下运行正常的条件是否满足, 满足之后再运行, 不满足的话则是另一种运行方案, 通过这样可以代替 try...catch 的使用.
冲动前:try
{
return _persons[index];
}
catch (IndexOutOfRangeException e)
{
//...
}
//... 冲动后:if (index < _persons.Length)
return _persons[index];
//...