代码走查1

线上问,和研发一起简单的从代码层面去解决问题,略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.直接去读代码,按照代码的逻辑,可能查不出来问题,走查代码的时候,更多的时候,需要返回到代码业务的层面,反复推敲代码是否符合规则

猜你喜欢

转载自blog.csdn.net/weixin_41585557/article/details/107846704