重构-改善既有代码的设计(十）：简化函数调用

1、函数改名（Rename Method）

（1）症状：函数胡名称未能揭示函数的用途

（2）解决：修改函数的名称

（3）函数的名称应该准确表达它的用途，可以先考虑给这个函数写上一句注释，然后想办法把注释变成函数名称

public String getTelephoneNumber() {  
    return ("(" + _officeAreaCode + ") " + _officeNumber);  
} 
----------------------------------------------------------------
重构（简化函数调用）之后的代码
把这个函数改名为getOfficeTelepnoneNumber()。
首先建立一个新函数，命名为getOfficeTelephoneNumber()，
并将原函数getTelephone- Number()的代码复制过来。
然后，让旧函数直接调用新函数：
class Person...  
  public String getTelephoneNumber(){  
      return getOfficeTelephoneNumber();  
  }  
  public String getOfficeTelephoneNumber() {  
      return ("(" + _officeAreaCode + ") " + _officeNumber);  
  } 

2、添加参数（Add Parameter）

（1）症状：某个函数需要从调用端得到更多信息

（2）解决：为此函数添加一个对象参数，让该对象带进函数所需信息

• 检查函数签名是否被超类或者子类实现，如果是，则需要针对每份实现分别进行下列步骤
• 声明一个新函数，名称与原函数相同，只是加上新添参数，将旧函数代码复制到新函数中
• 编译
• 修改旧函数，令它调用新函数
• 编译，测试
• 找出旧函数所有的引用点，将它们全部修改为对新函数的引用。每次修改后，编译并测试
• 删除旧函数，或者deprecated
• 编译，测试

class Customer{
    getContact(){
        // do something
    }
}
--------------------------------------------------------------
重构（添加参数）之后的代码
class Customer{
    getContact(Date d){
        // do something
    }
}

3、移除参数（Remove parameter）

（1）症状：函数本体不再需要某个参数

（2）解决：将该参数去除

（3）对于多态函数，可能多态函数的另一份（或多份）实现会使用这个参数，就不能去除它

• 检查函数签名是否被超类或者子类实现，如果是，则需要针对每份实现分别进行下列步骤
• 声明一个新函数，名称与原函数相同，只是去除不必要的参数。将旧函数代码复制到新函数中
• 编译
• 修改旧函数，令它调用新函数
• 编译，测试
• 找出旧函数所有的引用点，将它们全部修改为对新函数的引用。每次修改后，编译并测试删除旧函数，或者deprecated
• 编译，测试

 

4、将查询函数和修改函数分离（Separate Query from Modifier）

（1）症状：某个函数既返回对象状态值，又修改对象状态

（2）解决：建立两个不同的函数，其中一个负责查询，另一个负责修改

（3）明确“有副作用”与“无副作用”两种函数之间的差异

（4）如果遇到一个“既有返回值又有副作用”的函数，就应该试着将查询动作从修改动作中分割出来

一旦有人入侵安全系统，它会告诉我入侵者的名字，并发送一个警报。
如果入侵者不止一个，也只发送一条警报
String foundMiscreant(String[] people) {  
        for (int i = 0; i < people.length; i++) {  
            if (people[i].equals("Don")) {  
                sendAlert();  
                return "Don";  
            }  
            if (people[i].equals("John")) {  
                sendAlert();  
                return "John";  
            }  
        }  
        return "";  
    }  
    
该函数被下列代码调用：  
void checkSecurity(String[] people) {  
    String found = foundMiscreant(people);      
    someLaterCode(found);  
}  
--------------------------------------------------------------
重构（将查询函数和修改函数分离）之后的代码
（1）首先建立一个适当的查询函数，使其与修改函数返回相同的值，但不造成任何副作用：
String foundPerson(String[] people) {  
    for (int i = 0; i < people.length; i++) {  
        if (people[i].equals("Don")) {  
            return "Don";  
        }  
        if (people[i]