多个月之前,参与某项目开发,目前在进行代码评审,项目经理一直在吐槽以下几个问题,我资历尚浅,但仍感某些改进有些问题,请大侠们帮忙看看这评审意见是否可行。谢谢指导
代码评审发现的一些问题 1、职责不清晰:包含资产查询、收益查询、组合交易账号的查询逻辑(这个我认,一开始业务不大清晰,只是查资产,把一些要用到的数据一起查出来了) 2、一堆的 get/set,非常影响阅读(某些 vo 互相转换而已,由于属性多,影响阅读) 3、map 的 key 值含义不清晰(说是 key 都是 String 类型,要进入方法里面才看得到是什么,影响阅读) 4、方法体内处理入参(说是把传进去的入参改了,这样不妥,但是我的实现是需要改入参,说这是一个不好的习惯) 5、缺少复杂方法的算法注释(说方法行数太多,得说明这个方法的实现逻辑) 6、方法体入参过多(建议将入参封装成对象。。。)
觉得有些意见挺不靠谱的,希望能互相探讨互相学习一下。
1
kimqcn 2017-11-23 16:42:09 +08:00
项目经理能说出这些来是为你好。
|
2
hcymk2 2017-11-23 16:43:20 +08:00
这些个《重构:改善既有代码的设计》基本都说的了的。
|
3
owenliang 2017-11-23 17:01:35 +08:00
模块边界都拆不清楚,这代码质量肯定烂。
|
4
jadec0der 2017-11-23 17:37:57 +08:00
找工作都说要找大牛指导,有人指导又觉得不服,咋办
|