Large Class--过大的类--要重构的信号

如果想利用单个类做太多事情,其内往往就会出现太多实例变量。一旦如此,Duplicated Code也就接踵而至。
 
 
解决方法:
 
 
1.将类内彼此相关的变量,将它们放在一起。使用Extract Class手法,将彼此相关的变量提炼到新的类。
 
 
2.如果1中的新类适合作为一个子类,那么可以使用Extract Subclass手法。
 
11.对于太多代码的处理办法,分解函数,将大函数分解成若干小函数,这样可以消除重复代码。相关的函数,可以跟着变量,一起被提炼到一个新的类中去,使用Extract Class手法或者Extract Subclass。
 
12.实现11时,要确定客户端如何使用提炼的类,然后,使用Extract Interface手法,为每种使用方式提炼出一个接口。
 
111.如果当前过大的类是GUI类,那么,新提炼的类和当前类,两边要各保留一些重复数据,并保持同步。这要使用Duplicate Observed Data手法。
 
具体处理手法介绍:
 
 
1. Extract Subclass
类中的某些特性只被某些(而非全部)实例用到。
 
出现这种特征,可以创建一个子类,该子类只使用这部分特性。这样,就把子类专有的特性从原来的类中
提炼出来,从而让原来的类变得不那么大。
 
重点是要识别出这种情况的存在,比如:Job Item有一种使用情况,只是使用getUnitPrice,getEmployee,
不使用Job Item的其它特性。那么,就可以把使用getUnitPrice,getEmployee提炼为一个类,在这里使用
Extract Subclass手法。
 
再比如:
1: public class Registration
   2: {
   3:     public NonRegistrationAction Action { get; set; }
   4:     public decimal RegistrationTotal { get; set; }
   5:     public string Notes { get; set; }
   6:     public string Description { get; set; }
   7:     public DateTime RegistrationDate { get; set; }
   8: }
 
 
这个类,有两个使用场景,一个是注册的;一个是非注册的。作为注册实例的时候,它只使用到RegistrationDate,Description,RegistrationTotal方法,没有使用其它方法;
而作为非注册实例来使用的时候,只是使用NonRegistrationAction,Notes方法。
 
符合特征,类中的某些特性只被某些(而非全部)实例用到。这里如果使用Extract Subclass手法, 那么,
处理后,结果是这样:
   1: public class Registration
   2: {
   3:     public decimal RegistrationTotal { get; set; }
   4:     public string Description { get; set; }
   5:     public DateTime RegistrationDate { get; set; }
   6: }
   7:  
   8: public class NonRegistration : Registration
   9: {
  10:     public NonRegistrationAction Action { get; set; }
  11:     public string Notes { get; set; }
  12: }
 
2.Extract Interface
 
“使用一个类”的含义解读,下述情况之一:
1).使用该类的所有责任区。
2).某一组客户只使用类责任区中的一个特定子集。
3).该类需要与所有协助处理某些特定请求的类协作。
 
对于2),3)的情况,将一个类中的这部分责任分离出来,这样有意义。因为,
这样可以更容易看清楚类的责任划分。
 
所以,Extract Interface手法做的事情,就是把原来类中的部分子集,部分责任分离出来。
 
例子:
Employee类,提供了很多功能,但是,有部分类,只使用它的getRate()和hasSpecialSkill()功能。
那么,为了让Employee类变小,可以把它的这两个功能,getRate()和hasSpecialSkill()提炼出来。提炼为接口
interface Billable{
 
  public int getRate();
  public boolean hasSpecialSkill();
}
 
然后Employee类实现接口Billable。
 
这样,那些只需要使用到Employee类getRate(),hasSpecialSkill()功能的类,只需要使用接口Billable就可以,这样,只是暴露了Billable功能给使用者。也就是,只给使用者提供它们可以使用的功能,其它功能子集无需提供。
 
多个使用者,只关注getRate(),hasSpecialSkill()功能,不同的使用者,会要求不同的实现。这时,就可以进行不同的实现来满足使用者的要求。
3.Duplicate Observed Data(复制“被监视的数据”)
场景:
一个GUI类,包含了业务处理代码和用户界面代码。当业务逻辑的增加, 会让这个类越来越大,而且逻辑会复杂。这是因为,一个GUI类同时承担了两种责任,业务处理功能和界面渲染功能。这时候,就需要把业务处理代码从GUI类中分离出来。
 
业务处理代码和界面显示代码分离开之后,方便维护和方便开发。在进行业务处理时,需要使用到界面数据,而界面进行更新的时候,也会需要使用到业务处理的结果。这时,就需要数据在业务层和界面层之间进行同步。
 
而这个手法,可以实现上述同能,业务层(领域层)和界面层的数据同步。
 
 
例子:
下面的内容都是IntervalWindow类:
 
  public class IntervalWindow extends Frame...
    java.awt.TextField _startField;
    java.awt.TextField _endField;
    java.awt.TextField _lengthField;
 
    class SymFocus extends java.awt.event.FocusAdapter
    {
        public void focusLost(java.awt.event.FocusEvent event)
        {
            Object object = event.getSource();
            if (object == _startField)
                StartField_FocusLost(event);
            else if (object == _endField)
                EndField_FocusLost(event);
            else if (object == _lengthField)
                LengthField_FocusLost(event);
        }
    }


一个GUI界面,有三个文本框,当某个文本框输入内容,离开时,也就是失去焦点的时候,会触发一个计算。
Start文本框失去焦点的时候,执行StartField_FocustLost方法;
End文本框失去焦点的时候,执行EndField_FocusLost方法;
Length文本框失去焦点的时候,执行LengthField_FocusLost方法。
 
