八月份的第三个周日,写的第四篇文章,坚持写下去一定会有收获,正所谓,万事开头难、中间难、结尾难。。。

八月份的第三个周日,写的第四篇文章,坚持写下去一定会有收获,正所谓,万事开头难、中间难、结尾难。。。

okay,本周在实际工作中遇到了蛮多的新知识,鉴于每日工作繁忙,周日还要加班,因此只有周六一天可以‘忙中作乐’,不过这个周六依然过得十分‘充足’,上午陪同事找房子忙到下午2点,下午三点去学习游泳学到六点钟。此外,本周抽时间重新布局了我的卧室,安装了一张写字桌,组装一台电脑,现在终于可以在周日的晚上安安静静的写点东西。本周决定翻译一篇dzone.com上面的一篇文章练习一下,下周决定写一篇介绍 Android Notification 的文章,巩固学习。

 阅读本文大概需要 4.5 分钟

 

正文

这些违反设计原则和质量的代码结构简直就是臭虫。 

下面我们就列举一些常见的臭虫代码。

常量接口(Constant Interface)

常量接口只包含了静态常量数据成员,没有声明任何的方法在里面。建议的重构方式要根据常量接口里面常亮的种类:常量是类里面的成员可以被添加或者是可以被写入的枚举。接口 java.io.ObjectStreamConstant 就是一个很好的例子。

全局变量类(Global Variable Class)

这种类拥有一个公共静态成员(非final),由于这种变量可以被任何人修改,这就变成了类似于 C 语言里面的全局变量,不同的是,在使用前要加上类的名字来防止变量名冲突。下面是一个例子:

class Balls {
	public static long balls = 0;
}

全局方法类 (Global Function Class)

这是一种只有一个静态公共方法没有任何其他的成员或者方法的public修饰的类;他有一个可选的用private属性修饰的构造方法来禁止实例化。这相当于C语言里面的全局函数,除了在使用他时需要在前面加上类的名字。

公共裸露的成员 (Publicly Exposed Fields)

这是一种public修饰的,他的成员都是public修饰的,非final,非static,并且没有任何方法在里面(可能会有一个可选的构造方法),这是难以保持的public修饰、类似C的一些类,正如  Effective Java 中所写道的:“一些java平台库里面的类,忽视public类不应该直接暴露成员的建议。最突出的例子就是 java.awt 包里面的 Point 类和 Dimension 类,这些栗子不是简单的来说说,而是应当被视作严重警示。

孤立的抽象(Orphan Abstract)

这是一个抽象abstract 修饰的没有任何实现类的类,一个抽象类,只有当具体实现的时候,他才能获得‘人生价值’。如果一个抽象类没有任何用处,那么还留你何用(可以移除了)?如果这个类可以考虑实例化,那么就去具体实现;如果这代表一种有用的抽象,就提供若干具体实现类去实现他。

被遗忘的接口 (Forgotten Interface)

一个类用同样的签名实现了所有接口列举的方法。这很有可能是一种失误:类原本是想要实现一个接口,但是忘记列举接口作为他的基本类型。这种臭虫做法的后果就是:该类的对象不能被视作接口的子类,于是,子类型的好处和运行多态都不能被利用。

双胞胎类 (Clone Class)

一个类完完全全克隆另外一个类,不是继承关系。实质上,只是类的名字不太一样,但是他们的全部成员、签名其他等等都是一样的。约束他们的仅仅是至少有一个不同的成员。一个例外就是那个成员顺序可能不太一样。 如果有若干个类拥有共同的成员或者表现,他们应该继承一个共同的类,包含那些共同的数据和表现。因此, 一个解决方法就是看看有没有可能提供一个基类,并且让其他类成为他的派生类 ,克隆类常常发生在复制粘贴代码,在这种情况下,最好就是移除副本,用一个独一无二的类取代。

孤独的类 (Lonely Class)

这种类不依赖于或使用任何其他类,并且,任何其他的类也不依赖或者使用这个类。这也许并不是一个错误或者失误,但是,然而,很少能见到一个非常独立的类和其他的类没有任何的联系。因此,我相信这应该是一个设计错误。

抽象类在不重要的位置(Abstract Leaf)

(这个实在不知道怎么翻译)抽象类在继承树的根部时一般都有很大的意义和用处,但是在另外一种情况下,当抽象类在继承树的叶子时:通常代表无意义可以被淘汰的;通常都意味着设计失误。

标记类 (Tagged Class)

这种情况通常发生在那些出身结构编程背景的程序员写的程序。他们通常像定义结构体一样定义类,而不是提供继承相关类型和多态。他们通常用一系列枚举和switch case 语句,或者是一串长长的 is else 语句 来区分类型然后做相应的操作。 术语 ‘tag class’ 来自于  Effective Java :“这种类的实例有两个或者更多的‘口味’ 并且有一个标签域指明这种实例的‘口味’”。 通过以为几点来检查是不是  ‘tag class’ :  * 一个枚举(或者很多public 修饰的 static final 数据)来指出实例的flavour。  * 一个域来保存 enum/int 的值,典型的,通常在构造函数中类设置他。  * 一个switch语句在一个或者更多的方法里,基于tag来决定执行的代码。 很明显的,重构 tag class 的方法就是继承。

小结

根据你的经验,你认出上面的那些代码了吗?欢迎补充更多的类似常见的代码。 有很多的静态分析工具(FindBugs, PMD, etc) 可以用来分析检查java里面的bug模式,然而,并没有很多的工具用于专门分析这样的臭代码。。希望未来能很快出现这样的分析工具。

作者有话说

还说啥。。。已经零点了,明天还得早起搬砖,晚安。 附上原文链接