发表于: 2018-01-16 23:38:05

1 586


今日完成的事情:

今天让施星大佬对我写的接口进行了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



返回列表 返回列表
评论

    分享到