2020/10/10

十一假期前,进行了一次代码评审,发现我代码中存在的很多问题。确实令我汗颜,在此记录一下。

1、排序不能使用$符号,跟产品事先定义好有几个排序,然后把这些排序在mapper.xml中写固定。通过传一个flag来判断使用哪个排序。

2.不能使用 where 1=1,太low了,使用标签。

3.参数校验不能写在controller层,而是写在service层,因为service层不光可以用在控制层,也可以用在别的service中调用。在service层做校验,只需要做一遍就可以了。

11/12

今天好险,我看同事的代码的时候,觉得明显有问题,于是我想给他指出这个问题,但我没直接说你的代码有问题,而是说你的代码我有些看不懂,请你给我讲讲。结果人家一讲,我才发现写的其实没问题,是我没认真看!差点丢人了。。

以后都得这样,不能武断的认为别人的代码有问题,很可能是自己没看好。

2019/8/22

思维缜密,思路清晰,寥寥八字,但真的做到很不容易,昨天业务反应的一个问题,我简单一思考,就把原因往前端那边想,而对自己这边可能出现的原因,却怎么也想不起来,这就说明我思维还不够缜密,我的leader,稍加思索, 就能一语中的,指出问题所在,真的好惭愧!

问题大概如下:页面上一个查询功能,一个导出EXcel个功能,用户输入了一堆查询条件,查询好使,但导出就立刻报错,查询和导出 组装查询提交件的代码 是通用的一个,我当时就想到这是前端在传参的时候没传对。但却忽略了一个问题,查询和导出有个不同点,导出需要往一张record表里写入一条数据,这个表里有个字段 叫输入参数,这个字段的length是有限的,如果用户只是简单的查询,完全能胜任,但用户搞了一个很长的查询条件,结果写入数据库记录的时候,立马就报错了。

2019/8/20

非空校验:如果入参是一个List,那么需要做如下四种非空检验:

1.list本身是null,

2.list的size是0 这两个统一使用CollectionUtils.isNotEmpty()方法

3.list中的元素T是null

4如果需要使用list中的元素T的属性去调用方法,还需对T的get出来的属性值就行非空校验

5.根据阿里编程规范,字符串equal时,把字符串常量和你认为肯定不会为空的参数放到左边,因为一个null调用equal()方法,百分百会报错,但是把null放置到equal()方法内,就不会报错。

昨晚搞到凌晨1点,问题是这样的,一个导出Excel功能,我在我本地测试是好使的。我本地连测试环境的数据库,进行测试,也是好使的。但是代码部署到测试环境后,就不好使了,什么也导不出来……11点12点,我找了两个小时,想找出到底 我本地和测试环境,哪里不一样,但始终没找到、

最后,还是我经验丰富的leader,他看了下我的代码,几分钟就定位出了不同点!原因如下:代码是一样的,连的数据库也是一样的,但我在程序中调用了一个dubbo接口,这个外部接口,在我本地,和在测试环境,访问的IP不是一个环境,所以拿到的dubbo远程接口数据也就不一样,所以在我本地好使,但部署到测试环境,就嗝屁了!!

俗话说,吃一堑长一智,以后再遇到这种“找不同”类型的bug,我一定要思维更加缜密,更加冷静分析。

经过最近两天的实践,前些天一直困扰我的idea编辑代码时,内存飙升 卡死的问题,罪魁祸首就是阿里巴巴出的P3C插件的实时检测,右键,把实时检测关掉后,一切的卡顿都烟消云散!哈哈,没想到阿里巴巴这么大牛,也是一个坑爹货,P3C插件没有给我们用户任何关于实时检测占内存的提示!

2019/8/15

bean中的属性,布尔类型记得一定要用包装类大写的B:Boolean,而不要用小写的关键字boolean,还有命名布尔类型值的时候,尽量不要以is开头,不管是lombok,还是属性复制,还是序列化,这其中都有大坑,哈哈!

修改bean属性的时候,一定要有修改后building的意识,因为我不能知道这个被修改的bean都在哪用到了,修改后本地不building,直接提交的话,造成报错,到时就很不好了、

2019/8/14

今晚给一个新员工解决git无法拉代码的问题,什么都配置好了,但就是拉不下来,耗了一些时间,最后重启电脑才好了,总计下经验:还是重启治百病,哈哈!

如果项目本来用着好好的,更新过git以后,突然就无法启动,那十之八九肯定是上个提交人员,提交的代码引出的问题,这时,话不多说,直接看Git提交记录,把肇事者拎出来

Idea的常用快捷键经常会被别的一些软件所占用,我的态度就是绝不容忍,一定要把占用热键的幕后黑手揪出来,然后改掉。目前,我个人体会占用热键最多的就是,有道词典和搜狗输入法这两个渣渣,但都可以通过改他们的设置,把热键再改回来

最上方的import导包行上面,要注意一点,随着多次的开发,import会有一些不用的置灰的包,每次提交代码前,需要检查下这些import,把无用的去掉后再提交。Idea有一个自动导包(Auto Import)功能很好使,会把没用的包自动删除掉,建议开发团队一块儿用,如果只有一两个人用,会出现每次commit代码时,会多出来好多different

开发当中,一遇到问题,不要马上问别人,因为你问别人一次,就是打扰别人一次,就是打断别人的思路一次,一天多次打断一个程序员的思路,无异于谋财害命!我觉得,在不卡住自己开发进度的情况下,可以多积攒几个问题,一块问。

时不时的会在开发中遇到NullPointerException异常,主要实在dev和test环境,这两个环境的数据往往是不完整的,是有缺陷的。如何在开发中尽量避免空指针异常呢?

非空检验宁滥勿缺,加非空检验主要有以下几个注意事项:

调用外部的接口,比如通过dubbo调用远程服务,得到一个返回值,就一定要检验这个返回值。List 集合类型的返回值,校验集合本身不为空,并且集合的size大于0。通常项目中有封装好的方法,如:CollectionUtils.isEmpty();取反

遍历取出List中的元素,通过get方法获得元素field时,如果要调用该field,一定要加非空校验。要做最坏的假想,认为所有数据都有可能出现空值的情况。

平时在开发中,要有加非空检验,增加程序健壮性的意识,不要等到被测试人员测出了空指针,再去加。

编码中需要的七个意识:

多写注释的意识,为己为人,为了以后能快速看得懂代码,多写注释,特别是一些复杂的功能,一定要多写注释

多打log的意识,多打log,最大的好处就是方便在生产环境排错

非空校验的意识 记得用Optional

分拆代码的意识 长方法,分拆成若干个小方法,根据阿里的编程规范,每个方法最多80行

提高复用的意识 一些与业务无关的的util类方法,写到统一的一个util类中

使用新语法糖的意识 Optional和双冒号:: 在后续的开发中,一定要用起来

代码简洁的意识 尽可能的多使用兰慕妲lambda表达式

调用dubbo服务时,调用的入参size要可控,不能一次一调,也不能一次大批量的调用。


本文由转载于互联网,如有侵权请联系删除!