大话重构连载16：超级大函数

事情总是这样的：当我们对一个遗留系统一忍再忍，再忍，忍，还要忍……终于积攒到某一天，实在忍无可忍了，拍案而起，不能再忍了，重构！！！事情就这样发生了。然而，在这时你突然发现，重构的工作千头万绪，真不知从何开始。堆积如山的问题此起彼伏，期望修改的设计思绪万千。这里有个想法，那里有个思路，什么都想做，却什么都做不了，真是脑子里一团乱麻。这时候，没有一个合理的步骤，清晰的计划，瞎干蛮干是十分危险的，它会为你的重构带来不可预期的未来。无数次的经验告诉我，不论是什么系统，采用什么架构，从分解大函数开始，肯定没有错。

大函数，就是那些业务逻辑特别复杂、程序代码特别多、一提起就叫人头疼不已的超级方法。超级大函数，很难让人读懂，更难于维护与变更，毫无疑问是软件退化的重灾区。它起初可能并不复杂，也逻辑清晰、易于读懂，但随着业务逻辑的一次次变更，不停地往里面添加代码，再加上一些不合理的设计，经过天长日久，变得越来越臃肿，超级大函数就这样产生了。超级大函数的产生是有它内在的客观原因的。怎么这么说呢？前面我们谈过，软件发展的客观规律就是业务逻辑越来越复杂。随着业务逻辑越来越复杂，正确的办法就是适时地重构和优化我们的代码。但非常遗憾地是，几乎很少有人认识到这一点。这样的结果就是，随着业务逻辑越来越复杂，人们总是就着原有的程序结构不停地往里面添加新的代码。原有的清晰而简单的程序，随着新代码的不断添加，开始变得越来越复杂而难懂了。正因为如此，在大多数软件企业的遗留系统中，超级大函数就变成了一种通病。让我们用HelloWorld为例来演变一番它的历程吧。

如前面第三章所述，最开初的HelloWorld程序是这样的：

 1 /**
 2  * The Refactoring's hello-world program
 3  * @author fangang
 4  */
 5 public class HelloWorld {
 6     /**
 7      * Say hello to everyone
 8      * @param now
 9      * @param user
10      * @return the words what to say
11      */
12     public String sayHello(Date now, String user){
13         //Get current hour of day
14         Calendar calendar = Calendar.getInstance();
15         calendar.setTime(now);
16         int hour = calendar.get(Calendar.HOUR_OF_DAY);
17         
18         //Get the right words to say hello
19         String words = null;
20         if(hour>=6 && hour<12){
21             words = "Good morning!";
22         }else if(hour>=12 && hour<19){
23             words = "Good afternoon!";
24         }else{
25             words = "Good night!";
26         }
27         words = "Hi, "+user+". "+words;
28         return words;
29     }
30 }

 

了了数十行代码，简单明了。随后就开始变更了，首先是关于时间的问候变得复杂了，添加了一些特殊的节日的问题，如新年问候“Happy new year! ”、情人节问候“Happy valentine’s day! ”、三八妇女节问候“Happy women’s day! ”，等等。同时，对一天中的问候也变得更加精细。对于这样的需求，IT攻城狮们敲着键盘就开始改写了：

 1 /**
 2  * The Refactoring's hello-world program
 3  * @author fangang
 4  */
 5 public class HelloWorld {
 6     /**
 7      * Say hello to everyone
 8      * @param now
 9      * @param user
10      * @return the words what to say
11      */
12     public String sayHello(Date now, String user){
13         //Get current month, date and hour.
14         Calendar calendar = Calendar.getInstance();
15         calendar.setTime(now);
16         int hour = calendar.get(Calendar.HOUR_OF_DAY);
17         int month = calendar.get(Calendar.MONTH);
18         int day = calendar.get(Calendar.DAY_OF_MONTH);
19         
20         //Get the right words to say hello
21         String words = null;
22         if(month==1 && day==1){
23             words = "Happy new year!";
24         }else if(month==1 && day==14){
25             words = "Happy valentine's day!";
26         }else if(month==3 && day==8){
27             words = "Happy women's day!";
28         }else if(month==5 && day==1){
29             words = "Happy Labor day!";
30         
31         ……
32         
33         }else if(hour>=6 && hour<12){
34             words = "Good morning!";
35         }else if(hour==12){
36             words = "Good noon!";
37         }else if(hour>=12 && hour<19){
38             words = "Good afternoon!";
39         }else{
40             words = "Good night!";
41         }
42         words = "Hi, "+user+". "+words;
43         return words;
44     }
45 }

 

