参考《重构》第3章,代码的坏味道
1. 常量&变量
1.1 直接赋值常量值,禁止声明新变量
1 | //反例 |
1.2 当成员变量值无需改变时,尽量定义为静态变量
1 | //反例 |
1.3 尽量使用基本数据类型,避免自动装箱和拆箱
Java 中的基本数据类型double、float、long、int、short、char、boolean,分别对应包装类Double、Float、Long、Integer、Short、Character、Boolean。
JVM支持基本类型与对应包装类的自动转换,被称为自动装箱和拆箱。装箱和拆箱都是需要CPU和内存资源的,所以应尽量避免使用自动装箱和拆箱。
1 | //反例 |
1.4 如果变量的初值会被覆盖,就没有必要给变量赋初值
1 | //反例 |
1.5 尽量使用函数内的基本类型临时变量
在函数内,基本类型的参数和临时变量都保存在栈(Stack)中,访问速度较快;对象类型的参数和临时变量的引用都保存在栈(Stack)中,内容都保存在堆(Heap)中,访问速度较慢。在类中,任何类型的成员变量都保存在堆(Heap)中,访问速度较慢。
1 | //反例 |
1.6 尽量不要在循环体外定义变量
1 | //反例 |
1.7 不可变的静态常量,尽量使用非线程安全类
1 | //反例 |
1.8 不可变的成员变量,尽量使用非线程安全类
1 | //反例 |
2. 对象&类
2.1 禁止使用JSON转化对象
影响性能!
我经常用json转换
1 | //反例 |
2.2 尽量不适用反射赋值对象
影响性能!x2
1 | //反例 |
2.3 采用Lambda表达式替换内部匿名类
Lambda表达式在大多数虚拟机中采用invokeDynamic
指令实现,相对于匿名内部类在效率上会更高一些。
1 | //反例 |
2.4 避免不必要的子类定义
原因:多一个类就多一份类加载
2.5 尽量使用指定类的final修饰符
原因:类指定了final修饰符以后,该类不可以被继承。类为final,则该类的所有方法都是final,Java编译器会寻找机会内联所有的final方法。内联对于提升Java运行效率作用重大。
注:使用Spring的AOP特性时,需要对Bean进行动态代理,如果bean类添加final修饰,会导致异常
3. 方法
3.1 把跟类成员变量无关的方法声明成静态方法
静态方法本质上就不在属于某个对象,属于它所在的类。只要通过类名就可以访问,不用消耗资源反复创建对象。
注:即使在类内部的私有方法,没有用到类成员变量,也应该声明为静态方法。
1 | //正例 |
3.2 尽量使用基本数据类型作为方法参数类型
原因:避免不必要的装箱、拆箱和空指针判断
1 | //反例 |
3.3 尽量使用基本数据类型作为方法返回值类型
原因同上
3.4 协议方法参数值/返回值非空,避免不必要的空指针判断
协议编程,用@NonNull @Nullable标注参数
1 | //正例 |
3.5 被调用方法已支持判空处理,调用方法无需再进行判空处理
3.6 尽量避免不必要的函数封装
原因:方法调用会引起出栈和入栈,消耗CPU和内存。但可以为了代码简洁易维护,增加方法调用而牺牲性能。
3.7 尽量指定方法的final修饰符
原因,同类的原因。方法指定final修饰符,可以让方法不可以被重写,Java编译器会寻找机会内联所有的final方法。
注:所有的private方法会隐式地被指定final修饰符,所以无须再为其指定final修饰符。
1 | //正例 |
4. 表达式
4.1 尽量减少方法的重复调用
4.2.尽量避免不必要的方法调用
4.3 尽量使用移位来代替正整数乘除
原因:用移位操作可以极大地提高性能!
1 | int num1 = a << 2; //乘4 |
4.4 提取公共表达式,只计算一次值,重复利用值
4.5 尽量不在条件表达式里用!
取反
4.6.对于多常量选择分支,尽量使用switch语句而不是if-else语句
和面向对象原则不太一样。
原因:switch语句进行了跳转优化,Java中采用tableswitch
或lookupswitch
指令实现,对于多常量选择分支处理效率更高。
5. 字符串
5.1 尽量不用正则表达式匹配
注意是匹配,用字符串匹配操作
5.2 尽量使用字符替换字符串
字符串长度不确定,而字符长度固定为1
5.3 尽量使用StringBuilder进行字符串拼接
原因:String是final类,内容不可修改,所以每次字符串拼接都会生成一个新对象。StringBuilder在初始化时申请了一块内存,以后的字符串拼接都在这块内存中执行,不会申请新内存和生成新对象。
1 | //正例 |
5.4 不要使用””+转化字符串,建议使用String.valueOf.
使用””+进行字符串转化,使用方便但是效率低
1 | int i=12345; |
6. 数组
6.1 不要使用循环拷贝数组
尽量使用System.arraycopy
拷贝数组,也可以使用Arrays.copyOf
拷贝数组。
1 | //反例 |
6.2 集合转化为类型T数组时,尽量传入空数组T[0]
将集合转换为数组有2种形式:toArray(new T[n])
和toArray(new T[0])
。
原因:在旧的Java版本中,建议使用toArray(new T[n])
,因为创建数组时所需的反射调用非常慢。在OpenJDK6后,反射调用是内在的,使得性能得以提高,toArray(new T[0])
比toArray(new T[n])
效率更高。此外,toArray(new T[n])
比toArray(new T[0])
多获取一次列表大小,如果计算列表大小耗时过长,也会导致toArray(new T[n])
效率降低。
1 | //反例 |
6.3 集合转化为Object数组时,尽量使用toArray()方法
转化Object数组时,没有必要使用toArray[new Object[0]],可以直接使用toArray()。避免了类型的判断,也避免了空数组的申请,所以效率会更高。
1 | List<Object> objectList =Arrays.asList(1,"2",3,"4",5,...); |
7. 集合
7.1 初始化集合时,尽量指定集合大小
原因:当默认大小不再满足数据需求时就会扩容,每次扩容的时间复杂度有可能是O(n)。所以,尽量指定预知的集合大小,就能避免或减少集合的扩容次数。
7.2 不要用循环拷贝集合
原因:尽量使用JDK提供的方法拷贝集合,底层用System.arraycopy
实现,进行数据的批量拷贝效率高
1 | //反例 |
7.3 尽量使用Arrays.asList转化数组为列表
7.4 直接迭代需要使用的集合
1 | //反例 |
7.5 不要使用size方法检测空,必须使用isEmpty方法检测空
原因:使用size方法来检测空逻辑上没有问题,但使用isEmpty方法使得代码更易读,并且可以获得更好的性能。任何isEmpty方法实现的时间复杂度都是O(1),但是某些size方法实现的时间复杂度有可能是O(n)。
7.6 非随机访问的List,尽量使用迭代代替随机访问
7.7 尽量使用HashSet判断值存在
原因:在Java集合类库中,List的contains方法普遍时间复杂度是O(n),而HashSet的时间复杂度为O(1)。
7.8 避免先判断存在再进行获取
原因:如果需要先判断存在再进行获取,可以直接获取并判断空,从而避免了二次查找操作。
8. 异常
8.1 避免在循环中捕获异常
原因:当循环体抛出异常后,无需循环继续执行(注意前提)时,没有必要在循环体中捕获异常。因为,过多的捕获异常会降低程序执行效率。
8.2 禁止使用异常控制业务流程
原因:异常处理效率比条件表达式低
1 | //反例 |
9. 缓冲区
9.1 初始化时尽量指定缓冲区大小
原因同数组指定缓冲区一样。
9.2 尽量重复使用同一缓冲区
针对缓冲区,Java虚拟机需要花时间生成对象,还要花时间进行垃圾回收处理。所以,尽量重复利用缓冲区。
9.3 尽量使用缓冲流减少IO操作
使用缓冲流:
BufferedReader
BufferedWriter
- BufferedInputStream
- BufferedOutputStream
- …
可以大幅减少IO次数并提升IO速度。
1 | //反例 |
10. 线程
10.1 在单线程中,尽量使用非线程安全类
原因:避免不必要的同步开销
1 | //反例 |
10.2 在多线程中,尽量使用线程安全类
原因:代码更简洁高效
10.3 尽量减少同步代码块范围
原因:在一个方法中,可能只有一小部分的逻辑是需要同步控制的,如果同步控制了整个方法会影响执行效率。
1 | //反例 |
10.4 尽量合并为同一同步代码块
同步代码块是有性能开销的,如果确定可以合并为同一同步代码块,就应该尽量合并为同一同步代码块。
10.5 尽量使用线程池减少线程开销
原因:多线程中两个必要的开销:线程的创建和上下文切换。采用线程池,可以尽量地避免这些开销。
1 | //反例 |