本文介绍的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” //正确的应该如此 } |
(2) 避免使用不必要的嵌套
过多的嵌套会使你的代码复杂化,减弱可读性。
| public class test { string add (){ int c=(a=a+b)+b; //过于复杂 return c } } |
(3) 避免在同一行声明不同类型的多个变量
这样可以使程序更加清晰,避免混乱
| private int index, index1[]; |
正确的应该如此:
| private int index; private int index1[]; |
(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; } } |
(5)经常从finalize ()中调用super.finalize ()
这里的finalize ()是java在进行垃圾收集的时候调用的,和finally不一样。如果你的父类没有定义finally()的话,你也应该调用。这里有两个原因:(1)在不改变代码的情况下能够将父类的finally方法加到你的类中。 (2)以后你会养成习惯调用父类的finally方法,即使父类没有定义finally方法的时候。
正确的方法应该如此:
| public class parentfinalize { protected void finalize () throws throwable { super.finalize(); // fixed } |
(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 (); } |
(8)不要使用不推荐的api
尽量使用jdk1.3推荐的api。在类和方法或者java组件里有很多方法是陈旧的或者是可以选择的。有一些方法sun用了"deprecated“标记。最好不要使用例如:
| private list t_list = new list (); t_list.additem (str); |
如果查一下javadoc的话,会发现建议用add()来代替additem()。
(9)为所有序列化的类创建一个'serialversionuid'
可以避免从你各种不同的类破坏序列的兼容性。如果你不特别制订一个uid的话,那么系统为自动产生一个uid(根据类的内容)。如果uid在你新版本的类中改变了,即使那个被序列化的类没改变,你也不能反序列化老的版本了。
| public class duid implements java.io.serializable { public void method () {}} |
在里面加一个uid,当这个类的序列化形式改变的时候,你也改变这个uid就可以了。
| public class duidfixed implements java.io.serializable { public void method () {} private static final long serialversionuid = 1; } |
(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; |
闽公网安备 35060202000074号