线上问,和研发一起简单的从代码层面去解决问题,略low,
代码走查:
1.存在的问题
SQL查询
1.对于业务上没有用的字段没必要添加
比如 IsDel ,在具体的业务上,删除数据是直接硬删除,直接删除了物理数据,所以,在SQL查询的时候,SQL读着可能没有问题,但实际执行却没有意义,没必要添加
2.!=不等于,查空值,要和判断语句相符合,不要在SQL语句中否定,而if判断中肯定,很容易迷茫
方法名称
1.方法名称要用更为让人易懂的词汇,尤其是判断
checkTitleOnlyOne 这种返回,不明确的是否为true还是false,
不如这种isBlank(orgId) 是true,就返回true
方法调用规范
1.虽然都会调用dao,但是类不同,不能不同的类用相同的类名调用相同的方法,不易理解也不符合代码规范
2.不能图省事,直接复制代码
Debug调试
1.代码调试必须使用debug,这样才能去发现参数是否正常赋值、排查判断返回值具体出现什么位置
参数、判断---确定问题
判断重复
1.如果业务上不存在判断,就没必要添加,或者就不符合业务上的逻辑,比如,isBlank(orgId),orgId在全局都不会为空,就没必要在具体业务中增加这样的判断,没有任何的实际意义,只会影响代码阅读性,和代码编译的成本
2. 总结,
1.反复查看代码,需要日常写代码的积累,如果不写代码,读代码会很吃力,不会写代码,有什么资格指点代码
2.直接去读代码,按照代码的逻辑,可能查不出来问题,走查代码的时候,更多的时候,需要返回到代码业务的层面,反复推敲代码是否符合规则