我们接第一篇
来继续说明在代码 review 中,有哪些属于 "层次结构" 中的坏味道。
注:通过上图咱们看到了在层次结构中有九大问题点,咱们就从中找出三个典型的问题点给与分析和解释。
- public Insets getBorderInsets(Component c, Insets insets) {
- if(c instanceof AbstractButton) {
- margin = ((AbstractButton)c).getMargin();
- } else if(c instanceof JToolBar) {
- margin = ((JToolBar)c).getMargin();
- } else if(c instanceof JTextComponent) {
- margin = ((JTextComponent)c).getMargin();
- }
注:串接的 if else 语句显示的检查类型 AbstractButton,JToolBar 和 JTextCompont 并在各种条件下调用方法 getMargin(),这种造成的情况是将来可能在代码中的其他地方也会出现。
1、如果条件检查中的多个实现调用方法相同,可引入相关的接口来抽象共同的协议。
2、如果代码中包含可转换为类的条件语句,可采用重构手法 "提取层次结构" 来创建一个类层次结构,其中每个类都表示条件检查中的一种情形。
AbstractQueuedSynchronizer 和 AbstractQueuedLongSynchronizer 类都是直接从 AbstractOwnableSynchronizer 派生而来的(这些类都包含在 java.util.concurrent.locks 包),这二个子类的很多代码都是重复的,每个类都包含 2110 行代码,但重复的代码多达 1278 行。
显然,这二个类的代码绝大部分是相同的,只是在 AbstractQueuedLongSynchronizer 中使用的是 long 而不是 int,那么我们看这二个类的继承类图如下:
对于 AbstractOwnableSynchronizer,由于子类型中的方法定义相同,因此可采用重构手法上移,将相同的方法定义移到超类中。
这种层次结构主要体现在,虽然超类和子类之间不存在 is-a 的关系,但是超类的方法对于子类来说是适用或者相关的。
注:java.util.Date 这个类不仅提供了日期功能,如 getDate(),getYeah() 等方法,还提供了 getTime(),getHours() 等时间方法,但是它的二个子类 java.sql.Date 不支持与时间有关的功能,而 java.sql.Time 不支持与日期有关的功能,于是 java.sql.Date 拒绝了从超类继承的所有与时间有关的方法,java.sql.Time 拒绝了继承的所有与日期有关的方法。
看一段简单的代码:
- java.util.Date date = new java.util.Date();
- int dateValue = date.getDate(); //不报错,一切正常
- date = new java.sql.Time(10, 10, 10);
- dateValue = date.getDate(); //将引发IllegalArgumentException异常
超类和子类之间并不存在 is-a 的关系,它们在设计中使用继承只是为了能够利用抽象提供的功能,其实在相关类之间建立关联关系也可以达到这样的目的,采用重构手法 "以委拖取代继承",应用 hash-a 的关系取代 is-a 的关系。
在第二篇中我们重点介绍了关于类层次结构方面的坏味道,那么我们将在第三篇中介绍关于封装类方面的故事。
来源: http://www.bubuko.com/infodetail-1970057.html