编程规约是比较重要的部分,按照基础和高级,我分成了两篇来学习,下篇篇涉及到日期时间、集合处理、并发处理、控制语句和注释规约、前后端规约、性能规约都是一些编程规约的高级部分内容。红色加粗字体为自己可能会犯的错误以及不规范的地方,蓝色结论部分为几条规则的归纳或一条规则的阐述。
日期时间
1 【强制】日期格式化时,传入 pattern 中表示年份统一使用小写的 y。这里个人容易不符合规范
- 说明:日期格式化时,yyyy 表示当天所在的年,而大写的 YYYY 代表是 week in which year(JDK7 之后引入的概念),意思是当天所在的周属于的年份,一周从周日开始,周六结束,只要本周跨年,返回的 YYYY就是下一年。
- 正例:表示日期和时间的格式如下所示:
new SimpleDateFormat("yyyy-MM-dd HH:mm:ss")
2 【强制】在日期格式中分清楚大写的 M 和小写的 m,大写的 H 和小写的 h 分别指代的意义。
- 说明:日期格式中的这两对字母表意如下:表示月份是大写的 M;表示分钟则是小写的 m; 24 小时制的是大写的 H;12 小时制的则是小写的 h。
3 【强制】获取当前毫秒数:System.currentTimeMillis();
而不是 new Date().getTime()。
这里个人容易不符合规范
- 说明:如果想获取更加精确的纳秒级时间值,使用
System.nanoTime
的方式。在 JDK8 中,针对统计时间等场景,推荐使用 Instant 类。
4 【强制】不允许在程序任何地方中使用:java.sql.Date;java.sql.Time;java.sql.Timestamp。
- 说明:第 1 个不记录时间,getHours()抛出异常;第 2 个不记录日期,getYear()抛出异常;第 3 个在构造方法
super((time/1000)*1000)
,在 Timestamp 属性 fastTime 和 nanos 分别存储秒和纳秒信息。 - 反例:
java.util.Date.after(Date)
进行时间比较时,当入参是java.sql.Timestamp
时,会触发 JDK BUG(JDK9 已修复),可能导致比较时的意外结果。
5 【强制】不要在程序中写死一年为 365 天,避免在公历闰年时出现日期转换错误或程序逻辑错误
正例:
// 获取今年的天数
int daysOfThisYear = LocalDate.now().lengthOfYear();
// 获取指定某年的天数
LocalDate.of(2011, 1, 1).lengthOfYear();
反例:
// 第一种情况:在闰年 366 天时,出现数组越界异常
int[] dayArray = new int[365];
// 第二种情况:一年有效期的会员制,今年 1 月 26 日注册,硬编码 365 返回的却是 1 月 25 日
Calendar calendar = Calendar.getInstance();
calendar.set(2020, 1, 26);
calendar.add(Calendar.DATE, 365)
6 【推荐】避免公历闰年 2 月问题。闰年的 2 月份有 29 天,一年后的那一天不可能是 2 月 29日。
7 【推荐】使用枚举值来指代月份。如果使用数字,注意 Date,Calendar 等日期相关类的月份,month 取值在 0-11 之间。
- 说明:参考 JDK 原生注释,
Month value is 0-based. e.g., 0 for January.
- 正例: Calendar.JANUARY,Calendar.FEBRUARY,Calendar.MARCH 等来指代相应月份来进行传参或
比较。
结论:尽量用封装好的时间类,例如LocalDate等,时间判断等也不要以常理度之,而是使用Java封装好的方法
集合处理
1 【强制】关于 hashCode 和 equals 的处理,遵循如下规则:
- 只要覆写 equals,就必须覆写 hashCode。
- 因为 Set 存储的是不重复的对象,依据 hashCode 和 equals 进行判断,所以 Set 存储的对象必须覆写这两种方法。
- 如果自定义对象作为 Map 的键,那么必须覆写 hashCode 和 equals。
说明:String 因为覆写了 hashCode 和 equals 方法,所以可以愉快地将 String 对象作为 key 来使用。总之就是如果覆写就一起覆写了就好
2 【强制】判断所有集合内部的元素是否为空,使用 isEmpty()
方法,而不是 size()==0 的方式。这里个人容易不符合规范
- 说明:在某些集合中,前者的时间复杂度为 O(1),而且可读性更好。
3 【强制】在使用 java.util.stream.Collectors
类的 toMap()方法转为 Map 集合时,一定要使用含有参数类型为 BinaryOperator
,参数名为 mergeFunction 的方法,否则当出现相同 key值时会抛IllegalStateException
异常。
- 说明:参数 mergeFunction 的作用是当出现 key 重复时,自定义对 value 的处理策略。
正例:
List<Pair<String, Double>> pairArrayList = new ArrayList<>(3);
pairArrayList.add(new Pair<>("version", 12.10));
pairArrayList.add(new Pair<>("version", 12.19));
pairArrayList.add(new Pair<>("version", 6.28));
Map<String, Double> map = pairArrayList.stream().collect(
// 生成的 map 集合中只有一个键值对:{version=6.28}
Collectors.toMap(Pair::getKey, Pair::getValue, (v1, v2) -> v2));
反例:
String[] departments = new String[] {"iERP", "iERP", "EIBU"};
// 抛出 IllegalStateException 异常
Map<Integer, String> map = Arrays.stream(departments)
.collect(Collectors.toMap(String::hashCode, str -> str));
package cn.shentianlan.studytest.common;
import javafx.util.Pair;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.function.BinaryOperator;
import java.util.function.Function;
import java.util.stream.Collectors;
/**
* 将List集合转换成Map集合
*/
public class ListToMapTest {
public static void main(String[] args) {
List<Pair<String, Double>> pairArrayList = new ArrayList<>(3);
pairArrayList.add(new Pair<>("version", 6.19));
pairArrayList.add(new Pair<>("version", 10.24));
pairArrayList.add(new Pair<>("version", 13.14));
Map<String, Double> map = pairArrayList.stream().collect(
// 生成的 map 集合中只有一个键值对:{version=13.14}
Collectors.toMap(Pair::getKey, Pair::getValue, (v1, v2) -> v2)
);
//上面代码写详细之后如下所示
Map<String, Double> map1 = pairArrayList.stream().collect(
Collectors.toMap(new Function<Pair<String, Double>, String>() {
@Override
public String apply(Pair<String, Double> stringDoublePair) {
return stringDoublePair.getKey();
}
}, new Function<Pair<String, Double>, Double>() {
@Override
public Double apply(Pair<String, Double> stringDoublePair) {
return stringDoublePair.getValue();
}
}, new BinaryOperator<Double>() {
@Override
public Double apply(Double aDouble, Double aDouble2) {
return aDouble2;
}
})
);
System.out.println(map);
System.out.println(map1);
}
}
4 【强制】在使用 java.util.stream.Collectors
类的 toMap()方法转为 Map 集合时,一定要注意当 value 为 null 时会抛 NPE 异常。
- 说明:在 java.util.HashMap 的 merge 方法里会进行如下的判断:
if (value == null || remappingFunction == null)
throw new NullPointerException();
反例:
List<Pair<String, Double>> pairArrayList = new ArrayList<>(2);
pairArrayList.add(new Pair<>("version1", 8.3));
pairArrayList.add(new Pair<>("version2", null));
Map<String, Double> map = pairArrayList.stream().collect(
// 抛出 NullPointerException 异常
Collectors.toMap(Pair::getKey, Pair::getValue, (v1, v2) -> v2));
5 【强制】ArrayList 的 subList (截取一部分列表)结果不可强转成 ArrayList,否则会抛出 ClassCastException
异常:java.util.RandomAccessSubList cannot be cast to java.util.ArrayList。
- 说明:
subList()
返回的是 ArrayList 的内部类 SubList,并不是 ArrayList 本身,而是 ArrayList 的一个视图,对于 SubList 的所有操作最终会反映到原列表上。
6 【强制】使用 Map 的方法 keySet()/values()/entrySet()
返回集合对象时,不可以对其进行添加元素操作,否则会抛出 UnsupportedOperationException
异常。
7【强制】Collections 类返回的对象,如:emptyList()/singletonList()
等都是 immutable list,不可对其进行添加或者删除元素的操作。这里个人容易不符合规范
- 反例:如果查询无结果,返回 Collections.emptyList()空集合对象,调用方一旦进行了添加元素的操作,就会触发
UnsupportedOperationException
异常
8 【强制】在 subList 场景中,高度注意对父集合元素的增加或删除,均会导致子列表的遍历、增加、删除产生 ConcurrentModificationException 异常
9【强制】使用集合转数组的方法,必须使用集合的 toArray(T[] array)
,传入的是类型完全一致、长度为 0 的空数组。
- 反例:直接使用 toArray 无参方法存在问题,此方法返回值只能是
Object[]
类,若强转其它类型数组将出现ClassCastException
错误。正例:
List<String> list = new ArrayList<>(2);
list.add("guan");
list.add("bao");
String[] array = list.toArray(new String[0]);
说明:使用 toArray 带参方法,数组空间大小的 length:
- 等于 0,动态创建与 size 相同的数组,性能最好。
- 大于 0 但小于 size,重新创建大小等于 size 的数组,增加 GC 负担。
- 等于 size,在高并发情况下,数组创建完成之后,size 正在变大的情况下,负面影响与 2 相同。
- 大于 size,空间浪费,且在 size 处插入 null 值,存在 NPE 隐患。
10 【强制】在使用 Collection 接口任何实现类的 addAll()
方法时,都要对输入的集合参数进行NPE 判断。这里个人容易不符合规范
- 说明:在
ArrayList#addAll
方法的第一行代码即Object[] a = c.toArray();
其中 c 为输入集合参数,如果为 null,则直接抛出异常。
11【强制】使用工具类 Arrays.asList()
把数组转换成集合时,不能使用其修改集合相关的方法,它的 add/remove/clear
方法会抛出 UnsupportedOperationException
异常。
- 说明:asList 的返回对象是一个 Arrays 内部类,并没有实现集合的修改方法。Arrays.asList 体现的是适配器模式,只是转换接口,后台的数据仍是数组。
String[] str = new String[] { "chen", "yang", "hao" };
List list = Arrays.asList(str);
- 第一种情况
:list.add("tianmaolin");
运行时异常。 - 第二种情况:
str[0] = "change";
也会随之修改,反之亦然。
12【强制】泛型通配符<? extends T>
来接收返回的数据,此写法的泛型集合不能使用 add 方法,而<? super T>
不能使用 get 方法,两者在接口调用赋值的场景中容易出错。
- 说明:扩展说一下 PECS(Producer Extends Consumer Super)原则:第一、频繁往外读取内容的,适合用
<? extends T>
。第二、经常往里插入的,适合用<? super T>
13 【强制】在无泛型限制定义的集合赋值给泛型限制的集合时,在使用集合元素时,需要进行instanceof 判断,避免抛出 ClassCastException 异常。
- 说明:毕竟泛型是在 JDK5 后才出现,考虑到向前兼容,编译器是允许非泛型集合与泛型集合互相赋值。
- 反例如下:
List<String> generics = null;
List notGenerics = new ArrayList(10);
notGenerics.add(new Object());
notGenerics.add(new Integer(1));
generics = notGenerics;
// 此处抛出 ClassCastException 异常
String string = generics.get(0);
14【强制】不要在 foreach 循环里进行元素的 remove/add
操作。remove 元素请使用 Iterator方式,如果并发操作,需要对 Iterator 对象加锁。这里个人容易不符合规范
正例:
List<String> list = new ArrayList<>();
list.add("1");
list.add("2");
Iterator<String> iterator = list.iterator();
while (iterator.hasNext()) {
String item = iterator.next();
if (删除元素的条件) {
iterator.remove();
}
}
反例:
for (String item : list) {
if ("1".equals(item)) {
list.remove(item);
}
}
15 【强制】在 JDK7 版本及以上,Comparator 实现类要满足如下三个条件,不然 Arrays.sort,Collections.sort 会抛 IllegalArgumentException 异常。
-
说明:三个条件如下
- x,y 的比较结果和 y,x 的比较结果相反。
- x>y,y>z,则 x>z。
- x=y,则 x,z 比较结果和 y,z 比较结果相同。
-
反例:下例中没有处理相等的情况,交换两个对象判断结果并不互反,不符合第一个条件,在实际使用中可能会出现异常。
new Comparator<Student>() {
@Override
public int compare(Student o1, Student o2) {
return o1.getId() > o2.getId() ? 1 : -1;
}
};
16 .【推荐】集合泛型定义时,在 JDK7 及以上,使用 diamond 语法或全省略。这里个人容易不符合规范
- 说明:菱形泛型,即 diamond,直接使用
<>
来指代前边已经指定的类型。 - 正例:
// diamond 方式,即<>
HashMap<String, String> userCache = new HashMap<>(16);
// 全省略方式
ArrayList<User> users = new ArrayList(10)
17 【推荐】集合初始化时,指定集合初始值大小。
- 说明:HashMap 使用 HashMap(int initialCapacity) 初始化,如果暂时无法确定集合大小,那么指定默认值(16)即可。
- 正例:
initialCapacity = (需要存储的元素个数 / 负载因子) + 1
。注意负载因子(即 loader factor)默认为 0.75,如果暂时无法确定初始值大小,请设置为 16(即默认值)。 - 反例: HashMap 需要放置 1024 个元素,由于没有设置容量初始大小,随着元素增加而被迫不断扩容,resize()方法总共会调用 8 次,反复重建哈希表和数据迁移。当放置的集合元素个数达千万级时会影响程序性能。
18【推荐】使用 entrySet 遍历 Map 类集合 KV
,而不是 keySet 方式进行遍历。
- 说明:keySet 其实是遍历了 2 次,一次是转为 Iterator 对象,另一次是从 hashMap 中取出 key 所对应的value。而 entrySet 只是遍历了一次就把 key 和 value 都放到了 entry 中,效率更高。如果是 JDK8,使用
Map.forEach
方法。 - 正例:values()返回的是 V 值集合,是一个 list 集合对象;keySet()返回的是 K 值集合,是一个 Set 集合对象;entrySet()返回的是 K-V 值组合集合。
19 【推荐】高度注意 Map 类集合 K/V 能不能存储 null 值的情况,如下表格:
- 反例:由于 HashMap 的干扰,很多人认为 ConcurrentHashMap 是可以置入 null 值,而事实上,存储null 值时会抛出 NPE 异常
20.【参考】合理利用好集合的有序性(sort)和稳定性(order),避免集合的无序性(unsort)和不稳定性(unorder)带来的负面影响。
- 说明:有序性是指遍历的结果是按某种比较规则依次排列的。稳定性指集合每次遍历的元素次序是一定的。如:ArrayList 是 order/unsort;HashMap 是 unorder/unsort;TreeSet 是 order/sort。这里个人容易不符合规范
21.【参考】利用 Set 元素唯一的特性,可以快速对一个集合进行去重操作,避免使用 List 的contains()进行遍历去重或者判断包含操作,这里个人容易不符合规范
并发处理
1 【强制】获取单例对象需要保证线程安全,其中的方法也要保证线程安全。
- 说明:资源驱动类、工具类、单例工厂类都需要注意。
2 【强制】创建线程或线程池时请指定有意义的线程名称,方便出错时回溯。
- 正例:自定义线程工厂,并且根据外部特征进行分组,比如,来自同一机房的调用,把机房编号赋值给whatFeatureOfGroup
public class UserThreadFactory implements ThreadFactory {
private final String namePrefix;
private final AtomicInteger nextId = new AtomicInteger(1);
// 定义线程组名称,在利用 jstack 来排查问题时,非常有帮助
UserThreadFactory(String whatFeatureOfGroup) {
namePrefix = "From UserThreadFactory's " + whatFeatureOfGroup + "-Worker-";
}
@Override
public Thread newThread(Runnable task) {
String name = namePrefix + nextId.getAndIncrement();
Thread thread = new Thread(null, task, name, 0, false);
System.out.println(thread.getName());
return thread;
}
}
3 【强制】线程资源必须通过线程池提供,不允许在应用中自行显式创建线程。这里个人容易不符合规范
- 说明:线程池的好处是减少在创建和销毁线程上所消耗的时间以及系统资源的开销,解决资源不足的问题。如果不使用线程池,有可能造成系统创建大量同类线程而导致消耗完内存或者“过度切换”的问题。
4【强制】线程池不允许使用 Executors 去创建,而是通过ThreadPoolExecutor
的方式,这样的处理方式让写的同学更加明确线程池的运行规则,规避资源耗尽的风险。
- 说明:Executors 返回的线程池对象的弊端如下:
- FixedThreadPool 和 SingleThreadPool:允许的请求队列长度为
Integer.MAX_VALUE
,可能会堆积大量的请求,从而导致 OOM。 - CachedThreadPool:允许的创建线程数量为
Integer.MAX_VALUE
,可能会创建大量的线程,从而导致 OOM。
- FixedThreadPool 和 SingleThreadPool:允许的请求队列长度为
5 【强制】SimpleDateFormat 是线程不安全的类,一般不要定义为 static 变量,如果定义为 static,必须加锁,或者使用 DateUtils 工具类。
- 正例:注意线程安全,使用 DateUtils。亦推荐如下处理:
private static final ThreadLocal<DateFormat> df = new ThreadLocal<DateFormat>() {
@Override
protected DateFormat initialValue() {
return new SimpleDateFormat("yyyy-MM-dd");
}
};
- 说明:如果是 JDK8 的应用,可以使用 Instant 代替 Date,LocalDateTime 代替 Calendar,DateTimeFormatter 代替 SimpleDateFormat,官方给出的解释:
simple beautiful strong immutable thread-safe。
6 【强制】必须回收自定义的 ThreadLocal 变量,尤其在线程池场景下,线程经常会被复用,如果不清理自定义的 ThreadLocal 变量,可能会影响后续业务逻辑和造成内存泄露等问题。尽量在代理中使用 try-finally 块进行回收。正例:
objectThreadLocal.set(userInfo);
try {
// ...
} finally {
objectThreadLocal.remove();
}
7 【强制】高并发时,同步调用应该去考量锁的性能损耗。能用无锁数据结构,就不要用锁;能锁区块,就不要锁整个方法体;能用对象锁,就不要用类锁。
- 说明:尽可能使加锁的代码块工作量尽可能的小,避免在锁代码块中调用 RPC 方法。
8【强制】对多个资源、数据库表、对象同时加锁时,需要保持一致的加锁顺序,否则可能会造成死锁。
- 说明:线程一需要对表 A、B、C 依次全部加锁后才可以进行更新操作,那么线程二的加锁顺序也必须是 A、B、C,否则可能出现死锁。
9【强制】在使用阻塞等待获取锁的方式中,必须在 try 代码块之外,并且在加锁方法与 try 代码块之间没有任何可能抛出异常的方法调用,避免加锁成功后,在 finally 中无法解锁。这里个人容易不符合规范
- 说明一:如果在 lock 方法与 try 代码块之间的方法调用抛出异常,那么无法解锁,造成其它线程无法成功获取锁。
- 说明二:如果 lock 方法在 try 代码块之内,可能由于其它方法抛出异常,导致在 finally 代码块中,unlock对未加锁的对象解锁,它会调用 AQS 的
tryRelease
方法(取决于具体实现类),抛出IllegalMonitorStateException
异常。 - 说明三:在 Lock 对象的 lock 方法实现中可能抛出 unchecked 异常,产生的后果与说明二相同。
正例:
Lock lock = new XxxLock();
// ...
lock.lock();
try {
doSomething();
doOthers();
} finally {
lock.unlock();
}
反例:
Lock lock = new XxxLock();
try {
// 如果此处抛出异常,则直接执行 finally 代码块
doSomething();
// 无论加锁是否成功,finally 代码块都会执行
lock.lock();
doOthers();
} finally {
lock.unlock();
}
10 【强制】在使用尝试机制来获取锁的方式中,进入业务代码块之前,必须先判断当前线程是否持有锁。锁的释放规则与锁的阻塞等待方式相同。
- 说明:Lock 对象的 unlock 方法在执行时,它会调用 AQS 的 tryRelease 方法(取决于具体实现类),如果当前线程不持有锁,则抛出
IllegalMonitorStateException
异常。
正例:
Lock lock = new XxxLock();
// ...
boolean isLocked = lock.tryLock();
if (isLocked) {
try {
doSomething();
doOthers();
} finally {
lock.unlock();
}
}
11.【强制】并发修改同一记录时,避免更新丢失,需要加锁。要么在应用层加锁,要么在缓存加锁,要么在数据库层使用乐观锁,使用 version 作为更新依据。
- 说明:如果每次访问冲突概率小于 20%,推荐使用乐观锁,否则使用悲观锁。乐观锁的重试次数不得小于3 次。
12.【强制】多线程并行处理定时任务时,Timer 运行多个 TimeTask 时,只要其中之一没有捕获抛出的异常,其它任务便会自动终止运行,使用 ScheduledExecutorService
则没有这个问题。
13.【推荐】资金相关的金融敏感信息,使用悲观锁策略。
- 说明:乐观锁在获得锁的同时已经完成了更新操作,校验逻辑容易出现漏洞,另外,乐观锁对冲突的解决策略有较复杂的要求,处理不当容易造成系统压力或数据异常,所以资金相关的金融敏感信息不建议使用乐观锁更新。
- 正例:悲观锁遵循一锁、二判、三更新、四释放的原则。
14.【推荐】使用 CountDownLatch
进行异步转同步操作,每个线程退出前必须调用 countDown 方法,线程执行代码注意 catch 异常,确保 countDown 方法被执行到,避免主线程无法执行至await 方法,直到超时才返回结果。
- 说明:注意,子线程抛出异常堆栈,不能在主线程 try-catch 到
15 【推荐】避免 Random 实例被多线程使用,虽然共享该实例是线程安全的,但会因竞争同一 seed导致的性能下降。
- 说明:Random 实例包括
java.util.Random
的实例或者Math.random()
的方式。 - 正例:在 JDK7 之后,可以直接使用
API ThreadLocalRandom
,而在 JDK7 之前,需要编码保证每个线程持有一个单独的 Random 实例。
16 【推荐】通过双重检查锁(double-checked locking)(在并发场景下)存在延迟初始化的优化问题隐患(可参考 The “Double-Checked Locking is Broken” Declaration),推荐解决方案中较为简单一种(适用于 JDK5 及以上版本),将目标属性声明为 volatile 型,比如将 helper 的属性声明修改为private volatile Helper helper = null;
。正例:
public class LazyInitDemo {
private volatile Helper helper = null;
public Helper getHelper() {
if (helper == null) {
synchronized (this) {
if (helper == null)
{
helper = new Helper();
}
}
}
return helper;
}
// other methods and fields...
}
17【参考】volatile 解决多线程内存不可见问题。对于一写多读,是可以解决变量同步问题,但是如果多写,同样无法解决线程安全问题。
- 说明:如果是 count++操作,使用如下类实现:
AtomicInteger count = new AtomicInteger(); count.addAndGet(1);
如果是 JDK8,推荐使用 LongAdder 对象,比 AtomicLong 性能更好(减少乐观锁的重试次数)。
18【参考】HashMap 在容量不够进行 resize 时由于高并发可能出现死链,导致 CPU 飙升,在开发过程中注意规避此风险。
19【参考】ThreadLocal 对象使用 static 修饰,ThreadLocal 无法解决共享对象的更新问题。
- 说明:这个变量是针对一个线程内所有操作共享的,所以设置为静态变量,所有此类实例共享此静态变量,也就是说在类第一次被使用时装载,只分配一块存储空间,所有此类的对象(只要是这个线程内定义的)都可以操控这个变量。
控制语句
1 【强制】在一个 switch 块内,每个 case 要么通过 continue/break/return
等来终止,要么注释说明程序将继续执行到哪一个 case 为止;在一个 switch 块内,都必须包含一个 default 语句并且放在最后,即使它什么代码也没有。这里个人容易不符合规范
- 说明:注意 break 是退出 switch 语句块,而 return 是退出方法体。
2 【强制】当 switch 括号内的变量类型为 String 并且此变量为外部参数时,必须先进行 null判断。这里个人容易不符合规范
反例:如下的代码输出是什么?
public class test {
public static void main(String[] args) {
method(null);
}
public static void method(String param) {
switch (param) {
// 肯定不是进入这里
case "sth":
System.out.println("it's sth");
break;
// 也不是进入这里
case "null":
System.out.println("it's null");
break;
// 也不是进入这里
default:
System.out.println("default");
}
}
}
而是报错:
3 【强制】在 if/else/for/while/do
语句中必须使用大括号。
- 说明:即使只有一行代码,也禁止不采用大括号的编码方式:
if (condition) statements;
这里个人容易不符合规范
4 【强制】三目运算符 condition? 表达式 1 : 表达式 2
中,高度注意表达式 1 和 2 在类型对齐时,可能抛出因自动拆箱导致的 NPE 异常。
- 说明:以下两种场景会触发类型对齐的拆箱操作:
- 1) 表达式 1 或表达式 2 的值只要有一个是原始类型。
- 2) 表达式 1 或表达式 2 的值的类型不一致,会强制拆箱升级成表示范围更大的那个类型。
- 反例:以下表达式c会强制拆箱,因为是null所以抛NPE
Integer a = 1;
Integer b = 2;
Integer c = null;
Boolean flag = false;
// a*b 的结果是 int 类型,那么 c 会强制拆箱成 int 类型,抛出 NPE 异常
Integer result=(flag? a*b : c)
5 【强制】在高并发场景中,避免使用等于
判断作为中断或退出的条件。
- 说明:如果并发控制没有处理好,容易产生等值判断被
击穿
的情况,使用大于或小于的区间判断条件来代替 - 反例:判断剩余奖品数量等于 0 时,终止发放奖品,但因为并发处理错误导致奖品数量瞬间变成了负数,这样的话,活动无法终止
6 【推荐】当某个方法的代码总行数超过 10 行时,return / throw
等中断逻辑的右大括号后均需要加一个空行。这里个人容易不符合规范
- 说明:这样做逻辑清晰,有利于代码阅读时重点关注。
7 【推荐】表达异常的分支时,少用 if-else 方式,这种方式可以改写成:
if (condition) {
...
return obj;
}
// 接着写 else 的业务逻辑代码;
- 说明:如果非使用
if()...else if()...else...
方式表达逻辑,避免后续代码维护困难,请勿超过 3 层。 - 正例:超过 3 层的 if-else 的逻辑判断代码可以使用卫语句、策略模式、状态模式等来实现,其中卫语句示例如下:
public void findBoyfriend (Man man) {
if (man.isUgly()) {
System.out.println("本姑娘是外貌协会的资深会员");
return;
}
if (man.isPoor()) {
System.out.println("贫贱夫妻百事哀");
return;
}
if (man.isBadTemper()) {
System.out.println("银河有多远,你就给我滚多远");
return;
}
System.out.println("可以先交往一段时间看看");
}
8 【推荐】除常用方法(如 getXxx/isXxx)等外,不要在条件判断中执行其它复杂的语句,将复杂逻辑判断的结果赋值给一个有意义的布尔变量名,以提高可读性。这里个人容易不符合规范
- 说明:很多 if 语句内的逻辑表达式相当复杂,与、或、取反混合运算,甚至各种方法纵深调用,理解成本非常高。如果赋值一个非常好理解的布尔变量名字,则是件令人爽心悦目的事情。
正例:
// 伪代码如下
final boolean existed = (file.open(fileName, "w") != null) && (...) || (...);
if (existed) {
...
}
反例:
public final void acquire ( long arg) {
if (!tryAcquire(arg) &&
acquireQueued(addWaiter(Node.EXCLUSIVE), arg)) {
selfInterrupt();
}
9 【推荐】不要在其它表达式(尤其是条件表达式)中,插入赋值语句。
- 说明:赋值点类似于人体的穴位,对于代码的理解至关重要,所以赋值语句需要清晰地单独成为一行。
反例:
public Lock getLock(boolean fair) {
// 算术表达式中出现赋值操作,容易忽略 count 值已经被改变
threshold = (count = Integer.MAX_VALUE) - 1;
// 条件表达式中出现赋值操作,容易误认为是 sync==fair
return (sync = fair) ? new FairSync() : new NonfairSync();
}
10【推荐】循环体中的语句要考量性能,以下操作尽量移至循环体外处理,如定义对象、变量、获取数据库连接,进行不必要的 try-catch 操作(这个 try-catch 是否可以移至循环体外)。
11 【推荐】避免采用取反逻辑运算符。
- 说明:取反逻辑不利于快速理解,并且取反逻辑写法一般都存在对应的正向逻辑写法。
- 正例:使用
if (x < 628)
来表达 x 小于 628。 - 反例:使用
if (!(x >= 628))
来表达 x 小于 628。
12【推荐】公开接口需要进行入参保护,尤其是批量操作的接口。
- 反例:某业务系统,提供一个用户批量查询的接口,API 文档上有说最多查多少个,但接口实现上没做任何保护,导致调用方传了一个 1000 的用户 id 数组过来后,查询信息后,内存爆了。
13 【参考】下列情形,需要进行参数校验:
- 调用频次低的方法。
- 执行时间开销很大的方法。此情形中,参数校验时间几乎可以忽略不计,但如果因为参数错误导致 中间执行回退,或者错误,那得不偿失。
- 需要极高稳定性和可用性的方法。
- 对外提供的开放接口,不管是 RPC/API/HTTP 接口。
- 敏感权限入口。
14 【参考】下列情形,不需要进行参数校验:这里个人容易不符合规范
- 极有可能被循环调用的方法。但在方法说明里必须注明外部参数检查。
- 底层调用频度比较高的方法。毕竟是像纯净水过滤的最后一道,参数错误不太可能到底层才会暴露问题。一般 DAO 层与 Service 层都在同一个应用中,部署在同一台服务器中,所以 DAO 的参数校验,可以省略。
- 被声明成 private 只会被自己代码所调用的方法,如果能够确定调用方法的代码传入参数已经做过检查或者肯定不会有问题,此时可以不校验参数
注释规约
1 【强制】类、类属性、类方法的注释必须使用 Javadoc 规范,使用/**内容*/格式
,不得使用// xxx
方式。
- 说明:在 IDE 编辑窗口中,Javadoc 方式会提示相关注释,生成Javadoc 可以正确输出相应注释;在 IDE中,工程调用方法时,不进入方法即可悬浮提示方法、参数、返回值的意义,提高阅读效率。
2 【强制】所有的抽象方法(包括接口中的方法)必须要用 Javadoc 注释、除了返回值、参数、异常说明外,还必须指出该方法做什么事情,实现什么功能。
- 说明:对子类的实现要求,或者调用注意事项,请一并说明。
3 【强制】所有的类都必须添加创建者和创建日期。
- 说明:在设置模板时,注意 IDEA 的
@author
为${USER}
,日期的设置统一为yyyy/MM/dd
的格式。
/**
* @author tianmaolin
* @date 2021/11/14
*/
4 【强制】方法内部单行注释,在被注释语句上方另起一行,使用//
注释。方法内部多行注释使用/* */
注释,注意与代码对齐。
5 【强制】所有的枚举类型字段必须要有注释,说明每个数据项的用途。
6【推荐】与其“半吊子”英文来注释,不如用中文注释把问题说清楚。专有名词与关键字保持英文原文即可。
- 反例:“TCP 连接超时”解释成“传输控制协议连接超时”,理解反而费脑筋。
7 【推荐】代码修改的同时,注释也要进行相应的修改,尤其是参数、返回值、异常、核心逻辑等的修改。
- 说明:代码与注释更新不同步,就像路网与导航软件更新不同步一样,如果导航软件严重滞后,就失去了导航的意义。
8 【推荐】在类中删除未使用的任何字段、方法、内部类;在方法中删除未使用的任何参数声明与内部变量。这里个人容易不符合规范
9 【参考】谨慎注释掉代码。在上方详细说明,而不是简单地注释掉。如果无用,则删除。
- 说明:代码被注释掉有两种可能性:1)后续会恢复此段代码逻辑。2)永久不用。前者如果没有备注信息,难以知晓注释动机。后者建议直接删掉即可,假如需要查阅历史代码,登录代码仓库即可
10 【推荐】及时清理不再使用的代码段或配置信息。上一条的引申这里个人容易不符合规范
- 说明:对于垃圾代码或过时配置,坚决清理干净,避免程序过度臃肿,代码冗余。
- 正例:对于暂时被注释掉,后续可能恢复使用的代码片断,在注释代码上方,统一规定使用三个斜杠(///)来说明注释掉代码的理由。如:
public static void hello() {
/// 业务方通知活动暂停
// Business business = new Business();
// business.active();
System.out.println("it's finished");
11 【参考】对于注释的要求:第一、能够准确反映设计思想和代码逻辑;第二、能够描述业务含义,使别的程序员能够迅速了解到代码背后的信息。完全没有注释的大段代码对于阅读者形同天书,注释是给自己看的,即使隔很长时间,也能清晰理解当时的思路;注释也是给继任者看的,使其能够快速接替自己的工作。
12【参考】好的命名、代码结构是自解释的,注释力求精简准确、表达到位。避免出现注释的一个极端:过多过滥的注释,代码的逻辑一旦修改,修改注释又是相当大的负担。反例:这里个人容易不符合规范
// put elephant into fridge
put(elephant, fridge);
方法名 put,加上两个有意义的变量名 elephant 和 fridge,已经说明了这是在干什么,语义清晰的代码不需要额外的注释。
结论:隔行注释,多用JavaDoc格式注释,代码没用直接干掉,实在要留也要详细说明,避免过度注释,尽量让代码自解释
13【参考】特殊注释标记,请注明标记人与标记时间。注意及时处理这些标记,通过标记扫描,经常清理此类标记。线上故障有时候就是来源于这些标记处的代码。这里个人容易不符合规范
- 待办事宜(TODO):(标记人,标记时间,[预计处理时间])表示需要实现,但目前还未实现的功能。这实际上是一个 Javadoc 的标签,目前的 Javadoc 还没有实现,但已经被广泛使用。只能应用于类,接口和方法(因为它是一个 Javadoc 标签)。
- 错误,不能工作(FIXME):(标记人,标记时间,[预计处理时间])在注释中用 FIXME 标记某代码是错误的,而且不能工作,需要及时纠正的情况。
前后端规约
1 【强制】前后端交互的 API,需要明确协议、域名、路径、请求方法、请求内容、状态码、响应体。这里个人容易不符合规范
- 协议:生产环境必须使用 HTTPS。
- 路径:每一个 API 需对应一个路径,表示 API 具体的请求地址:
- 代表一种资源,只能为名词,推荐使用复数,不能为动词,请求方法已经表达动作意义。
- URL 路径不能使用大写,单词如果需要分隔,统一使用下划线。
- 路径禁止携带表示请求内容类型的后缀,比如"
.json
",".xml
",通过 accept 头表达即可。
- 请求方法:对具体操作的定义,常见的请求方法如下:
- GET:从服务器取出资源。
- POST:在服务器新建一个资源。
- PUT:在服务器更新资源。
- DELETE:从服务器删除资源。
- 请求内容:URL 带的参数必须无敏感信息或符合安全要求;body 里带参数时必须设置 Content-Type。
- 响应体:响应体 body 可放置多种数据类型,由 Content-Type 头来确定。
2 【强制】前后端数据列表相关的接口返回,如果为空,则返回空数组[]
或空集合{}
。这里个人容易不符合规范
- 说明:此条约定有利于数据层面上的协作更加高效,减少前端很多琐碎的 null 判断。
3 【强制】服务端发生错误时,返回给前端的响应信息必须包含 HTTP 状态码,errorCode、errorMessage、用户提示信息四个部分。
- 说明:四个部分的涉众对象分别是浏览器、前端开发、错误排查人员、用户。其中输出给用户的提示信息要求:简短清晰、提示友好,引导用户进行下一步操作或解释错误原因,提示信息可以包括错误原因、上下文环境、推荐操作等。 errorCode:参考附表 3,大致错误方向。errorMessage:简要描述后端出错原因,便于错误排查人员快速定位问题,注意不要包含敏感数据信息。常见的 HTTP 状态码如下
- 200 OK: 表明该请求被成功地完成,所请求的资源发送到客户端。
- 401 Unauthorized: 请求要求身份验证,常见对于需要登录而用户未登录的情况。
- 403 Forbidden:服务器拒绝请求,常见于机密信息或复制其它登录用户链接访问服务器的情况。
- 404 Not Found: 服务器无法取得所请求的网页,请求资源不存在。
- 500 Internal Server Error: 服务器内部错误。
4 【强制】在前后端交互的 JSON 格式数据中,所有的 key 必须为小写字母开始的lowerCamelCase
风格,符合英文表达习惯,且表意完整。
- 正例:
errorCode / errorMessage / assetStatus / menuList / orderList / configFlag
- 反例:
ERRORCODE / ERROR_CODE / error_message / error-message / errormessage / ErrorMessage / msg
5 【强制】errorMessage 是前后端错误追踪机制的体现,可以在前端输出到 type="hidden"文字类控件中,或者用户端的日志中,帮助我们快速地定位出问题。
6 【强制】对于需要使用超大整数的场景,服务端一律使用 String 字符串类型返回,禁止使用Long 类型。
- 说明:Java 服务端如果直接返回 Long 整型数据给前端,JS 会自动转换为 Number 类型(注:此类型为双精度浮点数,表示原理与取值范围等同于 Java 中的 Double)。Long 类型能表示的最大值是 2 的 63 次方-1,在取值范围之内,超过 2 的 53 次方 (9007199254740992)的数值转化为 JS 的 Number 时,有些数值会有精度损失。扩展说明,在 Long 取值范围内,任何 2 的指数次整数都是绝对不会存在精度损失的,所以说精度损失是一个概率问题。若浮点数尾数位与指数位空间不限,则可以精确表示任何整数,但很不幸,双精度浮点数的尾数位只有 52 位。
- 反例:通常在订单号或交易号大于等于 16 位,大概率会出现前后端单据不一致的情况,比如,
"orderId": 362909601374617692
,前端拿到的值却是: 362909601374617660。
7 【强制】HTTP 请求通过 URL 传递参数时,不能超过 2048 字节。
- 说明:不同浏览器对于 URL 的最大长度限制略有不同,并且对超出最大长度的处理逻辑也有差异,2048字节是取所有浏览器的最小值
- 反例:某业务将退货的商品 id 列表放在 URL 中作为参数传递,当一次退货商品数量过多时,URL 参数超长,传递到后端的参数被截断,导致部分商品未能正确退货。
8 【强制】HTTP 请求通过 body 传递内容时,必须控制长度,超出最大长度后,后端解析会出错。
- 说明:nginx 默认限制是 1MB,tomcat 默认限制为 2MB,当确实有业务需要传较大内容时,可以通过调大服务器端的限制。
9 【强制】在翻页场景中,用户输入参数的小于 1,则前端返回第一页参数给后端;后端发现用户输入的参数大于总页数,直接返回最后一页。这里个人容易不符合规范
10【强制】服务器内部重定向必须使用 forward;外部重定向地址必须使用 URL 统一代理模块生成,否则会因线上采用 HTTPS 协议而导致浏览器提示“不安全”,并且还会带来 URL 维护不一致的问题。
11【推荐】服务器返回信息必须被标记是否可以缓存,如果缓存,客户端可能会重用之前的请求结果。
- 说明:缓存有利于减少交互次数,减少交互的平均延迟。
- 正例:http 1.1 中,s-maxage 告诉服务器进行缓存,时间单位为秒,用法如下,
response.setHeader("Cache-Control", "s-maxage=" + cacheSeconds);
12 【推荐】服务端返回的数据,使用 JSON 格式而非 XML。
- 说明:尽管 HTTP 支持使用不同的输出格式,例如纯文本,JSON,CSV,XML,RSS 甚至 HTML。如果我们使用的面向用户的服务,应该选择 JSON 作为通信中使用的标准数据交换格式,包括请求和响应。此外,
application/JSON 是一种通用的 MIME 类型,具有实用、精简、易读的特点。
13【推荐】前后端的时间格式统一为"yyyy-MM-dd HH:mm:ss"
,统一为 GMT。
14.【参考】在接口路径中不要加入版本号,版本控制在 HTTP 头信息中体现,有利于向前兼容。
- 说明:当用户在低版本与高版本之间反复切换工作时,会导致迁移复杂度升高,存在数据错乱风险。
15 【强制】velocity 调用 POJO 类的属性时,直接使用属性名取值即可,模板引擎会自动按规范调用 POJO 的 getXxx(),如果是 boolean 基本数据类型变量(boolean 命名不需要加 is 前缀),会自动调用 isXxx()方法。
- 说明:注意如果是 Boolean 包装类对象,优先调用 getXxx()的方法
16 【强制】后台输送给页面的变量必须加$!{var}
——中间的感叹号。
- 说明:如果 var 等于 null 或者不存在,那么
${var}
会直接显示在页面上。
17 【推荐】不要在视图模板中加入任何复杂的逻辑。
- 说明:根据 MVC 理论,视图的职责是展示,不要抢模型和控制器的活。
性能规约
1 【强制】在使用正则表达式时,利用好其预编译功能,可以有效加快正则匹配速度。
- 说明:不要在方法体内定义:
Pattern pattern = Pattern.compile(“规则”);
2 【强制】避免用 Apache Beanutils 进行属性的 copy。
- 说明:Apache BeanUtils 性能较差,可以使用其他方案比如 Spring BeanUtils, Cglib BeanCopier,注意均是浅拷贝
3 【强制】注意 Math.random()
这个方法返回是 double 类型,注意取值的范围 0≤x<1
(能够取到零值,注意除零异常),如果想获取整数类型的随机数,不要将 x 放大 10 的若干倍然后取整,直接使用 Random 对象的 nextInt
或者 nextLong
方法。
4 【推荐】任何数据结构的构造或初始化,都应指定大小,避免数据结构无限增长吃光内存,这里个人容易不符合规范
总结一下
今天这篇Blog主要是对Java编程的一些高级特性进行学习,也有不小收获,例如自己之前不太明白时间format的具体含义,大写HH和小写hh的区别;获取时间总是使用new Date().getTime()
;返回给前端的部分内容也不符合规范,没有明确哪些是给用户看的,哪些是排查问题看到 ;入参的非空判断也没有章法,例如其实DAO层就无需非空判断;还有就是觉得自己写的代码舍不得删,即使已经过期了不再用了,注释也写的过于冗余,代码明明可以自解释了;集合的addAll时也从来不判断入参的集合是不是null,用集合的时候貌似脑子里也只有ArrayList和HashMap这两个概念,忘记了一些包含特殊功能的集合,还有些概念没有完全消化,在实践中锻炼吧!