一个优秀的工程师和一个普通的工程师的区别,不是满天飞舞的构架图,他的功底体现在所写的每一行代码上。
编程规约
命名风格
- 【强制】代码中的命名均不能以下划线或美元符号开始,也不能以下划线或美元符号结束 
- 【强制】代码中的命名严禁使用拼音与英文混合的方式,更不允许直接使用中文的方式 
- 【强制】类名使用UpperCamelCase风格,但以下情形例外:DO / BO / DTO / VO / AO / PO等 - 1 
 2- 正例:MarcoPolo / UserDO / XmlService / TcpUdpDeal / TaPromotion 
 反例:macroPolo / UserDo / XMLService / TCPUDPDeal / TAPromotion- 1 
 2
 3
 4
 5
 6
 7
 8
 9
 10
 11
 12
 13
 14
 15
 16
 17
 18
 19
 20
 21
 22
 23
 24- 分层领域模型规约: 
 DO( Data Object):与数据库表结构一一对应,通过DAO层向上传输数据源对象。
 DTO( Data Transfer Object):数据传输对象,Service或Manager向外传输的对象。
 BO( Business Object):业务对象。 由Service层输出的封装业务逻辑的对象。
 AO( Application Object):应用对象。 在Web层与Service层之间抽象的复用对象模型,极为贴近展示层,复用度不高。
 VO( View Object):显示层对象,通常是Web向模板渲染引擎层传输的对象。
 POJO( Plain Ordinary Java Object):在本手册中, POJO专指只有setter/getter/toString的简单类,包括DO/DTO/BO/VO等。
 Query:数据查询对象,各层接收上层的查询请求。 注意超过2个参数的查询封装,禁止使用Map类来传输。
 领域模型命名规约:
 数据对象:xxxDO,xxx即为数据表名。
 数据传输对象:xxxDTO,xxx为业务领域相关的名称。
 展示对象:xxxVO,xxx一般为网页名称。
 POJO是DO/DTO/BO/VO的统称,禁止命名成xxxPOJO
- 【强制】方法名、参数名、成员变量、局部变量都统一使用lowerCamelCase风格,必须遵从驼峰形式 - 1 - 正例: localValue / getHttpMessage() / inputUserId 
- 【强制】常量命名全部大写,单词间用下划线隔开,力求语义表达完整清楚,不要嫌名字长。 - 1 - 正例:MAX_STOCK_COUNT 反例:MAX_COUNT 
- 【强制】抽象类命名使用Abstract或Base开头;异常类命名使用Exception结尾;测试类命名以它要测试的类名开始,以Test结尾 
- 【强制】类型与中括号紧挨相连来定义数组。 - 1 
 2- 正例: 定义整形数组 int[] arrayDemo; int[] arrayDemo; 
 反例: 在 main 参数中,使用 String args[] 来定义
- 【强制】POJO(简单的Java对象,实际就是普通JavaBeans)类中布尔类型的变量,都不要加is前缀,否则部分框架解析会引起序列化错误。 - 1 
 2- 反例:定义为基本数据类型Boolean isDeleted;的属性,它的方法也是isDeleted(),RPC 
 框架在反向解析的时候,“误以为”对应的属性名称是deleted,导致属性获取不到,进而抛出异常。
- 【强制】包名统一使用小写,点分隔符之间有且仅有一个自然语义的英语单词。包名统一使用单数形式,但是类名如果有复数含义,类名可以使用复数形式。 - 1 - 正例:应用工具类包名为com.alibaba.ai.util、类名为MessageUtils(此规则参考spring的框架结构) 
- 【强制】杜绝完全不规范的缩写,避免望文不知义。 - 1 - 反例:AbstractClass“缩写”命名成AbsClass;condition“缩写”命名成 condi,此类随意缩写严重降低了代码的可阅读性。 
- 【推荐】为了达到代码自解释的目标,任何定义编程元素在命名时使用尽量完整单词 组合来表达其意。 - 1 
 2- 正例: 从远程 仓库 拉取代码的类 命名 为 PullCodeFromRemoteRepository 。 
 反例: 变量 int a; 的随意命名 方式。
- 【推荐】如果模块、接口、类、方法使用了设计模式,在命名时体现出具体模式,将设计模式体现在名字中,有利于阅读者快速理解架构设计理念 - 1 
 2
 3
 4- 正例: 
 public class OrderFactory;
 public class LoginProxy;
 public class ResourceObserver;
- 【推荐】接口类中的方法和属性不要加任何修饰符号(public 也不要加),保持代码的简洁性,并加上有效的Javadoc注释。尽量不要在接口里定义变量,如果一定要定义变量,肯定是与接口方法相关,并且是整个应用的基础常量。 - 1 
 2
 3
 4- 正例:接口方法签名void commit(); 
 接口基础常量String COMPANY = "alibaba";
 反例:接口方法定义public abstract void commit();
 说明:JDK8中接口允许有默认实现,那么这个default方法,是对所有实现类都有价值的默认实现。
