赞
踩
目录
大家好,今年下半年,小编有个重点工作任务:帮助公司制定一个符合公司实际与发展的【Java 编码规范】。亲身参与制定之后,我本身对编码规范的重视和理解也升到了好几个层次。此外,还根据公司规范出了一套面试题,来考察2-4年程序员的编码素质。
现代软件行业的高速发展对开发者的综合素质要求越来越高,因为不仅是编程知识点,其它维度的知识点也会影响到软件的最终交付质量。比如:数据库的表结构和索引设计缺陷可能带来软件上的架构缺陷或性能风险;工程结构混乱导致后续维护艰难;没有鉴权的漏洞代码易被黑客攻击等等。所以,统一的行为规范,旨在码出高效,码出质量。
对软件来说,适当的规范和标准绝不是消灭代码内容的创造性、优雅性,而是限制过度个性化,以一种普遍认可的统一方式一起做事,提升协作效率,降低沟通成本。质量的提升,直接收益就是是尽可能少踩坑,杜绝了踩重复的坑,切实提升系统稳定性。
(以下内容已经删除了个别涉及公司敏感信息的条款,请谅解)
1.【强制】代码中的命名均不能以下划线或美元符号开始,也不能以下划线或美元符号结束。
反例:_name / __name / $name / name_ / name$ / name__
2.【强制】代码中的命名严禁使用拼音与英文混合的方式,更不允许直接使用中文的方式。
说明:正确的英文拼写和语法可以让阅读者易于理解,避免歧义。注意,即使纯拼音命名方式也要避免采用。
正例:powerpeak / taobao / youku / hangzhou 等国际通用的名称,可视同英文。反例:DaZhePromotion [打折] / getPingfenByName() [评分] / int 某变量 = 3
3.【强制】类名使用 UpperCamelCase风格,但以下情形例外:DO / BO / DTO / VO / AO /PO / UID等。
正例:MarcoPolo / UserDO / XmlService / TcpUdpDeal / TaPromotion反例:macroPolo / UserDo / XMLService / TCPUDPDeal / TAPromotion
经验分享:
4.【强制】方法名、参数名、成员变量、局部变量都统一使用 lowerCamelCase风格,必须遵从
驼峰形式。
正例: localValue / getHttpMessage() / inputUserId
经验分享:
5.【强制】常量命名全部大写,单词间用下划线隔开,力求语义表达完整清楚,不要嫌名字长。
正例:MAX_STOCK_COUNT反例:MAX_COUNT
6.【强制】抽象类命名使用 Abstract或 Base开头;异常类命名使用 Exception结尾;测试类命名以它要测试的类的名称开始,以 Test结尾。
7.【强制】类型与中括号紧挨相连来表示数组。
正例:定义整形数组 int[] arrayDemo;
反例:在 main 参数中,使用 String args[]来定义。
8.【强制】POJO类中布尔类型的变量,都不要加 is前缀,否则部分框架解析会引起序列化错误。
反例:定义为基本数据类型 Boolean isDeleted的属性,它的方法也是 isDeleted(),RPC框架在反向解析的时候,“误以为”对应的属性名称是 deleted,导致属性获取不到,进而抛出异常。
9.【强制】包名统一使用小写,点分隔符之间有且仅有一个自然语义的英语单词。包名统一使用单数形式,但是类名如果有复数含义,类名可以使用复数形式。
正例:应用工具类包名为 com.powerpeak.ai.util、类名为 MessageUtils(此规则参考 spring的框架结构)
经验分享:
10.【强制】杜绝完全不规范的缩写,避免望文不知义。
反例:AbstractClass“缩写”命名成 AbsClass;condition“缩写”命名成 condi,此类随意缩写严重降低了代码的可阅读性。
11.【推荐】为了达到代码自解释的目标,任何自定义编程元素在命名时,使用尽量完整的单词组合来表达其意。
正例:在 JDK 中,表达原子更新的类名为:AtomicReferenceFieldUpdater。
反例:变量 int a 的随意命名方式。
12.【推荐】如果模块、接口、类、方法使用了设计模式,在命名时需体现出具体模式。
说明:将设计模式体现在名字中,有利于阅读者快速理解架构设计理念。
正例:
- public class OrderFactory;
-
- public class LoginProxy;
-
- public class ResourceObserver;
13.【推荐】接口类中的方法和属性不要加任何修饰符号(public 也不要加),保持代码的简洁性,并加上有效的 Javadoc注释。尽量不要在接口里定义变量,如果一定要定义变量,肯定是与接口方法相关,并且是整个应用的基础常量。
正例:接口方法签名 void commit();
接口基础常量 String COMPANY = "powerpeak";
反例:接口方法定义 public abstract void f();
说明:JDK8中接口允许有默认实现,那么这个 default方法,是对所有实现类都有价值的默认实现。
14.接口和实现类的命名有两套规则:
1)【强制】对于 Service和 DAO类,基于 SOA的理念,暴露出来的服务一定是接口,内部的实现类用 Impl的后缀与接口区别。
正例:CacheServiceImpl实现 CacheService接口。
2)【推荐】 如果是形容能力的接口名称,取对应的形容词为接口名(通常是–able的形式)
正例:AbstractTranslator实现 Translatable接口。
15.【参考】枚举类名建议带上 Enum后缀,枚举成员名称需要全大写,单词间用下划线隔开。
说明:枚举其实就是特殊的类,域成员均为常量,且构造方法被默认强制是私有。
正例:枚举名字为 ProcessStatusEnum的成员名称:SUCCESS / UNKNOWN_REASON。
16.【参考】各层命名规约:
A) Service/DAO层方法命名规约
1) 获取单个对象的方法用 get做前缀;
2) 获取多个对象的方法用 list做前缀,复数形式结尾如listObjects;
3) 获取统计值的方法用 count做前缀;
4) 插入的方法用 save/insert做前缀;
5) 删除的方法用 remove/delete做前缀;
6) 修改的方法用 update做前缀。
B) 领域模型命名规约
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);
2.【强制】在 long或者 Long赋值时,数值后使用大写的 L,不能是小写的 l,小写容易跟数字1混淆,造成误解。
说明:Long a = 2l; 写的是数字的 21,还是 Long型的 2l
3.【推荐】不要使用一个常量类维护所有常量,要按常量功能进行归类,分开维护。
说明:大而全的常量类,杂乱无章,使用查找功能才能定位到修改的常量,不利于理解和维护。
正例:缓存相关常量放在类 CacheConsts下;系统配置相关常量放在类 ConfigConsts下。
4.【推荐】常量的复用层次有五层:跨应用共享常量、应用内共享常量、子工程内共享常量、包内共享常量、类内共享常量。
1)跨应用共享常量:放置在二方库中,通常是 client.jar中的 constant目录下。
2)应用内共享常量:放置在一方库中,通常是子模块中的 constant目录下。
反例:易懂变量也要统一定义成应用内共享常量,两位工程师在两个类中分别定义了表示“是”的变量:
类 A中:public static final String YES = "yes";
类 B中:public static final String YES = "y";
A.YES.equals(B.YES),预期是 true,但实际返回为 false,导致线上问题。
3)子工程内部共享常量:即在当前子工程的 constant目录下。4)包内共享常量:即在当前包下单独的 constant目录下。5)类内共享常量:直接在类内部 private static final定义。
5.【推荐】如果变量值仅在一个固定范围内变化用 enum 类型来定义。
说明:如果存在名称之外的延伸属性应使用 enum 类型,下面正例中的数字就是延伸信息,表示一年中的第几个季节。
正例:
- public enum SeasonEnum {
-
- SPRING(1), SUMMER(2), AUTUMN(3), WINTER(4);
-
- private int seq;
-
- SeasonEnum(int seq){
-
- this.seq = seq;
-
- }
- }
6.【推荐】模块的常量类存放在模块中util包下,名称采用【模块名称】+ “Constants”表示方法
1.【强制】大括号的使用约定。如果是大括号内为空,则简洁地写成{}即可,不需要换行;如果是非空代码块则:
1)左大括号前不换行。
2)左大括号后换行。
3)右大括号前换行。
4)右大括号后还有 else等代码则不换行;表示终止的右大括号后必须换行。
2.【强制】左小括号和字符之间不出现空格;同样,右小括号和字符之间也不出现空格;而左大括号前需要空格。详见第 5条下方正例提示。
反例:if (空格 a == b空格)
3.【强制】if/for/while/switch/do等保留字与括号之间都必须加空格。
4.【强制】任何二目、三目运算符的左右两边都需要加一个空格。
说明:运算符包括赋值运算符=、逻辑运算符&&、加减乘除符号等。
5.【强制】采用 4个空格缩进,禁止使用 tab字符。
说明:如果使用 tab缩进,必须设置 1个 tab为 4个空格。IDEA设置 tab为 4个空格时,请勿勾选 Use tab character;而在 eclipse中,必须勾选 insert spaces for tabs。
正例:(涉及 1-5点)
- public static void main(String[] args) {
-
- // 缩进 4个空格
-
- String say = "hello";
-
- // 运算符的左右必须有一个空格
-
- int flag = 0;
-
- // 关键词 if与括号之间必须有一个空格,括号内的 f与左括号,0与右括号不需要空格
-
- if (flag == 0) {
-
- System.out.println(say);
- }
-
- // 左大括号前加空格且不换行;左大括号后换行
-
- if (flag == 1) {
-
- System.out.println("world");// 右大括号前换行,右大括号后有 else,不用换行
-
- } else {
-
- System.out.println("ok");
-
- // 在右大括号后直接结束,则必须换行
-
- }
-
- }
6.【强制】注释的双斜线与注释内容之间有且仅有一个空格。
正例:
- // 这是示例注释,请注意在双斜线之后有一个空格
-
- String ygb = new String();
7.【强制】单行字符数限制不超过 120 个,超出需要换行,换行时遵循如下原则:
1)第二行相对第一行缩进 4 个空格,从第三行开始,不再继续缩进,参考示例。
2)运算符与下文一起换行。
3)方法调用的点符号与下文一起换行。4)方法调用中的多个参数需要换行时,在逗号后进行。5)在括号前不要换行,见反例。
正例:
- StringBuffer sb = new StringBuffer();
-
- // 超过120个字符的情况下,换行缩进4个空格,点号和方法名称一起换行
- sb.append("zi").append("xin")…
- .append("huang")…
- .append("huang")…
- .append("huang");
反例:
- StringBuffer sb = new StringBuffer();// 超过 120个字符的情况下,不要在括号前换行
- sb.append("zi").append("xin")...append ("huang");
// 参数很多的方法调用可能超过 120个字符,不要在逗号前换行method(args1, args2, args3, ... , argsX);
8.【强制】方法参数在定义和传入时,多个参数逗号后边必须加空格。
正例:下例中实参的 args1,后边必须要有一个空格。
method(args1, args2, args3);
9.【强制】IDE的 text file encoding设置为 UTF-8; IDE中文件的换行符使用Unix格式,不要使用 Windows格式。
10.【推荐】单个方法的总行数不超过 80 行。
说明:包括方法签名、结束右大括号、方法内代码、注释、空行、回车及任何不可见字符的总行数不超过 80 行。
正例:代码逻辑分清红花和绿叶,个性和共性,绿叶逻辑单独出来成为额外方法,使主干代码更加清晰;共性逻辑抽取成为共性方法,便于复用和维护。
11.【推荐】没有必要增加若干空格来使某一行的字符与上一行对应位置的字符对齐。
正例:int one = 1;
long two = 2L;
float three = 3F;
StringBuffer sb = new StringBuffer();
说明:增加 sb这个变量,如果需要对齐,则给 a、b、c都要增加几个空格,在变量比较多的情况下,是非常累赘的事情。
12.【推荐】不同逻辑、不同语义、不同业务的代码之间插入一个空行分隔开来以提升可读性。
说明:任何情形,没有必要插入多个空行进行隔开。
13.【推荐】类和接口中元素按顺序来布局,具体布局顺序如下:
正例:
14.【推荐】避免长的布尔表达式,应换成多个更容易理解的表达式。示例如下:
正例:
- bool isFinished = (elementNo < 0) || (elementNo > maxElement);
-
- bool isRepeatedEntry = elementNo == lastElement;
-
- if (isFinished || isRepeatedEntry) {
-
- …
-
- }
反例:
- if ((elementNo < 0) || (elementNo > maxElement)|| elementNo ==lastElement){
-
- …
-
- }
15.【推荐】方法的修饰关键字按规则顺序书写,其中访问标示符一定要在最前面。规则如下:
<public, protected, private > static abstract synchronized unuaual final native methodName
正例:
public static double square(double a);
反例:
static public double square(double a);
16.【推荐】字符串进行频繁拼接时,使用StringBuffer 对象的append() 方法。sql 语句使用 pl/sql 的复制模板
1.【强制】避免通过一个类的对象引用访问此类的静态变量或静态方法,无谓增加编译器解析成本,直接用类名来访问即可。
2.【强制】所有的覆写方法,必须加@Override注解。
说明:getObject()与 get0bject()的问题。一个是字母的 O,一个是数字的 0,加@Override可以准确判断是否覆盖成功。另外,如果在抽象类中对方法签名进行修改,其实现类会马上编译报错。
3.【强制】相同参数类型,相同业务含义,才可以使用 Java的可变参数,避免使用 Object。
说明:可变参数必须放置在参数列表的最后。(提倡同学们尽量不用可变参数编程)
正例:public List<User> listUsers(String type, Long... ids) {...}
4.【强制】外部正在调用或者二方库依赖的接口,不允许修改方法签名,避免对接口调用方产生影响。接口过时必须加@Deprecated注解,并清晰地说明采用的新接口或者新服务是什么。
5.【强制】不能使用过时的类或方法。
说明:java.net.URLDecoder 中的方法 decode(String encodeStr) 这个方法已经过时,应该使用双参数 decode(String source, String encode)。接口提供方既然明确是过时接口,那么有义务同时提供新的接口;作为调用方来说,有义务去考证过时方法的新实现是什么。
6.【强制】Object的 equals方法容易抛空指针异常,应使用常量或确定有值的对象来调用equals。
正例:"test".equals(object); 反例:object.equals("test");
说明:推荐使用 java.util.Objects#equals(JDK7引入的工具类)
7.【强制】所有的相同类型的包装类对象之间值的比较,全部使用 equals方法比较。
说明:对于 Integer var = ? 在-128至 127范围内的赋值,Integer对象是在 IntegerCache.cache产生,会复用已有对象,这个区间内的 Integer值可以直接使用==进行 判断,但是这个区间之外的所有数据,都会在堆上产生,并不会复用已有对象,这是一个大坑, 推荐使用 equals方法进行判断。
8.关于基本数据类型与包装数据类型的使用标准如下:
1)【强制】所有的 POJO类属性必须使用包装数据类型。
2)【强制】RPC方法的返回值和参数必须使用包装数据类型。
3)【推荐】所有的局部变量使用基本数据类型。
说明:POJO类属性没有初值是提醒使用者在需要使用时,必须自己显式地进行赋值,任何NPE问题,或者入库检查,都由使用者来保证。
正例:数据库的查询结果可能是 null,因为自动拆箱,用基本数据类型接收有 NPE风险。
反例:比如显示成交总额涨跌情况,即正负 x%,x为基本数据类型,调用的 RPC服务,调用不成功时,返回的是默认值,页面显示为 0%,这是不合理的,应该显示成中划线。所以包装 数据类型的 null值,能够表示额外的信息,如:远程调用失败,异常退出。
9.【强制】定义 DO/DTO/VO等 POJO类时,不要设定任何属性默认值。
反例:POJO类的 gmtCreate默认值为 new Date(),但是这个属性在数据提取时并没有置入具体值,在更新其它字段时又附带更新了此字段,导致创建时间被修改成当前时间。
10.【强制】序列化类新增属性时,请不要修改 serialVersionUID字段,避免反序列失败;如果完全不兼容升级,避免反序列化混乱,那么请修改 serialVersionUID值。
说明:注意 serialVersionUID不一致会抛出序列化运行时异常。
11.【强制】构造方法里面禁止加入任何业务逻辑,如果有初始化逻辑,请放在 init方法中。
12.【强制】POJO类必须写 toString方法。使用 IDE中的工具:source>generate toString时,如果继承了另一个 POJO类,注意在前面加一下 super.toString。
说明:在方法执行抛出异常时,可以直接调用 POJO的 toString()方法打印其属性值,便于排查问题。
13.【强制】禁止在 POJO类中,同时存在对应属性 xxx的 isXxx()和 getXxx()方法。
说明:框架在调用属性 xxx 的提取方法时,并不能确定哪个方法一定是被优先调用到。
14.【推荐】使用索引访问用 String的 split方法得到的数组时,需做最后一个分隔符后有无内容的检查,否则会有抛 IndexOutOfBoundsException的风险。
说明:
- String str = "a,b,c,,"; String[] ary = str.split(",");
-
- // 预期大于 3,结果是 3
-
- System.out.println(ary.length);
15.【推荐】当一个类有多个构造方法,或者多个同名方法,这些方法应该按顺序放置在一起,便于阅读,此条规则优先于第 16条规则。
16.【推荐】 类内方法定义的顺序依次是:公有方法或保护方法 > 私有方法 >getter/setter方法。
说明:公有方法是类的调用者和维护者最关心的方法,首屏展示最好;保护方法虽然只是子类 关心,也可能是“模板设计模式”下的核心方法;而私有方法外部一般不需要特别关心,是一个黑盒实现;因为承载的信息价值较低,所有 Service和 DAO的 getter/setter方法放在类体最后。
17.【推荐】setter方法中,参数名称与类成员变量名称一致,this.成员名 = 参数名。在getter/setter方法中,不要增加业务逻辑,增加排查问题的难度。
反例:
- public Integer getData() {
-
- if (condition) {
-
- return this.data + 100;
-
- } else {
-
- return this.data - 100;
-
- }
- }
18.【推荐】循环体内,字符串的连接方式,使用 StringBuilder的 append方法进行扩展。
说明:下例中,反编译出的字节码文件显示每次循环都会 new出一个 StringBuilder对象,然后进行 append操作,最后通过 toString方法返回 String对象,造成内存资源浪费。
反例:
- String str = "start";
- for (int i = 0; i < 100; i++) {
- str = str + "hello";
- }
19.【推荐】final可以声明类、成员变量、方法、以及本地变量,下列情况使用 final关键字:
1)不允许被继承的类,如:String类;
2)不允许修改引用的域对象;
3)不允许被重写的方法,如:POJO类的 setter方法
4)不允许运行过程中重新赋值的局部变量;
5)避免上下文重复使用一个变量,使用 final描述可以强制重新定义一个变量,方便更好地进行重构。
20.【推荐】慎用 Object的 clone方法来拷贝对象。
说明:对象的 clone方法默认是浅拷贝,若想实现深拷贝需要重写 clone方法实现域对象的深度遍历式拷贝。
21.【推荐】类成员与方法访问控制从严:
1)如果不允许外部直接通过 new来创建对象,那么构造方法必须是 private;
2)工具类不允许有 public或 default构造方法;
3)类非 static成员变量并且与子类共享,必须是 protected;
4)类非 static成员变量并且仅在本类使用,必须是 private;
5)类 static成员变量如果仅在本类使用,必须是 private;
6)若是 static成员变量,考虑是否为 final;
7)类成员方法只供类内部调用,必须是 private;
8)类成员方法只对继承类公开,那么限制为 protected;
说明:任何类、方法、参数、变量,严控访问范围。过于宽泛的访问范围,不利于模块解耦。
思考:如果是一个 private的方法,想删除就删除,可是一个 public的 service成员方法或成员变量,删除一下,不得手心冒点汗吗?变量像自己的小孩,尽量在自己的视线内,变量作用域太大,无限制的到处跑,那么你会担心的。
1.【强制】关于 hashCode和 equals的处理,遵循如下规则:
1)只要重写 equals,就必须重写 hashCode;
2)因为 Set存储的是不重复的对象,依据 hashCode和 equals进行判断,所以 Set存储的对象必须重写这两个方法;
3)如果自定义对象作为 Map的键,那么必须重写 hashCode和 equals;
说明:String重写了 hashCode和 equals方法,所以我们可以非常愉快地使用 String对象作为 key来使用。
2.【强制】ArrayList的subList结果不可强转成ArrayList,否则会抛出ClassCastException异常,即 java.util.RandomAccessSubList cannot be cast to java.util.ArrayList。
说明:subList 返回的是 ArrayList 的内部类 SubList,并不是 ArrayList而是 ArrayList的一个视图,对于 SubList子列表的所有操作最终会反映到原列表上。
3.【强制】在 subList场景中,高度注意对原集合元素的增加或删除,均会导致子列表的遍历、增加、删除产生 ConcurrentModificationException 异常。
4.【强制】使用集合转数组的方法,必须使用集合的 toArray(T[] array),传入的是类型完全一样的数组,大小就是 list.size()。
说明:使用 toArray带参方法,入参分配的数组空间不够大时,toArray方法内部将重新分配内存空间,并返回新数组地址;如果数组元素个数大于实际所需,下标为[ list.size() ]的数组元素将被置为 null,其它数组元素保持原值,因此最好将方法入参数组大小定义与集合元素个数一致。
正例:
- List<String> list = new ArrayList<String>(2);list.add("guan"); list.add("bao");
- String[] array = new String[list.size()];array = list.toArray(array);
反例:直接使用 toArray无参方法存在问题,此方法返回值只能是 Object[]类,若强转其它类型数组将出现 ClassCastException错误。
5.【强制】使用工具类 Arrays.asList()把数组转换成集合时,不能使用其修改集合相关的方法,它的 add/remove/clear方法会抛出 UnsupportedOperationException异常。
说明:asList的返回对象是一个 Arrays内部类,并没有实现集合的修改方法。Arrays.asList 体现的是适配器模式,只是转换接口,后台的数据仍是数组。
- String[] str = new String[] { "you", "wu" };
- List list = Arrays.asList(str);
第一种情况:list.add("yangguanbao"); 运行时异常。
第二种情况:str[0] = "gujin"; 那么 list.get(0)也会随之修改。
6.【强制】泛型通配符<? extends T>来接收返回的数据,此写法的泛型集合不能使用 add方法,而<? super T>不能使用 get方法,作为接口调用赋值时易出错。
说明:扩展说一下 PECS(Producer Extends Consumer Super)原则:
第一、频繁往外读取内容的,适合用<? extends T>。
第二、经常往里插入的,适合用<? super T>。
7.【强制】不要在 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);
- }
- }
说明:以上代码的执行结果肯定会出乎大家的意料,那么试一下把“1”换成“2”,会是同样的结果吗?
8.【强制】 在 JDK7版本及以上,Comparator实现类要满足如下三个条件,不然 Arrays.sort,Collections.sort会报 IllegalArgumentException异常。
说明:三个条件如下
1)x,y的比较结果和 y,x的比较结果相反;
2)x>y,y>z,则 x>z。
3)x=y,则 x,z比较结果和 y,z比较结果相同。
反例:下例中没有处理相等的情况,实际使用中可能会出现异常:
- new Comparator<Student>() {
- @Override
- public int compare(Student o1, Student o2) {
- return o1.getId() > o2.getId() ? 1 : -1;
- }
- };
9.【推荐】集合泛型定义时,在 JDK7及以上,使用 diamond语法或全省略。
说明:菱形泛型,即 diamond,直接使用<>来指代前边已经指定的类型。
正例:
- // <> diamond方式
-
- HashMap<String, String> userCache = new HashMap<>(16);
-
- // 全省略方式
-
- ArrayList<User> users = new ArrayList(10);
10.【推荐】集合初始化时,指定集合初始值大小。
说明:HashMap使用 HashMap(int initialCapacity) 初始化。
正例:initialCapacity= (需要存储的元素个数 / 负载因子)+1。注意负载因子(即 loaderfactor)默认为 0.75,如果暂时无法确定初始值大小,请设置为 16(即默认值)。
反例:HashMap需要放置 1024个元素,由于没有设置容量初始大小,随着元素不断增加,容 量 7 次被迫扩大,resize需要重建 hash表,严重影响性能。
11.【推荐】使用 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值组合集合。
12.【推荐】高度注意 Map类集合 K/V能不能存储 null值的情况,如下表格:
集合类 | Key | Value | Super | 说明 |
Hashtable | 不允许为 null | 不允许为 null | Dictionary | 线程安全 |
ConcurrentHashMap | 不允许为 null | 不允许为 null | AbstractMap | 锁分段技术(JDK8:CAS) |
TreeMap | 不允许为 null | 允许为 null | AbstractMap | 线程不安全 |
HashMap | 允许为 null | 允许为 null | AbstractMap | 线程不安全 |
反例: 由于 HashMap的干扰,很多人认为 ConcurrentHashMap是可以置入 null值,而事实上,存储 null值时会抛出 NPE异常。
13.【参考】合理利用好集合的有序性(sort)和稳定性(order),避免集合的无序性(unsort)和不稳定性(unorder)带来的负面影响。说明:有序性是指遍历的结果是按某种比较规则依次排列的。稳定性指集合每次遍历的元素次序是一定的。如:ArrayList是 order/unsort;HashMap是 unorder/unsort;TreeSet是order/sort。
14.【参考】利用 Set元素唯一的特性,可以快速对一个集合进行去重操作,避免使用 List的contains方法进行遍历、对比、去重操作。
1.【强制】获取单例对象需要保证线程安全,其中的方法也要保证线程安全。
说明:资源驱动类、工具类、单例工厂类都需要注意。
2.【强制】创建线程或线程池时请指定有意义的线程名称,方便出错时回溯。
正例:
- public class TimerTaskThread extends Thread {
-
- public TimerTaskThread() {
-
- super.setName("TimerTaskThread");
- ...
-
- }
- }
3.【强制】线程资源必须通过线程池提供,不允许在应用中自行显式创建线程。
说明:使用线程池的好处是减少在创建和销毁线程上所消耗的时间以及系统资源的开销,解决资源不足的问题。如果不使用线程池,有可能造成系统创建大量同类线程而导致消耗完内存或者“过度切换”的问题。
4.【强制】线程池不允许使用 Executors去创建,而是通过 ThreadPoolExecutor的方式,这样的处理方式让写的同学更加明确线程池的运行规则,规避资源耗尽的风险。
说明:Executors返回的线程池对象的弊端如下:
1)FixedThreadPool和 SingleThreadPool:
允许的请求队列长度为 Integer.MAX_VALUE,可能会堆积大量的请求,从而导致 OOM。
2)CachedThreadPool和 ScheduledThreadPool:
允许的创建线程数量为 Integer.MAX_VALUE,可能会创建大量的线程,从而导致 OOM。
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 strongimmutable thread-safe。
6.【强制】高并发时,同步调用应该去考量锁的性能损耗。能用无锁数据结构,就不要用锁;能锁区块,就不要锁整个方法体;能用对象锁,就不要用类锁。
说明:尽可能使加锁的代码块工作量尽可能的小,避免在锁代码块中调用 RPC 方法。
7.【强制】对多个资源、数据库表、对象同时加锁时,需要保持一致的加锁顺序,否则可能会造成死锁。
说明:线程一需要对表 A、B、C依次全部加锁后才可以进行更新操作,那么线程二的加锁顺序也必须是 A、B、C,否则可能出现死锁。
8.【强制】并发修改同一记录时,避免更新丢失,需要加锁。要么在应用层加锁,要么在缓存加锁,要么在数据库层使用乐观锁,使用 version作为更新依据。
说明:如果每次访问冲突概率小于 20%,推荐使用乐观锁,否则使用悲观锁。乐观锁的重试次数不得小于 3次。
9.【强制】多线程并行处理定时任务时,Timer运行多个 TimeTask时,只要其中之一没有捕获抛出的异常,其它任务便会自动终止运行,使用 ScheduledExecutorService则没有这个问题。
10.【推荐】使用 CountDownLatch进行异步转同步操作,每个线程退出前必须调用 countDown方法,线程执行代码注意 catch异常,确保 countDown方法被执行到,避免主线程无法执行至 await方法,直到超时才返回结果。
说明:注意,子线程抛出异常堆栈,不能在主线程 try-catch到。
11.【推荐】避免 Random实例被多线程使用,虽然共享该实例是线程安全的,但会因竞争同一seed 导致的性能下降。
说明:Random实例包括 java.util.Random 的实例或者 Math.random()的方式。 正例:在 JDK7之后,可以直接使用 API ThreadLocalRandom,而在 JDK7之前,需要编码保 证每个线程持有一个实例。
12.【推荐】在并发场景下,通过双重检查锁(double-checked locking)实现延迟初始化的优化问题隐患(可参考 The "Double-Checked Locking is Broken" Declaration),推荐解决方案中较为简单一种(适用于 JDK5及以上版本),将目标属性声明为 volatile型。
反例:
- class LazyInitDemo {
-
- private Helper helper = null; public Helper getHelper() {
- if (helper == null) synchronized(this) {
- if (helper == null) helper = new Helper();
- }
- return helper;
- }
- // other methods and fields...
- }
13.【参考】volatile解决多线程内存不可见问题。对于一写多读,是可以解决变量同步问题,但是如果多写,同样无法解决线程安全问题。
如果是 count++操作,使用如下类实现:AtomicInteger count = new AtomicInteger(); count.addAndGet(1);
如果是 JDK8,推 荐使用 LongAdder对象,比 AtomicLong性能更好(减少乐观锁的重试次数)。
14.【参考】 HashMap在容量不够进行 resize时由于高并发可能出现死链,导致 CPU飙升,在开发过程中可以使用其它数据结构或加锁来规避此风险。
15.【参考】ThreadLocal无法解决共享对象的更新问题,ThreadLocal对象建议使用 static修饰。这个变量是针对一个线程内所有操作共享的,所以设置为静态变量,所有此类实例共享此静态变量 ,也就是说在类第一次被使用时装载,只分配一块存储空间,所有此类的对象(只要是这个线程内定义的)都可以操控这个变量。
1.【强制】在一个 switch块内,每个 case要么通过 break/return等来终止,要么注释说明程序将继续执行到哪一个 case为止;在一个 switch块内,都必须包含一个 default语句并且放在最后,即使空代码。
2.【强制】在 if/else/for/while/do语句中必须使用大括号。即使只有一行代码,避免采用单行的编码方式:if (condition) statements;
3.【强制】在高并发场景中,避免使用”等于”判断作为中断或退出的条件。
说明:如果并发控制没有处理好,容易产生等值判断被“击穿”的情况,使用大于或小于的区间判断条件来代替。
反例:判断剩余奖品数量等于 0 时,终止发放奖品,但因为并发处理错误导致奖品数量瞬间变成了负数,这样的话,活动无法终止。
4.【推荐】表达异常的分支时,少用 if-else方式,这种方式可以改写成:
- if (condition) {
- ...
- return obj;
- }
- // 接着写 else的业务逻辑代码;
说明:如果非得使用 if()...else if()...else...方式表达逻辑,【强制】避免后续代码维护困难,请勿超过 3层。
正例:超过 3层的 if-else 的逻辑判断代码可以使用卫语句、策略模式、状态模式等来实现,其中卫语句示例如下:
- public void today() {
-
- if (isBusy()) {
- System.out.println(“change time.”);
- return;
- }
-
- if (isFree()) {
- System.out.println(“go to travel.”);
- return;
- }
-
- System.out.println(“stay at home to learn powerpeak Java Coding Guidelines.”);
- return;
- }
5.【推荐】除常用方法(如 getXxx/isXxx)等外,不要在条件判断中执行其它复杂的语句,将复杂逻辑判断的结果赋值给一个有意义的布尔变量名,以提高可读性。
说明:很多 if 语句内的逻辑相当复杂,阅读者需要分析条件表达式的最终结果,才能明确什么样的条件执行什么样的语句,那么,如果阅读者分析逻辑表达式错误呢?
正例:
// 伪代码如下
final boolean existed = (file.open(fileName, "w") != null) && (...) || (...); if (existed) {
...}
反例:
if ((file.open(fileName, "w") != null) && (...) || (...)) {
...}
6.【推荐】循环体中的语句要考量性能,以下操作尽量移至循环体外处理,如定义对象、变量、获取数据库连接,进行不必要的 try-catch操作(这个 try-catch是否可以移至循环体外)。
7.【推荐】避免采用取反逻辑运算符。
说明:取反逻辑不利于快速理解,并且取反逻辑写法必然存在对应的正向逻辑写法。
正例:使用 if (x < 628) 来表达 x 小于 628。
反例:使用 if (!(x >= 628)) 来表达 x 小于 628。
8.【推荐】接口入参保护,这种场景常见的是用作批量操作的接口。
9.【参考】下列情形,需要进行参数校验:
1)调用频次低的方法;
2)执行时间开销很大的方法。此情形中,参数校验时间几乎可以忽略不计,但如果因为参数错误导致中间执行回退,或者错误,那得不偿失;
3)需要极高稳定性和可用性的方法;
4)对外提供的开放接口,不管是 RPC/API/HTTP接口;
5)敏感权限入口。
10.【参考】下列情形,不需要进行参数校验:
1)极有可能被循环调用的方法。但在方法说明里必须注明外部参数检查要求;
2)底层调用频度比较高的方法。毕竟是像纯净水过滤的最后一道,参数错误不太可能到底层才会暴露问题。一般 DAO层与 Service层都在同一个应用中,部署在同一台服务器中,所以 DAO的参数校验,可以省略;
3)被声明成 private只会被自己代码所调用的方法,如果能够确定调用方法的代码传入参数已经做过检查或者肯定不会有问题,此时可以不校验参数。
1.【强制】类、类属性、类方法的注释必须使用 Javadoc规范,使用 /**内容*/ 格式,不得使用 // xxx 方式。
说明:在 IDE编辑窗口中,Javadoc方式会提示相关注释,生成 Javadoc可以正确输出相应注释;在 IDE中,工程调用方法时,不进入方法即可悬浮提示方法、参数、返回值的意义,提高阅读效率。
2.【强制】所有的抽象方法(包括接口中的方法)必须要用 Javadoc注释、除了返回值、参数、异常说明外,还必须指出该方法做什么事情,实现什么功能。
说明:对子类的实现要求,或者调用注意事项,请一并说明。
3.【强制】所有的类都必须添加创建者和创建日期。
4.【强制】方法内部单行注释,在被注释语句上方另起一行,使用//注释。方法内部多行注释使用/* */注释,注意与代码对齐。
5.【强制】所有的枚举类型字段必须要有注释,说明每个数据项的用途。
6.【推荐】与其“半吊子”英文来注释,不如用中文注释把问题说清楚。专有名词与关键字保持英文原文即可。
反例:“TCP连接超时”解释成“传输控制协议连接超时”,理解反而费脑筋。
7.【推荐】代码修改的同时,注释也要进行相应的修改,尤其是参数、返回值、异常、核心逻辑等的修改。
说明:代码与注释更新不同步,就像路网与导航软件更新不同步一样,如果导航软件严重滞后,就失去了导航的意义。
8.【参考】谨慎注释掉代码。在上方详细说明,而不是简单地注释掉。如果无用,则删除。
说明:代码被注释掉有两种可能性:1)后续会恢复此段代码逻辑。2)永久不用。前者如果没有备注信息,难以知晓注释动机。后者建议直接删掉(代码仓库保存了历史代码)。
9.【参考】对于注释的要求:第一、能够准确反应设计思想和代码逻辑;第二、能够描述业务含义,使别的程序员能够迅速了解到代码背后的信息。完全没有注释的大段代码对于阅读者形同天书,注释是给自己看的,即使隔很长时间,也能清晰理解当时的思路;注释也是给继任者看的,使其能够快速接替自己的工作。
10.【参考】好的命名、代码结构是自解释的,注释力求精简准确、表达到位。避免出现注释的一个极端:过多过滥的注释,代码的逻辑一旦修改,修改注释是相当大的负担。
反例:
// put elephant into fridgeput(elephant, fridge);
方法名 put,加上两个有意义的变量名 elephant和 fridge,已经说明了这是在干什么,语义清晰的代码不需要额外的注释。
11.【参考】特殊注释标记,请注明标记人与标记时间。注意及时处理这些标记,通过标记扫描,经常清理此类标记。线上故障有时候就是来源于这些标记处的代码。
1)待办事宜(TODO):( 标记人,标记时间,[预计处理时间])表示需要实现,但目前还未实现的功能。这实际上是一个 Javadoc的标签,目前的 Javadoc还没有实现,但已经被广泛使用。只能应用于类,接口和方法(因为它是一个 Javadoc标签);
2)错误,不能工作(FIXME):(标记人,标记时间,[预计处理时间])在注释中用 FIXME标记某代码是错误的,而且不能工作,需要及时纠正的情况。
1.【强制】在使用正则表达式时,利用好其预编译功能,可以有效加快正则匹配速度。
说明:不要在方法体内定义:Pattern pattern = Pattern.compile(“规则”);
2.【强制】velocity调用 POJO类的属性时,建议直接使用属性名取值即可,模板引擎会自动按规范调用 POJO的 getXxx(),如果是 boolean基本数据类型变量(boolean命名不需要加 is前缀),会自动调用 isXxx()方法。
说明:注意如果是 Boolean包装类对象,优先调用 getXxx()的方法。
3.【强制】后台输送给页面的变量必须加$!{var}——中间的感叹号。
说明:如果 var等于 null或者不存在,那么${var}会直接显示在页面上。
4.【强制】注意 Math.random() 这个方法返回是 double类型,注意取值的范围 0≤x<1(能够取到零值,注意除零异常),如果想获取整数类型的随机数,不要将 x放大 10的若干倍然后取整,直接使用 Random对象的 nextInt或者 nextLong方法。
5.【强制】获取当前毫秒数 System.currentTimeMillis(); 而不是 new Date().getTime();
说明:如果想获取更加精确的纳秒级时间值,使用 System.nanoTime()的方式。在 JDK8中,针对统计时间等场景,推荐使用 Instant类。
6.【推荐】不要在视图模板中加入任何复杂的逻辑。
说明:根据 MVC 理论,视图的职责是展示,不要抢模型和控制器的活。
7.【推荐】任何数据结构的构造或初始化,都应指定大小,避免数据结构无限增长吃光内存。
8.【推荐】及时清理不再使用的代码段或配置信息。
说明:对于垃圾代码或过时配置,坚决清理干净,避免程序过度臃肿,代码冗余。
正例:对于暂时被注释掉,后续可能恢复使用的代码片断,在注释代码上方,统一规定使用三个斜杠(///)来说明注释掉代码的理由。
1.【强制】提交代码中不能存在空循环体和调试语句;
2.【强制】用SVN提交代码时写清楚备注,备注内容同文件头部的修改信息;
3.【强制】提交粒度是测试之后的一个完整功能;
4.【强制】提交代码前确保使用SoTower自带的代码格式化功能进行一次Format。
1.【强制】Java 类库中定义的可以通过预检查方式规避的 RuntimeException异常不应该通过catch 的方式来处理,比如:NullPointerException,IndexOutOfBoundsException等等。
说明:无法通过预检查的异常除外,比如,在解析字符串形式的数字时,不得不通过 catchNumberFormatException来实现。
正例:if (obj != null) {...}
反例:try { obj.method(); } catch (NullPointerException e) {…}
2.【强制】异常不要用来做流程控制,条件控制。
说明:异常设计的初衷是解决程序运行中的各种意外情况,且异常的处理效率比条件判断方式要低很多。
3.【强制】catch时请分清稳定代码和非稳定代码,稳定代码指的是无论如何不会出错的代码。对于非稳定代码的 catch尽可能进行区分异常类型,再做对应的异常处理。
说明:对大段代码进行 try-catch,使程序无法根据不同的异常做出正确的应激反应,也不利于定位问题,这是一种不负责任的表现。
正例:用户注册的场景中,如果用户输入非法字符,或用户名称已存在,或用户输入密码过于简单,在程序上作出分门别类的判断,并提示给用户。
4.【强制】捕获异常是为了处理它,不要捕获了却什么都不处理而抛弃之,如果不想处理它,请将该异常抛给它的调用者。最外层的业务使用者,必须处理异常,将其转化为用户可以理解的内容。
5.【强制】有 try块放到了事务代码中,catch异常后,如果需要回滚事务,一定要注意手动回滚事务。
6.【强制】finally块必须对资源对象、流对象进行关闭,有异常也要做 try-catch。
说明:如果 JDK7及以上,可以使用 try-with-resources方式。
7.【强制】不要在 finally块中使用 return。
说明:finally块中的 return返回后方法结束执行,不会再执行 try块中的 return语句。
8.【强制】捕获异常与抛异常,必须是完全匹配,或者捕获异常是抛异常的父类。
说明:如果预期对方抛的是绣球,实际接到的是铅球,就会产生意外情况。
9.【推荐】方法的返回值可以为 null,不强制返回空集合,或者空对象等,必须添加注释充分说明什么情况下会返回 null值。
说明:本手册明确防止 NPE是调用者的责任。即使被调用方法返回空集合或者空对象,对调用者来说,也并非高枕无忧,必须考虑到远程调用失败、序列化失败、运行时异常等场景返回null的情况。
10.【推荐】防止 NPE,是程序员的基本修养,注意 NPE产生的场景:
1)返回类型为基本数据类型,return包装数据类型的对象时,自动拆箱有可能产生 NPE。
反例:public int f() { return Integer对象}, 如果为 null,自动解箱抛 NPE;
2)数据库的查询结果可能为 null;
3)集合里的元素即使 isNotEmpty,取出的数据元素也可能为 null;
4)远程调用返回对象时,一律要求进行空指针判断,防止 NPE;
5)对于 Session中获取的数据,建议 NPE检查,避免空指针;
6)级联调用 obj.getA().getB().getC();一连串调用,易产生 NPE;
正例:使用 JDK8的 Optional类来防止 NPE问题。
11.【推荐】定义时区分 unchecked/checked 异常,避免直接抛出 new RuntimeException(),更不允许抛出 Exception或者 Throwable,应使用有业务含义的自定义异常。推荐业界已定义过的自定义异常,如:DAOException / ServiceException等。
12.【参考】对于公司外的 http/api开放接口必须使用“错误码”;而应用内部推荐异常抛出;跨应用间 RPC调用优先考虑使用 Result方式,封装 isSuccess()方法、“错误码”、“错误简短信息”。
说明:关于 RPC方法返回方式使用 Result方式的理由:
1)使用抛异常返回方式,调用方如果没有捕获到就会产生运行时错误;
2)如果不加栈信息,只是 new自定义异常,加入自己的理解的 error message,对于调用端解决问题的帮助不会太多。
如果加了栈信息,在频繁调用出错的情况下,数据序列化和传输的性能损耗也是问题。
13.【参考】避免出现重复的代码(Don’t Repeat Yourself),即 DRY原则。
说明:随意复制和粘贴代码,必然会导致代码的重复,在以后需要修改时,需要修改所有的副本,容易遗漏。必要时抽取共性方法,或者抽象公共类,甚至是组件化。
正例:一个类中有多个 public方法,都需要进行数行相同的参数校验操作,这个时候请抽取:private boolean checkParam(DTO dto) {...}
1.【强制】应用中不可直接使用日志系统(Log4j、Logback)中的 API,而应依赖使用日志框架SLF4J中的 API,使用门面模式的日志框架,有利于维护和各个类的日志处理方式统一。
- import org.slf4j.Logger;
-
- import org.slf4j.LoggerFactory;
-
- private static final Logger logger = LoggerFactory.getLogger(Abc.class);
2.【强制】日志文件至少保存 15天,因为有些异常具备以“周”为频次发生的特点。
3.【强制】应用中的扩展日志(如打点、临时监控、访问日志等)命名方式:appName_logType_logName.log。logType:日志类型,如 stats/monitor/access等;logName:日志描述。这种命名的好处:通过文件名就可知道日志文件属于什么应用,什么类型,什么目的,也有利于归类查找。
正例:mppserver应用中单独监控时区转换异常,如:mppserver_monitor_timeZoneConvert.log
说明:推荐对日志进行分类,如将错误日志和业务日志分开存放,便于开发人员查看,也便于通过日志对系统进行及时监控。
4.【强制】对 trace/debug/info级别的日志输出,必须使用条件输出形式或者使用占位符的方式。
说明:logger.debug("Processing trade with id:"+id +"and symbol:"+symbol);如果日志级别是 warn,上述日志不会打印,但是会执行字符串拼接操作,如果 symbol是对象,会执行 toString()方法,浪费了系统资源,执行了上述操作,最终日志却没有打印。
正例:(条件)建设采用如下方式
- if (logger.isDebugEnabled()) {
-
- logger.debug("Processing trade with id: " + id + " and symbol: " + symbol);
-
- }
正例:(占位符)logger.debug("Processing trade with id: {} and symbol : {} ", id, symbol);
5.【强制】避免重复打印日志,浪费磁盘空间,务必在 log4j.xml中设置 additivity=false。
正例:<logger name="com.taobao.dubbo.config" additivity="false">
6.【强制】异常信息应该包括两类信息:案发现场信息和异常堆栈信息。如果不处理,那么通过关键字 throws往上抛出。
正例:logger.error(各类参数或者对象 toString() + "_" + e.getMessage(), e);
7.【推荐】谨慎地记录日志。生产环境禁止输出 debug日志;有选择地输出 info日志;如果使用 warn来记录刚上线时的业务行为信息,一定要注意日志输出量的问题,避免把服务器磁盘撑爆,并记得及时删除这些观察日志。
说明:大量地输出无效日志,不利于系统性能提升,也不利于快速定位错误点。记录日志时请思考:这些日志真的有人看吗?看到这条日志你能做什么?能不能给问题排查带来好处?
8.【推荐】可以使用 warn 日志级别来记录用户输入参数错误的情况,避免用户投诉时,无所适从。如非必要,请不要在此场景打出 error 级别,避免频繁报警。
说明:注意日志输出的级别,error 级别只记录系统逻辑出错、异常或者重要的错误信息。
9.【推荐】尽量用英文来描述日志错误信息,如果日志中的错误信息用英文描述不清楚的话使用中文描述即可,否则容易产生歧义。国际化团队或海外部署的服务器由于字符集问题,【强制】使用全英文来注释和描述日志错误信息。
1.【强制】好的单元测试必须遵守 AIR原则。
说明:单元测试在线上运行时,感觉像空气(AIR)一样并不存在,但在测试质量的保障上,却是非常关键的。好的单元测试宏观上来说,具有自动化、独立性、可重复执行的特点。
l A:Automatic(自动化)
l I:Independent(独立性)
l R:Repeatable(可重复)
2.【强制】单元测试应该是全自动执行的,并且非交互式的。测试用例通常是被定期执行的,执行过程必须完全自动化才有意义。输出结果需要人工检查的测试不是一个好的单元测试。单元测试中不准使用 System.out来进行人肉验证,必须使用 assert来验证。
3.【强制】保持单元测试的独立性。为了保证单元测试稳定可靠且便于维护,单元测试用例之间决不能互相调用,也不能依赖执行的先后次序。
反例:method2需要依赖 method1的执行,将执行结果作为 method2的输入。
4.【强制】单元测试是可以重复执行的,不能受到外界环境的影响。
说明:单元测试通常会被放到持续集成中,每次有代码 check in时单元测试都会被执行。如果单测对外部环境(网络、服务、中间件等)有依赖,容易导致持续集成机制的不可用。
正例:为了不受外界环境影响,要求设计代码时就把 SUT的依赖改成注入,在测试时用 spring这样的 DI框架注入一个本地(内存)实现或者 Mock实现。
5.【强制】对于单元测试,要保证测试粒度足够小,有助于精确定位问题。单测粒度至多是类级别,一般是方法级别。
说明:只有测试粒度小才能在出错时尽快定位到出错位置。单测不负责检查跨类或者跨系统的交互逻辑,那是集成测试的领域。
6.【强制】核心业务、核心应用、核心模块的增量代码确保单元测试通过。
说明:新增代码及时补充单元测试,如果新增代码影响了原有单元测试,请及时修正。
7.【强制】单元测试代码必须写在如下工程目录:src/test/java,不允许写在业务代码目录下。
说明:源码构建时会跳过此目录,而单元测试框架默认是扫描此目录。
8.【推荐】单元测试的基本目标:语句覆盖率达到 70%;核心模块的语句覆盖率和分支覆盖率都要达到 100%
说明:在工程规约的应用分层中提到的 DAO层,Manager层,可重用度高的 Service,都应该进行单元测试。
9.【推荐】编写单元测试代码遵守 BCDE原则,以保证被测试模块的交付质量。
l B:Border,边界值测试,包括循环边界、特殊取值、特殊时间点、数据顺序等。
l C:Correct,正确的输入,并得到预期的结果。
l D:Design,与设计文档相结合,来编写单元测试。
l E:Error,强制错误信息输入(如:非法数据、异常流程、非业务允许输入等),并得到预期的结果。
10.【推荐】对于数据库相关的查询,更新,删除等操作,不能假设数据库里的数据是存在的,或者直接操作数据库把数据插入进去,请使用程序插入或者导入数据的方式来准备数据。
反例:删除某一行数据的单元测试,在数据库中,先直接手动增加一行作为删除目标,但是这一行新增数据并不符合业务插入规则,导致测试结果异常。
11.【推荐】和数据库相关的单元测试,可以设定自动回滚机制,不给数据库造成脏数据。或者对单元测试产生的数据有明确的前后缀标识。
正例:在 RDC内部单元测试中,使用 RDC_UNIT_TEST_的前缀标识数据。
12.【推荐】对于不可测的代码建议做必要的重构,使代码变得可测,避免为了达到测试要求而书写不规范测试代码。
13.【推荐】在设计评审阶段,开发人员需要和测试人员一起确定单元测试范围,单元测试最好覆盖所有测试用例。
14.【推荐】单元测试作为一种质量保障手段,不建议项目发布后补充单元测试用例,建议在项目提测前完成单元测试。
15.【参考】为了更方便地进行单元测试,业务代码应避免以下情况:
l构造方法中做的事情过多。
l存在过多的全局变量和静态方法。
l存在过多的外部依赖。
l存在过多的条件语句。
说明:多层条件语句建议使用卫语句、策略模式、状态模式等方式重构。
16.【参考】不要对单元测试存在如下误解:
l那是测试同学干的事情。本文是开发手册,凡是本文内容都是与开发同学强相关的。
l单元测试代码是多余的。系统的整体功能与各单元部件的测试正常与否是强相关的。
l单元测试代码不需要维护。一年半载后,那么单元测试几乎处于废弃状态。
l单元测试与线上故障没有辩证关系。好的单元测试能够最大限度地规避线上故障。
1.【强制】隶属于用户个人的页面或者功能必须进行权限控制校验。
说明:防止没有做水平权限校验就可随意访问、修改、删除别人的数据,比如查看他人的私信内容、修改他人的订单。
2.【强制】用户敏感数据禁止直接展示,必须对展示数据进行脱敏。
说明:中国大陆个人手机号码显示为:158****9119,隐藏中间 4位,防止隐私泄露。
3.【强制】用户输入的 SQL参数严格使用参数绑定或者 METADATA字段值限定,防止 SQL注入,禁止字符串拼接 SQL访问数据库。
4.【强制】用户请求传入的任何参数必须做有效性验证。
说明:忽略参数校验可能导致:
lpage size过大导致内存溢出
l恶意 order by导致数据库慢查询
l任意重定向
lSQL注入
l反序列化注入
l正则输入源串拒绝服务 ReDoS
说明:Java 代码用正则来验证客户端的输入,有些正则写法验证普通用户输入没有问题,但是如果攻击人员使用的是特殊构造的字符串来验证,有可能导致死循环的结果。
5.【强制】禁止向 HTML页面输出未经安全过滤或未正确转义的用户数据。
6.【强制】表单、AJAX提交必须执行 CSRF安全验证。
说明:CSRF(Cross-site request forgery)跨站请求伪造是一类常见编程漏洞。对于存在CSRF漏洞的应用/网站,攻击者可以事先构造好 URL,只要受害者用户一访问,后台便在用户 不知情的情况下对数据库中用户参数进行相应修改。
7.【强制】在使用平台资源,譬如短信、邮件、电话、下单、支付,必须实现正确的防重放的机制,如数量限制、疲劳度控制、验证码校验,避免被滥刷而导致资损。
说明:如注册时发送验证码到手机,如果没有限制次数和频率,那么可以利用此功能骚扰到其它用户,并造成短信平台资源浪费。
8.【强制】组装sql语句的时候为防止sql注入应避免直接拼接字符串,采用预编译的方式先把sql语句传入,而后传入参数。例:
正例:
- Object[] params={"f",”小刘”};
-
- List list=hibernateDao.executeSqlQuery("SELECT SEX,NAME FROM TEST WHERE SEX=? AND NAME = ? ", params);
-
- //Hql用法:
- List list=hibernateDao.findAll("select sex,name from TEST t WHERE t.sex=? and t.name = ? ", params);
9.【强制】发贴、评论、发送即时消息等用户生成内容的场景必须实现防刷、文本内容违禁词过滤等风控策略。
10.【强制】当在项目中需要自定义新的xml类型时,需要指定xml文档类型,文档类型定义分为内部声明DTD和外部引入DTD,当使用外部引用时需要判断路径合法性,因为外部引入会导致Xml外部实体注入:
反例:
- <?xml version="1.0" encoding="utf-8"?>
-
- <!DOCTYPE xdsec [
-
- <!ELEMENT methodname ANY >
-
- <!ENTITY xxe(实体引用名) SYSTEM "file:///etc/passwd"(实体内容) >]>
-
- <methodcall>
-
- <methodname>&xxe;</methodname>
-
- </methodcall>
这种写法则调用了本地计算机的文件/etc/passwd,如果file路径为恶意地址,XML内容会被解析,文件内容便通过&xxe被存放在了methodname元素中,造成了敏感信息的泄露。
正例:
11.【强制】当页面有输入框且数据需要与后台交互时,要对用户输入的信息在前端进行校验,前端校验可避免存储型xss攻击,如下所示:
正例:
(1)对用户的输入进行合理验证(如年龄只能是数字)。
(2)根据数据将要置于HTML上下文中的不同位置(HTML标签、HTML属性、
JavaScript脚本、CSS、URL),对所有不可信数据进行恰当的输出编码。
(3)设置HttpOnly属性,避免攻击者利用跨站脚本漏洞进行Cookie劫持攻击。在JavaEE中,给Cookie添加HttpOnly的代码如下:
- ...
-
- response.setHeader(“Set-Cookie”,”cookiename=cookievalue;path= /;
-
- Domain=dom ain vaule;Max-age= seconds;HttpOnly”);
-
- ...
例如:采用OWASPESAPI对数据输出HTML上下文中不同位置,编码方法如下:
- //HTMLencode
-
- ESAPI.encoder().encodeForHTML("inputData");
-
- //HTML attribute encode
-
- ESAPI.encoder().encodeForHTMLAttribute("inputData ");
-
- //JavaScriptencode
-
- ESAPI.encoder().encodeForJavaScript("inputData");
-
- //CSS encode
-
- ESAPI.encoder().encodeForCSS(""inputData");
- //URL encode
- ESAPI.encoder().encodeForURL("inputData");
12.【强制】验证请求来源合法性,防止当cookie被劫持时,利用网站对用户标识信任
跨站请求伪造(csrf)
说明:
攻击者盗用了你的身份,以你的名义发送恶意请求,对服务器来说这个请求是完全合法的,但是却完成了攻击者所期望的一个操作,比如以你的名义发送邮件、发消息,盗取你的账号,添加系统管理员,甚至于购买商品、虚拟货币转账等。 如下:其中Web A为存在CSRF漏洞的网站,Web B为攻击者构建的恶意网站,User C为Web A网站的合法用户。
正例:referer头过滤
正例:
Token认证,对每一次请求的a标签或form生产一个token,token信息放在请求头中。
13.【强制】如数组声明为final的public static,需保证数组对象本身只分配一次,且不可以改变数组中存储的值
14.【强制】JavaEE程序:session中存储非序列化对象,在session中存储的对象所属类以及引用对象所属类,都需要实现Serializable接口,对封装到jar包里的非序列化实体类,可以考虑转化为自己定义的类,然后再存入到session中
15.【强制】硬编码文件分隔符,在使用路径分隔符时,要用File.separator,不能直接使用“/”或“\”。
16.【强制】比较locale相关的数据未指定适当的locale,在调用与国际化有关的函数时,应指明语言,避免发生意外,正例:"TITLE".toLowerCase(Locale.ENGLISH)
17.【强制】在使用可能为null引用的对象前,应判断对象不为null。避免出现空指针异常
18.【强制】重定向后要结束程序,不应该在重定向代码后面写其他的业务代码,应该立即执行返回,避免额外代码被利用的风险 。
19.【强制】可序列化类中存在可序列化的敏感信息,对敏感信息字段使用transient关键字修饰,该字段将不会参与序列化。
20.【强制】JavaEE程序:遗留的调试代码包括:入口函数main或@test,在发布到正式环境的war包中部应出现,建议删除调试遗留代码
21.【强制】日志记录使用系统输出流, 避免程序中出现system.out或system.err等控制台输出语句,应使用日志记录工具代替
22.【强制】public static字段必须声明为final,否则声明为private
23.【强制】使用equles进行字符串比较
24.【强制】如构造方法中需调用本实例方法,需将方法声明为final或private,避免被继承,造成初始化构造方法时,子类未实例化;避免构造方法中调用可覆盖的方法。
25.【强制】精确计算时建议将浮点型转化为整型进行计算,或使用bigdecimal进行计算
26.【强制】系统捕获异常时不应该将异常信息直接输出e.printStackTrace,应该使用日志工具输出具体的错误例如:log.error(“订单异常”)
27.【强制】if表达式不可以出现永远为false或true
28.【强制】在类中不应出现未使用的字段,未使用字段建议删除
29.【强制】在类中不应出现未使用的变量,未使用变量建议删除
30.【强制】在类中不应出现未使用的方法,未使用方法建议删除,如果该方法为预留方法,请在注释中说明
31.【强制】避免重复的null检查
32.【强制】采用判断字符串长度的方法,来判断字符串是否为空,会提升程序性能
正例:“”.equals(“xxx”) 改为 xxx.lenght()>0 判断字符串是否为空,
33.【强制】在循环中拼接字符串,应使用StringBuffer代替String
34.【强制】使用new String()创建String对象,会占用额外的空间,降低性能;直接赋值java会在底层寻址,充分利用字符串池;
正例:String str1 = "abc";
35.【强制】无效的冗余代码,应该删除
反例1:
- // 空代码块
- if (i > 10) {
- 空
- }
反例2:
- // 如果后续使用,请在TODO 中写明用意,否则应该删除
- public void queryXX(String arg) {
- 空
- }
36.【强制】遗留的调试代码,调试代码应避免提交到svn,从而发布到正式环境
37.【强制】通过有权限的类更改访问权限修饰符,并确保修改的访问权限修饰符参数不能被攻击者控制;
38.【强制】前端输入数据,要在前端校验同时也需要在后台校验
39.【强制】程序中所需密码从配置文件获取经过加密的密码值
说明:在jdbc配置文件中 数据库密码是明文或者为空;
40.【强制】应用程序注释中不应保留密码等敏感信息。同时禁止使用password、pass、pwd等敏感字作为变量名、表单名或者字段名
反例:
//password = 123456
41.【强制】finally块中的不应出现return语句,会导致 try块中抛出的异常丢失,要求删除finally语句块中的return语句
42.【强制】异常捕获时异常尽量准确,不应直接抛出Exception或者捕获Exception;应尽量使用其子类,避免泛化的捕获异常
43.【强制】在finally代码块中抛出异常时会消除从try或catch程序段抛出的任何异常,并影响后续代码的执行,不应在finally代码块中抛出异常,应直接处理掉可能发生的异常
44.【强制】程序异常中断了预期中的程序流程,应恰当地进行异常处理,要么从异常中恢复,将异常重新抛出,由下一个和try语句对应的catch段来处理,要么根据catch程序段的上下文抛出另一个合适的异常,避免空的catch代码块
45.【强制】程序不应依赖于finalize()回 收 流资源,应在finally代码块中手动释放流资源。下面代码片段中,在finally 代码块中对流资源进行了合理的释放
46.【强制】忽略方法的返回值会导致程序无法发现意外状况。应确保方法返回的是期望的数据值
47.【强制】程序应对可能返回null的方法的返回值进行检查,避免产生NullPointException
48.【强制】应用程序所解析的JSON数据来源于不可信赖的数据源,程序没有对这些不可信赖的数据进行验证、过滤,如果应用程序使用未经验证的输入构造 JSON,则可以更改 JSON 数据的语义,避免JSON注入;
正例:
① 验证json的结构是正确,没有多余的key和value;
② 验证 json的内容是否有不合理的值,例如xss;
③ 验证 发送请求是合法的;
④ 也可以使用 JsonStringEncoder 对json字符串转码确保安全;
49.【强制】功能编码过程中,要时刻注意防范sql注入
防范措施:
① where 后面条件 都交给框架拼接(?占位符或者在mapper中循环),不要自己拼接sql字符串;
② sql中部要出现一些永为真的条件 1=1;
③ 根据项目整理一个sql指令白名单,如果拦截的sql语句中包含白名单中部存在的指令则视为 不合法sql;
50.【强制】在设计po类的时候要注意敏感字段的使用,避免直接绑定敏感字段
正例:
在实体类中 不要出现 name username password role 这些敏感字段,特别是在po中;将这些字段取一些别的名字
51.【强制】路径遍历
说明:
目录遍历(路径遍历)是由于Web服务器或者Web应用程序对用户输入的文件名称的安全性验证不足而导致的一种安全漏洞,使得攻击者通过利用一些特殊字符就可以绕过服务器的安全限制,访问任意的文件(可以是Web根目录以外的文件),甚至执行系统命令。该漏洞常常出现在文件读取或者展示图片等对文件读取交互的功能块。
解决办法:
① 对用户的输入进行验证,特别是路径替代字符如“../”和“~/”;
② 尽可能采用白名单的形式,验证所有的输入;
③ 合理配置Web服务器的目录权限;
④ 当程序出错时,不要显示内部相关配置细节;
⑤ 对用户传过来的文件名参数进行统一编码,对包含恶意字符或者空字符的参数进行拒绝;
52.【强制】当使用重定向时,重定向的地址不能是外部传入的
① 在servlet中 response.sendirect(URL) 这个url 尽量不要是用户带过来的参数;
② 制定重定向白名单,非白名单中地址都不允许跳转;
③ 重定向的地址定义到属性文件中,需要重定向时到文件中读取对应的地址;
53.【强制】不安全的框架绑定
说明:
当框架把前端页面中的输入信息转化成一个 实体类 传给controller层的一个方法,有的时候我们会直接把着 实体类传给 service 层 增/改的方法,这时这个实体类存在数据被篡改的风险。
解决办法:
① 不传实体类 在controller中手动拼成实体类或者传map;
② 对实体类的内容验证,确保是安全的.
54.【强制】http响应截断
说明:
HTTP响应截断是由于应用程序未对用户提交的数据进行严格过滤,当用户恶意提交包含 CR(回车,即URL编码%0d或\r)和 LF(换行符,即URL编码%0a或\n)的HTTP请求,服务器可能会创建两个 HTTP 响应,攻击者可以控制第二个响应并加载攻击。攻击者可控制响应的内容构造 XSS 攻击,其中响应中包含恶意的 JavaScript 或其它代码在用户的浏览器中执行,也有可能让用户重定向到攻击者控制的Web内容或在用户的主机上执行恶意操作。本篇文章以JAVA语言源代码为例,分析HTTP响应截断漏洞产生的原因以及修复方法。
正例:
(1)对用户的输入进行合理验证,对特殊字符(如<、>、’、”等)等进行编码。
(2)创建一份安全字符白名单,只接受完全由这些受认可的字符组成的输入出现在 HTTP 响应头文件中。
(3)使用源代码静态分析工具,进行自动化的检测,可以有效的发现源代码中的 HTTP 响应截断问题。
55.【强制】javascript 劫持
正例:
(1) referer 验证
(2)token 验证
(3)尽量不要直接执行javascript的响应结果
56.【强制】加密密钥不要在java代码中直接定义死 例如 xxxkey =’123456789ABCDE’;这样如果有人获得这个class文件的话,会反编译得到这个密钥
正例:
建议在 在初始化类的时候动态生成一组随机数
57.【强制】不受保护的cookie
正例:
当把重要的信息 缓存到cookie中的时候,对value值进行加密
58.【强制】当使用StringBuilder接收数据是,首先要判断数据大小
正例:
StringBuilder. Append() 接收数据的时候 对这个数据的大限进行判断,如果这个值过大可拒绝执行操作
59.【强制】无论是前端发送数据到后台,还是后台查询数据库中的信息,敏感信息一律不许出现明文
60.【强制】登录功能验证码漏洞
说明:
① 前端首先对密码加密
② 一个通过request参数获得,另一个可以放到http请求头的其它地方
③ 在controller中 首先对比着两个加密的字符串是否一致,防止请求过程中被篡改;
④ 数据库中有两种表存储密码,获取两种表里的密码 在controller中验证以否一致
⑤ 对jsp页面的密码解密(也可不解密看设计),然后用一种加密方式加密,最后使用js带过来的密码 与数据库中的做对比
1.【强制】表达是与否概念的字段,必须使用 is_xxx的方式命名,数据类型是 unsigned tinyint(1 表示是,0 表示否)。
说明:任何字段如果为非负数,必须是 unsigned。
注意:POJO类中的任何布尔类型的变量,都不要加 is前缀,所以,需要在<resultMap>设置从 is_xxx到 Xxx的映射关系。数据库表示是与否的值,使用 tinyint类型,坚持 is_xxx的命名方式是为了明确其取值含义与取值范围。
正例:表达逻辑删除的字段名 is_deleted,1 表示删除,0 表示未删除。
2.【强制】表名、字段名必须使用小写字母或数字,禁止出现数字开头,禁止两个下划线中间只出现数字。数据库字段名的修改代价很大,因为无法进行预发布,所以字段名称需要慎重考虑。
说明:MySQL在 Windows下不区分大小写,但在 Linux 下默认是区分大小写。因此,数据库名、表名、字段名,都不允许出现任何大写字母,避免节外生枝。
正例:powerpeak _admin,rdc_config,level3_name 反例:Powerpeak Admin,rdcConfig,level_3_name
3.【强制】表名不使用复数名词。
说明:表名应该仅仅表示表里面的实体内容,不应该表示实体数量,对应于 DO类名也是单数形式,符合表达习惯。
4.【强制】禁用保留字,如 desc、range、match、delayed等,请参考 MySQL官方保留字。
5.【强制】主键索引名为 pk_字段名;唯一索引名为 uk_字段名;普通索引名则为 idx_字段名。
说明:pk_ 即 primary key;uk_ 即 unique key;idx_ 即 index的简称。
6.【强制】小数类型为 decimal,禁止使用 float和 double。
说明:float和 double在存储的时候,存在精度损失的问题,很可能在值的比较时,得到不正确的结果。如果存储的数据范围超过 decimal的范围,建议将数据拆成整数和小数分开存储。
7.【强制】如果存储的字符串长度几乎相等,使用 char定长字符串类型。
8.【强制】varchar是可变长字符串,不预先分配存储空间,长度不要超过 5000,如果存储长度大于此值,定义字段类型为 text,独立出来一张表,用主键来对应,避免影响其它字段索引效率。
9.【强制】表必备三字段:id, gmt_create, gmt_modified。
说明:其中 id必为主键,类型为 bigint unsigned、单表时自增、步长为 1。gmt_create,gmt_modified的类型均为 datetime类型,前者现在时表示主动创建,后者过去分词表示被动更新。
10.【推荐】表的命名最好是加上“业务名称_表的作用”。
正例:powerpeak_task / force_project / trade_config
11.【推荐】库名与应用名称尽量一致。
12.【推荐】如果修改字段含义或对字段表示的状态追加时,需要及时更新字段注释。
13.【推荐】字段允许适当冗余,以提高查询性能,但必须考虑数据一致。冗余字段应遵循:
1)不是频繁修改的字段。
2)不是 varchar超长字段,更不能是 text字段。
正例:商品类目名称使用频率高,字段长度短,名称基本一成不变,可在相关联的表中冗余存储类目名称,避免关联查询。
14.【推荐】单表行数超过 500万行或者单表容量超过 2GB,才推荐进行分库分表。
说明:如果预计三年后的数据量根本达不到这个级别,请不要在创建表时就分库分表。
15.【参考】合适的字符存储长度,不但节约数据库表空间、节约索引存储,更重要的是提升检索速度。
正例:如下表,其中无符号值可以避免误存负数,且扩大了表示范围。
对象 | 年龄区间 | 类型 | 字节 | 表示范围 |
人 | 150岁之内 | tinyint unsigned | 1 | 无符号值:0到 255 |
龟 | 数百岁 | smallint unsigned | 2 | 无符号值:0到 65535 |
恐龙化石 | 数千万年 | int unsigned | 4 | 无符号值:0到约 42.9亿 |
太阳 | 约 50亿年 | bigint unsigned | 8 | 无符号值:0到约 10的 19次方 |
1.【强制】业务上具有唯一特性的字段,即使是多个字段的组合,也必须建成唯一索引。
说明:不要以为唯一索引影响了 insert速度,这个速度损耗可以忽略,但提高查找速度是明显的;另外,即使在应用层做了非常完善的校验控制,只要没有唯一索引,根据墨菲定律,必然有脏数据产生。
2.【强制】超过三个表禁止 join。需要 join的字段,数据类型必须绝对一致;多表关联查询时,保证被关联的字段需要有索引。
说明:即使双表 join也要注意表索引、SQL性能。
3.【强制】在 varchar字段上建立索引时,必须指定索引长度,没必要对全字段建立索引,根据实际文本区分度决定索引长度即可。
说明:索引的长度与区分度是一对矛盾体,一般对字符串类型数据,长度为 20的索引,区分度会高达 90%以上,可以使用 count(distinct left(列名, 索引长度))/count(*)的区分度 来确定。
4.【强制】页面搜索严禁左模糊或者全模糊,如果需要请走搜索引擎来解决。
说明:索引文件具有 B-Tree的最左前缀匹配特性,如果左边的值未确定,那么无法使用此索引。
5.【推荐】如果有 order by的场景,请注意利用索引的有序性。order by 最后的字段是组合索引的一部分,并且放在索引组合顺序的最后,避免出现 file_sort的情况,影响查询性能。正例:where a=? and b=? order by c; 索引:a_b_c
反例:索引中有范围查找,那么索引有序性无法利用,如:WHERE a>10 ORDER BY b; 索引a_b无法排序。
6.【推荐】利用覆盖索引来进行查询操作,避免回表。
说明:如果一本书需要知道第 11章是什么标题,会翻开第 11章对应的那一页吗?目录浏览一下就好,这个目录就是起到覆盖索引的作用。
正例:能够建立索引的种类分为主键索引、唯一索引、普通索引三种,而覆盖索引只是一种查询的一种效果,用 explain的结果,extra列会出现:using index。
7.【推荐】利用延迟关联或者子查询优化超多分页场景。
说明:MySQL并不是跳过 offset行,而是取 offset+N行,然后返回放弃前 offset行,返回N行,那当 offset特别大的时候,效率就非常的低下,要么控制返回的总页数,要么对超过特定阈值的页数进行 SQL改写。
正例:先快速定位需要获取的 id段,然后再关联:SELECT a.* FROM 表 1 a, (select id from 表 1 where 条件 LIMIT 100000,20 ) b where a.id=b.id
8.【推荐】SQL性能优化的目标:至少要达到 range 级别,要求是 ref级别,如果可以是 consts最好。
说明:
1)consts 单表中最多只有一个匹配行(主键或者唯一索引),在优化阶段即可读取到数据。
2)ref 指的是使用普通的索引(normal index)。
3)range 对索引进行范围检索。
反例:explain表的结果,type=index,索引物理文件全扫描,速度非常慢,这个 index级别比较 range还低,与全表扫描是小巫见大巫。
9.【推荐】建组合索引的时候,区分度最高的在最左边。
正例:如果 where a=? and b=? ,如果 a列的几乎接近于唯一值,那么只需要单建 idx_a索引即可。
说明:存在非等号和等号混合时,在建索引时,请把等号条件的列前置。
如:where c>? andd=? 那么即使 c的区分度更高,也必须把 d放在索引的最前列,即索引 idx_d_c。
10.【推荐】防止因字段类型不同造成的隐式转换,导致索引失效。
11.【参考】创建索引时避免有如下极端误解:
1)宁滥勿缺。认为一个查询就需要建一个索引。
2)宁缺勿滥。认为索引会消耗空间、严重拖慢更新和新增速度。
3)抵制惟一索引。认为业务的惟一性一律需要在应用层通过“先查后插”方式解决。
1.【强制】不要使用 count(列名)或 count(常量)来替代 count(*),count(*)是 SQL92定义的标准统计行数的语法,跟数据库无关,跟 NULL和非 NULL无关。
说明:count(*)会统计值为 NULL的行,而 count(列名)不会统计此列为 NULL值的行。
2.【强制】count(distinct col) 计算该列除 NULL之外的不重复行数,注意count(distinct col1, col2) 如果其中一列全为 NULL,那么即使另一列有不同的值,也返回为 0。
3.【强制】当某一列的值全是 NULL时,count(col)的返回结果为 0,但 sum(col)的返回结果为NULL,因此使用 sum()时需注意 NPE问题。
正例:可以使用如下方式来避免 sum的 NPE问题:SELECT IF(ISNULL(SUM(g)),0,SUM(g))FROM table;
4.【强制】使用 ISNULL()来判断是否为 NULL值。
说明:NULL与任何值的直接比较都为 NULL。
1) NULL<>NULL的返回结果是 NULL,而不是 false。
2) NULL=NULL的返回结果是 NULL,而不是 true。
3) NULL<>1的返回结果是 NULL,而不是 true。
5.【强制】 在代码中写分页查询逻辑时,若 count为 0应直接返回,避免执行后面的分页语句。
6.【强制】不得使用外键与级联,一切外键概念必须在应用层解决。
说明:以学生和成绩的关系为例,学生表中的 student_id是主键,那么成绩表中的 student_id则为外键。如果更新学生表中的 student_id,同时触发成绩表中的 student_id更新,即为 级联更新。外键与级联更新适用于单机低并发,不适合分布式、高并发集群;级联更新是强阻 塞,存在数据库更新风暴的风险;外键影响数据库的插入速度。
7.【强制】禁止使用存储过程,存储过程难以调试和扩展,更没有移植性。
8.【强制】数据订正(特别是删除、修改记录操作)时,要先 select,避免出现误删除,确认无误才能执行更新语句。
9.【推荐】in操作能避免则避免,若实在避免不了,需要仔细评估 in后边的集合元素数量,控制在 1000个之内。
10.【参考】如果有国际化需要,所有的字符存储与表示,均以 utf-8编码,注意字符统计函数的区别。
说明:
SELECT LENGTH("轻松工作"); 返回为 12
SELECT CHARACTER_LENGTH("轻松工作"); 返回为 4
如果需要存储表情,那么选择 utf8mb4来进行存储,注意它与 utf-8编码的区别。
11.【参考】TRUNCATE TABLE 比 DELETE 速度快,且使用的系统和事务日志资源少,但 TRUNCATE无事务且不触发 trigger,有可能造成事故,故不建议在开发代码中使用此语句。
说明:TRUNCATE TABLE 在功能上与不带 WHERE 子句的 DELETE 语句相同。
1.【强制】在表查询中,一律不要使用 * 作为查询的字段列表,需要哪些字段必须明确写明。
说明:
1)增加查询分析器解析成本。
2)增减字段容易与 resultMap配置不一致。
3)无用字段增加网络消耗,尤其是 text类型的字段。
2.【强制】POJO类的布尔属性不能加 is,而数据库字段必须加 is_,要求在 resultMap中进行字段与属性之间的映射。
说明:参见定义 POJO类以及数据库字段定义规定,在<resultMap>中增加映射,是必须的。在 MyBatis Generator生成的代码中,需要进行对应的修改。
3.【强制】不要用 resultClass当返回参数,即使所有类属性名与数据库字段一一对应,也需要定义;反过来,每一个表也必然有一个 POJO类与之对应。
说明:配置映射关系,使字段与 DO类解耦,方便维护。
4.【强制】sql.xml配置参数使用:#{},#param# 不要使用${} 此种方式容易出现 SQL注入。
5.【强制】iBATIS自带的 queryForList(String statementName,int start,int size)不推荐使用。
说明:其实现方式是在数据库取到statementName对应的SQL语句的所有记录,再通过subList取 start,size的子集合。
正例:
- Map<String, Object> map = new HashMap<>();
-
- map.put("start", start);map.put("size", size);
6.【强制】不允许直接拿 HashMap与 Hashtable作为查询结果集的输出。
说明:resultClass=”Hashtable”,会置入字段名和属性值,但是值的类型不可控。
7.【强制】更新数据表记录时,必须同时更新记录对应的 gmt_modified字段值为当前时间。
8.【推荐】不要写一个大而全的数据更新接口。传入为 POJO类,不管是不是自己的目标更新字段,都进行 update table set c1=value1,c2=value2,c3=value3; 这是不对的。执行 SQL时,不要更新无改动的字段,一是易出错;二是效率低;三是增加 binlog存储。
9.【参考】@Transactional事务不要滥用。事务会影响数据库的 QPS,另外使用事务的地方需要考虑各方面的回滚方案,包括缓存回滚、搜索引擎回滚、消息补偿、统计修正等。
10.【参考】<isEqual>中的 compareValue是与属性值对比的常量,一般是数字,表示相等时带上此条件;<isNotEmpty>表示不为空且不为 null时执行;<isNotNull>表示不为 null值时执行。
更多精彩,请关注我的"今日头条号":Java云笔记
随时随地,让你拥有最新,最便捷的掌上云服务
Copyright © 2003-2013 www.wpsshop.cn 版权所有,并保留所有权利。