|
|
本文介绍的JAVA规则的说明分为5个级别，级别1是最基本也是最重要的级别，在今后将陆续写出其他的规则。遵守了这些规则可以提高程序的效率、使代码有更好的可读性等。
（1） 避免使用NEW关键字来创建String对象。
把一个String常量copy到String 对象中通常是多余、浪费时间的
Public class test{
Public void method(){
System.out.print (str);
}
private String str = new String ("1"); //这里新建对象是完全没有必要的
private String str2=”2” //正确的应该如此
}
参考：Joshua Bloch: "Effective Java - Programming Language Guide"
（2） 避免使用不必要的嵌套。
过多的嵌套会使你的代码复杂化，减弱可读性。
Public class test {
String add (){
Int c=(a=a+b)+b; //过于复杂
Return c
}
}
参考：http://java.sun.com/docs/codeconv/html/CodeConventions.doc9.html#177
（3） 避免在同一行声明不同类型的多个变量
这样可以使程序更加清晰，避免混乱
private int index, index1[];
正确的应该如此：
private int index;
private int index1[];
参考：http://java.sun.com/docs/codeconv/html/CodeConventions.doc5.html#2992
(4) 在每一行里写一条语句
这条规则不包括for语句：比如：'for (int i = 0; i < 10; i++) x--;’可以增加代码的可读性。
public class OSPL {
int method (int a, int b) {
int i = a + b; return i; // 可读性不强
}
正确的：
public class OSPLFixed {
int method (int a, int b) {
int i = a + b;
return i;
}
}
参考：Section 7.1 of http://java.sun.com/docs/codeconv/html/CodeConventions.doc6.html#431
（5）经常从finalize ()中调用super.finalize ()
这里的finalize ()是java在进行垃圾收集的时候调用的，和finally不一样。如果你的父类没有定义finally（）的话，你也应该调用。这里有两个原因：（1）在不改变代码的情况下能够将父类的finally方法加到你的类中。 （2）以后你会养成习惯调用父类的finally方法，即使父类没有定义finally方法的时候。
正确的方法应该如此：
public class parentFinalize {
protected void finalize () throws Throwable {
super.finalize(); // FIXED
}
参考："The Java Programming Language" by Ken Arnold and James Gosling, page 49.
（6） 不要在finalize ()中注销listeners
不要再finalize ()方法中中注销listeners，finalize ()只有再没有对象引用的时候调用，如果listeners从finalize()方法中去除了，被finalize的对象将不会在垃圾收集中去除。
public void finalize () throws Throwable {
bButton.removeActionListener (act);
}
(7) 不要显式的调用finalize ()方法
虽然显式的调用这个方法可以使你确保你的调用，但是当这个方法收集了以后垃圾收集会再收集一次。
- public class T7 {
- public void finalize() throws Throwable {
- close_resources ();
- super.finalize ();
- }
- public void close_resources() {}
- }
-
- class Test {
- void cleanup () throws Throwable {
- t71.finalize(); // 调用
- t71 = null;
- }
- private t71 = new T7 ();
- }
对于这样的调用我们应该自己创建一个释放的方法，做最初finalize ()所作的事情，当你每次想显式的调用finalize ()的时候实际上调用了释放方法。然后再使用一个判断字段来确保这个方法只执行一次，以后再调用就没关系了。
- public class T7 {
- public synchronized void release () throws Throwable{
- if (!_released) {
- close_resources (); // do what the old 'finalize ()' did
- _released = true;
- }
- }
- public void finalize () throws Throwable {
- release ();
- super.finalize ();
- }
- public void close_resources() {}
- private boolean _released = false;
- }
-
- class TestFixed {
- void closeTest () throws Throwable {
- t71 .release (); // FIXED
- t71 = null;
- }
- private T7 t71 = new T7 ();
- }
参考：Nigel Warren, Philip Bishop: "Java in Practice - Design Styles and Idioms
for Effective Java". Addison-Wesley, 1999. pp.110-111
(8)不要使用不推荐的API
尽量使用JDK1.3推荐的API。在类和方法或者java组件里有很多方法是陈旧的或者是可以选择的。有一些方法SUN用了"deprecated“标记。最好不要使用例如：
private List t_list = new List ();
t_list.addItem (str);
如果查一下javadoc的话，会发现建议用add()来代替addItem()。
参考：http://java.sun.com/j2se/1.3/docs/api/index.html
（9）为所有序列化的类创建一个'serialVersionUID'
可以避免从你各种不同的类破坏序列的兼容性。如果你不特别制订一个UID的话，那么系统为自动产生一个UID（根据类的内容）。如果UID在你新版本的类中改变了，即使那个被序列化的类没改变，你也不能反序列化老的版本了。
- public class DUID implements java.io.Serializable {
- public void method () {}
- }
- public class DUIDFixed implements java.io.Serializable {
- public void method () {}
- private static final long serialVersionUID = 1;
- }
参考：Joshua Bloch: "Effective Java - Programming Language Guide"
Addison Wesley, 2001, pp. 223
（10）对于private常量的定义
比较好的做法是对于这样的常量，加上final标记，这样的常量从初始化到最后结束值都不会改变。
private int size = 5;
改变后的做法是：
private final int size = 5;
(11)避免把方法本地变量和参数定义成和类变量相同的名字。
这样容易引起混扰，建议把任何的变量字都定义成唯一的。这样看来，SCJP里的那些题目在现实中就用不到了：）
- public void method (int j) {
- final int i = 5; // VIOLATION
- }
- private int j = 2;
建议：
- public void method (int j1) {
- final int i = 5; // VIOLATION
- }
- private int j = 2;
参考：Michael Daconta, Eric Monk, J Keller, Keith Bohnenberger: "Java Pitfalls"
John Wiley & Sons, ISBN: 0-471-36174-7 pp.17 - 25
|
|