- 接口和实现类的命名有两套规则: - 【强制】对于Service和DAO类,基于SOA的理念,暴露出来的服务一定是接口,内部的实现类用Impl的后缀与接口区别 - 1 - 正例:CacheServiceImpl实现CacheService接口。 
- 【推荐】 如果是形容能力的接口名称,取对应的形容词为接口名(通常是–able的形式) - 1 - 正例:AbstractTranslator实现 Translatable。 
 
- 【参考】枚举类名建议带上Enum后缀,枚举成员名称需要全大写,单词间用下划线隔开。 - 1 
 2- 说明:枚举其实就是特殊的常量类,且构造方法被默认强制是私有。 
 正例:枚举名字为ProcessStatusEnum的成员名称:SUCCESS / UNKNOWN_REASON。
- 【参考】各层命名规约: - 1 
 2
 3
 4
 5
 6
 7
 8
 9
 10
 11
 12
 13- Service/DAO层方法命名规约 
 1) 获取单个对象的方法用get作前缀。
 2) 获取多个对象的方法用list作前缀。
 3) 获取统计值的方法用count作前缀。
 4) 插入的方法用save/insert作前缀。
 5) 删除的方法用remove/delete作前缀。
 6) 修改的方法用update作前缀。
 领域模型命名规约
 1) 数据对象:xxxDO,xxx即为数据表名。
 2) 数据传输对象:xxxDTO,xxx为业务领域相关的名称。
 3) 展示对象:xxxVO,xxx一般为网页名称。
 4) POJO是DO/DTO/BO/VO的统称,禁止命名成xxxPOJO。
常量定义
- 【强制】不允许任何魔法值(即未经预先定义的常量)直接出现在代码中。 - 1 - 反例:String key = "Id#taobao_" + tradeId; cache.put(key, value); 
- 【强制】long或者Long初始赋值时,使用大写的L,不能是小写的l,小写容易跟数字1混淆,造成误解。 - 1 - 说明:Long a = 2l; 写的是数字的21,还是Long型的2? 
- 【推荐】不要使用一个常量类维护所有常量,按常量功能进行归类,分开维护。 - 1 
 2- 说明:大而全的常量类,非得使用查找功能才能定位到修改的常量,不利于理解和维护。 
 正例:缓存相关常量放在类CacheConsts下;系统配置相关常量放在类ConfigConsts下。
- 【推荐】常量的复用层有5层:跨应用共享常量,应用内共享常量,子工程内共享常量,包内共享常量,类内共享常量 - 跨应用共享常量:防止在二方库中,通常是在Client.jar中的constant目录下 
- 应用内共享常量:放置在一方库中,通常是在子模块中的constant目录下 - 1 
 2
 3
 4- 反例:易懂变量也要统一定义成应用内共享变量,两位工程师在两个类中分别定义了表示"是"的变量: 
 类A中:public static final String YES = "yes";
 类B中:public static final String YES = "y";
 A.YES.equals(B.YES),预期是true,但返回的是false,导致了先上问题
- 子工程内部共享变量:即在当前子工程的constant目录下 
- 包内共享变量:即在当前包下单独的constant目录下 
- 类内共享常量:直接在类内部private static final 定义 
 
- 【推荐】如果变量值仅在一个固定范围内变化用enum类型来定义。 - 1 
 2
 3
 4
 5
 6
 7
 8- 说明:如果存在名称之外的延伸属性使用enum类型,下面正例中的数字就是延伸信息,表示一年中的第几个季节。 
 正例:
 public enum SeasonEnum {
 SPRING(1), SUMMER(2), AUTUMN(3), WINTER(4);
 int seq;
 SeasonEnum(int seq){ this.seq = seq;
 }
 }
代码格式
- 【强制】大括号的使用约定。如果是大括号内为空,则简洁地写成{}即可,不需要换行;如果是非空代码块则: - 左大括号前不换行。
- 左大括号后换行。
- 右大括号前换行。
- 右大括号后还有else等代码则不换行;
- 表示终止的右大括号后必须换行。
 