代码量开始翻倍。接着，客户要求所有的用户信息应当来源于数据库的用户表，同时设计了问候语规则表，所有关于时间的问候都应来源于对该表的查询。这时我们继续膨胀sayHello()这个方法：

 1 /**
 2  * The Refactoring's hello-world program
 3  * @author fangang
 4  */
 5 public class HelloWorld {
 6     /**
 7      * Say hello to everyone
 8      * @param now
 9      * @param user
10      * @return the words what to say
11      */
12     public String sayHello(Date now, long userId){
13         //Get database connection.
14         try {
15             Class.forName("oracle.jdbc.driver.OracleDriver");
16         } catch (ClassNotFoundException e1) {
17             throw new RuntimeException("No found JDBC driver");
18         }
19         String url = "jdbc:oracle:thin:@localhost:1521:helloworld";
20         String username = "test";
21         String password = "testpwd";
22         Connection connection;
23         try {
24             connection = DriverManager.getConnection(url,username,password);
25         } catch (SQLException e1) {
26             throw new RuntimeException("Connect database failed!");
27         }
28         
29         //Get current month, date and hour.
30         Calendar calendar = Calendar.getInstance();
31         calendar.setTime(now);
32         int hour = calendar.get(Calendar.HOUR_OF_DAY);
33         int month = calendar.get(Calendar.MONTH);
34         int day = calendar.get(Calendar.DAY_OF_MONTH);
35         
36         //Get the right words to say hello
37         String words = null;
38         String greetingRuleSql = 
39             "select words, month, day, hourLower, hourUpper from greeting_rules";
40         try {
41             PreparedStatement statement = 
42                 connection.prepareStatement(greetingRuleSql);
43             ResultSet resultSet = statement.executeQuery();
44             while(!resultSet.isLast()){
45                 int monthOfRule = resultSet.getInt("month");
46                 int dayOfRule = resultSet.getInt("day");
47                 if(month==monthOfRule && day==dayOfRule){
48                     words = resultSet.getString("words");
49                     break;
50                 }
51                 int hourLower = resultSet.getInt("hourLower");
52                 int hourUpper = resultSet.getInt("hourUpper");
53                 if(hour>=hourLower && hour<hourUpper){
54                     words = resultSet.getString("words");
55                     break;
56                 }
57             }
58             if(words==null) 
59             throw new RuntimeException("Error when searching greeting rules.");
60         } catch (SQLException e1) {
61             throw new RuntimeException("Error when getting greeting rules.");
62         }
63         
64         //Get user's name
65         String user = "";
66         String userSql = "select name from rms_user where user_id=?";
67         try {
68             PreparedStatement statement = connection.prepareStatement(userSql);
69             statement.setLong(1, userId);
70             ResultSet resultSet = statement.executeQuery();
71             user = resultSet.getString(1);
72         } catch (SQLException e) {
73             throw new RuntimeException("Error when getting user's name.");
74         }
75         
76         words = "Hi, "+user+". "+words;
77         return words;
78     }
79 }

 

这是一个十分简单的示例，但我们可以看到它已经由短短十来行膨胀成了60多行，膨胀了4倍之多。在最后这个版本中，sayHello()既要负责连接数据库、查询数据，又要获得当前时间的月份、日期与小时，还要完成相应的业务逻辑的判断，使程序变得相当复杂。我们可以继续想象，如果继续提出新的需求，比如支持多语言、支持多数据库，程序质量将继续下滑，直到我们无法忍受。

解决超级大函数问题最有效的办法就是分解，按照功能一步一步分解，还原其应有的优化结构。在这个过程中我们常用的重构方法叫“抽取方法（Extract Method）”。重构是一个探索的过程，因为我们总是起初对要重构的系统并不了解，没有设计文档（即使有，对不对还是一说呢），没有熟悉系统的人（哪怕只是在不明白的时候问一问）。你，只是接到任务才开始接手这个系统，你对这个系统的了解简直就是一猫黑。而这时，抽取方法是我们开始这种探索，了解这个系统最有效的工具，它往往是这样进行的：

当我们在阅读一段大函数时，我们可以自觉不自觉地为这些代码分段，为一段功能相对独立的代码编写注释。有时我们可能还需要调整代码的先后顺序，将一些有更多关联的代码放在一起，如将变量的声明与变量真正使用的代码放在一起，或者将有明显前后关系的代码放在一起。这样的调整是一个好的开端，因为它让我们的代码开始变得有序，开始变得可读。

随后，我们就可以使用“抽取方法”了。将被我们分段、加上注释的代码从原函数中抽取出来，放在另外一个独立的函数中，为这个函数取个易懂的名称——这是一个非常重要的好习惯。我常常为了给一个函数取一个正确的名字而思索很长时间，甚至修改好几次。不要认为这是在浪费时间，它也是优化代码重要的一个环节。但起初我们对这段代码的理解可能不那么深，因此我们往往选择用结果变量为其命名。随着我们对这段代码理解的深入，可以运用重构中的“重命名方法（Rename Method）”，根据其代码意图重新为其命名（许多开发工具，如eclipse，都支持该重构方法，它们使你在重命名的同时，同步修改了所有对该方法的调用）。经过这样对代码段的抽取，原代码在这里就变成了对这个新函数的调用。

