Java后端开发规范-

Java后端开发规范

　　一、技术栈规约

　　二、命名规范

　　三、Java代码规范（注释规范、异常与日志、代码逻辑规范）

　　四、Mybatis与SQL规范

　　五、结果检查（单元测试及代码扫描）

　　六、安全规范

一、技术栈规约

二、命名规范

命名使用英文词组合，严禁使用中文拼音或拼音首字母组合命名（专有名词例外） - OrganizationTreeNode, OrganizationVO ; 不推荐使用PSTree , Tlogs
groupId，package包名前缀统一为: com.wiwj
包名第三位为产品分类名，如com.wiwj.cbs
常量命名全大写，单词间下划线分隔。如: DEFAULT_PAGE_SIZE
其他命名遵循驼峰式命名法:类名：首字母大写的UpperCamelCase，如: Organization方法名、变量名：首字母小写的lowerCamelCase，如: orgName
特定标识命名:领域模型增加类型后缀标识，如xxVO, yyDAO基类/抽象类使用Base/Abstract等前缀标识设计模式类添加Factory，Builder，Proxy等标识Controller, Service, Mapper统一添加到对应分层目录接口实现类添加Impl后缀标识枚举类添加Enum后缀标识CRUD接口采用统一前缀: get, count, create, delete, update, batchCreate …

三、Java代码规范

　　注释规范

Java文件统一添加固定Header，通过IDE统一配置（code templates）

/**
*  

* @author mazhicheng@5i5j.com

* @version 1.0

* @date ${YEAR}/${MONTH}/${DAY} 

 */

接口和方法统一添加Java Doc标准注释

/**
*缓存key-value并设定过期时间
* @param key 缓存对象的key
* @param valueList 缓存对象
* @return 缓存是否成功
*/
 boolean addList(String key, List valueList);

需暂留的弃用类/方法添加 @Deprecated 废弃标记和 @see 链接指向新接口

* @see com.wiwj.common.cache.redis.JedisSentinelPoolUtil
*/
@Deprecated
public class JedisUtils {…}

异常与日志

调用外部服务等可能异常的代码块，用 try/catch 代码块捕获并在catch中记录异常跟踪日志及业务逻辑处理
禁止吞掉异常信息

　　　　禁止catch里不做任何记录和处理，吞掉异常及其堆栈信息

　　　　禁止: logger.error(“XXX操作异常”) 或 logger.error(“XXX操作异常”+e) 或 e.printStackTrace()

　　　　正确: logger.error("XXX操作异常", e)

对于非预期的条件，尽量增加else记录跟踪日志
禁止通过System.*.out()打印日志（单元测试例外）
日志记录logger需使用Slf4J代理声明，禁止绑死具体日志系统的API，避免后期更换日志组件导致代码的大量改动

　　　　private static final Logger log = LoggerFactory.getLogger(OrganizationServiceImpl.class);

　　　　如采用了lombok，可用 @Slf4j 注解替代以上声明。

对 trace/debug/info 级别的日志输出，必须使用占位符形式，避免直接String拼接异常信息（即使日志级别不匹配也会执行拼接操作空耗资源）。

　　　　正确写法如:

log.debug("当前用户id: {} ，操作对象: {}=>{} ", userId, objectType, objectId);
或条件输出形式如:
if(log.isDebugEnabled()){
log.debug("当前用户id: “+id+” ，操作对象: “+ objectType +”=> “+ objectId);
}

逻辑代码规范

废弃的/无用的代码一律直接删除，禁止以注释等方式保留。如需查看历史代码，通过SVN/Git的history找回

　　　　(无用的代码会干扰团队成员的阅读/或被误调，越积越多会导致代码维护成本增高)

接口类中的方法不需添加 public 修饰符
需要序列化的Bean类统一实现Serializable接口并用IDE生成serialVersionUID

public class MyEntity implements Serializable {     
private static final long serialVersionUID = 123456L;     
...
}

常用字符串统一定义在常量类里，如: “utf-8”, “yyyyMMdd”
避免数字类型比较的坑: 统一采用equals进行比较其值，不用==进行比较，避免踩坑。
if/else/for/while语句后必须使用大括号，即使只有一行代码。(需求总是变化的，一行是暂时的)
嵌套层次过多的代码块利用反向思维缩减层次
方法单一职责: 单个方法代码行数控制在100行以内，超长的需要拆分（拆分成多个方法或类）
避免NPE(NullPointException)的一些建议:

equals比较将非空对象前置:如 "true".equals(request.getParameter("isXx"))，即使后者为空也不会导致NPE。
数据库字段可空的映射属性使用包装类型定义:如基本数据类型的int映射到数据库的null值将产生NPE，而用吧包装类型 Integer 则不会。
可能为空的变量进行必要判空，并在非预期条件下打印必要的跟踪日志，不但避免NPE，还非常便于跟踪调试。如:
级联调用 obj.getA().getB().getC() 易产生 NPE，先进行判空或使用 JDK8 的 Optional 类包装。
调用Dubbo接口拿到返回值时，进行判空。
封装统一的判空类用于常用类型的判空，代码需要判空时统一调用即可。如 XX.isEmpty(), XX.isNotEmpty()

遵循: Don’t Repeat Yourself，即 DRY 原则。避免进行简单的复制粘贴修改，当出现重复代码时思考是否封装

　　　　当代码中存在大量重复代码时，一旦代码逻辑变动将很容易导致顾此失彼，产生bug，非常不利于维护。

Bean属性拷贝推荐用Spring BeanCopier或者Mapstruct,避免Apache BeanUtils或调用setter
禁止在循环中执行耗时的操作，如在循环中执行SQL语句/调用外部服务等

// 错误的示例：
for(Long id : idList){     
// 循环执行SQL查询或调用外部系统接口，产生性能问题     
Entity entity = xxService.getEntityById(id);    
 ...
}
 // 此案例的更优方案是 通过idList一次性查询获取到Entity集合，然后转换为Map供后续获取。

需要多次使用的可复用对象将对象单独定义，禁止多次调用取不同属性。如：

String name = userService.getUser(id).getName();
Long deptId = userService.getUser(id).getDepeId();
替换为:
User user = userService.getUser(id); String name = user.getName(), ….

可异步执行的耗时操作采用异步处理：使用Spring @Async 或 MQ，或夜间Timer定时
常用数据考虑缓存，存入Redis，设置缓存过期时间
需要保证写一致性的逻辑，在外层方法上添加事务 @Transactional(rollbackFor = Exception.class)

四、Mybatis与SQL规范

表名、字段名、索引等数据结构定义大小写: Oracle大写, MySQL小写。名称使用英文+下划线，并控制总长度，如 user_name。
表名建议采用“模块标识_”前缀，如 bas_user（如果模块库独立可省略模块名标识）
禁止程序中的SQL使用并行计算 /*+parallel(t,n)*/
SQL使用标准SQL，避免出现数据库特定的语法
未经评审不可直接使用视图、触发器、存储过程 SQL JOIN表数量不超过3张，超过3张表需要经过评审 (拆分成多次单表查询、主表冗余、程序绑定id-name映射、根据条件动态JOIN等)。
合理创建索引，并尽量避免不走索引的情况: 如LIKE右/任意匹配(‘%xx’, ‘%xx%’)不走索引, 换为“精确匹配=”或固定前缀的左匹配’张%’不等条件（!=、<>、NOT）不走索引，应尽量避免（转换成IN/BETWEEN等）IS (NOT) NULL 不走索引，应尽量避免（如字段给定默认值，避免NULL）索引列使用函数或隐式转换都将导致索引失效，如 to_char(create_date,'yyyymmdd') = '20190102'
禁止手动拼接SQL语句，利用Mybatis等ORM框架的动态SQL实现。参数使用#{} (避免${}产生SQL注入问题)。
禁止使用数据库处理函数 decode()，改为Java枚举或Map定义，通过id进行绑定 decode(client.TYPE, 1, '私客', 2, '店组公客', 3, '组团公客‘)
禁止动态拼接时强加 1=1 之类的写法，如WHERE 1=1。使用Mybatis动态SQL标签实现，如，，
SQL中的参数类型确保与列定义一致，避免数据库隐式转换开销且无法使用索引，如:列定义为日期类型，参数要转换为Date日期类型进行比较:

CREATE_TIME <= '2019-04-14 23:59:59’
CREATE_TIME <= to_date('2019-04-14 00:00:00','yyyy-MM-dd HH24:mi:ss’)

列定义为数字类型，参数不用String DEPT_ID = '123’
ID主键自增的情况下，按create_time排序改为按ID排序，效果一样效率更高

五、检查结果

后端服务及其他需要自测的代码，编写对应的单元测试类，统一采用Junit，禁止直接在原Java类中写main()方法自测。

单元测试会在打包前统一运行，可及时发现受影响的代码问题（比如新代码导致了之前的代码逻辑产生问题，如果有单元测试可在打包时及时发现）
Junit单元测试类示例:
public class TestApollo {
@Test // 标记为单元测试方法
public void testApolloConfig(){
String appId = Foundation.app().getAppId();
// 预期结果断言
Assert.assertNotNull(appId);
}
}