- 【强制】 左小括号和字符之间不出现空格;同样,右小括号和字符之间也不出现空格。 
- 【强制】if/for/while/switch/do等保留字与括号之间都必须加空格 
- 【强制】任何二目、三目运算符的左右两边都需要加一个空格。 
- 【强制】采用四个空格缩进,禁止使用tab控制符 - 1 - 如果采用tab缩进,必须设置1个tab为四个空格.idea设置tab为四个空格时,请勿勾选use tab character;而在Eclipse中,必须勾选insert space for tabs 
- 【强制】注释的双斜线与内容之间有且仅一个空格。 
- 【强制】单行字符数限不超过 120 个,超出需要换行时 遵循如下原则: - 第二行相对一缩进 4个空格,从第三行开始不再继续缩进
- 运算符与下文一起换行。
- 方法调用的点符号与下文一起换行。
- 方法调用时,多个参数,需要换行时,在逗号后进行。
- 在括号前不要换行,见反例。
 - 1 
 2
 3
 4
 5
 6
 7- 正例: 
 StringBuffer sb = new StringBuffer();
 // 超过120个字符的情况下,换行缩进4个空格,点号和方法名称一起换行
 sb.append("zi").append("xin")...
 .append("huang")...
 .append("huang")...
 .append("huang");- 1 
 2
 3
 4
 5
 6
 7
 8- 反例: 
 StringBuffer sb = new StringBuffer();
 // 超过120个字符的情况下,不要在括号前换行
 sb.append("zi").append("xin")...append
 ("huang");
 // 参数很多的方法调用可能超过120个字符,不要在逗号前换行
 method(args1, args2, args3, ...
 , argsX);
- 【强制】方法参数在定义和传入时,多个参数逗号后边必须加空格。 - 1 
 2- 正例:下例中实参的"a",后边必须要有一个空格。 
 method("a", "b", "c");
- 【强制】IDE的text file encoding设置为UTF-8; IDE中文件的换行符使用Unix格式,不要使用Windows格式。 
- 【推荐】没有必要增加若干空格来使某一行的字符与上一行对应位置的字符对齐。 - 1 
 2
 3
 4
 5
 6- 正例: 
 int a = 3;
 long b = 4L;
 float c = 5F;
 StringBuffer sb = new StringBuffer();
 说明:增加sb这个变量,如果需要对齐,则给a、b、c都要增加几个空格,在变量比较多的情况下,是非常累赘的事情。
- 【推荐】不同逻辑、不同语义、不同业务的代码之间插入一个空行分隔开来以提升可读性。 
OOP规约
- 【强制】避免通过一个类的对象引用访问此类的静态变量或静态方法,无谓增加编译器解析成本,直接用类名来访问即可 
- 【强制】所有的覆写方法,必须加@Override注解。 - 1 - @Override注解可以准确判断是否覆盖成功,另外,如果在抽象类中对方法签名进行了修改,其实现类会马上编译报错 
- 相同参数类型,相同业务含义,才可以使用Java的可变参数,避免使用Object - 1 
 2- 说明:可变参数必须放置在参数列表的最后(建议工程师们尽量不适用可变参数编程) 
 正例:public User listUsers(String type, Long ...ids){...}
- 【强制】外部正在调用或者二方库依赖的接口,不允许修改方法签名,避免对接口调用方产生影响。接口过时必须加@Deprecated注解,并清晰地说明采用的新接口或者新服务是什么。 
- 【强制】不能使用过时的类或方法。 - 1 - 说明:java.net.URLDecoder中的方法decode(String encodeStr)已经过时,应该使用双参数decode(String source, String encode)。接口提供方既然明确是过时接口,那么有义务同时提供新的接口;作为接口的调用方来说,有义务去考证过时方法的新实现是什么 
- 【强制】Object的equals方法容易抛空指针异常,应使用常量或确定有值的对象来调用equals。 - 1 
 2
 3- 正例:"test".equals(object); 
 反例:object.equals("test");
 说明:推荐使用java.util.Objects#equals(JDK7引入的工具类)
- 【强制】所有的相同类型的包装类对象之间值的比较,全部使用equals方法比较。 - 1 
 2- 说明: 
 对于Integer var = ? 在-128至127范围内的赋值,Integer对象是在IntegerCache.cache产生,会复用已有对象,这个区间内的Integer值可以直接使用==进行判断,但是这个区间之外的所有数据,都会在堆上产生,并不会复用已有对象,这是一个大坑,推荐使用equals方法进行判断。
- 关于基本数据类型与包装数据类型的使用标准如下: - 【强制】所有的POJO属性必须使用包装数据类型
- 【强制】RPC方法的返回值和参数必须使用包装数据类型
- 【推荐】所有的局部变量使用基本数据类型
 - 1 
 2
 3- 说明:POJO类属性没有初值,是要提醒使用者在使用时,必须自己显式的进行赋值,任何NPE问题,或者入库检查,都由使用者来保证 
 正例:数据库的查询结果可能是null,因为自动拆箱,所以用基本数据类型接收有NPE风险
 反例:如果显示成交总额涨跌情况,即正负X%,x为基本数据类型,调用的RPC服务在调用不成功时,返回的是默认值,页面显示0%,这是不合理的,应该显示成中画线,所以包装数据类型的null值,能够表示额外的信息,如:远程调用失败,异常退出等