Start文本框和End文本框,失去焦点时,做的事情是,检查文本框的输入是否是整数,如果不是,则将文本框的内容设置为0;然后,计算Length文本框的内容。
 
Length文本框失去焦点的时候,做的事情是,检查文本框的输入是否是整数,如果不是,则将文本框的内容设置为0;然后,计算当前文本框的值。
这是界面操作代码。
 
    void StartField_FocusLost(java.awt.event.FocusEvent event) {
        if (isNotInteger(_startField.getText()))
            _startField.setText("0");
        calculateLength();
    }
 
    void EndField_FocusLost(java.awt.event.FocusEvent event) {
        if (isNotInteger(_endField.getText()))
            _endField.setText("0");
        calculateLength();
    }
 
    void LengthField_FocusLost(java.awt.event.FocusEvent event) {
        if (isNotInteger(_lengthField.getText()))
            _lengthField.setText("0");
        calculateEnd();
    }
void calculateLength(){
      try {
        int start = Integer.parseInt(_startField.getText());
        int end = Integer.parseInt(_endField.getText());
        int length = end - start;
        _lengthField.setText(String.valueOf(length));
      } catch (NumberFormatException e) {
        throw new RuntimeException ("Unexpected Number Format Error");
      }
}
void calculateEnd() {
    try {
      int start = Integer.parseInt(_startField.getText());
      int length = Integer.parseInt(_lengthField.getText());
      int end = start + length;
      _endField.setText(String.valueOf(end));
    } catch (NumberFormatException e) {
      throw new RuntimeException ("Unexpected Number Format Error");
    }
}
calculateLength()方法和calculateEnd()方法,它包含了计算的逻辑,还包含了对界面组件的引用。我们现在想做的事情,就是,让这两个方法与界面组件(_startField,_endField,_lengthField)解耦。
 
这里就可以使用到Duplicate Observed Data手法。创建一个领域类,领域类和界面类的数据同步;领域类的数据发生更新后,会将结果传递给界面类。
 
-------------------------------------------------------------------------------------------------------
应用Duplicate Observed Data手法之后,得到如下代码:
 
整个流程就变成这样:
1.GUI类,IntervalWindow初始化时,使用的是Interval这个类的值。
2.当GUI类发生事件变化的时候,把组件的值传递给_subject实例,并进行计算。
3._subject计算完毕,再通知GUI类界面进行更新。
 
在这里,计算的逻辑,被分割到_subject实例(领域类)。
数据的传递流程:
 
领域类,界面初始值数据------>{界面生成(GUI类)------>领域类------>界面}------>{界面生成(GUI类)--->领域类--->界面}
 
 
 
 
public class IntervalWindow extends Frame implements Observer{
 
 private Interval _subject;
 
  public IntervalWindow(){
 
    _subject = new Interval();
    _subject.addObserver(this);
    update(_subject, null);
    
     }
 
  String getEnd() {
        return _subject.getEnd();
    }
 
    void setEnd (String arg) {
        _subject.setEnd(arg);
    }
 
 
 
 
 
 
   void StartField_FocusLost(java.awt.event.FocusEvent event) {
        if (isNotInteger(_startField.getText()))
            _startField.setText("0");
          
        _subject.setStart(_startField.getText());
        _subject.calculateLength();
    }
 
    void EndField_FocusLost(java.awt.event.FocusEvent event) {
        if (isNotInteger(_endField.getText()))
            _endField.setText("0");
 
        _subject.setEnd (_endField.getText());
        _subject.calculateLength();
    }
 
    void LengthField_FocusLost(java.awt.event.FocusEvent event) {
        if (isNotInteger(_lengthField.getText()))
            _lengthField.setText("0");
 
       _subject.setLength(_lengthField.getText());
        _subject.calculateEnd();
    }
 
 
 
   @Override
   public void update(Observable observed, Object arg) {
    
       _endField.setText(_subject.getEnd());
       _startField.setText(_subject.getStart());
       _lengthField.setText(_subject.getLength());
   }
}
 
 
-------------------------------
 
 class Interval extends Observable {
 
    private String _end = "0";
    private String _start = "0";
    private String _length ="0";
 
 
    String getEnd() {
        return _end;
    }
    void setEnd (String arg) {
        _end = arg;
        setChanged();
        notifyObservers();
    }
 
 
    String getStart(){
   
       return _start;
    }
   void setStart(String arg){
        
         _start = arg;
         setChanged();
         notifyObservers();
    }
 
 
   String getLength(){
   
       return _length;
    }
   void setLength(String arg){
        
         _length = arg;
         setChanged();
         notifyObservers();
    }
 
      void calculateLength(){
      try {
        int start = Integer.parseInt(getStart());
        int end = Integer.parseInt(getEnd());
        int length = end - start;
        setLength(String.valueOf(length));
      } catch (NumberFormatException e) {
        throw new RuntimeException ("Unexpected Number Format Error");
      }
    }
 
    void calculateEnd() {
      try {
        int start = Integer.parseInt(getStart());
        int length = Integer.parseInt(getLength());
        int end = start + length;
        setEnd(String.valueOf(end));
      } catch (NumberFormatException e) {
        throw new RuntimeException ("Unexpected Number Format Error");
      }
  }
 

}
 
参考资料:
https://sourcemaking.com/refactoring/duplicate-observed-data
https://lostechies.com/seanchambers/2009/08/20/refactoring-day-20-extract-subclass/
http://www.refactoring.com/catalog/duplicateObservedData.html
http://www.refactoring.com/catalog/extractSubclass.html
原文地址:https://www.cnblogs.com/ttylinux/p/4567940.html