Java编程中写出好代码的建议

汪星人

最近在做应用的性能优化，在review代码的过程中积累了一些规则和经验。做到这些规则的目的很简单，就是写出“优美”的代码来。

( I. {0 X0 y- z. l: H' e1、注释尽可能全面

# s) b! z$ J+ E' R1 i对于方法的注释应该包含详细的入参和结果说明，有异常抛出的情况也要详细叙述；类的注释应该包含类的功能说明、作者和修改者。

5 g) E" s2 b* i. ?2、多次使用的相同变量最好归纳成常量
, D6 [6 W  f( N9 E
* T6 F  D3 v2 b1 H5 ]5 K多处使用的相同值的变量应该尽量归纳为一个常量，方便日后的维护。

3、尽量少的在循环中执行方法调用
+ ]5 h: m! L  O" U! ?
! \& _$ z4 d9 b- W6 R! I1 X尽量在循环中少做一些可避免的方法调用，这样可以节省方法栈的创建。例如：
* o6 |2 }. Y+ U9 Q/ E7 g7 \+ m
/ X8 o/ A: T8 r6 \$ Pfor(int i=0;i<list.size();i++){  
! L+ _9 a6 F/ P% F) R8 J7 ?  System.out.println(i);  
}
可以修改为：

5 d( s0 Z3 q3 }/ T! gfor(int i=0,size=list.size();i<size;i++){  
' R: g+ r0 b$ P: M8 i* `  System.out.println(i);  
}
4、常量的定义可以放到接口中
, u' f& W- s! |! S7 k
在java中，接口里只允许存在常量，因此把常量放到接口中声明就可以省去public static final这几个关键词。
3 u' a, v6 K; y. E3 V
5、ArrayList和LinkedList的选择

3 W. m0 w# e0 A4 B  }这个问题比较常见。通常程序员最好能够对list的使用场景做出评估，然后根据特性作出选择。ArrayList底层是使用数组实现的，因此随机读取数据会比LinkedList快很多，而LinkedList是使用链表实现的，新增和删除数据的速度比ArrayList快不少。

/ q, {+ s3 J! h: O+ s4 }* P6、String，StringBuffer和StringBuilder
( O% _  I# R, k/ G- j2 W2 b
0 x0 T1 \9 s. x, f# m; f6 j2 d这个问题也比较常见。在进行字符串拼接处理的时候，String通常会产生多个对象，而且将多个值缓存到常量池中。例如：
! J  O, q7 _4 e( X
String a="a";  
% w0 S; \9 D9 }7 v3 M- K" L8 n9 DString b="b";  
  \2 x; |0 b- ]* \& Q, |a=a+b;
" s6 z8 o8 @1 C% t- K( L这种情况下jvm会产生"a","b","ab"三个对象。而且字符串拼接的性能也很低。因此通常需要做字符串处理的时候尽量采用StringBuffer和StringBuilder来。

7、包装类和基本类型的选择

在代码中，如果可以使用基本数据类型来做局部变量类型的话尽量使用基本数据类型，因为基本类型的变量是存放在栈中的，包装类的变量是在堆中，栈的操作速度比堆快很多。

8、尽早的将不再使用的变量引用赋给null
" ?* \" f" F4 b; n. O: i
这样做可以帮助jvm更快的进行内存回收。当然很多人其实对这种做法并不感冒。
( x8 t* {" i; J. V/ K0 Z) `
' Q1 g  R6 [/ `. e; m& D* I9、在finally块中对资源进行释放
% I, e& Y8 b8 s- T3 u7 L+ J/ }0 b
2 b: D* ~" C6 W& y7 n& I  \典型的场景是使用io流的时候，不论是否出现异常最后都应该在finally中对流进行关闭。
  ~( J; W0 s' g: Q2 V0 v- g8 @3 ?
) ]! L4 z3 D* h10、在HashMap中使用一个Object作为key时要注意如何区分Object是否相同
$ U: E& s1 B; {( i0 G  e
9 A: R; G4 m) P  d5 t& n在jdk的HashMap实现中，判断两个Object类型的key是否相同的标准是hashcode是否相同和equals方法的返回值。如果业务上需要对两个数据相同的内存对象当作不同的key存储到hashmap中就要对hashcode和equals方法进行覆盖。