- 【强制】在定义DO/DTO/VO等POJO类时,不要设定任何属性默认值 - 1 - 反例:POJO类的gmtCreate默认值为new Date();,但是这个属性在数据提取时并没有置入具体值,在更新其他字段时又附带更新了此字段,导致创建时间被修改成为当前时间 
- 【强制】当序列化类新增属性时,请不要修改serialVersionUID字段,以避免反序列化失败;如果完全不兼容升级,避免反序列化混乱,那么请修改serialVersionUID值。 - 1 - 说明:注意serialVersionUID不一致会抛出序列化运行时异常。 
- 【强制】构造方法里面禁止加入任何业务逻辑,如果有初始化逻辑,请放在init方法中。 
- 【强制】POJO类必须写toString方法。在使用IDE中的工具Source>generate toString时,如果继承了另一个POJO类,注意在前面加一下super.toString。 - 1 - 说明:在方法执行抛出异常时,可以直接调用POJO的toString()方法打印其属性值,便于查找问题。 
- 【推荐】当使用索引访问用String的split方法得到的数组时,需在最后一个分隔符后做有无内容的检查,否则会有抛出IndexOutOfBoundsException的风险。 - 1 
 2
 3
 4- 说明:String str = "a,b,c,,"; 
 String[] ary = str.split(",");
 //预期大于3,结果是3
 System.out.println(ary.length);
- 【推荐】当一个类有多个构造方法,或者多个同名方法,这些方法应该按顺序放置在一起,便于阅读,此条规则优先于第15条规则。 
- 【推荐】 类内方法定义的顺序依次是:公有方法或保护方法 > 私有方法 > getter/setter方法。 - 1 - 说明:公有方法是类的调用者和维护者最关心的方法,首屏展示最好;保护方法虽然只是子类关心,也可能是“模板设计模式”下的核心方法;而私有方法外部一般不需要特别关心,是一个黑盒实现;因为承载的信息价值较低,所有Service和DAO的getter/setter方法放在类体最后。 
- 【推荐】setter方法中,参数名称与类成员变量名称一致,this.成员名 = 参数名。在getter/setter方法中,不要增加业务逻辑,否则会增加排查问题的难度。 - 1 
 2
 3
 4
 5
 6
 7- public Integer getData(){ 
 if(condition) {
 return this.data + 100;
 }else {
 return this.data - 100;
 }
 }
- 【推荐】循环体内,字符串的连接方式,使用StringBuilder的append方法进行扩展。 - 1 
 2
 3
 4
 5
 6- 说明:反编译出的字节码文件显示每次循环都会new出一个StringBuilder对象,然后进行append操作,最后通过toString方法返回String对象,造成内存资源浪费。 
 反例:
 String str = "start";
 for(int i = 0; i < 100; i++){
 str = str+"hello";
 }
- 【推荐】类成员与方法访问控制从严: - 如果不允许外部直接通过new来创建对象,那么构造方法必须限制为private
- 工具类不允许有public或者defult构造方法
- 类非static成员变量并且与子类共享,必须限制为protected
- 类非static成员变量并且仅在本类使用,必须限制为private
- 类static成员变量如果只在本类使用必须限制为private
- 若是static成员变量,必须考虑是否为final
- 类成员方法只供类内部调用,必须限制为private
- 类成员方法只对继承公开,限制为protected
 - 1 - 说明:对任何类、方法、参数、变量,严格控制访问范围,过于宽泛的访问范围,不利于模块解耦。思考,如果是一个private的方法,想删除就删除,可如果是一个public的service方法,或者一个public的成员变量,删除时手心不应该冒点汗吗?变量像自己的小孩,应尽量让它在自己的视线范围内,变量作用域太大,如果任其无限制的到处跑,你会担心的。 
集合处理
- 【强制】关于hashCode和equals的处理,遵循如下规则: - 只要重写equals,就必须重写hashCode
- 因为Set集合存储的不是重复的对象,依据hashCode和equals进行判断,所以Set存储的对象必须重写这两种方法
- 如果自定义对象作为Map的键,那么必须重写hashCode和equals
 - 1 - 说明:String重写了hashCode和equals方法,所以我们可以非常愉快的将String对象作为key来使用 
