一、避免在循環條件中使用復雜表達式 在不做編譯優化的情況下，在循環中，循環條件會被反復計算，如果不使用復雜表達式，而使循環條件值不變的話，程序將會運行的更快。 例子： import java.util.vector; class cel { void method (vector vector) { for (int i = 0; i < vector.size (); i++) // violation ; // 
} } 更正： class cel_fixed { void method (vector vector) { int size = vector.size () for (int i = 0; i < size; i++) ; // } } 二、為'vectors' 和 'hashtables'定義初始大小 jvm為vector擴充大小的時候需要重新創建一個更大的數組，將原原先數組中的內容復制過來，最后，原先的數組再被回收。可見vector容量的擴大是一個頗費時間的事。 通常，默認的10個元素大小是不夠的。你最好能準確的估計你所需要的最佳大小。 例子： import java.util.vector;
public class dic { public void addobjects (object[] o) { // if length > 10, vector needs to expand for (int i = 0; i< o.length;i++) { v.add(o); // capacity before it can add more elements. } } public vector v = new vector(); // no initialcapacity. } 更正： 自己設定初始大小。 public vector v = new vector(20); public hashtable hash = new hashtable(10);
三、在finally塊中關閉stream 程序中使用到的資源應當被釋放，以避免資源泄漏。這最好在finally塊中去做。不管程序執行的結果如何，finally塊總是會執行的，以確保資源的正確關閉。 例子： import java.io.*; public class cs { public static void main (string args[]) { cs cs = new cs (); cs.method (); } public void method () { try { fileinputstream fis = new fileinputstream ("cs.java"); int count = 0; while (fis.read () != -1) count++; system.out.println (count); fis.close (); } catch (filenotfoundexception e1) { } catch (ioexception e2) { } } } 更正： 在最后一個catch后添加一個finally塊
四、使用'system.arraycopy ()'代替通過來循環復制數組 'system.arraycopy ()' 要比通過循環來復制數組快的多。 例子： public class irb { void method () { int[] array1 = new int [100]; for (int i = 0; i < array1.length; i++) { array1 [i] = i; } int[] array2 = new int [100]; for (int i = 0; i < array2.length; i++) { array2 [i] = array1 [i]; // violation } } } 更正： public class irb{ void method () { int[] array1 = new int [100]; for (int i = 0; i < array1.length; i++) { array1 [i] = i; } int[] array2 = new int [100]; system.arraycopy(array1, 0, array2, 0, 100); } }
五、讓訪問實例內變量的getter/setter方法變成”final” 簡單的getter/setter方法應該被置成final，這會告訴編譯器，這個方法不會被重載，所以，可以變成”inlined” 例子： class maf { public void setsize (int size) { _size = size; } private int _size; } 更正： class daf_fixed { final public void setsize (int size) { _size = size; } private int _size; }