改善代码设计 —— 优化函数的构成

改善代码设计 —— 优化函数的构成(Composing Methods)

1. Extract Method (提炼函数)

解释:

      如果发现一个函数的代码很长, 很可能的一种情况是这个函数做了很多事情, 找找看函数中有没有注释, 往往注释都是为了解释下面一块代码做的什么事情, 可以考虑将这块代码提炼(Extract)成一个独立的函数.

      这样做的好处不言而喻, 是面向对象五大基本原则中的单一职责原则 (Single Responsibility Principle), 比较长的函数被拆分成一个个小函数, 将有利于代码被复用.

冲动前:

00public void Print(Employee employee)
01{
02    //print employee's information
03    Console.WriteLine("Name:" + employee.Name);
04    Console.WriteLine("Sex:" + employee.Sex);
05    Console.WriteLine("Age:" + employee.Age);
06 
07    //print employee's salary
08    Console.WriteLine("Salary:" + employee.Salary);
09    Console.WriteLine("Bonus:" + employee.Bonus);
10}

冲动后:

00public void Print(Employee employee)
01{
02    //print employee's information
03    PrintInfo(employee);
04 
05    //print employee's salary
06    PrintSalary(employee);
07}
08 
09public void PrintInfo(Employee employee)
10{
11    Console.WriteLine("Name:" + employee.Name);
12    Console.WriteLine("Sex:" + employee.Sex);
13    Console.WriteLine("Age:" + employee.Age);
14}
15public void PrintSalary(Employee employee)
16{
17    Console.WriteLine("Salary:" + employee.Salary);
18    Console.WriteLine("Bonus:" + employee.Bonus);
19}

2. Inline Method (将函数内联)

解释:

      有些函数很短, 只有一两行, 而且代码的意图也非常明显, 这时可以考虑将这个函数干掉, 直接使用函数中的代码.

      物件中过多的方法会让人感到不舒服, 干掉完全不必要的函数后代码会更简洁.

冲动前:

0public bool IsDeserving(int score)
1{
2    return IsScoreMoreThanSixty(score);
3}
4 
5public bool IsScoreMoreThanSixty(int score)
6{
7    return (score > 60);
8}

冲动后:

0public bool IsDeserving(int score)
1{
2    return (score > 60) ;
3}

3. Inline Temp (将临时变量内联)

解释:

      如果有一个临时变量 (Temp)用来表示某个函数的返回值, 一般来说, 这样的做法挺好的. 但如果这个临时变量实在多余, 将这个临时变量内联之后毫不影响代码的阅读, 甚至这个临时变量妨碍了其它重构工作, 就应该将这个临时变量内联化.

      把这个临时变量干掉的好处在于减少了函数的长度, 有时可以让其它重构工作更顺利的进行.

冲动前:

0int salary = employee.Salary;
1return (salary > 10000);

冲动后:

0return (employee.Salary > 10000);

4. Replace Temp With Query (用查询式代替临时变量)

解释:

      程序中有一个临时变量(Temp)用来保存某个表达式的计算结果, 将这个计算表达式提炼(Extract)到一个独立的函数(即查询式Query)中, 将这个临时变量所有被调用的地方换成对新函数(Query)的调用, 新函数还可以被其它函数使用.

      好处在于减少函数长度, 增加代码复用率, 有利于代码进一步的重构. 并且注意 Replace Temp With Query 往往是 Extract Method 之前必不可少的步骤, 因为局部变量会使代码不太容易被提炼, 所以在进行类似的重构前可以将它们替换成查询式.

      下面的这个例子不是很有必要使用Replace Temp With Query, 主要展示如何 Replace Temp With Query. 试想"冲动前"函数中有很多个代码块都使用到totalPrice, 突然有一天我发现这个函数太长, 我需要将这一块块的代码提炼成单独的函数, 这样就需要将 totalPrice = price * num; 放到每一个提炼出来的函数中. 而如果原来函数中使用的是查询式, 就不存在这个问题. 如果查询式中的计算量很大, 也不建议使用 Replace Temp With Query.

冲动前:

0public double FinalPrice(double price, int num)
1{
2    double totalPrice = price * num;
3    if (totalPrice > 100)
4        return totalPrice * 0.8;
5    else
6        return totalPrice * 0.9;
7}

冲动后:

00public double FinalPrice(double price, int num)
01{
02    if (TotalPrice(price, num) > 100)
03        return TotalPrice(price, num) * 0.8;
04    else
05        return TotalPrice(price, num) * 0.9;
06}
07public double TotalPrice(double price, int num)
08{
09    return price * num;
10}