- 【强制】ArrayList的subList结果不可以强转为ArrayList,否则会抛出ClassCastException异常,即java.util.RandomAccessSubList cannot be cast to java.util.ArrayList。 - 1 - 说明:sublist返回的是ArrayList的内部类subList,并不是ArrayList,而是ArrayList的一个视图,对与subList子列表的操作,最终都会反应到原始列表上。 
- 【强制】在subList场景中,高度注意对原集合元素个数的修改,会导致子列表的遍历、增加、删除均产生ConcurrentModificationException异常。 
- 【强制】使用集合转数组的方法,必须使用集合的toArray(T[] arrray),传入的是类型完全一样的数组,大小就是list.size()。 - 1 - 说明:当使用toArray带参方法,入参分配的数组空间不够大时,toArray方法内部将重新分配内存空间,并返回新数组地址,如果数组元素大于实际所需,下标为[list.size()]的数组元素将被置为null,其它数组元素保持原值,因此最好将方法入参数组大小定义为与集合元素个数一致。 - 1 
 2
 3
 4
 5
 6- 正例: 
 List<String> list = new ArrayList<Sting> (2);
 list.add("guan");
 list.add("bao");
 String[] array = new String[list.size()];
 array = list.toArray(array);- 1 
 2- 反例: 
 直接使用toArray无参方法存在问题,此方法返回的值只能是Object[]类,若强制转化为其他类型,数据将出现ClassCastException错误。
- 【强制】在使用工具Arrays.asList()把数组转化为集合时,不能使用其修改集合相关的方法,他的add/remove/clear方法会抛出 UnsupportedOperationException异常。 - 1 - 说明:asList的返回对象是一个Arrays内部类,并没有实现集合的修改方法,Arrays.asList体现的是适配器模式,只是转化接口,后台数据仍是数组。 - 1 
 2
 3
 4
 5
 6- String[] str = new String[] {"you","wu"}; 
 List list = Arrays.asList(str);
 第一种情况:
 list.add("hhh") 运行时异常。
 第二种情况:
 如果str[0] = "metoo"; 那么list.get(0) 也会随之改变。
- 【强制】泛型通配符<? extends T>用来接收返回的数据,此写法的泛型集合不能使用add方法,而<? super T>不能使用get方法,因为其作为接口调用赋值时容易出错。 - 1 
 2- 说明: 
 扩展说一下 PESC(Producer Extends Consumer Super)原则:第一,频繁往外读取内容的,适合用<? extends T>;第二,经常往里插入的,适合使用<? super T>。
- 【强制】不要在foreach循环里进行元素的remove/add操作,remove元素请使用Iterator方式,如果并发操作,需要对Iterator对象加锁。 - 1 
 2
 3
 4
 5
 6
 7
 8- 正例: 
 Iterator<Sting> iterator = list.iterator();
 while(iterator.hasNext()) {
 String item = iterator.next();
 if(删除元素的条件) {
 iterator.remove();
 }
 }- 1 
 2
 3
 4
 5
 6
 7
 8
 9- 反例: 
 List<String> list = new ArrayList<String>();
 list.add("1");
 list.add("2");
 for(String item : list) {
 if("1".equals(item)){
 list.remove(item); // 直接调用集合的remove方法,会导致迭代器的hasnext方法出现故障,导致异常。
 }
 }
- 【强制】在JDk 7 及以上版本中,Comparator要满足如下三个条件,不然Arrays.sort,Collections.sort会报IllegalArgumentException异常。 - 1 
 2
 3
 4- 说明:三个条件如下。 
 1. x,y的比较结果和y,x的比较结果相反
 2. x>y,y>z,则x>z。
 3. x=y,则x,z比较结果和y,z比较结果相同- 1 
 2
 3
 4
 5
 6
 7
 8- 反例: 
 下例中没有处理相等的情况,交换两个对象判断结果相同。不符合第三个条件,在实际使用中可能会出现异常。
 new Comparator<Student> () {
 @Override
 public int compare(Student o1,Student o2) {
 return o1.getId()>o2.getId() ? 1 : -1;
 }
 };
- 【推荐】在集合初始化时,指定集合初始值大小 - 1 
 2
 3
 4
 5- 说明:HashMap使用HashMap(int initialCapacity)初始化 
 正例:
 initialCapacity = (需要存储的元素的个数/负载因子) + 1。注意负载因子(即loader factor) 默认为0.75,如果暂时无法确定初始值大小,请设置为16(即默认值)
 反例:
 HashMap 需要放置1024个元素,由于没有设置容量初始大小,随着元素不断增加,容量被迫扩大7次,resize需要重新建hash表,这严重影响性能。
- 【推荐】使用entrySet遍历Map类集合K/V,而不是用KeySet方式遍历。 - 1 
 2
 3- 说明:keySet其实遍历了2次,一次是转为Iterator对象,另一次是从hashMap中取出key所对应的value。如果是jdk 8 ,使用map.foreach方法。 
 正例:
 values()返回的是V值集合,是一个list集合对象,;keySet()返回的是一个K值集合,是一个Set集合对象;entrySet()返回的是K/V值组合集合。
