CR小结

        自从mentor去年12月修产假之后，整个营销系统由我和XX扛了下来（架构师直接领导），只记得那几个月每天都过着乌云笼罩的日子，小心翼翼却又出各种问题，自身的原因很多，客观的原因也不少，总之一句话，顶住了系统的压力！近期人员变动后（架构师调岗和组内增加了新成员），忽然压力减了下来，这下给了自己喘息的机会，技术债也得拟补一下了，从对java8的恨相见晚到XX工具包的理解，再从Java知识的复习和并发编程的进一步熟悉，心里面又燃起了一把火......

        项目压力虽然比较大，可还是抽时间做了一部分CR，这次换了leader，又收获了一些，日后希望多做些类似的CR，下面是这次CR的一些总结，有些问题虽小，但仍要做到规范，写得比较粗糙，目的主要是做记录用：

1. perf4j

2. spring计时工具类stopwatch用法

3. mapstruct

4. 废弃的方法必须加上替代方法(没有话，也得写说明)，另外标志废弃的方法也要维护

5. 代码折叠，多加换行(代码分块)

6. 格式化代码最好只格式化自己写的，要全部格式化，单独commit一次，比较好追溯

7. 一次commit只做了一件事

8. git commit --fixup=“commitId”

9. service层的变量不要依赖上一层（如DTO，如Db的依赖，service建议不要以Db为返回对象）

10. service层函数若是Integer,建议int类弄，不用校验

11. 常量魔法值

12. 函数参数长时，可随意换行

13. 防御性编程

14. fastjson及Json转换

15. 代码整洁之道