Code Review

1、字符串拼接问题
一般带有复杂逻辑的字符串拼接,建议使用StringBuilder和StringBuffer,两者的区别

image.png

2、有常量时,字符串相等方法.equals()
一般使用常量在前
优雅:"target".equals(data.getTarget());
不优雅:data.getTarget().equals("target");


image.png

3、字符串是否控制判断
优雅:StringUtils.isEmpty(getCatalogueDirectoryVo.getName());
不优雅:null != getCatalogueDirectoryVo.getName() && !"".equals(getCatalogueDirectoryVo.getName())


image.png

4、魔法值问题——错误码


image.png

5、service代码缩减问题


image.png

原因:跨层调用, A service层中直接调用B模块或实体的dao层,造成本A service处理了大量B的业务逻辑,造成代码过长
解决方法:每个entity对应一个service,使用其他entity时,只调用该entity的service,而不是dao

6、无关传参


image.png

影响代码结构和整体架构

7、if没有大括号


image.png

8、主键参数和外键参数必须表名所属实体


image.png

image.png
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
平台声明:文章内容(如有图片或视频亦包括在内)由作者上传并发布,文章内容仅代表作者本人观点,简书系信息发布平台,仅提供信息存储服务。

推荐阅读更多精彩内容

  • JAVASE基础总结 Java运行环境 安装JDK文件, 进入计算机环境变量 添加新变量JAVA_HOME,变量值...
    半俗半雅_f42a阅读 1,826评论 0 0
  • 1. Java基础部分 基础部分的顺序:基本语法,类相关的语法,内部类的语法,继承相关的语法,异常的语法,线程的语...
    子非鱼_t_阅读 31,946评论 18 399
  • JAVA面试题 1、作用域public,private,protected,以及不写时的区别答:区别如下:作用域 ...
    JA尐白阅读 4,860评论 1 0
  • 一. Java基础部分.................................................
    wy_sure阅读 9,247评论 0 11
  • 久违的晴天,家长会。 家长大会开好到教室时,离放学已经没多少时间了。班主任说已经安排了三个家长分享经验。 放学铃声...
    飘雪儿5阅读 12,192评论 16 22