- 【推荐】高度注意Map类集合K/V能不能存储null值的情况。 - | 集合类 | Key | Value | Super | 说明 | 
 | —————– | ———— | ———— | ———– | ———————– |
 | Hashtable | 不允许为null | 不允许为null | Dictionary | 线程安全 |
 | ConcurrentHashMap | 不允许为null | 不允许为null | AbstractMap | 锁分段技术(JDK 8:CAS) |
 | TreeMap | 不允许为null | 允许为null | AbstractMap | 线程不安全 |
 | HashMap | 允许为null | 允许为null | AbstractMap | 线程不安全 |- 1 - 反例:由于HashMap的干扰,很多人认为ConcurrentHashMap是可以置入null值的,而事实上存储null值时会抛出NPE异常 
- 【参考】合理利用集合的有序性(sort)和稳定性(order),避免集合的无序性(unsort)和不稳定性(unorder)带来的负面影响。 - 1 - 说明:有序性是指遍历的结果是按某种比较规则依次排列的。稳定性是值集合每次遍历的元素次序是一定的。如:ArrayList是order/unsort;HashMap是 unorder/unsort; TreeSet是order/sort。 
- 【参考】利用Set元素唯一的特性,可以快速对一个集合进行去重操作,避免使用List的contains方法进行遍历、对比、去重操作 
并发处理
- 【强制】获取单例对象需要保证线程安全,其中的方法也要保证线程安全。 - 1 - 说明:资源驱动类、工具类、单例工厂类都需要注意。 
- 【强制】在创建对象或者线程池时,请指定有意义的线程名称,方便出错时回溯。 - 1 
 2
 3
 4
 5
 6
 7- 正例: 
 public class TimerTashThread extends Thread {
 public TimerTashThread() {
 super.setName("TimerTaskThread");
 ...
 }
 }
- 【强制】线程资源必须通过线程池提供,不允许在应用中自己显式的创建线程。 - 1 - 说明:使用线程池的好处是减少在创建和销毁线程上所消耗的时间和系统资源,解决资源不足的问题。如果不使用线程池,有可能造成系统创建大量同类线程而导致消耗完内存或者“过度切换”的问题。 
- 【强制】线程池不允许使用Executors创建,而是通过ThreadPoolExecutor的方式创建,这样的处理方式能够让编写代码的工程师更加明确线程池的运行规则,规避资源耗尽的风险。 - 1 
 2
 3- 说明:Executors返回的线程池的弊端如下: 
 1. FixedThreadPool和SingleThreadPool: 允许的请求队列长度为Integer.MAX_VALUE,可能会堆积大量的请求,从而导致OOM。
 2. CachedThreadPool和ScheduledThreadPool: 允许的创建线程数量为Integer.MAX_VALUE,可能会创建大量的线程,从而导致OOM。
- 【强制】SimpleDateFormat 是线程不安全的类,一般不要定义为static变量,如果定义为static,必须加锁,或者使用DateUtils工具类。 - 1 
 2
 3
 4
 5
 6
 7- 正例:注意线程安全,使用DateUtils,推荐处理如下: 
 private static final ThreadLocal<DateFormat> df = new ThreadLocal <DateFormat>() {
 @Override
 protected DateFormat initialValue() {
 return new SimpleDateFormat("yyyy-MM-dd");
 }
 };- 1 - 说明:如果是JDK 8的应用,可以使用Instant 代替Date,LocalDateTime代替Calendar,DateTimeFormatter代替SimpleDateFormat,官方给出的解释:simple beautiful strong immutable thread-safe。 
- 【强制】在高并发场景中,同步调用应该去考量锁的性能损耗,能用无锁的数据结构,就不要用锁;能锁区块,就不要锁整个方法体;能用对象锁,就不要用类锁。 - 1 - 说明:使加锁的代码块工作量尽可能小,避免在锁代码块中调用RPC方法。 
- 【强制】在对多个资源、数据库表、对象同时加锁时,需要保持一致的加锁顺序,否则可能会造成死锁。 - 1 - 说明:如果线程1需要对表A、B、C一次全部加锁后才可以进行更新操作,那么线程2的加锁顺序也必须是A、B、C,否则可能出现死锁。 
- 【强制】在并发修改同一个记录时,为了避免更新丢失,需要加锁。要么在应用层加锁,要么在缓存层加锁,要么在数据库层使用乐观锁,使用version作为更新依据。 - 1 - 说明:如果每次访问冲突概率小于20%,推荐使用乐观锁,否则使用悲观锁,乐观锁的重试次数不得小于三次 
- 【强制】对于多线程并行处理定时任务的情况,在Timer运行多个TimerTask时,只要其中之一没有捕获抛出的异常,洽谈任务便会自动终止运行,如果在处理定时任务时使用ScheduledExecutorService,则没有这个问题 
- 【推荐】使用CountDownLatch进行异步转同步操作,每个线程退出前必须调用countDown方法,线程执行代码注意catch异常,确保countDown方法被执行,避免主线程无法执行至await方法,直到超时才返回结果 - 1 - 说明:子线程抛出异常堆栈,不能在主线程try-catch到 
- 【推荐】避免Random实例被多个线程使用,虽然共享该实例是线程安全的,但会因竞争同一个seed导致性能下降。 - 1 
 2- 说明:Random实例包括java.util.Random的实例或者Math.random()的方式。 
 正例:在JDK 7 之后可以直接使用API ThreadLocalRandom,而在JDK 7 之前,需要编码保证每个线程持有一个实例。
