这是一个创建于 2345 天前的主题，其中的信息可能已经有所发展或是发生改变。
多个月之前，参与某项目开发，目前在进行代码评审，项目经理一直在吐槽以下几个问题，我资历尚浅，但仍感某些改进有些问题，请大侠们帮忙看看这评审意见是否可行。谢谢指导
代码评审发现的一些问题 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
找工作都说要找大牛指导，有人指导又觉得不服，咋办
|
Select Language