多个月之前，参与某项目开发，目前在进行代码评审，项目经理一直在吐槽以下几个问题，我资历尚浅，但仍感某些改进有些问题，请大侠们帮忙看看这评审意见是否可行。谢谢指导
代码评审发现的一些问题 1、职责不清晰：包含资产查询、收益查询、组合交易账号的查询逻辑（这个我认，一开始业务不大清晰，只是查资产，把一些要用到的数据一起查出来了） 2、一堆的 get/set，非常影响阅读（某些 vo 互相转换而已，由于属性多，影响阅读） 3、map 的 key 值含义不清晰（说是 key 都是 String 类型，要进入方法里面才看得到是什么，影响阅读） 4、方法体内处理入参（说是把传进去的入参改了，这样不妥，但是我的实现是需要改入参，说这是一个不好的习惯） 5、缺少复杂方法的算法注释（说方法行数太多，得说明这个方法的实现逻辑） 6、方法体入参过多（建议将入参封装成对象。。。）
觉得有些意见挺不靠谱的，希望能互相探讨互相学习一下。
这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到 V2EX 上打开本讨论主题的完整版本。
V2EX 是创意工作者们的社区，是一个分享自己正在做的有趣事物、交流想法，可以遇见新朋友甚至新机会的地方。
V2EX is a community of developers, designers and creative people.