- 【推荐】在并发场景下,通过双重检查锁(double-checked locking)实现延迟初始化的优化问题隐患,推荐解决方案中较为简单的一种(适用于JDK 5及之后),即目标属性声明为volatile型。 - 1 
 2
 3
 4
 5
 6
 7
 8
 9
 10
 11
 12
 13- 反例: 
 class Singleton{
 private Helper helper = null;
 public Helper getHelper() {
 if(helper == null)
 synchronizde(this) {
 if(helper == null){
 helper = new Helper();
 }
 }
 return helper;
 }
 }
- 【参考】volatile 解决多线程内存不可见的问题,对于一写多读,可以解决变量同步的问题,但是如果多写,同样无法解决线程安全问题。如果是count++操作,使用如下类实现:AtomicInteger count = new AtomicInteger();count.addAndGet(1);如果是JDK 8 ,推荐使用LongAdder对象,它比AtomicLong性能更好(减少乐观锁的重试次数)。 
- 【参考】HashMap在容量不够时进行resize时,由于高并发可能出现死链,导致CPU占用飙升,在开发过程中可以使用其他数据结构或者加锁来规避此风险。 
- 【参考】ThreadLocal无法解决共享对象的更新问题,ThreadLocal对象建议使用static修饰,这个变量是针对一个线程内所有操作共享的,所以设置为静态变量,所有此类实例共享此静态变量,也就是说,在类第一次被使用时加载,只分配一块存储空间,所有此类的对象(只要是这个线程内定义的)都可以操控这个变量。 
控制语句
- 【强制】在一个switch块内,每个case要么通过break/return等来终止,要么注释说明程序将继续执行到哪一个case为止;在一个switch块内,都必须包含一个default语句并且放在最后,即使它没有任何内容。 
- 【强制】在if/else/for/while/do语句中必须使用大括号。即使只有一行代码,也应避免采用单行的编码方式:if(condition) statements;。 
- 【强制】在高并发场景中,避免使用 ”等于 ”判断作为中断或退出的条件。 判断作为中或退出的条件。 - 1 
 2- 说明: 如果并发控制没有处理好,容易产生等值判断被“击穿 ”的情况,使用大于或小区间判断条件来代替。 
 反例: 判断剩余奖品数量等于 0时,终止发放奖品但因为并处理错误导致数量瞬间变成了负数, 这样的话,活动无法终止。
- 【推荐】表达异常的分支时,少用if-else方式,这种方式可以改写成: - 1 
 2
 3
 4
 5
 6- if (condition) { 
 ...
 return obj;
 }
 // 接着写else的业务逻辑代码;
 说明:如果非得使用if()...else if()...else...方式表达逻辑,【强制】避免后续代码维护困难,请勿超过3层。
- 【推荐】除常用方法(如getXxx/isXxx)等外,不要在条件判断中执行其它复杂的语句,将复杂逻辑判断的结果赋值给一个有意义的布尔变量名,以提高可读性。 - 1 
 2
 3
 4
 5
 6
 7
 8
 9
 10
 11- 说明:很多if语句内的逻辑相当复杂,阅读者需要分析条件表达式的最终结果,理解成本非常高。如果赋值给一个非常好理解的布尔变量则是件令人赏心悦目的事情。 
 正例:
 // 伪代码如下
 final boolean existed = (file.open(fileName, "w") != null) && (...) || (...);
 if (existed) {
 ...
 }
 反例:
 if ((file.open(fileName, "w") != null) && (...) || (...)) {
 ...
 }
- 【推荐】循环体中的语句要考量性能,以下操作尽量移至循环体外处理,如定义对象、变量、获取数据库连接,避免进行不必要的try-catch操作。 - 1 - 说明:对循环体内的try-catch操作,需要思考是否可以移至循环体外。 
- 【推荐】避免采用取反逻辑运算符。 - 1 
 2
 3- 说明: 取反逻辑不利于快速理解,并且写法必然存在对应的正向。 
 正例: 使用 if (x < 628) if (x < 628) if (x < 628) 来表达 x 小于 628 。
 反例: 使用 if (!(x >= 628)) if (!(x >= 628)) if (!(x >= 628)) 来表达 x 小于 628 。