举一个例子吧，这时一段真实的遗留系统，原有程序是这样写的：

 1     ......
 2     int iCtbz = -1;
 3     ElecObj elecObj = null;
 4     //获取办理时间
 5     int iCzyf = Integer.valueOf(Stream[9]).intValue();
 6     String czMonth = String.valueOf(iCzyf % 20).toString().trim();
 7     String czYear = String.valueOf( (iCzyf - (iCzyf % 20)) / 20 + 2000).
 8                  toString();
 9     if (czMonth.length() == 1){
10       czMonth = "0" + czMonth;
11     }
12     long lBlsj = Long.valueOf(czYear + czMonth).longValue();
13     String dqyear = sysDate.toString().substring(0, 4);
14     ......

 

这段代码写得并不好，有许多需要我们优化的地方。但记住“小步快跑”原则，此时不是解决其它问题的时候，现在我们首先是运用抽取方法优化程序结构。经过抽取以后，将以上加粗的部分改为了这样：

1     ......
2     int iCtbz = -1;
3     ElecObj elecObj = null;
4     //获取办理时间
5     int iCzyf = Integer.valueOf(Stream[9]).intValue();
6     long lBlsj = getBlsj(iCzyf);
7     String dqyear = sysDate.toString().substring(0, 4);
8     ......

 

加粗的部分就是被改写的内容，同时将抽取的内容放进了一个独立的函数：

 1 /**
 2 *@param iCzyf
 3 *@return 获取办理时间
 4 */
 5 private long getBlsj(int iCzyf) {
 6     String czMonth = String.valueOf(iCzyf % 20).toString().trim();
 7     String czYear = String.valueOf( (iCzyf - (iCzyf % 20)) / 20 + 2000).
 8               toString();
 9     if (czMonth.length() == 1){
10          czMonth = "0" + czMonth;
11     }
12     return Long.valueOf(czYear + czMonth).longValue();
13 }

 

在给这个新创建的函数命名时，我们对这段代码的理解并不深刻。在原函数中，这段代码执行的结果是获得了lBlsj这个结果变量，即获得了办理时间。为此，我们先为该函数命名为getBlsj这个函数名。但在后续的工作中，我们逐渐理解到，它不仅可以获取办理时间，还可以获取很多时间。它实质是将时间由一种表示方式转换成另一种表示方式，因此我们运用开发工具的重命名功能，将该函数命名为transformDate，其它调用它的代码也随之修改了。重命名后的函数就不再仅仅运用在此处获取办理时间，还应用在其它业务代码的处理中。

抽取方法可大可小，你可以将一段数百行的代码抽取走，也可能只抽取了数行。但不论怎样，被抽取走的代码一定是功能内聚的，也就是说它们执行的是一个说得清道得明、清楚明确的功能。同时，被抽取走的代码一定是执行的一个清晰的功能，而不是多个。它可大可小，大也许还能分解为多个功能，但至少在逻辑上，这些功能是这个大的功能的组成部分。

抽取方法是一个探索的过程，最关键是那个红线划在哪里，即抽取代码的范围。多一行也对，少一行也对，关键在于我们抽取出来的这个函数，它的功能我们是怎样定义的。公说公有理，婆说婆有理，没有一个定论，所以方法的抽取常常是反反复复。开始我们按照一个思路抽取出来，后来想想觉得不对，因此又放回原函数，重新划分，重新抽取，反复多次。

有一次，我在重构一个Servlet的时候，抽取出来的函数是在执行一大段业务逻辑操作。但在完成了一系列业务操作以后，原程序要将返回值转换成二进制代码写入response中，返回前端。起初，我将整个这一段都抽取出来。但令人很别扭的是，传参的时候必须要把response传进去，这使得业务逻辑与Web应用环境耦合，不利于日后的优化，编写自动化测试。随后我还原了这段代码，重新进行抽取，将写response的部分留在了原函数中，而将红线画在了完成业务逻辑操作之后，写入response之前。新重构的函数得以与web应用解耦，为后面的进一步优化做好了准备。重构的过程，是考验开发人员能力的过程，需要大家反复练习与钻研。

另外，抽取方法就像核裂变，开始由一个函数裂变为几个函数。分解出来的函数又裂变为另外几个函数，不断这样往复下去。同时，抽取方法总是在一个类中发生裂变。而当这个类分解出来的方法达到一定程度以后，随之而来的就是类的裂变，由一个类分解成多个类，分解出来的类再分解……类的分解我们采用的是另一个方法——“抽取类（Extract Class）”，我们将在后面讲述。

重构是一系列的等量变换，抽取方法是这些等量变换中最典型的例子。将一段代码从原函数中抽取出来，代码依然是那些代码，只是程序结构发生了变换。正因为如此，才能保证我们的重构过程的安全可靠。

大话重构连载首页：http://www.cnblogs.com/mooodo/p/talkAboutRefactoringHome.html

特别说明：希望网友们在转载本文时，应当注明作者或出处，以示对作者的尊重，谢谢！

posted @ 2014-10-18 22:05  充满诗意的联盟  阅读(1128)  评论(2)  编辑  收藏  举报