5. Introduce Explaining Variable (引入可以理解的变量)

解释:

      很多时候在条件逻辑表达式中, 很多条件令人难以理解它的意义, 为什么要满足这个条件? 不清楚. 可以使用Introduce Explaining Variable将每个条件子句提炼出来, 分别用一个恰当的临时变量名表示条件子句的意义.

      好处在于增加了程序的可读性.

冲动前:

0if((operateSystem.Contains("Windows"))&&
1    (browser.Contatins("IE")))
2{
3    //do something
4}

冲动后:

0bool isWindowsOS = operateSystem.Contains("Windows");
1bool isIEBrowser = browser.Contatins("IE");
2if (isWindowsOS && isIEBrowser)
3{
4    //do something
5}

6. Split Temporary Variable (撇清临时变量)

解释:

      例如代码中有个临时变量在函数上面某处表示长方形周长, 在函数下面被赋予面积, 也就是这个临时变量被赋值超过一次, 且表示的不是同一种量. 应该针对每次赋值, 分配一个独立的临时变量.

      一个变量只应表示一种量, 否则会令代码阅读者感到迷惑.

冲动前:

0double temp = (width + height) * 2;
1//do something
2temp = width * height;
3//do something

冲动后:

0double perimeter = (width + height) * 2;
1//do something
2double area = width * height;
3//do something

7. Remove Assignments to Parameters (消除对参数的赋值操作)

解释:

      传入参数分"传值"和"传址"两种, 如果是"传址", 在函数中改变参数的值无可厚非, 因为我们就是想改变原来的值. 但如果是"传值", 在代码中为参数赋值, 就会令人产生疑惑. 所以在函数中应该用一个临时变量代替这个参数, 然后对这个临时变量进行其它赋值操作.

冲动前:

0public double FinalPrice(double price, int num)
1{
2    price = price * num;
3    //other calculation with price
4    return price;
5}

冲动后:

0public double FinalPrice(double price, int num)
1{
2    double finalPrice = price * num;
3    //other calculation with finalPrice
4    return finalPrice;
5}

8. Replace Method with Method Object (用函数物件代替函数)

解释:

      冲动的写下一行行代码后, 突然发现这个函数变得非常大, 而且由于这个函数包含了很多局部变量, 使得无法使用 Extract Method, 这时 Replace Method with Method Object 就起到了杀手锏的效果. 做法是将这个函数放入一个单独的物件中, 函数中的临时变量就变成了这个物件里的值域 (field).

冲动前:

00class Bill
01{
02    public double FinalPrice()
03    {
04        double primaryPrice;
05        double secondaryPrice;
06        double teriaryPrice;
07        //long computation
08        ...
09    }
10}

冲动后:

00class Bill
01{
02    public double FinalPrice()
03    {
04        return new PriceCalculator(this).compute();
05    }
06}
07class PriceCalculator
08{
09    double primaryPrice;
10    double secondaryPrice;
11    double teriaryPrice;
12    public PriceCalculator(Bill bill)
13    {
14        //initial
15    }
16    public double compute()
17    {
18        //computation
19    }
20}

9. Substitute Algorithm (替换算法)

解释:

      有这么一个笑话:

      某跨国日化公司, 肥皂生产线存在包装时可能漏包肥皂的问题, 肯定不能把空的肥皂盒卖给顾客, 于是该公司总裁命令组成了以博士牵头的专家组对这个问题进行攻关, 该研发团队使用了世界上最高精尖的技术 (如红外探测, 激光照射等), 在花费了大量美金和半年的时间后终于完成了肥皂盒检测系统, 探测到空的肥皂盒以后, 机械手会将空盒推出去. 这一办法将肥皂盒空填率有效降低至5%以内, 问题基本解决.

      而某乡镇肥皂企业也遇到类似问题, 老板命令初中毕业的流水线工头想办法解决之, 经过半天的思考, 该工头拿了一台电扇到生产线的末端对着传送带猛吹, 那些没有装填肥皂的肥皂盒由于重量轻就都被风吹下去了...

      这个笑话可以很好的解释 Substitute Algorithm, 对于函数中复杂的算法, 尽量想办法将这个算法简单化, 从而达到与之前同样甚至更好的效果.

posted on 2011-05-13 23:50  滴d  阅读(272)  评论(0)  编辑  收藏  举报

导航