- 【推荐】接口入参保护,这种场景常见的是用作批量操作的接口。 
- 【参考】下列情形,需要进行参数校验: - 1 
 2
 3
 4
 5- 1. 调用频次低的方法。 
 2. 执行时间开销很大的方法。此情形中,参数校验时间几乎可以忽略不计,但如果因为参数错误导致中间执行回退,或者错误,那得不偿失。
 3. 需要极高稳定性和可用性的方法。
 4. 对外提供的开放接口,不管是RPC/API/HTTP接口。
 5. 敏感权限入口。
- 【参考】下列情形,不需要进行参数校验: - 1 
 2
 3- 1. 极有可能被循环调用的方法。但在方法说明里必须注明外部参数检查要求。 
 2. 底层调用频度比较高的方法。毕竟是像纯净水过滤的最后一道,参数错误不太可能到底层才会暴露问题。一般DAO层与Service层都在同一个应用中,部署在同一台服务器中,所以DAO的参数校验,可以省略。
 3. 被声明成private只会被自己代码所调用的方法,如果能够确定调用方法的代码传入参数已经做过检查或者肯定不会有问题,此时可以不校验参数
注释规约
- 【强制】类、类属性、类方法的注释必须使用Javadoc规范,使用 - /**内容*/格式,不得使用// xxx方式。- 1 - 说明:说明:在IDE编辑窗口中,Javadoc方式会提示相关注释,生成Javadoc可以正确输出相应注释;在IDE中,工程调用方法时,不进入方法即可悬浮提示方法、参数、返回值的意义,提高阅读效率。 
- 【强制】所有的抽象方法(包括接口中的方法),必须要用Javadoc注释,除了返回值、参数、异常说明外,还必须指出该方法做什么事情,实现什么功能。 - 1 - 说明:对子类的实现要求,或者调用注意事项,请一并说明。 
- 【强制】所有的类都必须添加创建者和创建日期。 
- 【强制】方法内部单行注释,在被注释语句上方另起一行,使用//注释。方法内部多行注释使用/ /注释,注意与代码对齐。 
- 【强制】所有的枚举类型字段必须要有注释,说明每个数据项的用途。 
- 【推荐】与其“半吊子”英文来注释,不如用中文注释把问题说清楚。专有名词与关键字保持英文原文即可。 - 1 - 反例:“TCP连接超时”解释成“传输控制协议连接超时”,理解反而费脑筋。 
- 【推荐】代码修改的同时,注释也要进行相应的修改,尤其是参数、返回值、异常、核心逻辑等的修改。 - 1 - 说明:代码与注释更新不同步,就像路网与导航软件更新不同步一样,如果导航软件严重滞后,就失去了导航的意义。 
- 【参考】谨慎注释掉代码。在上方详细说明,而不是简单地注释掉。如果无用,则删除。 - 1 
 2
 3
 4- 说明:代码被注释掉有两种可能性: 
 1. 后续会恢复此段代码逻辑。
 2. 永久不用。
 前者如果没有备注信息,难以知晓注释动机。后者建议直接删掉(代码仓库保存了历史代码)。
- 【参考】对于注释的要求: - 1 
 2- 1. 第一、能够准确反应设计思想和代码逻辑; 
 2. 第二、能够描述业务含义,使别的程序员能够迅速了解到代码背后的信息。完全没有注释的大段代码对于阅读者形同天书,注释是给自己看的,即使隔很长时间,也能清晰理解当时的思路;注释也是给继任者看的,使其能够快速接替自己的工作。
- 【参考】好的命名、代码结构是自解释的,注释力求精简准确、表达到位。避免出现注释的一个极端:过多过滥的注释,代码的逻辑一旦修改,修改注释是相当大的负担。 - 1 
 2
 3
 4- 反例: 
 // put elephant into fridge
 put(elephant, fridge);
 方法名put,加上两个有意义的变量名elephant和fridge,已经说明了这是在干什么,语义清晰的代码不需要额外的注释。
- 【参考】特殊注释标记,请注明标记人与标记时间。注意及时处理这些标记,通过标记扫描,经常清理此类标记。线上故障有时候就是来源于这些标记处的代码。 - 1 
 2- 1. 待办事宜(TODO):( 标记人,标记时间,[预计处理时间]) 表示需要实现,但目前还未实现的功能。这实际上是一个Javadoc的标签,目前的Javadoc还没有实现,但已经被广泛使用。只能应用于类,接口和方法(因为它是一个Javadoc标签)。 
 2. 错误,不能工作(FIXME):(标记人,标记时间,[预计处理时间]) 在注释中用FIXME标记某代码是错误的,而且不能工作,需要及时纠正的情况。
 
       
        