发表于: 2018-01-16 23:38:05
1 589
今日完成的事情:
今天让施星大佬对我写的接口进行了codereview。
施星大佬对我的代码提出了几个问题:
1. 我的好几个接口,每次都有入参校验,还有一些判断标签,这样就造成了,代码非常冗余,修改起来也不容易找错,需要把一些重复的代码进行封装。
2. 有一些数字与信息对应的情况:
就像上面这种,需要写成常量,施星大佬的建议是放在core包里面,这个明天需要搞一下,我怕今晚自己搞了之后,项目就跑不起来了,因为修改core包需要重新deploy,前面几次每次修改core包就出问题。
3.入参校验的log日志,需要打印接口名,这样就比较容易判断出是哪个接口出问题了:
3. 多余的注释需要删除掉,因为在修改代码的时候,可能有一些东西不知道会不会到后来用上,所以就先暂时注释了, 到这个时候就需要把这些多余的代码删除掉。
4. 有一些变量名,判断标签的变量名:
在这里当时没有注意到,变量名没有做到见名知意,要修改成见名知意。
5.需要核对一下message信息,看看是不是一一对应了,不能有丝毫的疏忽。
6.拖动排序接口,我写的实现起来没有问题,但是就是太复杂了,今天施星大佬让我看了一下真实项目别人写的接口,需要进行修改。因为拖动排序接口对数据进行了好几次操作,可能这个接口会比较爆炸,不太好做性能优化。
明日计划的事情:
明天剩下来的问题一一解决,再次进行codereview,基本上就没有什么大的问题了,然后就是做性能优化,作为一个后端开发人员,最重要的可能就是性能优化了。
遇到的问题:
无
收获:
代码规范还是非常重要的,这个需要格外注意,今天又再次学习到了规范。
任务开始时间:2017/12/15
预计完成时间:2017/12/28
延期:第一次延期至2018/1/2
延期:第二次延期至2018/1/8
延期:第三次延期至2018/1/14
延期原因:自己太菜了,写接口太慢了,耽误了大家的进度。
解决办法:撸起袖子,加班加点敲代码
禅道:http://taskNaNteng.com/zentao/project-story-494.html
评论